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

Print warning if class ids differ #185

Merged
merged 4 commits into from Jan 23, 2014

Conversation

Projects
None yet
1 participant
@marchof
Member

marchof commented Jan 20, 2014

A frequent problem of JaCoCo users are classes showing 0% coverage while the classes are expected to be executed. In most cases this is caused by the fact that different classes are used at report generation time and at runtime.

Therefore the report and check tasks/goals should issue warnings, if the execution data does not contain a class with the same id but with the same name.

Also the core API should allow to provide such warning also in other tools see jacoco/eclemma#74

@ghost ghost assigned marchof Jan 20, 2014

@marchof

This comment has been minimized.

Show comment
Hide comment
@marchof

marchof Jan 20, 2014

Member

@Godin @mfriedenhagen Please feel free to add your review comments, thx!

Member

marchof commented Jan 20, 2014

@Godin @mfriedenhagen Please feel free to add your review comments, thx!

marchof added a commit that referenced this pull request Jan 23, 2014

Merge pull request #185 from jacoco/issue-185
Print warning if class ids differ

@marchof marchof merged commit 38a27cd into master Jan 23, 2014

1 check passed

default The Travis CI build passed
Details

@marchof marchof deleted the issue-185 branch Jan 23, 2014

@jacoco jacoco locked and limited conversation to collaborators Jan 11, 2017

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.