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


The following commit(s) were added to refs/heads/master by this push:
     new 2ccba31  [SUREFIRE-1531] Option to disable Java 9 modules
2ccba31 is described below

commit 2ccba319efc4248197ce307515540f1c56faa9c5
Author: Tibor17 <tibordig...@apache.org>
AuthorDate: Sat Nov 17 19:48:51 2018 +0100

    [SUREFIRE-1531] Option to disable Java 9 modules
---
 .../apache/maven/plugin/failsafe/IntegrationTestMojo.java  | 14 +++++++-------
 .../apache/maven/plugin/surefire/AbstractSurefireMojo.java | 14 +++++++-------
 .../plugin/surefire/AbstractSurefireMojoJava7PlusTest.java |  4 ++--
 .../maven/plugin/surefire/AbstractSurefireMojoTest.java    |  4 ++--
 .../org/apache/maven/plugin/surefire/MojoMocklessTest.java |  4 ++--
 .../org/apache/maven/plugin/surefire/SurefirePlugin.java   | 14 +++++++-------
 6 files changed, 27 insertions(+), 27 deletions(-)

diff --git 
a/maven-failsafe-plugin/src/main/java/org/apache/maven/plugin/failsafe/IntegrationTestMojo.java
 
b/maven-failsafe-plugin/src/main/java/org/apache/maven/plugin/failsafe/IntegrationTestMojo.java
index 42b4cc6..edc95c4 100644
--- 
a/maven-failsafe-plugin/src/main/java/org/apache/maven/plugin/failsafe/IntegrationTestMojo.java
+++ 
b/maven-failsafe-plugin/src/main/java/org/apache/maven/plugin/failsafe/IntegrationTestMojo.java
@@ -371,15 +371,15 @@ public class IntegrationTestMojo
     private String shutdown;
 
     /**
-     * Disables Jigsaw (Java 9) modular path even if <i>module-info.java</i> 
is used in project.
+     * Disables modular path (aka Jigsaw project since of Java 9) even if 
<i>module-info.java</i> is used in project.
      * <br>
      * Enabled by default.
      * If enabled, <i>module-info.java</i> exists and executes with JDK 9+, 
modular path is used.
      *
      * @since 3.0.0-M2
      */
-    @Parameter( property = "failsafe.useJigsawModules", defaultValue = "true" )
-    private boolean useJigsawModules;
+    @Parameter( property = "failsafe.useModulePath", defaultValue = "true" )
+    private boolean useModulePath;
 
     @Override
     protected int getRerunFailingTestsCount()
@@ -812,15 +812,15 @@ public class IntegrationTestMojo
     }
 
     @Override
-    protected boolean useJigsawModules()
+    protected boolean useModulePath()
     {
-        return useJigsawModules;
+        return useModulePath;
     }
 
     @Override
-    protected void setUseJigsawModules( boolean useJigsawModules )
+    protected void setUseModulePath( boolean useModulePath )
     {
-        this.useJigsawModules = useJigsawModules;
+        this.useModulePath = useModulePath;
     }
 
     @Override
diff --git 
a/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/AbstractSurefireMojo.java
 
b/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/AbstractSurefireMojo.java
index 4a1a213..b6ae1bb 100644
--- 
a/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/AbstractSurefireMojo.java
+++ 
b/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/AbstractSurefireMojo.java
@@ -791,9 +791,9 @@ public abstract class AbstractSurefireMojo
 
     protected abstract String getReportSchemaLocation();
 
-    protected abstract boolean useJigsawModules();
+    protected abstract boolean useModulePath();
 
-    protected abstract void setUseJigsawModules( boolean useJigsawModules );
+    protected abstract void setUseModulePath( boolean useModulePath );
 
     /**
      * This plugin MOJO artifact.
@@ -1240,9 +1240,9 @@ public abstract class AbstractSurefireMojo
         return new File( getClassesDirectory(), "module-info.class" );
     }
 
-    private boolean canExecuteProviderWithJigsaw( Platform platform )
+    private boolean canExecuteProviderWithModularPath( Platform platform )
     {
-        return useJigsawModules()
+        return useModulePath()
                 && platform.getJdkExecAttributesForTests().isJava9AtLeast()
                 && existsModuleDescriptor();
     }
@@ -1707,7 +1707,7 @@ public abstract class AbstractSurefireMojo
         {
             Set<Artifact> providerArtifacts = provider.getProviderClasspath();
             String providerName = provider.getProviderName();
-            if ( canExecuteProviderWithJigsaw( platform ) && !isInprocess )
+            if ( canExecuteProviderWithModularPath( platform ) && !isInprocess 
)
             {
                 return newStartupConfigWithModularPath( 
classLoaderConfiguration, providerArtifacts, providerName,
                         getModuleDescriptor(), scanResult );
@@ -2160,7 +2160,7 @@ public abstract class AbstractSurefireMojo
 
         Classpath bootClasspath = getArtifactClasspath( shadeFire != null ? 
shadeFire : surefireBooterArtifact );
 
-        if ( canExecuteProviderWithJigsaw( platform ) )
+        if ( canExecuteProviderWithModularPath( platform ) )
         {
             return new ModularClasspathForkConfiguration( bootClasspath,
                     tmpDir,
@@ -2429,7 +2429,7 @@ public abstract class AbstractSurefireMojo
         checksum.add( getForkedProcessExitTimeoutInSeconds() );
         checksum.add( getRerunFailingTestsCount() );
         checksum.add( getTempDir() );
-        checksum.add( useJigsawModules() );
+        checksum.add( useModulePath() );
         addPluginSpecificChecksumItems( checksum );
         return checksum.getSha1();
     }
diff --git 
a/maven-surefire-common/src/test/java/org/apache/maven/plugin/surefire/AbstractSurefireMojoJava7PlusTest.java
 
b/maven-surefire-common/src/test/java/org/apache/maven/plugin/surefire/AbstractSurefireMojoJava7PlusTest.java
index b3dc339..6903995 100644
--- 
a/maven-surefire-common/src/test/java/org/apache/maven/plugin/surefire/AbstractSurefireMojoJava7PlusTest.java
+++ 
b/maven-surefire-common/src/test/java/org/apache/maven/plugin/surefire/AbstractSurefireMojoJava7PlusTest.java
@@ -560,13 +560,13 @@ public class AbstractSurefireMojoJava7PlusTest
         }
 
         @Override
-        protected boolean useJigsawModules()
+        protected boolean useModulePath()
         {
             return false;
         }
 
         @Override
-        protected void setUseJigsawModules( boolean useJigsawModules )
+        protected void setUseModulePath( boolean useModulePath )
         {
 
         }
diff --git 
a/maven-surefire-common/src/test/java/org/apache/maven/plugin/surefire/AbstractSurefireMojoTest.java
 
b/maven-surefire-common/src/test/java/org/apache/maven/plugin/surefire/AbstractSurefireMojoTest.java
index 8514269..4ec49f4 100644
--- 
a/maven-surefire-common/src/test/java/org/apache/maven/plugin/surefire/AbstractSurefireMojoTest.java
+++ 
b/maven-surefire-common/src/test/java/org/apache/maven/plugin/surefire/AbstractSurefireMojoTest.java
@@ -705,13 +705,13 @@ public class AbstractSurefireMojoTest
         }
 
         @Override
-        protected boolean useJigsawModules()
+        protected boolean useModulePath()
         {
             return false;
         }
 
         @Override
-        protected void setUseJigsawModules( boolean useJigsawModules )
+        protected void setUseModulePath( boolean useModulePath )
         {
 
         }
diff --git 
a/maven-surefire-common/src/test/java/org/apache/maven/plugin/surefire/MojoMocklessTest.java
 
b/maven-surefire-common/src/test/java/org/apache/maven/plugin/surefire/MojoMocklessTest.java
index e555b00..f8ba15f 100644
--- 
a/maven-surefire-common/src/test/java/org/apache/maven/plugin/surefire/MojoMocklessTest.java
+++ 
b/maven-surefire-common/src/test/java/org/apache/maven/plugin/surefire/MojoMocklessTest.java
@@ -612,13 +612,13 @@ public class MojoMocklessTest
         }
 
         @Override
-        protected boolean useJigsawModules()
+        protected boolean useModulePath()
         {
             return false;
         }
 
         @Override
-        protected void setUseJigsawModules( boolean useJigsawModules )
+        protected void setUseModulePath( boolean useModulePath )
         {
 
         }
diff --git 
a/maven-surefire-plugin/src/main/java/org/apache/maven/plugin/surefire/SurefirePlugin.java
 
b/maven-surefire-plugin/src/main/java/org/apache/maven/plugin/surefire/SurefirePlugin.java
index 7fbbe20..6c86801 100644
--- 
a/maven-surefire-plugin/src/main/java/org/apache/maven/plugin/surefire/SurefirePlugin.java
+++ 
b/maven-surefire-plugin/src/main/java/org/apache/maven/plugin/surefire/SurefirePlugin.java
@@ -350,15 +350,15 @@ public class SurefirePlugin
     private String shutdown;
 
     /**
-     * Disables Jigsaw (Java 9) modular path even if <i>module-info.java</i> 
is used in project.
+     * Disables modular path (aka Jigsaw project since of Java 9) even if 
<i>module-info.java</i> is used in project.
      * <br>
      * Enabled by default.
      * If enabled, <i>module-info.java</i> exists and executes with JDK 9+, 
modular path is used.
      *
      * @since 3.0.0-M2
      */
-    @Parameter( property = "surefire.useJigsawModules", defaultValue = "true" )
-    private boolean useJigsawModules;
+    @Parameter( property = "surefire.useModulePath", defaultValue = "true" )
+    private boolean useModulePath;
 
     @Override
     protected int getRerunFailingTestsCount()
@@ -714,15 +714,15 @@ public class SurefirePlugin
     }
 
     @Override
-    protected boolean useJigsawModules()
+    protected boolean useModulePath()
     {
-        return useJigsawModules;
+        return useModulePath;
     }
 
     @Override
-    protected void setUseJigsawModules( boolean useJigsawModules )
+    protected void setUseModulePath( boolean useModulePath )
     {
-        this.useJigsawModules = useJigsawModules;
+        this.useModulePath = useModulePath;
     }
 
     @Override

Reply via email to