diff options
author | Jeremy Daer <jeremydaer@gmail.com> | 2015-10-03 19:14:37 -0700 |
---|---|---|
committer | Jeremy Daer <jeremydaer@gmail.com> | 2015-10-03 21:59:18 -0700 |
commit | 2356403330f2fa60045c858434cad550f6b3ee46 (patch) | |
tree | a82b9781bd1c2efb85252e376fa9d27fc94d1f29 /actionpack/test/dispatch/header_test.rb | |
parent | 24b1850130c68fe70b912277527e9c139ccc6742 (diff) | |
download | rails-2356403330f2fa60045c858434cad550f6b3ee46.tar.gz rails-2356403330f2fa60045c858434cad550f6b3ee46.tar.bz2 rails-2356403330f2fa60045c858434cad550f6b3ee46.zip |
Introduce `Headers#add`. Move `Response#add_header` upstream.
* Introduce `ActionDispatch::Http::Headers#add` to add a value to
a multivalued header.
* Move `Response#add_header` upstream: https://github.com/rack/rack/pull/957
* Match upstream `Response#have_header?` -> `#has_header?` name change.
Diffstat (limited to 'actionpack/test/dispatch/header_test.rb')
-rw-r--r-- | actionpack/test/dispatch/header_test.rb | 18 |
1 files changed, 18 insertions, 0 deletions
diff --git a/actionpack/test/dispatch/header_test.rb b/actionpack/test/dispatch/header_test.rb index 79600b654b..7f1ef121b7 100644 --- a/actionpack/test/dispatch/header_test.rb +++ b/actionpack/test/dispatch/header_test.rb @@ -42,6 +42,24 @@ class HeaderTest < ActiveSupport::TestCase assert_equal "127.0.0.1", @headers["HTTP_HOST"] end + test "add to multivalued headers" do + # Sets header when not present + @headers.add 'Foo', '1' + assert_equal '1', @headers['Foo'] + + # Ignores nil values + @headers.add 'Foo', nil + assert_equal '1', @headers['Foo'] + + # Converts value to string + @headers.add 'Foo', 1 + assert_equal '1,1', @headers['Foo'] + + # Case-insensitive + @headers.add 'fOo', 2 + assert_equal '1,1,2', @headers['foO'] + end + test "headers can contain numbers" do @headers["Content-MD5"] = "Q2hlY2sgSW50ZWdyaXR5IQ==" |