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

Investigate TCK failure when using updated SmallRye Config on Java 21 #26767

Closed
tevans78 opened this issue Oct 30, 2023 · 1 comment
Closed

Comments

@tevans78
Copy link
Member

tevans78 commented Oct 30, 2023

When SmallRye Config was updated from 3.3.0 to 3.4.1 (and SmallRye Common from 2.1.0 to 2.2.0), a TCK test fails when running on Java 21. We would still like to update but we might need to open a SmallRye issue first and get something fixed there.

CDIPropertyNameMatchingTest	testPropertyFromEnvironmentVariables	Failure	Expected: is "haha" but: was "woohoo"


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:136)
at org.testng.internal.invokers.MethodInvocationHelper$1.runTestMethod(MethodInvocationHelper.java:253)
at org.jboss.arquillian.testng.Arquillian$3.invoke(Arquillian.java:146)
at org.jboss.arquillian.container.test.impl.execution.LocalTestExecuter.execute(LocalTestExecuter.java:57)
at java.base/jdk.internal.reflect.DirectMethodHandleAccessor.invoke(DirectMethodHandleAccessor.java:103)
at java.base/java.lang.reflect.Method.invoke(Method.java:580)
at org.jboss.arquillian.core.impl.ObserverImpl.invoke(ObserverImpl.java:86) 
@jakub-pomykala
Copy link
Contributor

Tested SmallRye-config using java 21 locally and the TCKs passed.

Will start work on #26663.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

No branches or pull requests

2 participants