Not all reporters support recursion.
- 主要言語
- JavaScript
- スター
- 1.9k
- フォーク
- 359
- PR マージ指標
- 30日以内にマージされた PR はありません
説明
Just started using nodeunit. It seems not all the reporters support the recursive option.
It seems `recursive: options.recursive,` is missing from the call to `nodeunit.runFiles()` in at least eclipse, html, junit, minimal, nested, skip-passed and verbose.
I can see where the missing option is missing from lcov and machineout, but in my limited testing, I couldn't see a difference ... or ANY output!
I have amended my installed nodeunit and get the recursion as expected.
I can create a patch (https://github.com/caolan/nodeunit/pull/348), but I'm not sure about proof/testing. Ideally, I'd like to know how to add tests to prove that the recursion is operational, but if the fix is correct and a super obvious oversight, then maybe it wouldn't be necessary.
コントリビューションガイド
このリポジトリのコントリビューションガイドは索引されていません
調査の方向性
eclipse、html、junit、minimal、nested、skip-passed、verbose reporter における nodeunit.runFiles() の呼び出しを調査し、lcov および machineout と比較します。まず、既存の reporter テストが recursive runs をどのように実行しているかを確認し、次に、対象となる reporter 全体で recursive option が尊重されることを示すカバレッジを追加します。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- javascript, node.js
- 領域
- testing
- issue の種類
- バグ
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 活発さ
- 停滞
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 35/100