-
Notifications
You must be signed in to change notification settings - Fork 15
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
Make jacoco work with integration tests #459
Labels
enhancement
For changelog: Minor enhancement. use `major-rfe` for changes to be highlighted
good first issue
Good for newcomers
Comments
jonesbusy
added
the
enhancement
For changelog: Minor enhancement. use `major-rfe` for changes to be highlighted
label
Dec 15, 2024
Good first issue if someone interested by Maven config and jacoco coverage |
Dis some quick test without maven
Now we need to integration with failsafe, maven exec and jacoco plugin. And merge all together with previous unit tests |
Hi, as I was working on this. I wanted to know if I will have to also change pom.xml and pom-it.xml to integrate jacoco. |
Most likely yes. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
enhancement
For changelog: Minor enhancement. use `major-rfe` for changes to be highlighted
good first issue
Good for newcomers
What feature do you want to see added?
Right now coverage is only computed on unit tests
When we run integration tests with failsafe and 'java -jar..' we should add the argLine to the execution and make sure the execution is merged with previous jacoco execution.
Possible code change on
CommandLineIT
if we need to pass the jacoco argLine ?The text was updated successfully, but these errors were encountered: