micro-analytics/micro-analytics-cli

Connection leaks in adapter tests

Open

#75 geöffnet am 29. Juni 2017

Auf GitHub ansehen
 (0 Kommentare) (1 Reaktion) (0 zugewiesene Personen)JavaScript (52 Forks)batch import
adapter-testshelp wanted

Repository-Metriken

Stars
 (738 Stars)
PR-Merge-Metriken
 (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.

Contributor Guide

Research-Richtung
Untersuche das Test Setup und den Teardown. Stelle sicher, dass jeder beforeEach eine neue Client Verbindung herstellt und afterEach sie schließt. Das Problem erwähnt, dass 'call init should not throw' die Verbindung überschreibt. Überprüfe, ob alle Verbindungen ordnungsgemäß geschlossen werden, um Lecks zu vermeiden.
Tech Stack
javascriptnodejs
Domain
backendapi
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitNode.js
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65