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

Segregar pruebas unitarias de las pruebas de integración #56

Open
kilianpg opened this issue May 22, 2018 · 3 comments
Open

Segregar pruebas unitarias de las pruebas de integración #56

kilianpg opened this issue May 22, 2018 · 3 comments
Assignees
Labels
bug Something isn't working help wanted Extra attention is needed Integración Test Test del código, selenium, mockup, etc...

Comments

@kilianpg
Copy link
Contributor

kilianpg commented May 22, 2018

Las pruebas unitarias ejecutadas en la fase mvn test (Maven Surefire Plugin) están pensadas para ejecutarse de forma autónoma sin necesidad de depender de servicios externos como PostgreSQL, Apache Zookeeper, Apache Kafka u el resto de módulos. Por ese motivo deben hacer uso de los mockups o fixtures necesarios como por ejemplo la utilización de bases de datos en memoria sin necesidad de lanzar dichos servicios.

A su vez las pruebas de integración (entre las que se incluyen las de aceptación y rendimiento), ejecutadas en la fase mvn verify (Maven Failsafe Plugin) han sido configuradas para lanzar automáticamente la ejecución de los contenedores (mvn pre-integration-test) y una vez ejecutas exitosamente o no las pruebas, detener también automáticamente los contenedores implicados (mvn post-integration-test).

Dado que la fase de construcción del contenedor (mvn docker:build) precisa que se ejecute previamente la fase de empaquetado (mvn package) y esta a su vez depende de que se ejecuten correctamente las pruebas unitarias (mvn test), dichas pruebas no pueden depender de un contenedor que aun no se ha construido (dependencia cruzada).

Hasta que se solucione este problema, se han desactivado temporalmente la ejecución por defecto de todas las pruebas.

Referencias:

Ver también: Issue: #20, Arquisoft/Inci_e3b_modules#5

@kilianpg kilianpg added bug Something isn't working help wanted Extra attention is needed Test Test del código, selenium, mockup, etc... Integración labels May 22, 2018
@UO244730 UO244730 self-assigned this Jun 8, 2018
@UO244730
Copy link
Contributor

UO244730 commented Jun 8, 2018

He añadido la base de datos en memoria y los servidores locales de Kafka Y Zookeeper, funciona perfecto. Queda implementarlo en todos los tests, estoy pensando como hacerlo repetir el mínimo código posible.

@UO244730
Copy link
Contributor

¿Sería necesario añadir mockups a los tests de aceptación e integración?

@kilianpg
Copy link
Contributor Author

No, por definición no deberian.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working help wanted Extra attention is needed Integración Test Test del código, selenium, mockup, etc...
Projects
None yet
Development

No branches or pull requests

2 participants