micro-analytics / micro-analytics/micro-analytics-cli
Connection leaks in adapter tests
Personne n'a encore pris cette issue.
- Langage dominant
- JavaScript
- Étoiles
- 732
- Forks
- 39
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Description
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.
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez par les tests de l’adaptateur autour de call init should not throw, puis examinez les utilitaires de l’adaptateur et le cycle de vie de la connexion de beforeEach/afterEach. Reproduisez les tests de l’adaptateur Redis et déterminez quelle connexion est remplacée ; le travail est terminé lorsque chaque connexion créée par les tests est fermée et que les tests Redis se terminent proprement.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- javascript, node.js, redis
- Domaine
- backend, databases, testing-qa
- Type d'issue
- Bug
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 35/100