using __method__ for the command method calls

2 files changed, 64 insertions, 5 deletions

diff --git a/railties/lib/rails/configuration.rb b/railties/lib/rails/configuration.rb
index 3d66019e5e..5fa7f043c6 100644
--- a/railties/lib/rails/configuration.rb
+++ b/railties/lib/rails/configuration.rb
@@ -39,25 +39,25 @@ module Rails
       end
 
       def insert_before(*args, &block)
-        @operations << [:insert_before, args, block]
+        @operations << [__method__, args, block]
       end
 
       alias :insert :insert_before
 
       def insert_after(*args, &block)
-        @operations << [:insert_after, args, block]
+        @operations << [__method__, args, block]
       end
 
       def swap(*args, &block)
-        @operations << [:swap, args, block]
+        @operations << [__method__, args, block]
       end
 
       def use(*args, &block)
-        @operations << [:use, args, block]
+        @operations << [__method__, args, block]
       end
 
       def delete(*args, &block)
-        @operations << [:delete, args, block]
+        @operations << [__method__, args, block]
       end
 
       def merge_into(other) #:nodoc:
diff --git a/railties/test/configuration/middleware_stack_proxy_test.rb b/railties/test/configuration/middleware_stack_proxy_test.rb
new file mode 100644
index 0000000000..5984c0b425
--- /dev/null
+++ b/railties/test/configuration/middleware_stack_proxy_test.rb
@@ -0,0 +1,59 @@
+require 'minitest/autorun'
+require 'rails/configuration'
+require 'active_support/test_case'
+
+module Rails
+  module Configuration
+    class MiddlewareStackProxyTest < ActiveSupport::TestCase
+      def setup
+        @stack = MiddlewareStackProxy.new
+      end
+
+      def test_playback_insert_before
+        @stack.insert_before :foo
+        assert_playback :insert_before, :foo
+      end
+
+      def test_playback_insert_after
+        @stack.insert_after :foo
+        assert_playback :insert_after, :foo
+      end
+
+      def test_playback_swap
+        @stack.swap :foo
+        assert_playback :swap, :foo
+      end
+
+      def test_playback_use
+        @stack.use :foo
+        assert_playback :use, :foo
+      end
+
+      def test_playback_delete
+        @stack.delete :foo
+        assert_playback :delete, :foo
+      end
+
+      def test_order
+        @stack.swap :foo
+        @stack.delete :foo
+
+        mock = MiniTest::Mock.new
+        mock.expect :send, nil, [:swap, :foo]
+        mock.expect :send, nil, [:delete, :foo]
+
+        @stack.merge_into mock
+        mock.verify
+      end
+
+      private
+
+      def assert_playback(msg_name, args)
+        mock = MiniTest::Mock.new
+        mock.expect :send, nil, [msg_name, args]
+        @stack.merge_into(mock)
+        mock.verify
+      end
+    end
+  end
+end