diff options
author | Jay Pignata <john.pignata@gmail.com> | 2009-08-11 00:56:46 -0400 |
---|---|---|
committer | Jeremy Kemper <jeremy@bitsweat.net> | 2009-08-15 16:51:03 -0700 |
commit | 679128da58636f3ec96e24fcb7daac24666b2dad (patch) | |
tree | 51d16ff1b0476b6fbbb2bb453639a846adc9a894 /actionpack/test/dispatch/request/json_params_parsing_test.rb | |
parent | 911acc10de483e0d7a9e6b3c475aeaecad49bfc5 (diff) | |
download | rails-679128da58636f3ec96e24fcb7daac24666b2dad.tar.gz rails-679128da58636f3ec96e24fcb7daac24666b2dad.tar.bz2 rails-679128da58636f3ec96e24fcb7daac24666b2dad.zip |
Adding a call to logger from params_parser to give detailed debug information when invalid xml or json is posted
[#2481 state:committed]
Signed-off-by: Jeremy Kemper <jeremy@bitsweat.net>
Diffstat (limited to 'actionpack/test/dispatch/request/json_params_parsing_test.rb')
-rw-r--r-- | actionpack/test/dispatch/request/json_params_parsing_test.rb | 28 |
1 files changed, 24 insertions, 4 deletions
diff --git a/actionpack/test/dispatch/request/json_params_parsing_test.rb b/actionpack/test/dispatch/request/json_params_parsing_test.rb index a3dde72c4e..db6cf7b330 100644 --- a/actionpack/test/dispatch/request/json_params_parsing_test.rb +++ b/actionpack/test/dispatch/request/json_params_parsing_test.rb @@ -30,16 +30,36 @@ class JsonParamsParsingTest < ActionController::IntegrationTest ) end + test "logs error if parsing unsuccessful" do + with_test_routing do + begin + $stderr = StringIO.new + json = "[\"person]\": {\"name\": \"David\"}}" + post "/parse", json, {'CONTENT_TYPE' => 'application/json'} + assert_response :error + $stderr.rewind && err = $stderr.read + assert err =~ /Error occurred while parsing request parameters/ + ensure + $stderr = STDERR + end + end + end + private def assert_parses(expected, actual, headers = {}) + with_test_routing do + post "/parse", actual, headers + assert_response :ok + assert_equal(expected, TestController.last_request_parameters) + end + end + + def with_test_routing with_routing do |set| set.draw do |map| map.connect ':action', :controller => "json_params_parsing_test/test" end - - post "/parse", actual, headers - assert_response :ok - assert_equal(expected, TestController.last_request_parameters) + yield end end end |