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.

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

Exclude DefaultImpls from a report #517

Closed
BulatInDrive opened this issue Dec 5, 2023 · 3 comments
Closed

Exclude DefaultImpls from a report #517

BulatInDrive opened this issue Dec 5, 2023 · 3 comments
Assignees
Labels
Bug Bug issue type S: waiting for clarification Status: additional information required to proceed

Comments

@BulatInDrive
Copy link

Describe the bug
For interface with function with default params Kover generates 0% coverage report.

Expected behavior
Default params shouldn't be in a report.

Reports
image

Environment

  • Kover Gradle Plugin version: 0.7.4
  • Gradle version: 7.3.1
  • Kotlin project type: Kotlin/Android
  • Coverage Toolset (if customized in build script): Kover
@BulatInDrive BulatInDrive added Bug Bug issue type S: untriaged Status: issue reported but unprocessed labels Dec 5, 2023
@zuevmaxim
Copy link

@BulatInDrive I confirm that line 5 should not be marked as red as it contains no executable code -- this is a bug
Lines 6 and 7 contain the values of the default arguments, so kover treats them as uncovered, when the value is not used in tests. Consider adding tests with calls like makeCoffee()

@shanshin shanshin added S: waiting for clarification Status: additional information required to proceed and removed S: untriaged Status: issue reported but unprocessed labels Jan 29, 2024
@zuevmaxim
Copy link

The fix for line 5 is available in the agent version 1.0.745

@shanshin
Copy link
Collaborator

Fixed in 0.7.6

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Bug issue type S: waiting for clarification Status: additional information required to proceed
Projects
None yet
Development

No branches or pull requests

3 participants