caolan / caolan/nodeunit

test.throws incorrectly implemented

オープン
#156 コメント 2 件 リアクション 1 件 担当者 0 名 GitHub で見る
API Breaking Change
主要言語
JavaScript
スター
1.9k
フォーク
359
PR マージ指標
30日以内にマージされた PR はありません

説明

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.

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

調査の方向性

既存の assert.throws の実装とそのテストを読み、リンク先の Node.js の assert.throws ドキュメントと動作を比較してください。完了条件は、instanceof チェックだけでなく、コンストラクター、正規表現、検証関数のエラー引数をサポートし、それぞれの形式をカバーすることです。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
javascript, node.js
領域
testing-qa
issue の種類
バグ
難易度
3/5
見積もり時間
1〜2日
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
35/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。