Merge pull request #24781 from maclover7/rm-env

Remove last uses of `@env[]` and `@env[]=`
This commit is contained in:
Guillermo Iguaran 2016-04-28 23:13:07 -05:00
commit 97988829eb
3 changed files with 39 additions and 12 deletions

@ -164,7 +164,7 @@ def use_accept_header
end
def format_from_path_extension
path = @env['action_dispatch.original_path'] || @env['PATH_INFO']
path = get_header('action_dispatch.original_path') || get_header('PATH_INFO')
if match = path && path.match(/\.(\w+)\z/)
Mime[match.captures.first]
end

@ -22,23 +22,23 @@ def self.default_env
private_class_method :default_env
def request_method=(method)
@env['REQUEST_METHOD'] = method.to_s.upcase
set_header('REQUEST_METHOD', method.to_s.upcase)
end
def host=(host)
@env['HTTP_HOST'] = host
set_header('HTTP_HOST', host)
end
def port=(number)
@env['SERVER_PORT'] = number.to_i
set_header('SERVER_PORT', number.to_i)
end
def request_uri=(uri)
@env['REQUEST_URI'] = uri
set_header('REQUEST_URI', uri)
end
def path=(path)
@env['PATH_INFO'] = path
set_header('PATH_INFO', path)
end
def action=(action_name)
@ -46,24 +46,24 @@ def action=(action_name)
end
def if_modified_since=(last_modified)
@env['HTTP_IF_MODIFIED_SINCE'] = last_modified
set_header('HTTP_IF_MODIFIED_SINCE', last_modified)
end
def if_none_match=(etag)
@env['HTTP_IF_NONE_MATCH'] = etag
set_header('HTTP_IF_NONE_MATCH', etag)
end
def remote_addr=(addr)
@env['REMOTE_ADDR'] = addr
set_header('REMOTE_ADDR', addr)
end
def user_agent=(user_agent)
@env['HTTP_USER_AGENT'] = user_agent
set_header('HTTP_USER_AGENT', user_agent)
end
def accept=(mime_types)
@env.delete('action_dispatch.request.accepts')
@env['HTTP_ACCEPT'] = Array(mime_types).collect(&:to_s).join(",")
delete_header('action_dispatch.request.accepts')
set_header('HTTP_ACCEPT', Array(mime_types).collect(&:to_s).join(","))
end
end
end

@ -88,6 +88,33 @@ class TestRequestTest < ActiveSupport::TestCase
assert_equal 'GoogleBot', req.user_agent
end
test "setter methods" do
req = ActionDispatch::TestRequest.create({})
get = 'GET'
[
'request_method=', 'host=', 'request_uri=', 'path=', 'if_modified_since=', 'if_none_match=',
'remote_addr=', 'user_agent=', 'accept='
].each do |method|
req.send(method, get)
end
req.port = 8080
req.accept = 'hello goodbye'
assert_equal(get, req.get_header('REQUEST_METHOD'))
assert_equal(get, req.get_header('HTTP_HOST'))
assert_equal(8080, req.get_header('SERVER_PORT'))
assert_equal(get, req.get_header('REQUEST_URI'))
assert_equal(get, req.get_header('PATH_INFO'))
assert_equal(get, req.get_header('HTTP_IF_MODIFIED_SINCE'))
assert_equal(get, req.get_header('HTTP_IF_NONE_MATCH'))
assert_equal(get, req.get_header('REMOTE_ADDR'))
assert_equal(get, req.get_header('HTTP_USER_AGENT'))
assert_nil(req.get_header('action_dispatch.request.accepts'))
assert_equal('hello goodbye', req.get_header('HTTP_ACCEPT'))
end
private
def assert_cookies(expected, cookie_jar)
assert_equal(expected, cookie_jar.instance_variable_get("@cookies"))