caolan / caolan/nodeunit

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

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

説明

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⚡)

```

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

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

調査の方向性

node_modules/nodeunit/bin/nodeunit を使って動作を再現し、tests/my.test.js で minimal reporter と default reporter を比較します。minimal reporter のエントリーポイントと、未完了の onKeywords テストの処理を調べます。完了とは、test.done() がない場合に minimal reporter がステータス 1 で終了することを意味します。

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

評価

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

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

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