-
Notifications
You must be signed in to change notification settings - Fork 21.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Minitest upgrade broke build #13213
Minitest upgrade broke build #13213
Conversation
@@ -9,7 +9,7 @@ module Isolation | |||
def self.included(klass) #:nodoc: | |||
klass.extend(Module.new { | |||
def test_methods | |||
ParallelEach.new super | |||
Parallel.new super |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, but I'm not sure that's gonna work, it seems the API has changed and Parallel
is now a module. Maybe we should make use of the parallelize_me! method, not sure.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ahh yes
/cc @tenderlove |
This seems to be another fix for both 1.9.3 and 2.0.0 (which wasn't broken for me). I'm not sure if it's actually running in parallel or not, but the suite passes again. diff --git a/activesupport/lib/active_support/testing/isolation.rb b/activesupport/lib/active_support/testing/isolation.rb
index d5d31ce..18c48a7 100644
--- a/activesupport/lib/active_support/testing/isolation.rb
+++ b/activesupport/lib/active_support/testing/isolation.rb
@@ -1,5 +1,5 @@
require 'rbconfig'
-require 'minitest/parallel_each'
+require 'minitest/parallel'
module ActiveSupport
module Testing |
Doh, ignore me. @arunagw already had that in his diff. Sorry about the noise. |
minitest/minitest@34760e3 This PR should fix build
Minitest upgrade broke build
We made a change in rails#13213 that depends on a new file that's only in minitest 5.1.0+, so the version should be updated.
minitest/minitest@34760e3
This PR should fix build