Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Exclude testIntegration source set from coverage #108

Closed
SIMULATAN opened this issue Dec 15, 2024 · 1 comment · Fixed by #109
Closed

Exclude testIntegration source set from coverage #108

SIMULATAN opened this issue Dec 15, 2024 · 1 comment · Fixed by #109
Labels
bug Something isn't working

Comments

@SIMULATAN
Copy link
Contributor

Test source sets inflate the coverage percentage as they are pretty much covered 100% (duh).

Instead of the current excludedSourceSets.add of all JvmTestSuite source sets, use explicit opt-in and just select the main source sets.
@SIMULATAN SIMULATAN added the bug Something isn't working label Dec 15, 2024
SIMULATAN added a commit that referenced this issue Dec 15, 2024
@SIMULATAN
fix(tests): #108 exclude nonstandard sourceSets from test coverage
14b790c 
Fixes #108
@SIMULATAN
Copy link
Contributor Author

Temporarily fixed using afterEvaluate, awaiting the implementation of Kotlin/kotlinx-kover#714 in order to switch to the included-only approach.

@SIMULATAN SIMULATAN mentioned this issue Dec 15, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(tests): #108 exclude nonstandard sourceSets from test coverage SnotyMe/snoty-backend
1 participant
@SIMULATAN