aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/test/dispatch/request
diff options
context:
space:
mode:
authorBenjamin Fleischer <github@benjaminfleischer.com>2016-02-16 10:17:33 -0600
committerBenjamin Fleischer <github@benjaminfleischer.com>2016-02-16 21:44:23 -0600
commitc4d90b70f4f27a527e9c2cd3af6cebf560b8df98 (patch)
tree88ae67f566249f46d32e5aba48f158784cb24822 /actionpack/test/dispatch/request
parentb0b61b62f9c27eff5b9e0797a0de0d35d64d1620 (diff)
downloadrails-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/request')
-rw-r--r--actionpack/test/dispatch/request/json_params_parsing_test.rb11
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" },