Skip to content

Update embedded Maven plugin applications with GlassFish configuration#26041

Open
SavioFerreira wants to merge 3 commits into
eclipse-ee4j:mainfrom
SavioFerreira:fix-embedded-plugin-tests
Open

Update embedded Maven plugin applications with GlassFish configuration#26041
SavioFerreira wants to merge 3 commits into
eclipse-ee4j:mainfrom
SavioFerreira:fix-embedded-plugin-tests

Conversation

@SavioFerreira

@SavioFerreira SavioFerreira commented May 27, 2026

Copy link
Copy Markdown

Fixes #26036.

Changes

  • Updated all embedded maven plugin test modules with the required configuration changes.

Validation

  • Full embedded maven plugin test suite passes successfully.

Additional Investigation

I also tested non-forked execution (mvn embedded-glassfish:run).

While testing this mode, I encountered initialization/runtime issues related to embedded startup and command resolution.

Successful test suite execution

Screenshot From 2026-05-26 22-29-01

Non-forked execution issue

Screenshot From 2026-05-26 22-25-29

Comment thread appserver/tests/embedded/maven-plugin/pom.xml Outdated
Comment thread appserver/tests/embedded/maven-plugin/pom.xml
@SavioFerreira

Copy link
Copy Markdown
Author

Looks like the undeploy goal is having some issues with the expected response.

When running the test suite, the jmxUndeployEvent test fails because it expects a 404 response after undeployment, but the application still returns 200.

I've attached the test output and failure details for reference.
Screenshot From 2026-06-04 18-30-36
Screenshot From 2026-06-04 18-28-02

I noticed a possible typo in the module name:
jmxUndeployEvent vs jmsUndeploymentTest
Should the reference in the <name> element be updated to use jmx instead of jms?
Screenshot From 2026-06-04 18-35-18

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 this pull request may close these issues.

Test improvement: Convert Embedded GlassFish Maven Plugin Tests to use the latest plugin version

4 participants