[MENFORCER-323] - Improve output of "Some plugins are missing valid versions" error master
authorJosh Soref <jsoref@users.noreply.github.com>
Mon, 10 Dec 2018 19:39:20 +0000 (14:39 -0500)
committerEnrico Olivelli <eolivelli@apache.org>
Wed, 12 Dec 2018 17:31:39 +0000 (18:31 +0100)
Fix whitespaces around brackets in case of "Some plugins are missing valid versions"

enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequirePluginVersions.java
maven-enforcer-plugin/src/it/projects/require-plugin-versions-ci/verify.groovy

index f3df358..b63af6e 100644 (file)
@@ -265,7 +265,7 @@ public class RequirePluginVersions
                 newMsg.append( "Some plugins are missing valid versions:" );
                 if ( banLatest || banRelease || banSnapshots || banTimestamps )
                 {
-                    newMsg.append( "(" );
+                    newMsg.append( " (" );
                     if ( banLatest )
                     {
                         newMsg.append( "LATEST " );
@@ -278,8 +278,9 @@ public class RequirePluginVersions
                     {
                         newMsg.append( "SNAPSHOT " );
                     }
-                    newMsg.append( "are not allowed )\n" );
+                    newMsg.append( "are not allowed)" );
                 }
+                newMsg.append( "\n" );
                 for ( Plugin plugin : failures )
                 {
                     newMsg.append( plugin.getGroupId() );
@@ -1262,4 +1263,4 @@ public class RequirePluginVersions
     {
         return additionalPlugins;
     }
-}
\ No newline at end of file
+}
index 865bfeb..d098917 100644 (file)
@@ -18,4 +18,4 @@
  */
 File buildLog = new File( basedir, 'build.log' )
 assert buildLog.text.contains( '[WARNING] Rule 0: org.apache.maven.plugins.enforcer.RequirePluginVersions failed with message:' )
-assert buildLog.text.contains( 'Some plugins are missing valid versions:(LATEST RELEASE SNAPSHOT are not allowed )' )
+assert buildLog.text.contains( 'Some plugins are missing valid versions: (LATEST RELEASE SNAPSHOT are not allowed)' )