Ensure that status codes are logged properly

Needed to move AC::Metal::Instrumentation before AM::Metal::Rescue
so that status codes rendered from rescue_from blocks are logged
properly.

1 files changed, 19 insertions, 0 deletions

diff --git a/actionpack/test/controller/log_subscriber_test.rb b/actionpack/test/controller/log_subscriber_test.rb
index 80c4fa2ee5..ccdfcb0b2c 100644
--- a/actionpack/test/controller/log_subscriber_test.rb
+++ b/actionpack/test/controller/log_subscriber_test.rb
@@ -6,6 +6,13 @@ module Another
   class LogSubscribersController < ActionController::Base
     wrap_parameters :person, :include => :name, :format => :json
 
+    class SpecialException < Exception
+    end
+
+    rescue_from SpecialException do
+      head :status => 406
+    end
+
     def show
       render :nothing => true
     end
@@ -39,6 +46,10 @@ module Another
       raise Exception
     end
 
+    def with_rescued_exception
+      raise SpecialException
+    end
+
   end
 end
 
@@ -195,6 +206,14 @@ class ACLogSubscriberTest < ActionController::TestCase
     assert_match(/Completed 500/, logs.last)
   end
 
+  def test_process_action_with_rescued_exception_includes_http_status_code
+    get :with_rescued_exception
+    wait
+
+    assert_equal 2, logs.size
+    assert_match(/Completed 406/, logs.last)
+  end
+
   def logs
     @logs ||= @logger.logged(:info)
   end