caolan / caolan/nodeunit

Not all reporters support recursion.

未关闭
#347 7 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看
主要语言
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

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。