spec_helper: clarify comment and scope to all OS

Homebrew · Oct 27, 2024 · 46cb9ec · 46cb9ec
1 parent 92fee90
commit 46cb9ec
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/Library/Homebrew/test/spec_helper.rb b/Library/Homebrew/test/spec_helper.rb
@@ -10,10 +10,13 @@
   ]
   SimpleCov.formatters = SimpleCov::Formatter::MultiFormatter.new(formatters)
 
-  # Needed for outputting coverage reporting only once for parallel_tests
-  if RUBY_PLATFORM[/darwin/] && ENV["TEST_ENV_NUMBER"]
+  # Needed for outputting coverage reporting only once for parallel_tests.
+  # Otherwise, "Coverage report generated" will get spammed for each process.
+  if ENV["TEST_ENV_NUMBER"]
     SimpleCov.at_exit do
       result = SimpleCov.result
+      # `SimpleCov.result` calls `ParallelTests.wait_for_other_processes_to_finish`
+      # internally for you on the last process.
       result.format! if ParallelTests.last_process?
     end
   end