nodeSolidServer / nodeSolidServer/node-solid-server

Assertions in async callbacks will not mark tests as failed

Aperta

#1,203 aperta il 25/5/2019 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Lingua principale: JavaScript
Stelle: 1.8k
Fork: 308
Metriche di merge delle PR: Nessuna PR unita negli ultimi 30g

Descrizione

Looking through the tests for NSS I see some that don't appear to actually achieve anything (unless I'm missing something).

Some of them have been marked to skip, which is fair enough.

For example:

https://github.com/solid/node-solid-server/blob/master/test/integration/ldp-test.js#L178
https://github.com/solid/node-solid-server/blob/master/test/integration/ldp-test.js#L199
https://github.com/solid/node-solid-server/blob/master/test/integration/ldp-test.js#L214
https://github.com/solid/node-solid-server/blob/master/test/integration/ldp-test.js#L326 (This one isn't skipped)

If they're in tests like that, there could be a possibility that it's leaked into NSS itself as well.

Guida per i contributori

Apri la guida per i contributori

Come iniziare

Leggi tutta la issue e poi la guida ai contributi del progetto.
Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
Fai un fork del repository e lavora su un branch.
Apri una pull request che faccia riferimento al numero della issue.

Direzione di ricerca

Inizia leggendo i casi di test asincroni in test/integration/ldp-test.js alle righe 178, 199, 214 e 326. Controlla come vengono gestite le asserzioni all’interno dei relativi callback, quindi esamina il codice di test NSS correlato per lo stesso pattern. Il lavoro è completato quando i test interessati segnalano in modo affidabile gli errori di asserzione invece di superare il test senza segnalarli.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico: javascript
Ambito: testing-qa
Tipo di issue: Bug
Difficoltà: 3/5
Tempo stimato: 1-2 giorni
Stato di attività: Ferma
Chiarezza: Abbastanza chiara
Idoneità per principianti: 35/100