android / android/nowinandroid

[Bug]: Jacoco configuration ignores sourcesets of buildFlavors

Open
#682 2 comments 0 reactions 1 assignee Claimed by @alexvanyo View on GitHub
bug continuous-integration help wanted
Dominant language
Kotlin
Stars
21.8k
Forks
4.6k
Avg merge
19h 20m
Merged PRs (30d)
2

Description

### Is there an existing issue for this?

- [X] I have searched the existing issues

### Is there a StackOverflow question about this issue?

- [X] 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

- [X] I agree to follow this project's Code of Conduct

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.