-
Notifications
You must be signed in to change notification settings - Fork 111
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
Mojarra 4.0.3 is not Faces compatible as it fails the com/sun/ts/tests/jsf/api/jakarta_faces/context/facescontextwrapper/URLClient.java#facesCtxWrapper*
tests
#5308
Comments
More test output from the first test failure:
|
I haven't yet tested this, but the stack traces lead to #5207 being the cause of the failures. |
I ran against a Mojarra local snapshot build (using WildFly) that has every I cannot explain why I did see 143 (different) tests failing due to For reference the complete list of facescontextwrapper tests that are failing with 4.0.3 are listed in https://gist.github.com/scottmarlow/4c7c121470398cafcbf697195ed119fe |
The ant based scripts for running the TCKs are essentially al setup the same way. We just found similar issues with the connector TCK runner inside GlassFish. See eclipse-ee4j/glassfish@82fc12b @BalusC can you take a look at the #5207 change? I'll take a look soon as well. |
Fix #5308 ambiguous resolution during release of FacesContext when there is more than one implementation of FacesContext known to CDI in bean-discovery-mode=all state
Describe the bug
We need a Mojarra 4.0.4 release that passes the following Jakara Faces TCK tests that currently fail with Mojarra 4.0.3:
To Reproduce
Run https://ci.eclipse.org/jakartaee-tck/job/jakarta-faces-old-tck-glassfish/ against https://www.eclipse.org/downloads/download.php?file=/ee4j/glassfish/glassfish-7.0.8.zip which includes Mojarra 4.0.3. Example the started https://ci.eclipse.org/jakartaee-tck/job/10/job/jakarta-faces-tck-glassfish and search for
com/sun/ts/tests/jsf/api/jakarta_faces/context/facescontextwrapper/URLClient.java#facesCtxWrapper
Additional context
Also see recent comments on eclipse-ee4j/glassfish#24556
The text was updated successfully, but these errors were encountered: