android/nowinandroid

[Bug]: Jacoco configuration ignores sourcesets of buildFlavors

Open

#682 opened on 2023年4月25日

GitHub で見る
 (2 comments) (0 reactions) (1 assignee)Kotlin (4,404 forks)batch import
bugcontinuous-integrationhelp wanted

Repository metrics

Stars
 (21,223 stars)
PR merge metrics
 (30d に merged 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

コントリビューターガイド