cst1/openstreetmap-website
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Replace :nothing => true with :text => ""

Browse source 
Using `:nothing => true` confusingly actually causes a single space
to be sent as the content by rails.
This commit is contained in:
Tom Hughes 2013-07-19 11:14:55 +01:00
parent d492628f3e
commit 3880ac5de8
11 changed files with 49 additions and 49 deletions
Download patch file Download diff file Expand all files Collapse all files

8
test/functional/user_preference_controller_test.rb
View file

@ -108,7 +108,7 @@ class UserPreferenceControllerTest < ActionController::TestCase
end
assert_response :success
assert_equal "text/plain", @response.content_type
assert_equal " ", @response.body
assert_equal "", @response.body
assert_equal "new_value", UserPreference.find(1, "key").v
assert_equal "value", UserPreference.find(1, "new_key").v
assert_raises ActiveRecord::RecordNotFound do
@ -156,7 +156,7 @@ class UserPreferenceControllerTest < ActionController::TestCase
end
assert_response :success
assert_equal "text/plain", @response.content_type
assert_equal " ", @response.body
assert_equal "", @response.body
assert_equal "new_value", UserPreference.find(1, "new_key").v
# try changing the value of a preference
@ -166,7 +166,7 @@ class UserPreferenceControllerTest < ActionController::TestCase
end
assert_response :success
assert_equal "text/plain", @response.content_type
assert_equal " ", @response.body
assert_equal "", @response.body
assert_equal "newer_value", UserPreference.find(1, "new_key").v
end
@ -189,7 +189,7 @@ class UserPreferenceControllerTest < ActionController::TestCase
end
assert_response :success
assert_equal "text/plain", @response.content_type
assert_equal " ", @response.body
assert_equal "", @response.body
assert_raises ActiveRecord::RecordNotFound do
UserPreference.find(1, "key")
end