jdalvaradocol / cese-14co2021

Repositorio entrega de actividades del CESE 14Co2021

CESE-14Co2021

Repositorio entrega de actividades del CESE 14Co2021

cese-14co2021's Issues

La consulta del estado de un led debería tener dos casos de prueba

Para asegurarse de que una función que retorna el estado de un led funciona correctamente deberían plantearse dos casos de prueba: consultar un led encendido y consultar un led apagado.

CESE-14Co2021/Testing de software embebido/TP3_vs_A/test/test_leds.c

Lines 69 to 76 in a0eef5b

	/* Prueba para que se puedan consultar el estado de un LED. */
	void test_estado_leds (void)
	{
	int estado, led = 2;
	LedsTurnOn(led);
	estado = LedsState(led);
	TEST_ASSERT_EQUAL_HEX16(estado, ledsvirtuales);
	}

Se debe reformular esta prueba

Apagar leds apagados no es una prueba valida. Por esa razón la metodología propuesta exige que primero la prueba falle, cosa que claramente no sucedió en este caso.

CESE-14Co2021/Testing de software embebido/TP3_vs_A/test/test_leds.c

Lines 63 to 68 in a0eef5b

	/* Prueba para que se puedan apagar todos los LEDs de una vez. */
	void test_apagar_todos_los_leds (void)
	{
	LedsTurnOffall();
	TEST_ASSERT_EQUAL_HEX16(0x0000, ledsvirtuales);
	}

Una prueba que no prueba nada es peor que la falta de pruebas...

La interface propuesta para la función no es la mas adecuada

Si nos ponemos en el punto de vista de un usuario que quiere saber si un led esta encendido lo más razonable es que la función retorne un valor lógico. Así el propio de la siguiente función (o algo parecido)

bool LedsIsLedOn(int led)

es mucho más fácil de entender y de utilizar que el propuesto. Definir nombre y comportamientos naturales para las funciones es esencial para el desarrollo de un software robusto que se pueda mantener en el tiempo.

CESE-14Co2021/Testing de software embebido/TP3_vs_A/test/test_leds.c

Lines 69 to 76 in a0eef5b

	/* Prueba para que se puedan consultar el estado de un LED. */
	void test_estado_leds (void)
	{
	int estado, led = 2;
	LedsTurnOn(led);
	estado = LedsState(led);
	TEST_ASSERT_EQUAL_HEX16(estado, ledsvirtuales);
	}