Sobre inventarioDeEQuiapmiento en caso de Refactor para quitar el getEquipamiento() / obtenerEquipamiento()

[RodriguezNazareno56]

Sobre inventarioDeEQuiapmiento en caso de Refactor para quitar el getEquipamiento() / obtenerEquipamiento()

[hmaldonado12]

Por eso lo0 comente en el grupo ya que la primera solucion me hacia ruido. Hice una segunda solucion que se ajusta mas a lo que estamos buscando

[fcondori10]

Es cierto que añade complejidad al incluir la clase inventario y ser necesario entender su dependencia con gladiador y los equipos. Es un requerimiento el que el primer premio sea un casco y otro requerimiento el que la fiera provoque perder 15 puntos de energía con el casco. Al ser requerimientos es más difícil que cambien, en cambio los refactor que hagamos nosotros sobre nuestras implementaciones pueden cambiar como en inventario.

Mejor testear sobre el comportamiento esperado si tiene el casco el cual resulta ser que los ataques con las fieras resultan ser de 15 o que al mejorarse recibieses una armadura

Si nos quisiéramos deshacer también del getEnergia propondría lo siguiente. Podemos inicializar al jugador con 15 de energía, premiarlo y afectarlo con la fiera, luego probar si puede avanzar. Como la fiera quita 15 puntos de energía el jugador no puede avanzar y lanza una excepción, la cual por ejemplo, usamos en el caso de uso 3

@Test
public void verificarQueSiRecibeUnPremioPorPrimeraVezObtieneUnCasco() {
   
   // un gladiador inicializado con 15 puntos de energía y premiado por primera vez

   fiera.afectarGladiador(gladiador)
   //Assert
   Throwable exception= Assertions.assertThrows(MovimientoExeption.class, () -> {
        this.gladiador.mover();
    }) 

Lo que sucede es que tambien deberiamos probar el caso contrario, en el que tiene 16 de energía al inicio y aún con el ataque de la fiera puede avanzar. De esta forma sabriamos que el gladiador tiene el casco pues aún con un ataque de 15 de energía el gladiador aún puede moverse debido a la resistencia del casco
Parece algo rebuscado, pero el ataque de la fiera es algo establecido por contrato

[Jmsosaa]

Comparto la forma de testear el caso de uso como menciono Gustavo. El gladiador se inicia con Energia = 20 y al obtener por primera vez Equipamiento, se le agrega el casco.

Si es atacado por la fiera teniendo un casco, su energia disminuye a 15. Sabiendo que cada avance le cuesta al gladiador un punto de energia, podemos hacer que avace 5 veces para que agote toda su energia.

Al intentar avanzar una 6ta vez verificamos que se lanze la exepcion "El gladiador no se puede mover sin energia".

[RodriguezNazareno56]

El issue que genero la modificación primera fue este #18