nodejs/undici

Improve tests with more checks for unexpected `'disconnect'`

Open

#251 aberto em 11 de jul. de 2020

Ver no GitHub
 (2 comments) (0 reactions) (0 assignees)JavaScript (5.319 stars) (444 forks)batch import
enhancementgood first issue

Description

Some of our tests might emit disconnect, reconnect and then pass even though a disconnect was not expected.

Add more assertions (t.fail()) to catch this if it happens.

Guia do colaborador

Pilha de tecnologia
javascriptnodejs
Domain
backendtesting
Tipo Issue
test
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
under 1 hour
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
stale
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
clear
Prerequisites
basic Node.js testing knowledge
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
85
Direção de pesquisa
Search for test files in the 'test' directory of the undici repository. Identify tests that handle 'disconnect' events. Add a 't.fail()' call inside an event listener for 'disconnect' where the event is unexpected. For example, in tests that simulate reconnection, ensure that a disconnect event causes the test to fail unless it is expected. Look at existing patterns in test files using tap or tape.