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

Unignore 2 tests ignored with move to JDK 11 #203

Closed
rhusar opened this issue Sep 25, 2024 · 2 comments · Fixed by #207
Closed

Unignore 2 tests ignored with move to JDK 11 #203

rhusar opened this issue Sep 25, 2024 · 2 comments · Fixed by #207

Comments

@rhusar
Copy link
Collaborator

rhusar commented Sep 25, 2024

Basically revert this commit - f0a79e3

@WolfgangHG
Copy link
Contributor

Both tests contain a org.jboss.arquillian.warp.Inspection (snippet from org.jboss.arquillian.warp.jsf.ftest.BasicJsfTest), and injection does not seem to happen:

            .inspect(new Inspection() {
                         private static final long serialVersionUID = 1L;

                         @Inject
                         CdiBean myBean;

                         @AfterPhase(RENDER_RESPONSE)
                         public void initial_state_havent_changed_yet() {
                             assertEquals("John", myBean.getName());
                         }
                     }
            );

It works for me when switching back to the previous version in root "pom.xml":

<version.wildfly.arquillian.container>3.0.1.Final</version.wildfly.arquillian.container>

4.0.1.Final also works.

According to https://github.com/wildfly/wildfly-arquillian: "Versions 5.0 for for Jakarta EE 10 and work with WildFly 27 and higher and JBoss EAP 8.0 and higher".

So we were to fast by updating it :-)? See also #141

@rhusar
Copy link
Collaborator Author

rhusar commented Sep 26, 2024

Done, thanks @WolfgangHG !

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