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 2018/12/02 09:15:03 UTC

[maven-surefire] branch master updated: removed a hint from console warning

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 59d8f5b  removed a hint from console warning
59d8f5b is described below

commit 59d8f5be8fcd4b04d78e215ffc0fd33a5d7e2c77
Author: Tibor17 <ti...@apache.org>
AuthorDate: Sun Dec 2 10:14:20 2018 +0100

    removed a hint from console warning
---
 .../java/org/apache/maven/plugin/surefire/AbstractSurefireMojo.java   | 4 ++--
 .../maven/plugin/surefire/AbstractSurefireMojoJava7PlusTest.java      | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

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 6f973b8..b18463e 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
@@ -1832,9 +1832,9 @@ public abstract class AbstractSurefireMojo
         ResolvePathsResult<String> result = getLocationManager().resolvePaths( req );
         for ( Entry<String, Exception> entry : result.getPathExceptions().entrySet() )
         {
+            // Probably JDK version < 9. Other known causes: passing a non-jar or a corrupted jar.
             getConsoleLogger()
-                    .warning( "Exception for '" + entry.getKey() + "' (probably JDK version < 9).",
-                            entry.getValue() );
+                    .warning( "Exception for '" + entry.getKey() + "'.", entry.getValue() );
         }
 
         testClasspath = new Classpath( result.getClasspathElements() );
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 d2ef230..17cd930 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
@@ -174,7 +174,7 @@ public class AbstractSurefireMojoJava7PlusTest
         ArgumentCaptor<Exception> argument2 = ArgumentCaptor.forClass( Exception.class );
         verify( logger, times( 1 ) ).warn( argument1.capture(), argument2.capture() );
         assertThat( argument1.getValue() )
-                .isEqualTo( "Exception for 'java 1.8' (probably JDK version < 9)." );
+                .isEqualTo( "Exception for 'java 1.8'." );
         assertThat( argument2.getValue().getMessage() )
                 .isEqualTo( "low version" );
         ArgumentCaptor<String> argument = ArgumentCaptor.forClass( String.class );