Fixes perf improvement breaking ActionController::TestCase

Co-Authored-By: Hartley McGuire <[email protected]>

Author: Schwad

actionpack/lib/action_dispatch/http/request.rb

@@ -159,6 +159,7 @@ def request_method
     def route_uri_pattern
       unless pattern = get_header("action_dispatch.route_uri_pattern")
         route = get_header("action_dispatch.route")
+        return if route.nil?
         pattern = route.path.spec.to_s
         set_header("action_dispatch.route_uri_pattern", pattern)
       end

actionpack/test/controller/base_test.rb

@@ -72,6 +72,29 @@ def action_missing(action)
   end
 end
 
+class WithoutRouterController < ActionController::Base
+  after_action :log_request_details
+
+  def index
+    head :ok
+  end
+
+  private
+    def log_request_details
+      request.route_uri_pattern
+    end
+end
+
+class WithoutRouterTest < ActionController::TestCase
+  tests WithoutRouterController
+
+  def test_request_route_uri_pattern_in_after_action_callback
+    assert_nothing_raised do
+      get :index
+    end
+  end
+end
+
 class ControllerClassTests < ActiveSupport::TestCase
   def test_controller_path
     assert_equal "empty", EmptyController.controller_path