Not all reporters support recursion.
- Dominant language
- JavaScript
- Stars
- 1.9k
- Forks
- 359
- PR merge metrics
- No merged PRs in 30d
Description
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.
Contributor guide
No contributing guide indexed for this repository
Research direction
Inspect the calls to nodeunit.runFiles() in the eclipse, html, junit, minimal, nested, skip-passed, and verbose reporters, comparing them with lcov and machineout. First determine how existing reporter tests exercise recursive runs, then add coverage showing that the recursive option is honored across the affected reporters.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript, node.js
- Domain
- testing
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100