From c519d1c2ef28f42342ca4446b8211770b4f551cb Mon Sep 17 00:00:00 2001 From: Kasper Timm Hansen Date: Tue, 22 Mar 2016 21:51:47 +0100 Subject: [ci skip] Improve comment minitest plugin initialization. Didn't feel we were clear enough about our motivation for placing Rails first and why we needed to call `load_plugins`. --- railties/lib/rails/test_unit/minitest_plugin.rb | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) (limited to 'railties') diff --git a/railties/lib/rails/test_unit/minitest_plugin.rb b/railties/lib/rails/test_unit/minitest_plugin.rb index 5a24bcf288..e9195d5b4e 100644 --- a/railties/lib/rails/test_unit/minitest_plugin.rb +++ b/railties/lib/rails/test_unit/minitest_plugin.rb @@ -93,7 +93,9 @@ module Minitest mattr_accessor(:run_with_rails_extension) { false } end -# Let libraries override our reporter setup by loading other plugins then -# setting ourselves as the first plugin to be initialized. +# Put Rails as the first plugin minitest initializes so other plugins +# can override or replace our default reporter setup. +# Since minitest only loads plugins if its extensions are empty we have +# to call `load_plugins` first. Minitest.load_plugins Minitest.extensions.unshift 'rails' -- cgit v1.2.3