Merge pull request #26589 from kirs/ad-test-request-method

Fix memoization bug on ActionDispatch::TestRequest#request_method=

actionpack/lib/action_dispatch/testing/test_request.rb

@@ -22,7 +22,7 @@ def self.default_env
     private_class_method :default_env
 
     def request_method=(method)
-      set_header("REQUEST_METHOD", method.to_s.upcase)
+      super(method.to_s.upcase)
     end
 
     def host=(host)

actionpack/test/dispatch/test_request_test.rb

@@ -88,6 +88,13 @@ class TestRequestTest < ActiveSupport::TestCase
     assert_equal "GoogleBot", req.user_agent
   end
 
+  test "request_method getter and setter" do
+    req = ActionDispatch::TestRequest.create
+    req.request_method # to reproduce bug caused by memoization
+    req.request_method = "POST"
+    assert_equal "POST", req.request_method
+  end
+
   test "setter methods" do
     req = ActionDispatch::TestRequest.create({})
     get = "GET"