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
excludes package and patterns is not working at all #80
Comments
Hi, |
Relates #65 |
I found that following to kover doc 'com\.genoholding\.geno\.data\.model\..', does not work |
Hey guys, I have troubles with excluding my testclasses from the report. First question would be; why is this even included in the first place?! Seems to be related to the fact that other test-sources like itest or contractTest are not recognised for being test code. Related to #83 |
Now if you use However, in the next release, for the filters will be used the same format as for JaCoCo, with wildcards |
In our case the issue is not really about the regex but about the source sets it is looking at. So it's not looking inside all our source sets. |
These filters are not filters for the report, they determine which classes should be instrumented by the agent (estimate the coverage for them). In this case, to exclude a class from the report, you need to exclude it from all test tasks in your project. Excluding a class in only one test task may not have an effect, since this class can be instrumented by another task and so will get into the report. |
Hey @shanshin, I'm not sure that I understand what you mean. I don't really know what that would look like in practise. Currently I have my configuration set like this:
I made an example project to show what happens. As you can see the GreetingTest (which is part of itest) is still included in the report, whereas GreetingServiceTest (part of test) is not. |
Thanks for the example!
Unfortunately, at the moment the Kover plugin does not provide to specify which source set to exclude from the instrumentation, it excludes only the source set with the name
|
@periva101, have you tested version |
Hey @shanshin, do you have a reference to this feature you are talking about: I would like to subscribe myself to it if possible :) |
Now there is no separate task for this, because the API design has not been finished yet. |
non of the following is working
The text was updated successfully, but these errors were encountered: