Delete AS::Dependencies.safe_constantize

Author: fxn

activerecord/lib/active_record/inheritance.rb

@@ -239,7 +239,7 @@ def compute_type(type_name)
             type_name.constantize
           else
             type_candidate = @_type_candidates_cache[type_name]
-            if type_candidate && type_constant = ActiveSupport::Dependencies.safe_constantize(type_candidate)
+            if type_candidate && type_constant = type_candidate.safe_constantize
               return type_constant
             end
 
@@ -249,7 +249,7 @@ def compute_type(type_name)
             candidates << type_name
 
             candidates.each do |candidate|
-              constant = ActiveSupport::Dependencies.safe_constantize(candidate)
+              constant = candidate.safe_constantize
               if candidate == constant.to_s
                 @_type_candidates_cache[type_name] = candidate
                 return constant

activerecord/test/cases/inheritance_test.rb

@@ -1,5 +1,6 @@
 # frozen_string_literal: true
 
+require "active_support/inflector"
 require "cases/helper"
 require "models/author"
 require "models/company"
@@ -68,37 +69,33 @@ def test_compute_type_nonexistent_constant
   end
 
   def test_compute_type_no_method_error
-    ActiveSupport::Dependencies.stub(:safe_constantize, proc { raise NoMethodError }) do
-      assert_raises NoMethodError do
-        Company.send :compute_type, "InvalidModel"
-      end
-    end
+    # Done via autoload because you need the exception to happen when the file
+    # is required.
+    model = "RaisesNoMethodError"
+    Object.autoload(model, "#{MODELS_ROOT}/#{model.underscore}")
+    assert_raises(NoMethodError) { Company.send(:compute_type, model) }
+  ensure
+    Object.send(:remove_const, model)
   end
 
   def test_compute_type_on_undefined_method
-    error = nil
-    begin
-      Class.new(Author) do
-        alias_method :foo, :bar
-      end
-    rescue => e
-      error = e
-    end
-
-    ActiveSupport::Dependencies.stub(:safe_constantize, proc { raise e }) do
-      exception = assert_raises NameError do
-        Company.send :compute_type, "InvalidModel"
-      end
-      assert_equal error.message, exception.message
-    end
+    # Done via autoload because you need the exception to happen when the file
+    # is required.
+    model = "InvokesAnUndefinedMethod"
+    Object.autoload(model, "#{MODELS_ROOT}/#{model.underscore}")
+    assert_raises(NameError) { Company.send(:compute_type, model) }
+  ensure
+    Object.send(:remove_const, model)
   end
 
   def test_compute_type_argument_error
-    ActiveSupport::Dependencies.stub(:safe_constantize, proc { raise ArgumentError }) do
-      assert_raises ArgumentError do
-        Company.send :compute_type, "InvalidModel"
-      end
-    end
+    # Done via autoload because you need the exception to happen when the file
+    # is required.
+    model = "RaisesArgumentError"
+    Object.autoload(model, "#{MODELS_ROOT}/#{model.underscore}")
+    assert_raises(ArgumentError) { Company.send(:compute_type, model) }
+  ensure
+    Object.send(:remove_const, model)
   end
 
   def test_should_store_demodulized_class_name_with_store_full_sti_class_option_disabled

activerecord/test/config.rb

@@ -3,5 +3,6 @@
 TEST_ROOT       = __dir__
 ASSETS_ROOT     = TEST_ROOT + "/assets"
 FIXTURES_ROOT   = TEST_ROOT + "/fixtures"
+MODELS_ROOT     = TEST_ROOT + "/models"
 MIGRATIONS_ROOT = TEST_ROOT + "/migrations"
 SCHEMA_ROOT     = TEST_ROOT + "/schema"

activerecord/test/models/invokes_an_undefined_method.rb

@@ -0,0 +1,5 @@
+# frozen_string_literal: true
+
+class InvokesAnUndefinedMethod
+  this_method_is_undefined
+end

activerecord/test/models/raises_argument_error.rb

@@ -0,0 +1,5 @@
+# frozen_string_literal: true
+
+class RaisesArgumentError
+  raise ArgumentError
+end

activerecord/test/models/raises_no_method_error.rb

@@ -0,0 +1,5 @@
+# frozen_string_literal: true
+
+class RaisesNoMethodError
+  Object.new.calling_a_non_existing_method
+end

activesupport/lib/active_support/dependencies.rb

@@ -3,7 +3,6 @@
 require "set"
 require "active_support/core_ext/module/attribute_accessors"
 require "active_support/dependencies/interlock"
-require "active_support/inflector"
 
 module ActiveSupport # :nodoc:
   module Dependencies # :nodoc:
@@ -82,12 +81,6 @@ def search_for_file(path_suffix)
       nil # Gee, I sure wish we had first_match ;-)
     end
 
-    # Get the reference for class named +name+ if one exists.
-    # Otherwise returns +nil+.
-    def safe_constantize(name)
-      Inflector.safe_constantize(name)
-    end
-
     # Determine if the given constant has been automatically loaded.
     def autoloaded?(desc)
       return false if desc.is_a?(Module) && real_mod_name(desc).nil?

activesupport/lib/active_support/dependencies/zeitwerk_integration.rb

@@ -1,7 +1,6 @@
 # frozen_string_literal: true
 
 require "set"
-require "active_support/core_ext/string/inflections"
 require "zeitwerk"
 
 module ActiveSupport
@@ -16,10 +15,6 @@ def clear
           end
         end
 
-        def safe_constantize(cpath)
-          ActiveSupport::Inflector.safe_constantize(cpath)
-        end
-
         def autoloaded_constants
           Rails.autoloaders.main.unloadable_cpaths
         end

railties/test/application/zeitwerk_integration_test.rb

@@ -61,19 +61,6 @@ class RESTfulController < ApplicationController
     assert RESTfulController
   end
 
-  test "safe_constantize returns the value stored in the constant" do
-    app_file "app/models/admin/user.rb", "class Admin::User; end"
-    boot
-
-    assert_same Admin::User, deps.safe_constantize("Admin::User")
-  end
-
-  test "safe_constantize returns nil for unknown constants" do
-    boot
-
-    assert_nil deps.safe_constantize("Admin")
-  end
-
   test "autoloaded? and overridden class names" do
     invalid_constant_name = Module.new do
       def self.name