alu654 / tps-Parcial

0 stars 0 forks source link

Assert equals #7

Open LucasZibell opened 4 years ago

LucasZibell commented 4 years ago

En vez de comparar todo el tiempo usando assertThat, pueden usar assertEquals que recibe como primer parámetro el valor esperado y como segundo lo que iría antes del == en el assertThat.

tostrovsky commented 4 years ago

Hola, cuando intentamos hacer los test, nos da error en los test que estaban relacionados con calcularEnergia, que ustedes bien nos dijeron que lo corrijamos y omitamos los parámetros que no usa. El mensaje del test dice "burns no entiende calcularEnergia(param1, param2)"

tostrovsky commented 4 years ago

actualizo: estos errores son los únicos que tiran los test, y ocurre tanto cuando omitimos un parámetro o los dos.

LucasZibell commented 4 years ago

mmmm Puede ser que wollok lo tome como varios metodos distintos jajajajaja dependiendo de la cantidad de parametros. Tira error cuando usaron la variable anonima? (_, riqueza)

tostrovsky commented 4 years ago

no, pero porque toma "" como un nombre de un parametro, es como un identificador de un solo caracter ´´. Por eso, si ponemos calcular( , ) toma como que le estamos poniendo el mismo nombre a ambos parámetros también, y rompe

LucasZibell commented 4 years ago

Y esto: (_, __) 🤔

tostrovsky commented 4 years ago

sí, eso sirve porque toma _ y __ como dos identificadores diferentes. Ahora no rompe nada. Qué pasó? por qué no sirvió lo que nos corrigieron?

LucasZibell commented 4 years ago

Digamos que wollok es mistico