Coveralls标记if分支未覆盖但删除该分支致测试失败问题

我来帮你拆解这个看似矛盾的问题：你的if分支确实在测试的关键路径上生效了（否则删除它测试会直接报错），但Coveralls标记的“未覆盖”其实是因为存在未被执行的回调实例，这些实例里的分支从未被运行过。

核心原因分析 #

先看你的代码结构：

fns.forEach((f, idxF) => f(...cbs.map((cb, idxCb) => fVal => { 
  if(!fCache[idxF]) fCache[idxF] = {}; 
  fCache[idxF][idxCb] = fVal 
} )));

这里的cbs.map会生成一组回调函数（数量等于cbs数组的长度），然后把这些回调作为参数传给f。但你的测试用例里，传入的f是：

cb => cb(x => x*2)

这个函数只接受并调用第一个传入的回调参数，剩下的回调根本不会被执行。那些没被触发的回调里的if(!fCache[idxF])分支自然不会被执行，而Coveralls会统计所有代码分支的执行情况，所以会把这个分支标记为“未覆盖”。

而当你删除if语句时测试失败，是因为那个被实际调用的回调里尝试给fCache[idxF][idxCb]赋值，此时fCache[idxF]还未初始化，所以抛出TypeError: Cannot set property '0' of undefined——但其他未被调用的回调不会触发这个错误，因为它们根本没运行。

解决思路 #

针对这个问题，你有两个可行的方向：

1. 覆盖所有回调分支

如果业务逻辑要求cbs的所有回调都需要被触发，那调整测试用例，使用一个会调用所有传入回调的f函数，或者把cbs的长度设为1，确保cbs.map只生成一个回调，这样所有分支都会被执行到。

2. 调整代码结构，提前初始化

如果某些回调确实不需要被调用（符合业务逻辑），可以把fCache[idxF]的初始化逻辑移到回调外面，放在fns.forEach的循环体里：

fns.forEach((f, idxF) => {
  // 提前初始化，避免在回调里处理分支
  if(!fCache[idxF]) fCache[idxF] = {};
  f(...cbs.map((cb, idxCb) => fVal => { 
    fCache[idxF][idxCb] = fVal 
  }));
});

这样不管回调是否被执行，初始化逻辑都会在forEach循环中运行，覆盖率工具就能统计到这个分支的执行情况，同时也避免了回调内的分支判断。

内容的提问来源于stack exchange，提问作者Dmitri Zaitsev