Replace autoloader accessors with Rails.autoloaders.{main,once}

Rails.autoloader and Rails.once_autoloader was just tentative API good
enough for a first patch. Rails.autoloader is singular and does not
convey in its name that there is another autoloader. That might be
confusing, for example if you set a logger and miss traces. On the other
hand, the name `once_autoloader` is very close to being horrible.

Rails.autoloaders.main and Rails.autoloaders.once read better for my
taste, and have a nice symmetry. Also, both "main" and "once" are four
letters long, short and same length.

They are tagged as "rails.main" and "rails.once", respectively.

References #35235.

1 files changed, 17 insertions, 10 deletions

diff --git a/railties/test/application/configuration_test.rb b/railties/test/application/configuration_test.rb
index f3161ad2a8..73773602a3 100644
--- a/railties/test/application/configuration_test.rb
+++ b/railties/test/application/configuration_test.rb
@@ -1157,23 +1157,30 @@ module ApplicationTests
       end
     end
 
-    test "autoloader & autoloader=" do
+    test "autoloaders" do
       app "development"
 
       config = Rails.application.config
-      assert_instance_of Zeitwerk::Loader, Rails.autoloader
-      assert_instance_of Zeitwerk::Loader, Rails.once_autoloader
-      assert_equal [Rails.autoloader, Rails.once_autoloader], Rails.autoloaders
+      assert Rails.autoloaders.zeitwerk_enabled?
+      assert_instance_of Zeitwerk::Loader, Rails.autoloaders.main
+      assert_equal "rails.main", Rails.autoloaders.main.tag
+      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
 
       config.autoloader = :classic
-      assert_nil Rails.autoloader
-      assert_nil Rails.once_autoloader
-      assert_empty Rails.autoloaders
+      assert_not Rails.autoloaders.zeitwerk_enabled?
+      assert_nil Rails.autoloaders.main
+      assert_nil Rails.autoloaders.once
+      assert_equal 0, Rails.autoloaders.count
 
       config.autoloader = :zeitwerk
-      assert_instance_of Zeitwerk::Loader, Rails.autoloader
-      assert_instance_of Zeitwerk::Loader, Rails.once_autoloader
-      assert_equal [Rails.autoloader, Rails.once_autoloader], Rails.autoloaders
+      assert Rails.autoloaders.zeitwerk_enabled?
+      assert_instance_of Zeitwerk::Loader, Rails.autoloaders.main
+      assert_equal "rails.main", Rails.autoloaders.main.tag
+      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_raises(ArgumentError) { config.autoloader = :unknown }
     end