mirror of
https://github.com/rails/rails.git
synced 2022-11-09 12:12:34 -05:00
ea5f509643
Since #35709, `Response#conten_type` returns only MIME type correctly.
It is a documented behavior that this method only returns MIME type, so
this change seems appropriate.
39de7fac05/actionpack/lib/action_dispatch/http/response.rb (L245-L249)
But unfortunately, some users expect this method to return all
Content-Type that does not contain charset. This seems to be breaking
changes.
We can change this behavior with the deprecate cycle.
But, in that case, a method needs that include Content-Type with
additional parameters. And that method name is probably the
`content_type` seems to properly.
So I changed the new behavior to more appropriate `media_type` method.
And `Response#content_type` changed (as the method name) to return Content-Type
header as it is.
Fixes #35709.
[Rafael Mendonça França & Yuuji Yaginuma ]
25 lines
750 B
Ruby
25 lines
750 B
Ruby
# frozen_string_literal: true
|
|
|
|
require "action_dispatch/testing/request_encoder"
|
|
|
|
module ActionDispatch
|
|
# Integration test methods such as ActionDispatch::Integration::Session#get
|
|
# and ActionDispatch::Integration::Session#post return objects of class
|
|
# TestResponse, which represent the HTTP response results of the requested
|
|
# controller actions.
|
|
#
|
|
# See Response for more information on controller response objects.
|
|
class TestResponse < Response
|
|
def self.from_response(response)
|
|
new response.status, response.headers, response.body
|
|
end
|
|
|
|
def parsed_body
|
|
@parsed_body ||= response_parser.call(body)
|
|
end
|
|
|
|
def response_parser
|
|
@response_parser ||= RequestEncoder.parser(media_type)
|
|
end
|
|
end
|
|
end
|