Reapply "Merge pull request rails#27586 from maclover7/jm-fix-27584"

This reverts commit 6902ca5.

This was originally reverted due to a reported regression, however the
regression appeared to be present without this patch as well (and was
fixed in a later [commit][1].

[1]: ca937c5

Author: skipkayhil

actionpack/lib/action_controller/test_case.rb

@@ -429,9 +429,7 @@ def determine_default_controller_class(name)
       # Note that the request method is not verified. The different methods are
       # available to make the tests more expressive.
       def get(action, **args)
-        res = process(action, method: "GET", **args)
-        cookies.update res.cookies
-        res
+        process(action, method: "GET", **args)
       end
 
       # Simulate a POST request with the given parameters and set/volley the response.
@@ -639,6 +637,7 @@ def process_controller_response(action, cookies, xhr)
               unless @request.cookie_jar.committed?
                 @request.cookie_jar.write(@response)
                 cookies.update(@request.cookie_jar.instance_variable_get(:@cookies))
+                cookies.update(@response.cookies)
               end
             end
             @response.prepare!

actionpack/test/dispatch/cookies_test.rb

@@ -583,6 +583,15 @@ def test_deleted_cookie_predicate
     assert_equal false, cookies.deleted?("another")
   end
 
+  # Ensure all HTTP methods have their cookies updated
+  [:get, :post, :patch, :put, :delete, :head].each do |method|
+    define_method("test_deleting_cookie_#{method}") do
+      request.cookies[:user_name] = "Joe"
+      public_send method, :logout
+      assert_nil cookies[:user_name]
+    end
+  end
+
   def test_deleted_cookie_predicate_with_mismatching_options
     cookies[:user_name] = "Joe"
     cookies.delete("user_name", path: "/path")