spec_helper: clarify comment and scope to all OS

Homebrew · Oct 27, 2024 · c7a47b1 · c7a47b1
1 parent 92fee90
commit c7a47b1
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/Library/Homebrew/test/spec_helper.rb b/Library/Homebrew/test/spec_helper.rb
@@ -10,10 +10,12 @@
   ]
   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 calls `ParallelTests.wait_for_other_processes_to_finish` internally for you.
       result.format! if ParallelTests.last_process?
     end
   end