micro-analytics / micro-analytics/micro-analytics-cli
Connection leaks in adapter tests
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- JavaScript
- Sterne
- 732
- Forks
- 39
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
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.
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne mit den Adapter-Tests rund um call init should not throw und untersuche dann die Adapter-Hilfsfunktionen sowie den Verbindungslebenszyklus von beforeEach/afterEach. Stelle die Redis-Adapter-Tests nach und verfolge, welche Verbindung ersetzt wird; abgeschlossen ist die Aufgabe, wenn jede von den Tests erstellte Verbindung geschlossen wird und die Redis-Tests sauber beendet werden.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- javascript, node.js, redis
- Bereich
- backend, databases, testing-qa
- Issue-Typ
- Bug
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100