micro-analytics/micro-analytics-cli

Connection leaks in adapter tests

Open

#75 aberto em 29 de jun. de 2017

Ver no GitHub
 (0 comments) (1 reaction) (0 assignees)JavaScript (52 forks)batch import
adapter-testshelp wanted

Métricas do repositório

Stars
 (738 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

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.

Guia do colaborador

Direção de pesquisa
Investigue a configuração e limpeza do teste. Garanta que cada beforeEach conecte um novo cliente e o afterEach o feche. O problema menciona que 'call init should not throw' sobrescreve a conexão. Verifique se todas as conexões são fechadas corretamente para evitar vazamentos.
Pilha de tecnologia
javascriptnodejs
Domain
backendapi
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
GitNode.js
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65