commit 18e633dda4584739c404b245353b0a20db3f6f28
author Marc Wrobel <marc.wrobel@gmail.com> Tue Jul 19 21:03:12 2022 +0200
committer Michael Osipov <michaelo@apache.org> Tue Jul 26 19:52:21 2022 +0200
tree d2dff5c52af78313f162fcb4830b4f80da5b18ae
parent eff36ca9fed296136629c9d633dde08a18cc0458

[MNG-7517] Improve MavenITmng6391PrintVersionTest tests resiliency

Maven 4.0.x has some new code which takes care of the terminal width to adjust
the output. The number of dots can thus vary when running the tests locally.

Co-authored-by: Guillaume Nodet <gnodet@gmail.com>

This closes #187 and closes #175.

core-it-suite/src/test/java/org/apache/maven/it/MavenITmng6391PrintVersionTest.java

diff --git a/core-it-suite/src/test/java/org/apache/maven/it/MavenITmng6391PrintVersionTest.java b/core-it-suite/src/test/java/org/apache/maven/it/MavenITmng6391PrintVersionTest.java
index 94afe39..828351c 100644
--- a/core-it-suite/src/test/java/org/apache/maven/it/MavenITmng6391PrintVersionTest.java
+++ b/core-it-suite/src/test/java/org/apache/maven/it/MavenITmng6391PrintVersionTest.java
@@ -73,11 +73,19 @@
         assertEquals( 5, resultingLines.size() );
 
         // We expect those lines in the following exact order.
+        // Maven 4.0.x has some code new code which takes care of the terminal width to adjust the output.
+        // The number of dots can thus vary when running the tests.
         assertTrue( resultingLines.get( 0 ).startsWith( "[INFO] Reactor Summary for base-project 1.3.0-SNAPSHOT:" ) );
-        assertTrue( resultingLines.get( 1 ).startsWith( "[INFO] base-project ....................................... SUCCESS [" ) );
-        assertTrue( resultingLines.get( 2 ).startsWith( "[INFO] module-1 ........................................... SUCCESS [" ) );
-        assertTrue( resultingLines.get( 3 ).startsWith( "[INFO] module-2 ........................................... SUCCESS [" ) );
-        assertTrue( resultingLines.get( 4 ).startsWith( "[INFO] module-3 ........................................... SUCCESS [" ) );
+        assertTrue( resultingLines.get( 1 ).matches( "\\Q[INFO] base-project ............\\E\\.+\\Q SUCCESS [\\E.*" ) );
+        assertTrue( resultingLines.get( 2 ).matches( "\\Q[INFO] module-1 ................\\E\\.+\\Q SUCCESS [\\E.*" ) );
+        assertTrue( resultingLines.get( 3 ).matches( "\\Q[INFO] module-2 ................\\E\\.+\\Q SUCCESS [\\E.*" ) );
+        assertTrue( resultingLines.get( 4 ).matches( "\\Q[INFO] module-3 ................\\E\\.+\\Q SUCCESS [\\E.*" ) );
+
+        // We expect that line 1..4 have the same length
+        int line1Length = resultingLines.get( 1 ).length();
+        assertEquals( line1Length, resultingLines.get( 2 ).length() );
+        assertEquals( line1Length, resultingLines.get( 3 ).length() );
+        assertEquals( line1Length, resultingLines.get( 4 ).length() );
 
     }
 
@@ -109,11 +117,19 @@
         assertEquals( 5, resultingLines.size() );
 
         // We expect those lines in the following exact order.
+        // Maven 4.0.x has some code new code which takes care of the terminal width to adjust the output.
+        // The number of dots can thus vary when running the tests.
         assertTrue( resultingLines.get( 0 ).startsWith( "[INFO] Reactor Summary:" ) );
-        assertTrue( resultingLines.get( 1 ).startsWith( "[INFO] module-1 1.2.7.43.RELEASE .......................... SUCCESS [  " ) );
-        assertTrue( resultingLines.get( 2 ).startsWith( "[INFO] module-2 7.5-SNAPSHOT .............................. SUCCESS [  " ) );
-        assertTrue( resultingLines.get( 3 ).startsWith( "[INFO] module-3 1-RC1 ..................................... SUCCESS [  " ) );
-        assertTrue( resultingLines.get( 4 ).startsWith( "[INFO] base-project 1.0.0-SNAPSHOT ........................ SUCCESS [  " ) );
+        assertTrue( resultingLines.get( 1 ).matches( "\\Q[INFO] module-1 1.2.7.43.RELEASE ...............\\E\\.+\\Q SUCCESS [ \\E.*" ) );
+        assertTrue( resultingLines.get( 2 ).matches( "\\Q[INFO] module-2 7.5-SNAPSHOT ...................\\E\\.+\\Q SUCCESS [ \\E.*" ) );
+        assertTrue( resultingLines.get( 3 ).matches( "\\Q[INFO] module-3 1-RC1 ..........................\\E\\.+\\Q SUCCESS [ \\E.*" ) );
+        assertTrue( resultingLines.get( 4 ).matches( "\\Q[INFO] base-project 1.0.0-SNAPSHOT .............\\E\\.+\\Q SUCCESS [ \\E.*" ) );
+
+        // We expect that line 1..4 have the same length
+        int line1Length = resultingLines.get( 1 ).length();
+        assertEquals( line1Length, resultingLines.get( 2 ).length() );
+        assertEquals( line1Length, resultingLines.get( 3 ).length() );
+        assertEquals( line1Length, resultingLines.get( 4 ).length() );
 
     }