nodejs/undici

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

Open

Aperta il 11 lug 2020

Vedi su GitHub
 (2 commenti) (0 reazioni) (0 assegnatari)JavaScript (5319 star) (444 fork)batch import
enhancementgood first issue

Descrizione

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.

Guida contributor

Tech stack
javascriptnodejs
Dominio
backendtesting
Tipo issue
test
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
under 1 hour
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
stale
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
clear
Prerequisiti
basic Node.js testing knowledge
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
85
Direzione di ricerca
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.