No se aplicó correctamente la metodología

evolentini commented 6 months ago

La metodología de TDD exige que la prueba falle antes de escribir código de producción. Esta prueba nunca falló porque los leds ya inician apagados. Es decir que es una prueba que no prueba nada.

https://github.com/ceovejero/cese_tsse_tp4/blob/ee6729f05e0132bfb378c7b7a19c1ffa0c8e052b/test/test_leds.c#L142-L149

La secuencia correcta de prueba es

 void test_apagar_todos_y_verificar_todos_apagados(void) { 

     leds_turn_on_all(); 
     leds_turn_off_all(); 
     TEST_ASSERT_EQUAL_UINT16(ALL_LED_OFF, leds_virtuales); 
 }

Por esta razón la cobertura en el informe no es del 100%.

evolentini commented 6 months ago

Tampoco se cumple que todo el codigo de producción se haya escrito para resolver una prueba que falla, no hay una prueba que verifique el estado de un led valido pero apagado

ceovejero commented 6 months ago

Estimado Esteban. Muchas gracias por las devoluciones. Tomo nota de cada una de ellas. Es obvio que deberé dedicarle "su tiempo" para que estas herramientas y técnicas fluyan. Me gustó mucho el contenido y las prácticas de la materia.. Saludos!!

El jue, 18 abr 2024 a las 10:27, Esteban Volentini (< @.***>) escribió:

Tampoco se cumple que todo el codigo de producción se haya escrito para resolver una prueba que falla, no hay una prueba que verifique el estado de un led valido pero apagado image.png (view on web) https://github.com/ceovejero/cese_tsse_tp4/assets/8672347/76f0801a-09fa-46ff-b85e-f3cf6f482afb

— Reply to this email directly, view it on GitHub https://github.com/ceovejero/cese_tsse_tp4/issues/3#issuecomment-2063868766, or unsubscribe https://github.com/notifications/unsubscribe-auth/BAOMY5RCKWDW532IRM4SCPTY57C5ZAVCNFSM6AAAAABGNIOT36VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANRTHA3DQNZWGY . You are receiving this because you are subscribed to this thread.Message ID: @.***>

-- ceo

evolentini commented 6 months ago

De todas formas el TP esta muy bien, pero la idea es revisarlo “a cara de perro” para que tambien tengan la experiencia de un review de codigo.

Saludos

El 18 abr 2024, a las 12:58, Cesar Ovejero @.***> escribió:

Estimado Esteban. Muchas gracias por las devoluciones. Tomo nota de cada una de ellas. Es obvio que deberé dedicarle "su tiempo" para que estas herramientas y técnicas fluyan. Me gustó mucho el contenido y las prácticas de la materia.. Saludos!!

El jue, 18 abr 2024 a las 10:27, Esteban Volentini (< @.***>) escribió:

Tampoco se cumple que todo el codigo de producción se haya escrito para resolver una prueba que falla, no hay una prueba que verifique el estado de un led valido pero apagado image.png (view on web) https://github.com/ceovejero/cese_tsse_tp4/assets/8672347/76f0801a-09fa-46ff-b85e-f3cf6f482afb

— Reply to this email directly, view it on GitHub https://github.com/ceovejero/cese_tsse_tp4/issues/3#issuecomment-2063868766, or unsubscribe https://github.com/notifications/unsubscribe-auth/BAOMY5RCKWDW532IRM4SCPTY57C5ZAVCNFSM6AAAAABGNIOT36VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANRTHA3DQNZWGY . You are receiving this because you are subscribed to this thread.Message ID: @.***>

-- ceo — Reply to this email directly, view it on GitHub https://github.com/ceovejero/cese_tsse_tp4/issues/3#issuecomment-2064350913, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACCFIW2FJQY3BF3666K3ENDY57UQ3AVCNFSM6AAAAABGNIOT36VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANRUGM2TAOJRGM. You are receiving this because you authored the thread.

ceovejero / cese_tsse_tp4

No se aplicó correctamente la metodología #3