From 46cb9ec9479b81b9dd3a3e56ec33b0eb16eee8e6 Mon Sep 17 00:00:00 2001 From: Bo Anderson Date: Sun, 27 Oct 2024 15:38:23 +0000 Subject: [PATCH] spec_helper: clarify comment and scope to all OS --- Library/Homebrew/test/spec_helper.rb | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/Library/Homebrew/test/spec_helper.rb b/Library/Homebrew/test/spec_helper.rb index fb2922c699..831d8f7404 100644 --- a/Library/Homebrew/test/spec_helper.rb +++ b/Library/Homebrew/test/spec_helper.rb @@ -10,10 +10,13 @@ if ENV["HOMEBREW_TESTS_COVERAGE"] ] 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