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,然后添加覆盖率以显示 recursive option 在受影响的 reporter 中得到遵守。
由索引模型根据 Issue 内容生成。
评估
- 技术栈
- javascript, node.js
- 领域
- testing
- Issue 类型
- 缺陷
- 难度
- 3/5
- 预计耗时
- 1-2 天
- 活跃度
- 停滞
- 描述清晰度
- 基本清楚
- 新手友好度
- 35/100