test.throws incorrectly implemented
オープン
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