manujurado1 / SportsBar-IV

GNU General Public License v3.0
0 stars 1 forks source link

M3 - Gestión niveles post-partido. #126 #127

Closed manujurado1 closed 1 year ago

manujurado1 commented 1 year ago

En primer lugar se han cambiado las funciones de aumentar y decrementar nivel ya que estas solo se usan tras un partido y se aplican en conjunto a un equipo. Como no es algo ligado al comportamiento del usuario, se han eliminado los tests. Estas funciones se testean ahora dentro del test de modificar niveles post partido. Se ha establecido como criterio de modificación que se aumenta el nivel si se gana y se disminuye si se pierde, independientemente de lo abultado que sea el resultado. Si el partido queda en empate, todo queda igual. closes #126

manujurado1 commented 1 year ago

Ya estaría listo para revisión! @JJ Una vez aprobado esto se daría por terminado el milestone actual, el cual contiene la inclusión de la lógica de negocio.

manujurado1 commented 1 year ago

Vale, entonces entiendo que lo que tengo que hacer es quitar la parte del test que compruebo esa salida de error verdad? Y si es así y no lo testeo, ¿es necesario que haga esa comprobación y por consiguiente, que exista el atributo fecha en un equipo?

JJ commented 1 year ago

Vale, entonces entiendo que lo que tengo que hacer es quitar la parte del test que compruebo esa salida de error verdad? Y si es así y no lo testeo, ¿es necesario que haga esa comprobación y por consiguiente, que exista el atributo fecha en un equipo?

En las revisiones nunca se puede decir qué es lo que uno quiere que se haga. Simplemente haz lo que estimes conveniente, y se acepta o no. En todo caso ya está aceptado, como ves, el que lo cambies o no, antes de fusionar o más adelante, es cosa tuya.