diff options
| author | Brendon Murphy <xternal1+github@gmail.com> | 2013-01-28 23:18:16 -0800 |
|---|---|---|
| committer | Brendon Murphy <xternal1+github@gmail.com> | 2013-01-28 23:18:16 -0800 |
| commit | c302741d8f9c34acdeae773184b30c2ae9b58e73 (patch) | |
| tree | d68934268d7f15efada792453eff05f1ea13f375 /actionpack/test/controller | |
| parent | 744307e525203639bb7672ef1a767b5088104334 (diff) | |
| download | rails-c302741d8f9c34acdeae773184b30c2ae9b58e73.tar.gz rails-c302741d8f9c34acdeae773184b30c2ae9b58e73.tar.bz2 rails-c302741d8f9c34acdeae773184b30c2ae9b58e73.zip | |
Remove yaml Proc param parser test
I don't believe this test is exercising any explicit params_parser
behavior that the other two Proc tests aren't already doing. Given
that we now know it's a bad idea to load user input via YAML.load,
somebody reading this test might get a dangerous idea about building
out a YAML params parser.
Diffstat (limited to 'actionpack/test/controller')
| -rw-r--r-- | actionpack/test/controller/webservice_test.rb | 13 |
1 files changed, 0 insertions, 13 deletions
diff --git a/actionpack/test/controller/webservice_test.rb b/actionpack/test/controller/webservice_test.rb index 0480295056..19d5652d81 100644 --- a/actionpack/test/controller/webservice_test.rb +++ b/actionpack/test/controller/webservice_test.rb @@ -129,19 +129,6 @@ class WebServiceTest < ActionDispatch::IntegrationTest $stderr = STDERR end - def test_register_and_use_yaml - with_test_route_set do - with_params_parsers Mime::YAML => Proc.new { |d| YAML.load(d) } do - post "/", {"entry" => "loaded from yaml"}.to_yaml, - {'CONTENT_TYPE' => 'application/x-yaml'} - - assert_equal 'entry', @controller.response.body - assert @controller.params.has_key?(:entry) - assert_equal 'loaded from yaml', @controller.params["entry"] - end - end - end - def test_register_and_use_xml_simple with_test_route_set do with_params_parsers Mime::XML => Proc.new { |data| Hash.from_xml(data)['request'].with_indifferent_access } do |