Deprecate Module#reachable? method

activesupport/CHANGELOG.md

@@ -1,3 +1,7 @@
+*   Deprecate `Module#reachable?` method.
+
+    *bogdanvlviv*
+
 *   Add `config/credentials.yml.enc` to store production app secrets.
 
     Allows saving any authentication credentials for third party services

activesupport/lib/active_support/core_ext/module/reachable.rb

@@ -7,4 +7,5 @@ class Module
   def reachable? #:nodoc:
     !anonymous? && name.safe_constantize.equal?(self)
   end
+  deprecate :reachable?
 end

activesupport/test/core_ext/module/reachable_test.rb

@@ -5,14 +5,18 @@ require "active_support/core_ext/module/reachable"
 
 class AnonymousTest < ActiveSupport::TestCase
   test "an anonymous class or module is not reachable" do
+    assert_deprecated do
       assert !Module.new.reachable?
       assert !Class.new.reachable?
     end
+  end
 
   test "ordinary named classes or modules are reachable" do
+    assert_deprecated do
       assert Kernel.reachable?
       assert Object.reachable?
     end
+  end
 
   test "a named class or module whose constant has gone is not reachable" do
     c = eval "class C; end; C"
@@ -21,9 +25,11 @@ class AnonymousTest < ActiveSupport::TestCase
     self.class.send(:remove_const, :C)
     self.class.send(:remove_const, :M)
 
+    assert_deprecated do
       assert !c.reachable?
       assert !m.reachable?
     end
+  end
 
   test "a named class or module whose constants store different objects are not reachable" do
     c = eval "class C; end; C"
@@ -35,9 +41,11 @@ class AnonymousTest < ActiveSupport::TestCase
     eval "class C; end"
     eval "module M; end"
 
+    assert_deprecated do
       assert C.reachable?
       assert M.reachable?
       assert !c.reachable?
       assert !m.reachable?
     end
   end
+end