FedericoFuentesWeber / TPs-PdeP-2018

0 stars 0 forks source link

7) Dependencia innecesaria entre Rolando y la Máscara #39

Closed sanuzzi closed 5 years ago

sanuzzi commented 5 years ago

Tienen esto:

    test "El poder de lucha de la mascara de Rolando cuando es para nada oscura es de 4"{
        rolando.eliminaUnArtefacto(mascaraOscura)
        rolando.agregaUnArtefacto(mascaraParaNadaOscura)
        assert.equals(4, mascaraParaNadaOscura.poderDeLucha(rolando))
    }

Y es medio raro que haga falta que Rolando conozca a la máscara.

La máscara debería poder decirnos cuánta lucha le aportaría a Rolando, por más que Rolando aún no la tenga entre sus cosas.

Quizá no haga falta agregarla como artefacto, no verifiqué con el código aún. Pero viendo el tests, están dando una idea de dependencia que no debería hacer falta. Es decir, las primeras dos líneas no deberían existir.