You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by ti...@apache.org on 2020/05/31 15:41:06 UTC

[maven-surefire] 01/02: providerJpmsArguments are already controlled by isMainDescriptor in ProviderInfo

This is an automated email from the ASF dual-hosted git repository.

tibordigana pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/maven-surefire.git

commit 5925e3d8fdc52e3ad58d7fd35d679277f763a8bf
Author: tibordigana <ti...@apache.org>
AuthorDate: Sun May 31 15:09:16 2020 +0200

    providerJpmsArguments are already controlled by isMainDescriptor in ProviderInfo
---
 .../ModularClasspathForkConfiguration.java             | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/booterclient/ModularClasspathForkConfiguration.java b/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/booterclient/ModularClasspathForkConfiguration.java
index 16e8905..8d4d84c 100644
--- a/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/booterclient/ModularClasspathForkConfiguration.java
+++ b/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/booterclient/ModularClasspathForkConfiguration.java
@@ -207,21 +207,21 @@ public class ModularClasspathForkConfiguration
             }
             else
             {
-                for ( String[] entries : providerJpmsArguments )
-                {
-                    for ( String entry : entries )
-                    {
-                        args.append( entry )
-                            .append( NL );
-                    }
-                }
-
                 args.append( "--add-modules" )
                     .append( NL )
                     .append( moduleName )
                     .append( NL );
             }
 
+            for ( String[] entries : providerJpmsArguments )
+            {
+                for ( String entry : entries )
+                {
+                    args.append( entry )
+                        .append( NL );
+                }
+            }
+
             args.append( startClassName );
 
             String argsFileContent = args.toString();