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");

Reply via email to