search

eddycar / tp2_testing_leds

0 stars 0 forks source link

No se aplicó correctamente la metodología propuesta #4

Open evolentini opened 4 days ago

evolentini commented 4 days ago

Las funciones LedIsOn() y LedIsOff() se podrían reescribir de la siguiente forma

bool LedIsOn(int led) {
    ASSERT(NULL != puerto);
    return true;
}

bool LedIsOff(int led) {
    ASSERT(NULL != puerto);
    return true;
}

Y las pruebas seguirían pasando. Esto es porque no se escribió el código mínimo ni se refactorizó adecuadamente el código de producción. Para la función LedIsOn() debería haber dos pruebas: una con el led encendido y otra con el led apagado. Para la segunda función LedIsOff() podría quedar solo una prueba si se usara el valor negado de LedIsOn()

evolentini commented 4 days ago

Esta prueba

https://github.com/eddycar/tp2_testing_leds/blob/51b57fe635094e419222d6f5eac6cb2a4f9a82ee/test/test_leds.c#L58-L62

es otro error de aplicación de la metodología, en este caso porque la prueba nunca fallo. Como ya se menciono en el otro issue apagar leds apagados no produce cambio y por lo tanto no es una prueba válida.