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

Signature Test Error: API References Implementation Class #1957

Open
volosied opened this issue Aug 16, 2024 · 4 comments
Open

Signature Test Error: API References Implementation Class #1957

volosied opened this issue Aug 16, 2024 · 4 comments
Labels
accepted Accepted certification request challenge TCK challenge
Milestone

Comments

@volosied
Copy link
Contributor

volosied commented Aug 16, 2024

TCK Version:
Faces TCK 4.1.0

Tested Implementation:
Open Liberty (with MyFaces 4.1.0-RC2)

Description:
When Open Liberty is used against the Faces 4.1 signature test, it encounters a ClassNotFoundException by failing to find com.sun.faces.config.InitFacesContext. Upon looking at the Jakarta Faces API 4.1.0 source, we found the FacesContext imports the InitFacesContext class, and references it as a parameterized type in initContextServletContext ConcurrentHashMap.

As a follow up, we created the following bug in Mojarra: eclipse-ee4j/mojarra#5484
A snippet of the exception:

java.lang.reflect.InvocationTargetException
        at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77)
        at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.base/java.lang.reflect.Method.invoke(Method.java:568)
        at ee.jakarta.tck.faces.signaturetest.SigTestDriver.runSignatureTest(SigTestDriver.java:167)
        at ee.jakarta.tck.faces.signaturetest.SignatureTestDriver.executeSigTest(SignatureTestDriver.java:231)
        at ee.jakarta.tck.faces.signaturetest.JSFSigTestIT.signatureTest(JSFSigTestIT.java:223)
       ...
Caused by: java.lang.TypeNotPresentException: Type com.sun.faces.config.InitFacesContext not present
        at java.base/sun.reflect.generics.factory.CoreReflectionFactory.makeNamedType(CoreReflectionFactory.java:117)
        at java.base/sun.reflect.generics.visitor.Reifier.visitClassTypeSignature(Reifier.java:125)
        at java.base/sun.reflect.generics.tree.ClassTypeSignature.accept(ClassTypeSignature.java:49)
        at java.base/sun.reflect.generics.visitor.Reifier.reifyTypeArguments(Reifier.java:68)
        at java.base/sun.reflect.generics.visitor.Reifier.visitClassTypeSignature(Reifier.java:138)
        at java.base/sun.reflect.generics.tree.ClassTypeSignature.accept(ClassTypeSignature.java:49)
        ...
Caused by: java.lang.ClassNotFoundException: com.sun.faces.config.InitFacesContext
        at java.base/java.lang.Class.forNameImpl(Native Method)
        at java.base/java.lang.Class.forName(Class.java:428)
        at java.base/sun.reflect.generics.factory.CoreReflectionFactory.makeNamedType(CoreReflectionFactory.java:114)
        ... 101 more
@volosied
Copy link
Contributor Author

@BalusC @arjantijms Could you take a look whenever you have time (along with the other 2 challenges)? Thanks!

@BalusC
Copy link
Member

BalusC commented Aug 26, 2024

eclipse-ee4j/mojarra#5484 has been fixed, please retest after merge of eclipse-ee4j/mojarra#5485

@BalusC BalusC added this to the 4.1.1 milestone Aug 26, 2024
@BalusC BalusC added the accepted Accepted certification request label Aug 26, 2024
@scottmarlow
Copy link

Does the signature test pass or fail due to this issue? I expect that you would see a failure but would be good to include the test failure.

@pnicolucci
Copy link
Contributor

pnicolucci commented Oct 28, 2024 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accepted Accepted certification request challenge TCK challenge
Projects
None yet
Development

No branches or pull requests

4 participants