如何在 if 条件下增加 for-in 循环中 jest 的分支覆盖率？

Question

我有以下代码，它将几个字段从一个对象映射到另一个对象并复制值：

//Function params
var source= {"f1": "v1", "f2": "v2", "f3":"v3"};
var fieldsMapping = {"f1": "c1", "f2":"c2"};

//function definition starts
var copiedObj = {};    
for (var name in source) {
  if (source.hasOwnProperty(name)) { //Line X
    if(fieldsMapping[name]){
        copiedObj[fieldsMapping[name]] = source[name];
    }
  }
}

console.log(copiedObj); //outputs {c1: "v1", c2: "v2"}

我在 jest 中为该函数编写了测试用例，行覆盖率为 100%，但分支覆盖显示 Line X 未覆盖。根据 Standards similar in TSLint，for-in 循环后应紧跟 if condition。

任何人都可以建议如何有一个测试用例来增加branch coverage吗？

Answer 1

所以你看到这意味着这个 if 对你所有的测试用例都执行了。这就是线路覆盖显示 100% 的原因。但是要获得 100% 的分支覆盖率，对于某些特定的测试用例，您需要使 if 不是运行。

hasOwnProperty returns false 用于从原型继承的属性。所以我们将使用它来使该行不被执行：

const base = { a: 1};
const source = Object.create(base, {b: 2, c: 3} );
const fieldsMapping = {a: 'aa', b: 'bb', c: 'cc'};

expect(yourFunction(source, fieldsMapping)).toEqual({ bb: 2, cc: 3 });

PS 尽量不要沉迷于 100%-coverage-goal。

Answer 2

额外的 if 是为了代码的健壮性。这种稳健性检查无法以合理的方式进行测试。这也会发生在代码的其他各个部分 - 它经常发生在 switch 语句中，其中明确涵盖了所有可能的情况，并且添加了额外的默认情况只是为了抛出异常或以其他方式处理此 'impossible' 情况。或者，考虑添加到代码中的断言语句：由于断言永远不会失败，严格来说，您将永远无法覆盖隐藏在断言语句内的 else 分支——您如何测试断言内的表达式是否适合实际检测到你想要的问题？

删除此类健壮性代码和断言不是一个好主意，因为它们还可以帮助您检测未来更改的不良副作用。最后，您必须做出明智的决定（通过详细查看覆盖率报告，而不仅仅是总体百分比）您的代码中哪些 statements/branches 等确实需要覆盖，哪些不需要。

最后一点，请注意，高代码覆盖率并不一定表示您的测试套件具有高质量。如果您的测试套件能够检测到代码中可能存在的错误，那么它就是高质量的。您可以拥有一个 100% 覆盖率的测试套件，它不会检测到任何潜在的错误。

如何在 if 条件下增加 for-in 循环中 jest 的分支覆盖率？

How to increase branch coverage in jest in for-in loop with if condition?

javascript

typescript

unit-testing

jestjs

tslint