caolan / caolan/nodeunit

[Bug] reporter minimal doesn't throw "undone tests"

Offen
#163 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
JavaScript
Sterne
1.9k
Forks
359
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

Reporter minimal doesn't throw "undone tests" and so exit with 0 instead of 1.

``` shell
/test (master⚡) ./node_modules/nodeunit/bin/nodeunit --reporter minimal ./tests/my.test.js
my.test.js: ..% /test (master⚡) echo $?
0
/test (master⚡) ./node_modules/nodeunit/bin/nodeunit ./tests/my.test.js

my.test.js
✔ constructor
✔ _getKeywords

FAILURES: Undone tests (or their setups/teardowns):
- onKeywords

To fix this, make sure all tests call test.done()
/test (master⚡) echo $?
1
/test (master⚡)

```

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Rechercherichtung

Reproduziere das Verhalten mit node_modules/nodeunit/bin/nodeunit und vergleiche den minimal reporter mit dem default reporter für tests/my.test.js. Untersuche den Einstiegspunkt des minimal reporter und die Behandlung des nicht abgeschlossenen onKeywords-Tests; abgeschlossen bedeutet, dass der minimal reporter mit Status 1 beendet wird, wenn test.done() fehlt.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

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

Neue Issues direkt in Ihr Postfach

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