Test 2

[Steykwo]

Participantes:

Jeison Steven Guio Varon
Gerson David Cruz Rodríguez
María Fernanda Velez Benitez
Miguel Esteban Pinilla Leal

Descripción

Quitamos la duplicación del archivo Basic y el archivo Test, reorganizamos la estructura del código para que todas las funcionas estén dentro de una clase, y usamos el Framework Pytest para comprobar las funciones. NOTA IMPORTANTE: al momento de intentar probar el código en test provoca un error que no pudimos solucionar sin ayuda llamado: "ModuleNotFoundError: No module named 'ECSolver'", así que por favor quisiéramos un poco de ayuda con eso y ya solo faltarían unos pocos detalles para terminarlo por completo, gracias.

Tipo de cambio

• Nueva funcionalidad
• [ x] Corrección de error
• [ x] Refactorización
• [ x] Actualización de documentación

Checklist

• [ x] Mi código sigue el estilo del proyecto
• [ x] He realizado una auto-revisión
• [ x] He comentado el código en áreas complejas
• Mis cambios no generan nuevas advertencias

[Michael-Jimenez-C]

Ok, en general veo una mejoría respecto al ultimo PR, respecto a ese error se soluciona indicándole al interprete de Python que src y test son módulos agregando un fichero __init__.py en dichas carpetas, puede estar vacío, luego ustedes pueden hacer el importe directamente desde src ya que no va a reconocer ECSolver.

Los tests se definen a nivel de modulo, no se pueden definir de forma anidada de esa manera, pero en general cuando se dejan sin anidar y se agrega la variable calc, solo tiene un error en CONTARA pero se debe a que pusieron 4 en lugar de 3 en el assert.

Por otro lado, toca revisar si es importante tener una clase conteniendo esos métodos o es mejor dejarlo como funciones a nivel de modulo, pero en general esta bien.