Merge pull request #3578 from MikeMcQuaid/tests-no-duplicate-seed-output

tests: don't output seed multiple times.

1 files changed, 8 insertions, 3 deletions

diff --git a/Library/Homebrew/dev-cmd/tests.rb b/Library/Homebrew/dev-cmd/tests.rb
index af9dcc575..db2a6d33e 100644
--- a/Library/Homebrew/dev-cmd/tests.rb
+++ b/Library/Homebrew/dev-cmd/tests.rb
@@ -82,17 +82,20 @@ module Homebrew
         ]
       end
 
+      # Generate seed ourselves and output later to avoid multiple different
+      # seeds being output when running parallel tests.
+      seed = ARGV.include?("--seed") ? ARGV.next : rand(0xFFFF).to_i
+
       args = ["-I", HOMEBREW_LIBRARY_PATH/"test"]
       args += %W[
+        --seed #{seed}
         --color
         --require spec_helper
-        --format progress
+        --format NoSeedProgressFormatter
         --format ParallelTests::RSpec::RuntimeLogger
         --out #{HOMEBREW_CACHE}/tests/parallel_runtime_rspec.log
       ]
 
-      args << "--seed" << ARGV.next if ARGV.include? "--seed"
-
       unless OS.mac?
         args << "--tag" << "~needs_macos"
         files = files.reject { |p| p =~ %r{^test/(os/mac|cask)(/.*|_spec\.rb)$} }
@@ -102,6 +105,8 @@ module Homebrew
         files = files.reject { |p| p =~ %r{^test/os/linux(/.*|_spec\.rb)$} }
       end
 
+      puts "Randomized with seed #{seed}"
+
       if parallel
         system "bundle", "exec", "parallel_rspec", *opts, "--", *args, "--", *files
       else