Merge pull request #36018 from aricwalker/parallel-setup-exception

Catch exceptions raised in `parallelize_setup` and ensure tests fail when they occur

1 files changed, 10 insertions, 1 deletions

diff --git a/activesupport/lib/active_support/testing/parallelization.rb b/activesupport/lib/active_support/testing/parallelization.rb
index 08285b2f52..e760bf5ce3 100644
--- a/activesupport/lib/active_support/testing/parallelization.rb
+++ b/activesupport/lib/active_support/testing/parallelization.rb
@@ -71,7 +71,9 @@ module ActiveSupport
           fork do
             DRb.stop_service
 
-            after_fork(worker)
+            begin
+              after_fork(worker)
+            rescue => setup_exception; end
 
             queue = DRbObject.new_with_uri(@url)
 
@@ -83,6 +85,8 @@ module ActiveSupport
                 Minitest.run_one_method(klass, method)
               end
 
+              add_setup_exception(result, setup_exception) if setup_exception
+
               begin
                 queue.record(reporter, result)
               rescue DRb::DRbConnError
@@ -106,6 +110,11 @@ module ActiveSupport
         @queue_size.times { @queue << nil }
         @pool.each { |pid| Process.waitpid pid }
       end
+
+      private
+        def add_setup_exception(result, setup_exception)
+          result.failures.prepend Minitest::UnexpectedError.new(setup_exception)
+        end
     end
   end
 end