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

jacoco_test failing with openj9 jdk11 and 14 #2065

Closed
nikolamilijevic1 opened this issue Nov 20, 2020 · 7 comments
Closed

jacoco_test failing with openj9 jdk11 and 14 #2065

nikolamilijevic1 opened this issue Nov 20, 2020 · 7 comments
Labels
triage required Issue needs deeper triage to determine which repo to move issue into
Milestone

Comments

@nikolamilijevic1
Copy link
Contributor

While adding the jacoco test suite to the openj9 external group, I noticed that there was one test consistently failing with openj9 jdk11 and 14 on all platforms (note the test passes with hotspot).

Here is the error message produced by https://ci.adoptopenjdk.net/view/Test_grinder/job/Grinder/4940/
Failed tests: startup_should_not_create_duplicate_class_definition(org.jacoco.core.runtime.InjectedClassRuntimeTest)
and
10:56:10 [[1;34mINFO[m] JaCoCo :: Test :: Core ............................. [1;31mFAILURE[m [ 3.992 s]

The failing test can be found at https://github.com/jacoco/jacoco/blob/master/org.jacoco.core.test/src/org/jacoco/core/runtime/InjectedClassRuntimeTest.java#L50-L61

@smlambert
Copy link
Contributor

FYI @DanHeidinga @pshipton - passes on jdk8 j9

@smlambert smlambert added the triage required Issue needs deeper triage to determine which repo to move issue into label Nov 20, 2020
@smlambert smlambert added this to the December 2020 milestone Nov 20, 2020
@DanHeidinga
Copy link
Contributor

@pshipton
Copy link
Contributor

@tajila fyi

@tajila
Copy link
Contributor

tajila commented Nov 20, 2020

@hangshao0 Can you please take a look

@hangshao0
Copy link
Contributor

eclipse-openj9/openj9#11243 is resolved. The latest OpenJ9 build should be able to pass test org.jacoco.core.runtime.InjectedClassRuntimeTest now.

@llxia
Copy link
Contributor

llxia commented Nov 26, 2020

Thanks @hangshao0 . @nikolamilijevic1 could you please re-enable the test? Thanks.

@llxia
Copy link
Contributor

llxia commented Dec 10, 2020

The test is re-enabled. PR #2105

@llxia llxia closed this as completed Dec 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
triage required Issue needs deeper triage to determine which repo to move issue into
Projects
None yet
Development

No branches or pull requests

7 participants