android/nowinandroid

[Bug]: Jacoco configuration ignores sourcesets of buildFlavors

Open

#682 geöffnet am 25. Apr. 2023

Auf GitHub ansehen
 (2 Kommentare) (0 Reaktionen) (1 zugewiesene Person)Kotlin (4.404 Forks)batch import
bugcontinuous-integrationhelp wanted

Repository-Metriken

Stars
 (21.223 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

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

Contributor Guide