android/nowinandroid

[Bug]: Jacoco configuration ignores sourcesets of buildFlavors

Open

#682 创建于 2023年4月25日

在 GitHub 查看
 (2 评论) (0 反应) (1 负责人)Kotlin (4,404 fork)batch import
bugcontinuous-integrationhelp wanted

仓库指标

Star
 (21,223 star)
PR 合并指标
 (30 天内没有已合并 PR)

描述

Is there an existing issue for this?

  • I have searched the existing issues

Is there a StackOverflow question about this issue?

  • I have searched StackOverflow

What happened?

The sourceDirectories configuration in Jacoco.kt seems to be simply setting it from main sourceset and ignores other flavors: sourceDirectories.setFrom(files("$projectDir/src/main/java", "$projectDir/src/main/kotlin"))

Instead I think it should be including all sources of the current variant. This can be done in groovy by variant.sourceSets.java.srcDirs.collect { it.path }.flatten() but I'm not sure how this can be done in Kotlin which is using a different variant class. I came across this trying to migrate my own project scripts from groovy to kotlin, so any help is appreciated!

Relevant logcat output

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct

贡献者指南