Merge pull request #40172 from p8/shorten-inspect-on-controller

Shorten inspect on AbstractController::Base

actionpack/lib/abstract_controller/base.rb

@@ -204,6 +204,10 @@ def self.supports_path?
       true
     end
 
+    def inspect # :nodoc:
+      "#<#{self.class.name}:#{'%#016x' % (object_id << 1)}>"
+    end
+
     private
       # Returns true if the name can be considered an action because
       # it has a method defined in the controller.

actionpack/test/controller/base_test.rb

@@ -147,6 +147,10 @@ def test_response_has_default_headers
   ensure
     ActionDispatch::Response.default_headers = original_default_headers
   end
+
+  def test_inspect
+    assert_match(/\A#<EmptyController:0x[0-9a-f]+>\z/, @empty.inspect)
+  end
 end
 
 class PerformActionTest < ActionController::TestCase

actionpack/test/controller/metal_test.rb

@@ -29,4 +29,9 @@ def test_response_does_not_have_default_headers
   ensure
     ActionDispatch::Response.default_headers = original_default_headers
   end
+
+  def test_inspect
+    controller = SimpleController.new
+    assert_match(/\A#<MetalControllerInstanceTests::SimpleController:0x[0-9a-f]+>\z/, controller.inspect)
+  end
 end