roboart / tsse_tp3

Trabajo práctico N°3 - Testing de software usando Ceedling

TSSE_TP3

Trabajo práctico N°3 - Testing de software usando Ceedling

tsse_tp3's Issues

No se aplicó correctamente la metodología

Una de las reglas de TDD es "escribir el código mínimo y necesario para hacer pasar una prueba unitaria que falla". Para pasar esta prueba

TSSE_TP3/leds/test/test_leds.c

Lines 86 to 91 in 2753bf1

	//Revisar parámetros fuera de los limites.
	void test_InvalidUpperLimitTurnOnLed(void){
	RegistrarMensaje_Expect(ALERTA, "LedTurnOn",0,"Numero de led debe ser entre 1 y 16");
	RegistrarMensaje_IgnoreArg_linea();
	LedTurnOn(-17);
	}

el código mínimo sería

    if(led < LOWER_LIMIT){

y no el que esta en el archivo 'leds.c'

TSSE_TP3/leds/src/leds.c

Line 23 in 2753bf1

if(led > UPPER_LIMIT || led < LOWER_LIMIT){

Todas las funciones de la biblioteca deben tener un comportamiento uniforme

Si la funciones LedsOn() y LedsOff() no reciben la dirección del puerto como parámetro las dos nuevas funciones LedAllTurnOn () y LedAllTurnOff() tampoco deberían recibirla. Además si la función LedsOn() controla el límite superior del parámetro, entonces también debería controlar el límite inferior (que es 1 y no 0) y además todas las otras funciones de la biblioteca deberían hacer lo mismo, en lo posible sin repetir código y centralizando el control en una función (o macro) interna.

Se debe refactorizar el código de producción para eliminar repeticiones

En esta linea se debería llamar a la función LedTurnOffAll() en lugar de repetir la asignación de valores a la variable.

TSSE_TP3/leds/src/leds.c

Line 19 in 2753bf1

*puerto = LEDS_ALL_OFF;

Pienso que estas lineas debería eliminarse

Me parece que estas lineas de código no son necesarias para la prueba y deberían ser eliminadas.

TSSE_TP3/leds/test/test_leds.c

Lines 48 to 50 in 2753bf1

	LedTurnOn(11);
	LedTurnOn(1);
	LedTurnOff(3);

TSSE_TP3/leds/test/test_leds.c

Lines 57 to 59 in 2753bf1

	LedTurnOn(11);
	LedTurnOn(2);
	LedTurnOff(3);

TSSE_TP3/leds/test/test_leds.c

Lines 68 to 70 in 2753bf1

	LedTurnOn(2);
	LedTurnOn(3);
	LedTurnOff(2);

TSSE_TP3/leds/test/test_leds.c

Line 78 in 2753bf1

LedTurnOff(2);

TSSE_TP3/leds/test/test_leds.c

Line 80 in 2753bf1

LedTurnOn(3);