caolan / caolan/nodeunit

BUG: Asserting after `test.done()` is allowed

Offen
#172 1 Kommentar 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

``` javascript
exports.test1 = function(test) {
test.expect(1);
test.ok(true, 'is true');
test.done();

// this should throw an error, however it passes
test.ok(true, 'is true2');
};
```

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Rechercherichtung

Start by reproducing the example using test.ok after test.done and trace how those entry points handle completion and later assertions. The work is done when an assertion after test.done throws the expected error while assertions before completion retain their current behavior, with the regression covered by the project's tests.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

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

Neue Issues direkt in Ihr Postfach

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