[SUREFIRE-2055] Always show random seed

apache · Apr 5, 2022 · 0847b1d · 0847b1d
1 parent daf717e
commit 0847b1d
Show file tree

Hide file tree

Showing 2 changed files with 26 additions and 3 deletions.
diff --git a/...-surefire-common/src/main/java/org/apache/maven/plugin/surefire/AbstractSurefireMojo.java b/...-surefire-common/src/main/java/org/apache/maven/plugin/surefire/AbstractSurefireMojo.java
@@ -3114,9 +3114,12 @@ protected void warnIfIllegalFailOnFlakeCount() throws MojoFailureException
 
     private void printDefaultSeedIfNecessary()
     {
-        if ( getRunOrderRandomSeed() == null && getRunOrder().equals( RunOrder.RANDOM.name() ) )
+        if ( getRunOrder().equals( RunOrder.RANDOM.name() ) )
         {
-            setRunOrderRandomSeed( System.nanoTime() );
+            if ( getRunOrderRandomSeed() == null )
+            {
+                setRunOrderRandomSeed( System.nanoTime() );
+            }
             getConsoleLogger().info(
                 "Tests will run in random order. To reproduce ordering use flag -D"
                     + getPluginName() + ".runOrder.random.seed=" + getRunOrderRandomSeed() );

diff --git a/surefire-its/src/test/java/org/apache/maven/surefire/its/RunOrderIT.java b/surefire-its/src/test/java/org/apache/maven/surefire/its/RunOrderIT.java
@@ -97,7 +97,17 @@ public void testRandomJUnit4SameSeed()
             }
         }
     }
-
+
+    @Test
+    public void testRandomJUnit4PrintSeed()
+    {
+        long seed = 0L;
+        OutputValidator validator = executeWithRandomOrder( "junit4", seed );
+        validator.verifyTextInLog( "To reproduce ordering use flag" );
+        validator = executeWithRandomOrder( "junit4" );
+        validator.verifyTextInLog( "To reproduce ordering use flag" );
+    }
+
     @Test
     public void testReverseAlphabeticalJUnit4()
         throws Exception
@@ -187,6 +197,16 @@ private OutputValidator executeWithRunOrder( String runOrder, String profile )
             .verifyErrorFree( 3 );
     }
 
+    private OutputValidator executeWithRandomOrder( String profile )
+    {
+        return unpack()
+            .activateProfile( profile )
+            .forkMode( getForkMode() )
+            .runOrder( "random" )
+            .executeTest()
+            .verifyErrorFree( 3 );
+    }
+
     private OutputValidator executeWithRandomOrder( String profile, long seed  )
     {
         return unpack()