micro-analytics / micro-analytics/micro-analytics-cli
Connection leaks in adapter tests
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- JavaScript
- Estrellas
- 732
- Forks
- 39
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Descripción
The adapter tests does not close connection correctly resulting in tests for the redis adapter never quitting.
The problem is the at the moment is call init should not throw. Overwrites the connections with new one and then when the tests closes connection in afterEach the connection created in beforeEach is not closed.
I think it would be a good thing to require adapter.close() and handle the closing of database connections in the adapter utils.
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Empieza por las pruebas del adaptador relacionadas con call init should not throw y después inspecciona las utilidades del adaptador y el ciclo de vida de la conexión de beforeEach/afterEach. Reproduce las pruebas del adaptador de Redis y sigue qué conexión se reemplaza; el trabajo estará terminado cuando se cierre cada conexión creada por las pruebas y las pruebas de Redis finalicen correctamente.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- javascript, node.js, redis
- Área
- backend, databases, testing-qa
- Tipo de issue
- Error
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100