能找到這里,說明對jacoco的原理和使用有了一定的了解,而我寫這邊文章主要是網絡上基本沒有完整文檔加代碼的jaocco增量覆蓋說明,所以我想分享些東西讓需要這方面的人快速去實現自己想要的功能,那么如果想實現增量代碼覆蓋率需要做到哪些工作呢?
大家在網絡上找到的實現方式無外乎三種
- 獲取到增量代碼,在jacoco進行插樁時判斷是否是增量代碼后再進行插樁,這樣需要兩個步驟,一是獲取增量代碼,二是找到jacoco的插樁邏輯進行修改
- 獲取增量代碼,在report階段去判斷方法是否是增量,再去生成報告
- 獲取差異代碼,解析生成的report報告,再過濾出差異代碼的報告
首先第一種需要對java字節碼操作比較熟悉,難度較高,我們不談,第三種去解析生成的報告,可能存在誤差
所以我們一般選擇第二種,而網絡上所有的增量實現基本是基於第二種,我們先看看下面的圖
上圖說明了jacoco測試覆蓋率的生成流程,而我們要做的是在report的時候加入我們的邏輯
根據我們的方案,我們需要三個動作
- 計算出兩個版本的差異代碼(基於git)
- 將差異代碼在jacoco的report階段傳給jacoco
- 修改jacoco源碼,生成報告時判斷代碼是否是增量代碼,只有增量代碼才去生成報告
下面我們逐步講解上述步驟
計算差異代碼
計算差異代碼我實現了一個簡單的工程:差異代碼獲取
主要用到了兩個工具類
-
<dependency>
-
<groupId>org.eclipse.jgit</groupId>
-
<artifactId>org.eclipse.jgit</artifactId>
-
</dependency>
-
-
<!-- https://mvnrepository.com/artifact/com.github.javaparser/javaparser-core -->
-
<dependency>
-
<groupId>com.github.javaparser</groupId>
-
<artifactId>javaparser-core</artifactId>
-
</dependency>
org.eclipse.jgit主要用於從git獲取代碼,並獲取到存在變更的文件
javaparser-core是一個java解析類,能將class類文件解析成樹狀,方便我們去獲取差異類
-
/**
-
* 獲取差異類
-
*
-
* @param diffMethodParams
-
* @return
-
*/
-
public List<ClassInfoResult> diffMethods(DiffMethodParams diffMethodParams) {
-
try {
-
//原有代碼git對象
-
Git baseGit = cloneRepository(diffMethodParams.getGitUrl(), localBaseRepoDir + diffMethodParams.getBaseVersion(), diffMethodParams.getBaseVersion());
-
//現有代碼git對象
-
Git nowGit = cloneRepository(diffMethodParams.getGitUrl(), localBaseRepoDir + diffMethodParams.getNowVersion(), diffMethodParams.getNowVersion());
-
AbstractTreeIterator baseTree = prepareTreeParser(baseGit.getRepository(), diffMethodParams.getBaseVersion());
-
AbstractTreeIterator nowTree = prepareTreeParser(nowGit.getRepository(), diffMethodParams.getNowVersion());
-
//獲取兩個版本之間的差異代碼
-
List<DiffEntry> diff = nowGit.diff().setOldTree(baseTree).setNewTree(nowTree).setShowNameAndStatusOnly(true).call();
-
//過濾出有效的差異代碼
-
Collection<DiffEntry> validDiffList = diff.stream()
-
//只計算java文件
-
.filter(e -> e.getNewPath().endsWith( ".java"))
-
//排除測試文件
-
.filter(e -> e.getNewPath().contains( "src/main/java"))
-
//只計算新增和變更文件
-
.filter(e -> DiffEntry.ChangeType.ADD.equals(e.getChangeType()) || DiffEntry.ChangeType.MODIFY.equals(e.getChangeType()))
-
.collect(Collectors.toList());
-
if (CollectionUtils.isEmpty(validDiffList)) {
-
return null;
-
}
-
/**
-
* 多線程獲取舊代碼和新代碼的差異類及差異方法
-
*/
-
List<CompletableFuture<ClassInfoResult>> priceFuture = validDiffList.stream().map(item -> getClassMethods(getClassFile(baseGit, item.getNewPath()), getClassFile(nowGit, item.getNewPath()), item)).collect(Collectors.toList());
-
return priceFuture.stream().map(CompletableFuture::join).filter(Objects::nonNull).collect(Collectors.toList());
-
} catch (GitAPIException e) {
-
e.printStackTrace();
-
}
-
return null;
-
}
以上代碼為獲取差異類的核心代碼
-
-
/**
-
* 獲取類的增量方法
-
*
-
* @param oldClassFile 舊類的本地地址
-
* @param mewClassFile 新類的本地地址
-
* @param diffEntry 差異類
-
* @return
-
*/
-
private CompletableFuture<ClassInfoResult> getClassMethods(String oldClassFile, String mewClassFile, DiffEntry diffEntry) {
-
//多線程獲取差異方法,此處只要考慮增量代碼太多的情況下,每個類都需要遍歷所有方法,采用多線程方式加快速度
-
return CompletableFuture.supplyAsync(() -> {
-
String className = diffEntry.getNewPath().split("\\.")[0].split("src/main/java/")[1];
-
//新增類直接標記,不用計算方法
-
if (DiffEntry.ChangeType.ADD.equals(diffEntry.getChangeType())) {
-
return ClassInfoResult.builder()
-
.classFile(className)
-
.type(DiffEntry.ChangeType.ADD.name())
-
.build();
-
}
-
List<MethodInfoResult> diffMethods;
-
//獲取新類的所有方法
-
List<MethodInfoResult> newMethodInfoResults = MethodParserUtils.parseMethods(mewClassFile);
-
//如果新類為空,沒必要比較
-
if (CollectionUtils.isEmpty(newMethodInfoResults)) {
-
return null;
-
}
-
//獲取舊類的所有方法
-
List<MethodInfoResult> oldMethodInfoResults = MethodParserUtils.parseMethods(oldClassFile);
-
//如果舊類為空,新類的方法所有為增量
-
if (CollectionUtils.isEmpty(oldMethodInfoResults)) {
-
diffMethods = newMethodInfoResults;
-
} else { //否則,計算增量方法
-
List<String> md5s = oldMethodInfoResults.stream().map(MethodInfoResult::getMd5).collect(Collectors.toList());
-
diffMethods = newMethodInfoResults.stream().filter(m -> !md5s.contains(m.getMd5())).collect(Collectors.toList());
-
}
-
//沒有增量方法,過濾掉
-
if (CollectionUtils.isEmpty(diffMethods)) {
-
return null;
-
}
-
ClassInfoResult result = ClassInfoResult.builder()
-
.classFile(className)
-
.methodInfos(diffMethods)
-
.type(DiffEntry.ChangeType.MODIFY.name())
-
.build();
-
return result;
-
}, executor);
-
}
以上代碼為獲取差異方法的核心代碼
大家可以下載代碼后運行,下面我們展示下,運行代碼后獲取到的差異代碼內容(參數可以是兩次commitId,也可以是兩個分支,按自己的業務場景來)
-
{
-
"code": 10000,
-
"msg": "業務處理成功",
-
"data": [
-
{
-
"classFile": "com/dr/application/InstallCert",
-
"methodInfos": null,
-
"type": "ADD"
-
},
-
{
-
"classFile": "com/dr/application/app/controller/Calculable",
-
"methodInfos": null,
-
"type": "ADD"
-
},
-
{
-
"classFile": "com/dr/application/app/controller/JenkinsPluginController",
-
"methodInfos": null,
-
"type": "ADD"
-
},
-
{
-
"classFile": "com/dr/application/app/controller/LoginController",
-
"methodInfos": [
-
{
-
"md5": "2C9D2AE2B1864A2FCDDC6D47CEBEBD4C",
-
"methodName": "captcha",
-
"parameters": "HttpServletRequest request,HttpServletResponse response"
-
},
-
{
-
"md5": "3D6DFADD2171E893D99D3D6B335B22EA",
-
"methodName": "login",
-
"parameters": "@RequestBody LoginUserParam loginUserParam,HttpServletRequest request"
-
},
-
{
-
"md5": "90842DFA5372DCB74335F22098B36A53",
-
"methodName": "logout",
-
"parameters": ""
-
},
-
{
-
"md5": "D0B2397D04624D2D60E96AB97F679779",
-
"methodName": "testInt",
-
"parameters": "int a,char b"
-
},
-
{
-
"md5": "34219E0141BAB497DCB5FB71BAE1BDAE",
-
"methodName": "testInt",
-
"parameters": "String a,int b"
-
},
-
{
-
"md5": "F9BF585A4F6E158CD4475700847336A6",
-
"methodName": "testInt",
-
"parameters": "short a,int b"
-
},
-
{
-
"md5": "0F2508A33F719493FFA66C5118B41D77",
-
"methodName": "testInt",
-
"parameters": "int[] a"
-
},
-
{
-
"md5": "381C8CBF1F381A58E1E93774AE1AF4EC",
-
"methodName": "testInt",
-
"parameters": "AddUserParam param"
-
},
-
{
-
"md5": "64BF62C11839F45030198A8D8D7821C5",
-
"methodName": "testInt",
-
"parameters": "T[] a"
-
},
-
{
-
"md5": "D091AB0AD9160407AED4182259200B9B",
-
"methodName": "testInt",
-
"parameters": "Calculable calc,int n1,int n2"
-
},
-
{
-
"md5": "693BBA0A8A57F2FD19F61BA06F23365C",
-
"methodName": "display",
-
"parameters": ""
-
},
-
{
-
"md5": "F9DFE0E75C78A31AFB6A8FD46BDA2B81",
-
"methodName": "a",
-
"parameters": "InnerClass a"
-
}
-
],
-
"type": "MODIFY"
-
},
-
{
-
"classFile": "com/dr/application/app/controller/RoleController",
-
"methodInfos": null,
-
"type": "ADD"
-
},
-
{
-
"classFile": "com/dr/application/app/controller/TestController",
-
"methodInfos": [
-
{
-
"md5": "B1840C873BF0BA74CB6749E1CEE93ED7",
-
"methodName": "getPom",
-
"parameters": "HttpServletResponse response"
-
},
-
{
-
"md5": "9CEE68771972EAD613AF237099CD2349",
-
"methodName": "getDeList",
-
"parameters": ""
-
}
-
],
-
"type": "MODIFY"
-
},
-
{
-
"classFile": "com/dr/application/app/controller/UserController",
-
"methodInfos": [
-
{
-
"md5": "7F2AD08CE732ADDFC902C46D238A9EB3",
-
"methodName": "add",
-
"parameters": "@RequestBody AddUserParam addUserParam"
-
},
-
{
-
"md5": "D41D8CD98F00B204E9800998ECF8427E",
-
"methodName": "get",
-
"parameters": ""
-
},
-
{
-
"md5": "2B35EA4FB5054C6EF13D557C2ACBB581",
-
"methodName": "list",
-
"parameters": "@ApiParam(required = true, name = \"page\", defaultValue = \"1\", value = \"當前頁碼\") @RequestParam(name = \"page\") Integer page,@ApiParam(required = true, name = \"pageSize\", defaultValue = \"10\", value = \"每頁數量\") @RequestParam(name = \"pageSize\") Integer pageSize,@ApiParam(name = \"userId\", value = \"用戶id\") @RequestParam(name = \"userId\", required = false) Long userId,@ApiParam(name = \"username\", value = \"用戶名\") @RequestParam(name = \"username\", required = false) String username,@ApiParam(name = \"userSex\", value = \"性別\") @RequestParam(name = \"userSex\", required = false) Integer userSex,@ApiParam(name = \"mobile\", value = \"手機號\") @RequestParam(name = \"mobile\", required = false) String mobile"
-
}
-
],
-
"type": "MODIFY"
-
},
-
{
-
"classFile": "com/dr/application/app/controller/view/RoleViewController",
-
"methodInfos": null,
-
"type": "ADD"
-
},
-
{
-
"classFile": "com/dr/application/app/controller/view/UserViewController",
-
"methodInfos": [
-
{
-
"md5": "9A1DDA3F41B36026FC2F3ACDAE85C1DB",
-
"methodName": "user",
-
"parameters": ""
-
}
-
],
-
"type": "MODIFY"
-
},
-
{
-
"classFile": "com/dr/application/app/param/AddRoleParam",
-
"methodInfos": null,
-
"type": "ADD"
-
},
-
{
-
"classFile": "com/dr/application/app/vo/DependencyVO",
-
"methodInfos": null,
-
"type": "ADD"
-
},
-
{
-
"classFile": "com/dr/application/app/vo/JenkinsPluginsVO",
-
"methodInfos": null,
-
"type": "ADD"
-
},
-
{
-
"classFile": "com/dr/jenkins/vo/DeviceVo",
-
"methodInfos": null,
-
"type": "ADD"
-
},
-
{
-
"classFile": "com/dr/jenkins/vo/GoodsVO",
-
"methodInfos": null,
-
"type": "ADD"
-
},
-
{
-
"classFile": "com/dr/jenkins/vo/JobAddVo",
-
"methodInfos": null,
-
"type": "ADD"
-
},
-
{
-
"classFile": "com/dr/repository/user/dto/query/RoleQueryDto",
-
"methodInfos": null,
-
"type": "ADD"
-
},
-
{
-
"classFile": "com/dr/repository/user/dto/query/UserQueryDto",
-
"methodInfos": null,
-
"type": "ADD"
-
},
-
{
-
"classFile": "com/dr/repository/user/dto/result/MenuDTO",
-
"methodInfos": null,
-
"type": "ADD"
-
},
-
{
-
"classFile": "com/dr/repository/user/dto/result/RoleResultDto",
-
"methodInfos": null,
-
"type": "ADD"
-
},
-
{
-
"classFile": "com/dr/repository/user/dto/result/UserResultDto",
-
"methodInfos": null,
-
"type": "ADD"
-
},
-
{
-
"classFile": "com/dr/user/service/impl/RoleServiceImpl",
-
"methodInfos": [
-
{
-
"md5": "D2AAADF53B501AE6D2206B2951256329",
-
"methodName": "getRoleCodeByUserId",
-
"parameters": "Long id"
-
},
-
{
-
"md5": "47405162B3397D02156DE636059049F2",
-
"methodName": "getListByPage",
-
"parameters": "RoleQueryDto roleQueryDto"
-
}
-
],
-
"type": "MODIFY"
-
},
-
{
-
"classFile": "com/dr/user/service/impl/UserServiceImpl",
-
"methodInfos": [
-
{
-
"md5": "D41D8CD989ABCDEFFEDCBA98ECF8427E",
-
"methodName": "selectListByPage",
-
"parameters": "UserQueryDto userQueryDto"
-
}
-
],
-
"type": "MODIFY"
-
}
-
]
-
}
data部分為差異代碼的具體內容
將差異代碼傳遞到jaocco
大家可以參考:jacoco增量代碼改造
我們只需要找到Report類,加入可選參數
@Option(name = "--diffCode", usage = "input file for diff", metaVar = "<file>") String diffCode;
這樣,我們就可以在jacoco內部接受到傳遞的參數了,如果report命令加上--diffCode就計算增量,不加則計算全量,不影響正常功能,靈活性高
我們這里改造了analyze方法,將增量代碼塞給CoverageBuilder對象,我們需要用時直接去獲取
-
private IBundleCoverage analyze(final ExecutionDataStore data,
-
final PrintWriter out) throws IOException {
-
CoverageBuilder builder;
-
// 如果有增量參數將其設置進去
-
if (null != this.diffCode) {
-
builder = new CoverageBuilder( this.diffCode);
-
} else {
-
builder = new CoverageBuilder();
-
}
-
final Analyzer analyzer = new Analyzer(data, builder);
-
for (final File f : classfiles) {
-
analyzer.analyzeAll(f);
-
}
-
printNoMatchWarning(builder.getNoMatchClasses(), out);
-
return builder.getBundle(name);
-
}
差異代碼匹配
jacoco采用AMS類去解析class類,我們需要去修改org.jacoco.core包下面的Analyzer類
-
private void analyzeClass(final byte[] source) {
-
final long classId = CRC64.classId(source);
-
final ClassReader reader = InstrSupport.classReaderFor(source);
-
if ((reader.getAccess() & Opcodes.ACC_MODULE) != 0) {
-
return;
-
}
-
if ((reader.getAccess() & Opcodes.ACC_SYNTHETIC) != 0) {
-
return;
-
}
-
// 字段不為空說明是增量覆蓋
-
if (null != CoverageBuilder.classInfos
-
&& !CoverageBuilder.classInfos.isEmpty()) {
-
// 如果沒有匹配到增量代碼就無需解析類
-
if (!CodeDiffUtil.checkClassIn(reader.getClassName())) {
-
return;
-
}
-
}
-
final ClassVisitor visitor = createAnalyzingVisitor(classId,
-
reader.getClassName());
-
reader.accept(visitor, 0);
-
-
}
主要是判斷如果需要的是增量代碼覆蓋率,則匹配類是否是增量的(這里是jacoco遍歷解析每個類的地方)
然后修改ClassProbesAdapter類的visitMethod方法(這個是遍歷類里面每個方法的地方)
整個比較的代碼邏輯在這里,注釋寫的你叫詳細了
修改完成后,大家只要構建出org.jacoco.cli-0.8.7-SNAPSHOT-nodeps.jar包,然后report時傳入增量代碼即可
全量報告
增量報告
所遇到問題
- 差異方法的參數匹配
由於我們使用javaparser解析出的參數格式為String a,int b
而ASM解析出的 為Ljava/lang/String,I;在匹配參數的時候遇到了問題,最終我找到了Type類的方法
Type.getArgumentTypes(desc)
然后
argumentTypes[i].getClassName()
將AmS的參數解析成String,int(做了截取),然后再去匹配,就能正確匹配到參數的格式了
- 為什么不將整個生成報告做成一個平台
jacoco生成報告的時候,需要傳入源碼,編譯后的class文件,而編譯這些東西我們一般都有自己的ci平台去做,我們可以將我們的覆蓋率功能集成到我們的devops平台,從那邊去獲取源碼或編譯出的class文件,而且可以做業務上的整合,所以沒有像supper-jacoco那樣做成一個平台
考資料: super-jacoco 里面有些bug,使用的時候請注意