nodeSolidServer / nodeSolidServer/node-solid-server

Assertions in async callbacks will not mark tests as failed

Offen
#1,203 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Vorherrschende Sprache
JavaScript
Sterne
1.8k
Forks
308
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

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.

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne damit, die asynchronen Testfälle in test/integration/ldp-test.js in den Zeilen 178, 199, 214 und 326 zu lesen. Prüfe, wie Assertions innerhalb ihrer Callbacks behandelt werden, und sieh dir anschließend den zugehörigen NSS-Testcode auf dasselbe Muster hin an. Als abgeschlossen gilt die Aufgabe, wenn die betroffenen Tests Assertion-Fehler zuverlässig melden, statt sie stillschweigend zu bestehen.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
javascript
Bereich
testing-qa
Issue-Typ
Bug
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.