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

CDIPropertyNameMatchingTest fails when running with Java 21 #1038

Closed
tevans78 opened this issue Nov 3, 2023 · 7 comments · Fixed by #1043
Closed

CDIPropertyNameMatchingTest fails when running with Java 21 #1038

tevans78 opened this issue Nov 3, 2023 · 7 comments · Fixed by #1043

Comments

@tevans78
Copy link
Contributor

tevans78 commented Nov 3, 2023

MP Config TCK test fails using Java 21 - CDIPropertyNameMatchingTest.testPropertyFromEnvironmentVariables

Passes with Java 17. Fails with Java 21.
Passes with SmallRye Config 3.3.4. Fails on version 3.4.x.

java --version
openjdk 21 2023-09-19
OpenJDK Runtime Environment (build 21+35-2513)
OpenJDK 64-Bit Server VM (build 21+35-2513, mixed mode, sharing)
[ERROR] Tests run: 240, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 2.651 s <<< FAILURE! -- in TestSuite
[ERROR] org.eclipse.microprofile.config.tck.CDIPropertyNameMatchingTest.testPropertyFromEnvironmentVariables -- Time elapsed: 0.003 s <<< FAILURE!
java.lang.AssertionError:

Expected: is "haha"
     but: was "woohoo"
	at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:20)
	at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:8)
	at org.eclipse.microprofile.config.tck.CDIPropertyNameMatchingTest.testPropertyFromEnvironmentVariables(CDIPropertyNameMatchingTest.java:111)
	at java.base/jdk.internal.reflect.DirectMethodHandleAccessor.invoke(DirectMethodHandleAccessor.java:103)
	at java.base/java.lang.reflect.Method.invoke(Method.java:580)
	at org.testng.internal.invokers.MethodInvocationHelper.invokeMethod(MethodInvocationHelper.java:139)
	at org.testng.internal.invokers.MethodInvocationHelper$1.runTestMethod(MethodInvocationHelper.java:258)
	at org.jboss.arquillian.testng.Arquillian$3.invoke(Arquillian.java:146)
	at org.jboss.arquillian.container.test.impl.execution.LocalTestExecuter.execute(LocalTestExecuter.java:57)
@tevans78
Copy link
Contributor Author

tevans78 commented Nov 3, 2023

@radcortez could this be related to #987 ?

@tevans78 tevans78 changed the title MP Config TCK fails when running with Java 21 CDIPropertyNameMatchingTest fails when running with Java 21 Nov 3, 2023
@radcortez
Copy link
Member

No idea, but the change works for 11 and 17, I don't see a reason for it not to work with 21. I'll add a runner for 21 and check.

@radcortez
Copy link
Member

Added the Java 21 builder here and all the tests passed: #1039

@tevans78
Copy link
Contributor Author

tevans78 commented Nov 3, 2023

@radcortez ok, that is interesting ... do you know exactly which JVM that is using? Mine is a couple of months old now, maybe it needs updating?

@radcortez
Copy link
Member

I've tried locally with:

openjdk 21 2023-09-19 LTS
OpenJDK Runtime Environment Temurin-21+35 (build 21+35-LTS)
OpenJDK 64-Bit Server VM Temurin-21+35 (build 21+35-LTS, mixed mode, sharing)

Github actions v1 uses Azul Zulu OpenJDK.

@tevans78
Copy link
Contributor Author

tevans78 commented Nov 6, 2023

I've just tested locally with Zulu and Temurin and I still get the same failure. I will try to grab a Linux VM and try that instead of my Mac (Sonoma).

@tevans78
Copy link
Contributor Author

tevans78 commented Nov 6, 2023

Using a completely fresh RHEL 9.2 VM with Temurin 21, it still fails for me.

openjdk 21.0.1 2023-10-17 LTS
OpenJDK Runtime Environment Temurin-21.0.1+12 (build 21.0.1+12-LTS)
OpenJDK 64-Bit Server VM Temurin-21.0.1+12 (build 21.0.1+12-LTS, mixed mode, sharing)

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 a pull request may close this issue.

2 participants