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

Generate JaCoCo code coverage report for project build #7

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

dlhartveld
Copy link
Contributor

No description provided.

@avandeursen
Copy link

While JaCoCo is good for coverage within Eclipse, I think Cobertura is nicer for use in reports. JaCoCo does not count how often lines or loops are executed, and it does not correctly cover code in which exceptions are raised. See also my post on line coverage in JUnit. So after merging this pull request, we may later replace it by another one facilitating Cobertura :)

@dlhartveld
Copy link
Contributor Author

Ok, the pull request title is probably a bit misleading... This only adds coverage reports to the jenkins-ws-client build itself.

I'll open an issue for coverage support in created jobs.

It might still be a good idea to change to cobertura for this project's coverage reports, but that is maybe something for another day...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants