nodejs/undici

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

Open

#251 aperta il 11 lug 2020

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

Metriche repository

Star
 (5319 star)
Metriche merge PR
 (Merge medio 2g 10h) (82 PR mergiate in 30 g)

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

Direzione di ricerca
Esamina i file di test esistenti nel repository di undici, in particolare quelli che potrebbero emettere eventi di disconnessione imprevisti. Comprendi i pattern di test (probabilmente usando tap). Identifica le aree in cui potrebbe verificarsi una disconnessione inaspettata e aggiungi asserzioni `t.fail()` per catturare tali casi.
Tech stack
nodejsjavascript
Dominio
testing
Tipo issue
Test
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Node.jsGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80