micro-analytics/micro-analytics-cli

Connection leaks in adapter tests

Open

#75 aperta il 29 giu 2017

Vedi su GitHub
 (0 commenti) (1 reazione) (0 assegnatari)JavaScript (52 fork)batch import
adapter-testshelp wanted

Metriche repository

Star
 (738 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Investiga la configurazione e la pulizia del test. Assicurati che ogni beforeEach connetta un nuovo client e che afterEach lo chiuda. Il problema menziona che 'call init should not throw' sovrascrive la connessione. Verifica che tutte le connessioni siano correttamente chiuse per prevenire perdite.
Tech stack
javascriptnodejs
Dominio
backendapi
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitNode.js
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65