关于 gitlab 上 Java 项目测试覆盖率

gitlab 中是支持将项目的测试覆盖率解析的, 简单来说就是通过设置正则表达式从 CI 中的日志抓出覆盖率的相关信息(参见官方文档), 我虽然觉得这个方式挺 LOW 的, 但也没有什么其他办法. 记录一下 Java 项目的接入过程.

0x01 Java 测试覆盖率统计

Java 是有一些第三方工具进行测试覆盖率统计的, 原理也很简单:

  1. 使用 Java Instrumentation 技术将字节码改写, 添加一些标记代码, 以备在代码真正被执行的时候标记哪些逻辑/哪些分支被执行
  2. 执行 Unit Test 代码
  3. 收集标记代码产生的数据并统计覆盖率, 生成测试覆盖率报告

可选方案也有一些, 罗列几个我熟悉的:

  1. Clover, Atlassian 出品, 收费
  2. Cobertura, 开源, 最后更新是2015年(怀疑已经没人维护)
  3. Jacoco, 开源, 并且项目依旧活跃

选型原则也很简单, 一看是否有自己所用的 build 工具集成, 毕竟自己再去写一个 build 工具 plugin 也费力气; 二看结果是否满足需求.

由于 Cobertura 不怎么更新了, 我这个 Cobertura 老用户也切换到了 Jacoco.

0x02 接入 gitlab CI

接入也很容易, 由于使用 maven 作为 build 工具, 因此只需要在 pom.xml 中添加如下内容, 就可以在 mvn clean test 执行后自动获得覆盖率统计报告, 在 target/site/jacoco/index.html

<build>
    <plugins>
      <plugin>
         <groupId>org.jacoco</groupId>
         <artifactId>jacoco-maven-plugin</artifactId>
         <version>0.7.9</version>
         <executions>
            <execution>
               <id>pre-unit-test</id>
               <goals>
                  <goal>prepare-agent</goal>
               </goals>
            </execution>
            <execution>
               <id>post-unit-test</id>
               <phase>test</phase>
               <goals>
                  <goal>report</goal>
               </goals>
            </execution>
         </executions>
      </plugin>
    </plugins>
</build>

同样, 接入 gitlab CI 也是简单, 由于覆盖率统计结果在 html 报告中, 因此在项目的 .gitlab-ci.yml 中执行 mvn clean test && cat target/site/jacoco/index.html 即可将覆盖率信息打印到 LOG 中, 最终 gitlab 会通过 coverage 的正则解析到覆盖率. 正则也很简单:

Total.*?([0-9]{1,3})%
将上述正则填写到项目 Pipeline settings 中

0x02 采坑开始

用上诉方法接入了多个项目都没有问题, 遇到一个 Presto plugin 的项目就诡异的报错了, 现象就是:

  1. 没有 jacoco-maven-plugin 的情况下执行 mvn clean test 测试成功执行
  2. 添加了 jacoco-maven-plugin 就会报如下错误:
testValue(data.Test)  Time elapsed: 0.006 sec  <<< ERROR!
java.lang.NoClassDefFoundError: Could not initialize class com.facebook.presto.spi.block.BlockBuilderStatus
    at data.Test.testValue(Test.java:40)
    sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
    at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
    at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
    at java.lang.reflect.Method.invoke(Method.java:498)
    at org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50)
    at org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
    at org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47)
    at org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17)
    at org.junit.runners.ParentRunner.runLeaf(ParentRunner.java:325)
    at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:78)
    at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:57)
    at org.junit.runners.ParentRunner$3.run(ParentRunner.java:290)
    at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:71)
    at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:288)
    at org.junit.runners.ParentRunner.access$000(ParentRunner.java:58)
    at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:268)
    at org.junit.runners.ParentRunner.run(ParentRunner.java:363)
    at org.apache.maven.surefire.junit4.JUnit4Provider.execute(JUnit4Provider.java:252)
    at org.apache.maven.surefire.junit4.JUnit4Provider.executeTestSet(JUnit4Provider.java:141)
    at org.apache.maven.surefire.junit4.JUnit4Provider.invoke(JUnit4Provider.java:112)
    at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
    at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
    at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
    at java.lang.reflect.Method.invoke(Method.java:498)
    at org.apache.maven.surefire.util.ReflectionUtils.invokeMethodWithArray(ReflectionUtils.java:189)
    at org.apache.maven.surefire.booter.ProviderFactory$ProviderProxy.invoke(ProviderFactory.java:165)
    at org.apache.maven.surefire.booter.ProviderFactory.invokeProvider(ProviderFactory.java:85)
    at org.apache.maven.surefire.booter.ForkedBooter.runSuitesInProcess(ForkedBooter.java:115)
    at org.apache.maven.surefire.booter.ForkedBooter.main(ForkedBooter.java:75)

不过作为 Java 老司机, 看到 java.lang.NoClassDefFoundError: Could not initialize class com.facebook.presto.spi.block.BlockBuilderStatus 就知道是 com.facebook.presto.spi.block.BlockBuilderStatus 这个类的什么静态方法执行失败, 导致 com.facebook.presto.spi.block.BlockBuilderStatus 初始化不成功.

翻了一下日志果然发现如下错误:

Caused by: java.lang.IllegalArgumentException: Cannot determine size of boolean[] because it contains an array
    at com.facebook.presto.spi.block.BlockBuilderStatus.deepInstanceSize(BlockBuilderStatus.java:77)
    at com.facebook.presto.spi.block.BlockBuilderStatus.deepInstanceSize(BlockBuilderStatus.java:92)
    at com.facebook.presto.spi.block.BlockBuilderStatus.deepInstanceSize(BlockBuilderStatus.java:92)
    at com.facebook.presto.spi.block.BlockBuilderStatus.<clinit>(BlockBuilderStatus.java:26)
    ... 33 more

通过阅读源码发现 com.facebook.presto.spi.block.BlockBuilderStatus 根本就没有 boolean[] 类型的属性, 猜测是 jacoco 在改写字节码的时候添加了一些属性, 用于标记代码是否被执行. 通过 jacoco-maven-plugin 中添加 exclude 排除所有 preseto 相关代码, 问题解决.

<configuration>
    <excludes>
        <exclude>com/facebook/**/*</exclude>
    </excludes>
</configuration>

总结

gitlab 接入一下测试覆盖率还是很有必要的, gitlab 还支持在项目的 README.md 中添加覆盖率和 build status 的 badge, 从侧面敦促工程师尽量提升测试覆盖率. 反正我看到低于 80% 覆盖率的项目都是嗤之以鼻的.

覆盖率 badge

还有一个收获就是先了解原理, 在动手干活儿很重要, 遇到问题从原理角度定位问题比盲目的瞎尝试效率高得多.

-- EOF --

最后编辑于
©著作权归作者所有,转载或内容合作请联系作者
  • 序言:七十年代末,一起剥皮案震惊了整个滨河市,随后出现的几起案子,更是在滨河造成了极大的恐慌,老刑警刘岩,带你破解...
    沈念sama阅读 204,590评论 6 478
  • 序言:滨河连续发生了三起死亡事件,死亡现场离奇诡异,居然都是意外死亡,警方通过查阅死者的电脑和手机,发现死者居然都...
    沈念sama阅读 86,808评论 2 381
  • 文/潘晓璐 我一进店门,熙熙楼的掌柜王于贵愁眉苦脸地迎上来,“玉大人,你说我怎么就摊上这事。” “怎么了?”我有些...
    开封第一讲书人阅读 151,151评论 0 337
  • 文/不坏的土叔 我叫张陵,是天一观的道长。 经常有香客问我,道长,这世上最难降的妖魔是什么? 我笑而不...
    开封第一讲书人阅读 54,779评论 1 277
  • 正文 为了忘掉前任,我火速办了婚礼,结果婚礼上,老公的妹妹穿的比我还像新娘。我一直安慰自己,他们只是感情好,可当我...
    茶点故事阅读 63,773评论 5 367
  • 文/花漫 我一把揭开白布。 她就那样静静地躺着,像睡着了一般。 火红的嫁衣衬着肌肤如雪。 梳的纹丝不乱的头发上,一...
    开封第一讲书人阅读 48,656评论 1 281
  • 那天,我揣着相机与录音,去河边找鬼。 笑死,一个胖子当着我的面吹牛,可吹牛的内容都是我干的。 我是一名探鬼主播,决...
    沈念sama阅读 38,022评论 3 398
  • 文/苍兰香墨 我猛地睁开眼,长吁一口气:“原来是场噩梦啊……” “哼!你这毒妇竟也来了?” 一声冷哼从身侧响起,我...
    开封第一讲书人阅读 36,678评论 0 258
  • 序言:老挝万荣一对情侣失踪,失踪者是张志新(化名)和其女友刘颖,没想到半个月后,有当地人在树林里发现了一具尸体,经...
    沈念sama阅读 41,038评论 1 299
  • 正文 独居荒郊野岭守林人离奇死亡,尸身上长有42处带血的脓包…… 初始之章·张勋 以下内容为张勋视角 年9月15日...
    茶点故事阅读 35,659评论 2 321
  • 正文 我和宋清朗相恋三年,在试婚纱的时候发现自己被绿了。 大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
    茶点故事阅读 37,756评论 1 330
  • 序言:一个原本活蹦乱跳的男人离奇死亡,死状恐怖,灵堂内的尸体忽然破棺而出,到底是诈尸还是另有隐情,我是刑警宁泽,带...
    沈念sama阅读 33,411评论 4 321
  • 正文 年R本政府宣布,位于F岛的核电站,受9级特大地震影响,放射性物质发生泄漏。R本人自食恶果不足惜,却给世界环境...
    茶点故事阅读 39,005评论 3 307
  • 文/蒙蒙 一、第九天 我趴在偏房一处隐蔽的房顶上张望。 院中可真热闹,春花似锦、人声如沸。这庄子的主人今日做“春日...
    开封第一讲书人阅读 29,973评论 0 19
  • 文/苍兰香墨 我抬头看了看天上的太阳。三九已至,却和暖如春,着一层夹袄步出监牢的瞬间,已是汗流浃背。 一阵脚步声响...
    开封第一讲书人阅读 31,203评论 1 260
  • 我被黑心中介骗来泰国打工, 没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留,地道东北人。 一个月前我还...
    沈念sama阅读 45,053评论 2 350
  • 正文 我出身青楼,却偏偏与公主长得像,于是被迫代替她去往敌国和亲。 传闻我的和亲对象是个残疾皇子,可洞房花烛夜当晚...
    茶点故事阅读 42,495评论 2 343

推荐阅读更多精彩内容

  • Spring Cloud为开发人员提供了快速构建分布式系统中一些常见模式的工具(例如配置管理,服务发现,断路器,智...
    卡卡罗2017阅读 134,596评论 18 139
  • 软件的持续集成工具之一,易上手,功能强大,话不多说,干货奉上。我的博客地址:http://blog.lzoro.c...
    格子Lin阅读 12,194评论 10 41
  • 我很想你
    章欣宜阅读 92评论 0 0
  • 偶尔从网上找些图片,自己没事画着玩! 因为发现的简书,我更喜欢分享,虽然我并不聪明也不勤快,但值得开心的我一直没有丢弃
    半暖时光丫阅读 336评论 5 2
  • 来源:互联网自己整理发布给大家 简历模板莫名其妙就是面试用的,分享给那些还在找工作的小伙伴,以下是压缩包目录因为太...
    小向资源网阅读 333评论 0 3