diff options
author | Jim Herzberg <jimmiesh@gmail.com> | 2011-10-11 17:46:24 -0700 |
---|---|---|
committer | Jim Herzberg <jimmiesh@gmail.com> | 2011-10-11 17:46:24 -0700 |
commit | a78a75d67a8072bc7613edbed548d4b865daadf8 (patch) | |
tree | a1500200adbf85a4fc291a9261ec35a2c997ce9a /activeresource/test | |
parent | fa79408a559d250c59ab4dac34dc7e68e6b7533f (diff) | |
download | rails-a78a75d67a8072bc7613edbed548d4b865daadf8.tar.gz rails-a78a75d67a8072bc7613edbed548d4b865daadf8.tar.bz2 rails-a78a75d67a8072bc7613edbed548d4b865daadf8.zip |
activeresource should treat HTTP status 307 as redirection, same as 301 and 302; added missing test cases for statii 301 and 302.
Diffstat (limited to 'activeresource/test')
-rw-r--r-- | activeresource/test/cases/connection_test.rb | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/activeresource/test/cases/connection_test.rb b/activeresource/test/cases/connection_test.rb index 09df0fb678..6f33023832 100644 --- a/activeresource/test/cases/connection_test.rb +++ b/activeresource/test/cases/connection_test.rb @@ -38,6 +38,15 @@ class ConnectionTest < Test::Unit::TestCase assert_equal expected, handle_response(expected) end + # 301 is moved permanently (redirect) + assert_response_raises ActiveResource::Redirection, 301 + + # 302 is found (redirect) + assert_response_raises ActiveResource::Redirection, 302 + + # 307 is temporary redirect + assert_response_raises ActiveResource::Redirection, 307 + # 400 is a bad request (e.g. malformed URI or missing request parameter) assert_response_raises ActiveResource::BadRequest, 400 |