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

Support for class, method coverage koverVerify #128

Closed
Saketh-Karumuri opened this issue Jan 21, 2022 · 3 comments · Fixed by #190
Closed

Support for class, method coverage koverVerify #128

Saketh-Karumuri opened this issue Jan 21, 2022 · 3 comments · Fixed by #190
Labels
Feature Feature request issue type Kover Coverage Tool Kover Gradle Plugin Question Support request issue type
Milestone

Comments

@Saketh-Karumuri
Copy link

Are there any plans to add rules for class, method, branch coverage for the koverVerify task? Currently only line coverage is being supported.

@shanshin shanshin added the Question Support request issue type label Jan 21, 2022
@shanshin
Copy link
Collaborator

shanshin commented Jan 21, 2022

Hi,
yes, in the near future we have plans to implement a verifier for the IntelliJ Coverage Engine, and after that we will improve the API for adding validation rules.

@SimonedeGijt
Copy link

I think the additional value of Kover will improve significantly if we can also add verification on branch% !

Speaking of which, I did notice that in the report you can not really see how many branches you've missed per line. Only branch% of the entire class. This makes improving your tests a lot harder as you don't really know how many branches you are missing for specific functions (especially when the file is quite big).

Is this something you guys are looking at in the future as well or shall I send in a new request for that? :)

@husam-e
Copy link

husam-e commented Apr 2, 2022

+++ Really need branch coverage support (report + verification) to transition to this from JaCoCo, which has lots of issues dealing with "ghost" kotlin branches in bytecode. Should actually mention branch coverage explicitly in the title of this issue as it's not clear that it is in scope until you read the comments. Was about to open a separate issue for it :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Feature request issue type Kover Coverage Tool Kover Gradle Plugin Question Support request issue type
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants