This is an automated email from the ASF dual-hosted git repository. elharo pushed a commit to branch guice in repository https://gitbox.apache.org/repos/asf/maven-jlink-plugin.git
The following commit(s) were added to refs/heads/guice by this push: new 1a4657b comment 1a4657b is described below commit 1a4657ba3985b11d8646f90b5a1d643254fc74f2 Author: Elliotte Rusty Harold <elh...@ibiblio.org> AuthorDate: Mon Dec 2 12:22:10 2024 -0500 comment --- .../java/org/apache/maven/plugins/jlink/MultipleLauncherTest.java | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/test/java/org/apache/maven/plugins/jlink/MultipleLauncherTest.java b/src/test/java/org/apache/maven/plugins/jlink/MultipleLauncherTest.java index 1eec1be..adf655b 100644 --- a/src/test/java/org/apache/maven/plugins/jlink/MultipleLauncherTest.java +++ b/src/test/java/org/apache/maven/plugins/jlink/MultipleLauncherTest.java @@ -51,6 +51,7 @@ public class MultipleLauncherTest { @Test void testOneMultipleLauncher() throws Exception { + // It's OK to specify one launcher with "<launchers>" Field launchers = mojo.getClass().getDeclaredField("launchers"); launchers.setAccessible(true); launchers.set(mojo, List.of("l=com.example.Launch")); @@ -64,6 +65,7 @@ public class MultipleLauncherTest { @Test void testMultipleLaunchers() throws Exception { + // It's OK to specify multiple launchers with the "<launchers>" element Field launchers = mojo.getClass().getDeclaredField("launchers"); launchers.setAccessible(true); launchers.set(mojo, List.of("l1=com.example.Launch1", "l2=com.example.Launch2")); @@ -77,7 +79,7 @@ public class MultipleLauncherTest { @Test void testInvalidLauncherConfig() throws Exception { - // It's OK to specify one launcher with "<launcher>" given + // It's an error to specify both "<launcher>" and "<launchers>" Field launcher = mojo.getClass().getDeclaredField("launcher"); launcher.setAccessible(true); launcher.set(mojo, "l3=com.example.Launch3");