diff options
author | Benjamin Fleischer <github@benjaminfleischer.com> | 2016-02-16 10:17:33 -0600 |
---|---|---|
committer | Benjamin Fleischer <github@benjaminfleischer.com> | 2016-02-16 21:44:23 -0600 |
commit | c4d90b70f4f27a527e9c2cd3af6cebf560b8df98 (patch) | |
tree | 88ae67f566249f46d32e5aba48f158784cb24822 /actionpack/test/dispatch | |
parent | b0b61b62f9c27eff5b9e0797a0de0d35d64d1620 (diff) | |
download | rails-c4d90b70f4f27a527e9c2cd3af6cebf560b8df98.tar.gz rails-c4d90b70f4f27a527e9c2cd3af6cebf560b8df98.tar.bz2 rails-c4d90b70f4f27a527e9c2cd3af6cebf560b8df98.zip |
The JSON API media type should only work wih a JSON API handler
Since the media type 'application/vnd.api+json' is a spec,
it is inappropriate to handle it with the JSON renderer.
This PR removes support for a JSON API media type.
I would recommend the media type be registered on its own as `jsonapi`
when a jsonapi Renderer and deserializer (Http::Parameters::DEFAULT_PARSERS) are added.
Is related to work in https://github.com/rails/rails/pull/21496
Diffstat (limited to 'actionpack/test/dispatch')
-rw-r--r-- | actionpack/test/dispatch/request/json_params_parsing_test.rb | 11 |
1 files changed, 2 insertions, 9 deletions
diff --git a/actionpack/test/dispatch/request/json_params_parsing_test.rb b/actionpack/test/dispatch/request/json_params_parsing_test.rb index a3992ad008..71ad31ee15 100644 --- a/actionpack/test/dispatch/request/json_params_parsing_test.rb +++ b/actionpack/test/dispatch/request/json_params_parsing_test.rb @@ -37,9 +37,9 @@ class JsonParamsParsingTest < ActionDispatch::IntegrationTest ) end - test "parses json params for application/vnd.api+json" do + test "does not parses unregistered media types such as application/vnd.api+json" do assert_parses( - {"person" => {"name" => "David"}}, + {}, "{\"person\": {\"name\": \"David\"}}", { 'CONTENT_TYPE' => 'application/vnd.api+json' } ) end @@ -143,13 +143,6 @@ class RootLessJSONParamsParsingTest < ActionDispatch::IntegrationTest ) end - test "parses json params for application/vnd.api+json" do - assert_parses( - {"user" => {"username" => "sikachu"}, "username" => "sikachu"}, - "{\"username\": \"sikachu\"}", { 'CONTENT_TYPE' => 'application/vnd.api+json' } - ) - end - test "parses json with non-object JSON content" do assert_parses( {"user" => {"_json" => "string content" }, "_json" => "string content" }, |