最近测试的同学反馈一个问题，他在测试完项目以后查看覆盖率的结果，发现有个逻辑明明已经覆盖了，但是覆盖率的报告里面显示的却是没有覆盖。

问题分析

以下就是源码的内容，并且现场演示了接口返回库存不足的逻辑。

在这里插入图片描述

看到这个问题就比较捉急了。因为关于jacoco目前比较难的就是这样子的现象，很大程度是要去了解jacoco的逻辑才行。

因为覆盖率的情况基本都是在jacoco.exec这个文件中, 所以我们第一步要去了解的是exec文件是否确定已经有覆盖了这个代码逻辑。 这里我们可以借助jacococli.jar来辅助分析。

java -jar jacococli.jar execinfo [<execfiles> ...]

所以我们直接拉取了被测主服务的exec文件进行查看。得到了这样子的结果。

在这里插入图片描述

这个信息至少能够得到一个信息就是这个代码被覆盖了至少有1/3。就已经跟实际覆盖率的报告不符合了。

我们再看下对应的TradeOrderServiceImpl 类中probes 探针覆盖的情况。

在这里插入图片描述

这个时候测试同学告诉了一个很有意义的信息。这次的测试计划是有两个服务都用到了同一个仓库，测试同学问了对应的开发，开发给的意见是有没有可能是同一份代码在两个仓库里面都用到了，所以导致冲突一直没有被覆盖到。

所以又拉取了另外一个服务的exec文件进行查看。

在这里插入图片描述

确实在另外一份exec的数据里面也出现了这个类，但是它的覆盖率情况就比较糟糕了。所以从这里就大概能够知道覆盖率的报告用的是这份的数据了。

原因

那为什么会出现这样子的情况呢? 理论上这两份数据应该会被合并成最后一个才对吧？带着这个疑问我们去看了下逻辑。

public void put(final ExecutionData data) throws IllegalStateException {
        final Long id = Long.valueOf(data.getId());
        final ExecutionData entry = entries.get(id);
        if (entry == null) {
            entries.put(id, data);
            names.add(data.getName());
        } else {
            entry.merge(data);
        }
    }

从这里看出来，其实是否会合并覆盖率数据是根据data中的id来进行判断的，如果是相同的情况下，数据才会被合并。这个id其实是classId.难道说这两个classId是不相同的吗？
我们对比前两个截图其实就会发现了，其实这两个classId是不一样的。所以这就是导致了为什么覆盖率数据不正确的原因了，不过也不能说不正确，只能说这个可能更多是一个概率性的问题，如果说最后分析的是主服务中的exec文件，其实覆盖的数据就正确了。

所以针对上述这种情况我们只能够暂时忽略掉classId这种区别，只是根据className来做判断处理。

public void put(final ExecutionData data) throws IllegalStateException {
        final Long id = Long.valueOf(data.getId());
        final String className= data.getName();
        // 这里可能存在有相同的类名的 最好是做合并的处理
        ExecutionData entry = entries.get(id);
        if (entry == null) {
            entry = classEntries.get(className);
            // 如果probe不相同 说明是两个同名的class但是内容不相同
            if (entry != null && entry.getProbes().length != data.getProbes().length) {
                entry = null;
            }
        }
//      final ExecutionData classEntry = classEntries.get(className);
        if (entry == null) {
            entries.put(id, data);
            names.add(data.getName());
            classEntries.put(className, data);
        } else {
            entry.merge(data);
            classEntries.put(className, entry);
        }


//      if (classEntry == null) {
//          classEntries.put(className, data);
//          names.add(data.getName());
//      }else {
//          classEntries.put(className, )
////            classEntry.merge(data);
//      }
    }

所以就是通过上述的方式就可以解决了，不过这个肯定有相应的弊端，因为本身jacoco的设计是通过classId来进行区分不同的类的，而我们这里是通过className就行数据合并 还是有一定问题的。