Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ajuste en Test de M7T26 para Probar Caso de Undefined Correctamente #71

Open
Thayrov opened this issue Feb 14, 2024 · 0 comments · May be fixed by #75
Open

Ajuste en Test de M7T26 para Probar Caso de Undefined Correctamente #71

Thayrov opened this issue Feb 14, 2024 · 0 comments · May be fixed by #75

Comments

@Thayrov
Copy link
Contributor

Thayrov commented Feb 14, 2024

Descripción del problema

El test para verificar si se retorna undefined cuando no se encuentra un múltiplo en M7T26.test.js incluye un múltiplo que podría hacer que la función devuelva un resultado correcto, cuando se espera probar el comportamiento de retorno de undefined.

Archivo y cambio afectado

Archivo: M7T26.test.js
Cambio: Se modifica la secuencia para no incluir un múltiplo que haría que la función devuelva un resultado incorrectamente correcto.
Antes: const secuencia = [1, 2, 3, 4, 5, 6, 7, 8, 9];
Después: const secuencia = [1, 2, 3, 4, 5, 6, 8, 9];

Acción requerida

Actualizar el test en el archivo M7T26.test.js para asegurar que se prueba correctamente el caso en el que debe retornarse undefined.

Thayrov added a commit to Thayrov/fe-ct-prepcourse-fs-updated that referenced this issue Feb 14, 2024
Este commit ajusta la secuencia de entrada en el test de  para asegurar que el caso de prueba 'should return undefined if no multiple is found' funcione correctamente. Al remover el múltiplo que coincidía inesperadamente, se garantiza que la función se pruebe bajo la condición correcta de no encontrar un múltiplo de N en la secuencia. Relacionado con la issue soyHenry#71.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant