caolan / caolan/nodeunit

test.throws incorrectly implemented

Offen
#156 2 Kommentare 1 Reaktion 0 zugewiesene Personen Auf GitHub ansehen
API Breaking Change
Vorherrschende Sprache
JavaScript
Sterne
1.9k
Forks
359
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

According to http://nodejs.org/docs/v0.4.2/api/assert.html#assert.throws
... assert.throws can be passed three arguments:
`assert.throws(block, [error], [message])`

The `error` parameter can be constructor, regexp or validation function. Nodeunit assumes that `error` is a constructor and simply uses instanceof to validate. This behavior is incorrect. Regexp or validation function should be supported.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Rechercherichtung

Lies die vorhandene assert.throws-Implementierung und ihre Tests und vergleiche anschließend das Verhalten mit der verlinkten Node.js assert.throws-Dokumentation. Erledigt bedeutet, Konstruktor-, reguläre-Ausdruck- und Validierungsfunktions-Fehlerargumente zu unterstützen, statt nur instanceof-Prüfungen durchzuführen, mit Abdeckung für jede Form.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
javascript, node.js
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.