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

IllegalStateException: Incompatible execution data for class org/junit/runner/Description #286

Closed
don-vip opened this Issue Feb 23, 2015 · 4 comments

Comments

Projects
None yet
2 participants
@don-vip

don-vip commented Feb 23, 2015

Hi,
We're using Jacoco on JOSM project. We switched from Jacoco 0.7.2 to 0.7.3 in this commit:
https://josm.openstreetmap.de/changeset/8090/josm/
Since, I have a Sonar analysis that fails every time with the same error:
http://donvip.fr/jenkins/job/JOSM-Plugins-Sonar/211/console

java.lang.IllegalStateException: Incompatible execution data for class org/junit/runner/Description with id c3b4a36e785ee4be.
at org.jacoco.core.data.ExecutionData.assertCompatibility(ExecutionData.java:181)
at org.jacoco.core.data.ExecutionData.merge(ExecutionData.java:144)
at org.jacoco.core.data.ExecutionData.merge(ExecutionData.java:117)
at org.jacoco.core.data.ExecutionDataStore.put(ExecutionDataStore.java:53)
at org.jacoco.core.data.ExecutionDataStore.visitClassExecution(ExecutionDataStore.java:175)
at org.jacoco.core.data.ExecutionDataReader.readExecutionData(ExecutionDataReader.java:149)
at org.jacoco.core.data.ExecutionDataReader.readBlock(ExecutionDataReader.java:113)
at org.jacoco.core.data.ExecutionDataReader.read(ExecutionDataReader.java:87)
at org.sonar.plugins.jacoco.JaCoCoOverallSensor.loadSourceFiles(JaCoCoOverallSensor.java:124)
at org.sonar.plugins.jacoco.JaCoCoOverallSensor.mergeReports(JaCoCoOverallSensor.java:99)
at org.sonar.plugins.jacoco.JaCoCoOverallSensor.analyse(JaCoCoOverallSensor.java:90)
at org.sonar.batch.phases.SensorsExecutor.executeSensor(SensorsExecutor.java:79)

Jacoco calls Junit from Ant as follows:

<junit printsummary="yes" fork="true" forkmode="once" dir="${basedir}">

(See https://trac.openstreetmap.org/browser/subversion/applications/editors/josm/plugins/build-common.xml#L443 )

I have no idea what's wrong but it looks like somewhat a regression or a change of behaviour between 0.7.2 and 0.7.3.

@marchof

This comment has been minimized.

Show comment
Hide comment
@marchof

marchof Feb 24, 2015

Member

With 0.7.3 we're actually missing a probe at the beginning of a try/catch block when the first instruction of the block is only reached via jump instruction. Validation test: https://gist.github.com/marchof/d0f53b31f1bd3ca4dfa6

Member

marchof commented Feb 24, 2015

With 0.7.3 we're actually missing a probe at the beginning of a try/catch block when the first instruction of the block is only reached via jump instruction. Validation test: https://gist.github.com/marchof/d0f53b31f1bd3ca4dfa6

@marchof marchof added this to the 0.7.4 milestone Feb 24, 2015

marchof added a commit that referenced this issue Feb 24, 2015

GitHub #286: Fix missing probes
Fix missing probes at the beginning of try/catch blocks when the block
starts as a jump target. This should restore exec file compatibility
with 0.7.2. Basically a revert of commit e4a474c
@marchof

This comment has been minimized.

Show comment
Hide comment
@marchof

marchof Feb 25, 2015

Member

@don-vip The problem should be fixed on trunk. Can you please re-test with 0.7.4-SNAPSHOT?

Member

marchof commented Feb 25, 2015

@don-vip The problem should be fixed on trunk. Can you please re-test with 0.7.4-SNAPSHOT?

@don-vip

This comment has been minimized.

Show comment
Hide comment
@don-vip

don-vip commented Feb 25, 2015

Sure :)

don-vip added a commit to openstreetmap/josm that referenced this issue Feb 25, 2015

@don-vip

This comment has been minimized.

Show comment
Hide comment
@don-vip

don-vip Feb 25, 2015

I confirm: the problem is fixed! Thanks for the quick fix :)

don-vip commented Feb 25, 2015

I confirm: the problem is fixed! Thanks for the quick fix :)

@marchof marchof closed this Feb 26, 2015

ToeBee pushed a commit to ToeBee/myjosm that referenced this issue Mar 2, 2015

wiktorn pushed a commit to wiktorn/josm that referenced this issue Jul 4, 2015

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

floscher pushed a commit to floscher/josm that referenced this issue Feb 21, 2018

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