jonathanloscalzo / batalla_naval_ttps_ruby Goto Github PK

Les voy a pedir algunas correcciones para terminar de cerrar el trabajo, con estas correcciones estaría aprobado:

• Cuando un usuario intenta ver un juego que no le pertenece la aplicación lanza una excepción (NoMethodError at /games/2 undefined method `user' for false:FalseClass). La secuencia de pasos que seguí fue crear un usuario y después ir al link para acceder al juego que hay entre otros 2 usuarios.
• app_test.rb quedó vacío habría que sacarlo.
• Los tests de la lógica del juego quizá sea más apropiado que se hagan directamente sobre el modelo, de hecho eso les evitaría tener que hacer logouts, logins y seguir las redirecciones para testear cosas que no tienen que ver con eso.
• models/Board.rb en la linea 34 array + self.waters.to_a no tiene efecto, lo que buscan es <<.
• Todas las vistas deberían ser html, /games y /players por ejemplo son JSON.
• Por la parte de eliminar games, no se preocupen no se pedía en el enunciado.

Los siguiente es un comentario, dado el tiempo que hay no les voy a pedir que lo corrijan:

• Los tests de la lógica del juego quizá sea más apropiado que se hagan directamente sobre el modelo, de hecho eso les evitaría tener que hacer logouts, logins y seguir las redirecciones para testear cosas que no tienen que ver con eso.