diff options
author | Xavier Noria <fxn@hashref.com> | 2019-03-08 18:03:29 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2019-03-08 18:03:29 +0100 |
commit | 32d2a63bf4c77e11f8b7930629fdc206af51c51e (patch) | |
tree | 8f3679839ebdcf5780f7abf68a82c90d0298fdcc /railties | |
parent | 199de6bee261dd816b68c841c7775fdcd02b68d2 (diff) | |
parent | 6d624f2de94623b4e94375edb882ae0ccfc0cb88 (diff) | |
download | rails-32d2a63bf4c77e11f8b7930629fdc206af51c51e.tar.gz rails-32d2a63bf4c77e11f8b7930629fdc206af51c51e.tar.bz2 rails-32d2a63bf4c77e11f8b7930629fdc206af51c51e.zip |
Merge pull request #35533 from Shopify/zeitwerk-custom-inflectors
Allow autoloader inflectors to be swaped out
Diffstat (limited to 'railties')
-rw-r--r-- | railties/lib/rails.rb | 1 | ||||
-rw-r--r-- | railties/lib/rails/autoloaders.rb | 10 | ||||
-rw-r--r-- | railties/test/application/configuration_test.rb | 4 |
3 files changed, 13 insertions, 2 deletions
diff --git a/railties/lib/rails.rb b/railties/lib/rails.rb index 1f533a8c04..663960bb77 100644 --- a/railties/lib/rails.rb +++ b/railties/lib/rails.rb @@ -6,6 +6,7 @@ require "pathname" require "active_support" require "active_support/dependencies/autoload" +require "active_support/dependencies/zeitwerk_integration" require "active_support/core_ext/kernel/reporting" require "active_support/core_ext/module/delegation" require "active_support/core_ext/array/extract_options" diff --git a/railties/lib/rails/autoloaders.rb b/railties/lib/rails/autoloaders.rb index a6974cc207..36444ef05e 100644 --- a/railties/lib/rails/autoloaders.rb +++ b/railties/lib/rails/autoloaders.rb @@ -7,13 +7,19 @@ module Rails def main if zeitwerk_enabled? - @main ||= Zeitwerk::Loader.new.tap { |loader| loader.tag = "rails.main" } + @main ||= Zeitwerk::Loader.new.tap do |loader| + loader.tag = "rails.main" + loader.inflector = ActiveSupport::Dependencies::ZeitwerkIntegration::Inflector + end end end def once if zeitwerk_enabled? - @once ||= Zeitwerk::Loader.new.tap { |loader| loader.tag = "rails.once" } + @once ||= Zeitwerk::Loader.new.tap do |loader| + loader.tag = "rails.once" + loader.inflector = ActiveSupport::Dependencies::ZeitwerkIntegration::Inflector + end end end diff --git a/railties/test/application/configuration_test.rb b/railties/test/application/configuration_test.rb index 73773602a3..9ba5517afd 100644 --- a/railties/test/application/configuration_test.rb +++ b/railties/test/application/configuration_test.rb @@ -1167,6 +1167,8 @@ module ApplicationTests assert_instance_of Zeitwerk::Loader, Rails.autoloaders.once assert_equal "rails.once", Rails.autoloaders.once.tag assert_equal [Rails.autoloaders.main, Rails.autoloaders.once], Rails.autoloaders.to_a + assert_equal ActiveSupport::Dependencies::ZeitwerkIntegration::Inflector, Rails.autoloaders.main.inflector + assert_equal ActiveSupport::Dependencies::ZeitwerkIntegration::Inflector, Rails.autoloaders.once.inflector config.autoloader = :classic assert_not Rails.autoloaders.zeitwerk_enabled? @@ -1181,6 +1183,8 @@ module ApplicationTests assert_instance_of Zeitwerk::Loader, Rails.autoloaders.once assert_equal "rails.once", Rails.autoloaders.once.tag assert_equal [Rails.autoloaders.main, Rails.autoloaders.once], Rails.autoloaders.to_a + assert_equal ActiveSupport::Dependencies::ZeitwerkIntegration::Inflector, Rails.autoloaders.main.inflector + assert_equal ActiveSupport::Dependencies::ZeitwerkIntegration::Inflector, Rails.autoloaders.once.inflector assert_raises(ArgumentError) { config.autoloader = :unknown } end |