From 3674dd90e3f2de352d48cee10f12f540817d020c Mon Sep 17 00:00:00 2001 From: r7kamura Date: Wed, 5 Jul 2023 08:01:25 +0900 Subject: [PATCH] Rename `parameter_overwite_by_rails_rule` to `parameter_overwrite_by_rails_rule` --- README.md | 2 +- lib/committee/schema_validator/open_api_3.rb | 2 +- lib/committee/schema_validator/option.rb | 4 ++-- test/middleware/request_validation_open_api_3_test.rb | 6 +++--- 4 files changed, 7 insertions(+), 7 deletions(-) diff --git a/README.md b/README.md index 40ab63d8..ca3802cb 100644 --- a/README.md +++ b/README.md @@ -353,7 +353,7 @@ Important changes are also described below. Committee 5.* has few breaking changes so we recommend upgrading to the latest release on 4.* and fixing any deprecation errors you see before upgrading. - set `parse_response_by_content_type=true` by default (old versions set `false`) -- set `parameter_overwite_by_rails_rule=true` by default (old version set `false`) +- set `parameter_overwrite_by_rails_rule=true` by default (old version set `false`) #### Future Updates (5.*~) OpenAPI3 Schema Users: Newly-added `strict_reference_validation` option defaults to `false` if not set. diff --git a/lib/committee/schema_validator/open_api_3.rb b/lib/committee/schema_validator/open_api_3.rb index 494ba7a1..7e9468d0 100644 --- a/lib/committee/schema_validator/open_api_3.rb +++ b/lib/committee/schema_validator/open_api_3.rb @@ -97,7 +97,7 @@ def request_unpack(request) end def copy_coerced_data_to_params(request) - order = if validator_option.parameter_overwite_by_rails_rule + order = if validator_option.parameter_overwrite_by_rails_rule # (high priority) path_hash_key -> query_param -> request_body_hash [validator_option.request_body_hash_key, validator_option.query_hash_key, validator_option.path_hash_key] else diff --git a/lib/committee/schema_validator/option.rb b/lib/committee/schema_validator/option.rb index 7baa5fe2..ee8dd79c 100644 --- a/lib/committee/schema_validator/option.rb +++ b/lib/committee/schema_validator/option.rb @@ -17,7 +17,7 @@ class Option :optimistic_json, :validate_success_only, :parse_response_by_content_type, - :parameter_overwite_by_rails_rule + :parameter_overwrite_by_rails_rule # Non-boolean options: attr_reader :headers_key, @@ -45,7 +45,7 @@ def initialize(options, schema, schema_type) @coerce_recursive = options.fetch(:coerce_recursive, true) @optimistic_json = options.fetch(:optimistic_json, false) @parse_response_by_content_type = options.fetch(:parse_response_by_content_type, true) - @parameter_overwite_by_rails_rule = options.fetch(:parameter_overwite_by_rails_rule, true) + @parameter_overwrite_by_rails_rule = options.fetch(:parameter_overwrite_by_rails_rule, true) # Boolean options and have a different value by default @allow_get_body = options.fetch(:allow_get_body, schema.driver.default_allow_get_body) diff --git a/test/middleware/request_validation_open_api_3_test.rb b/test/middleware/request_validation_open_api_3_test.rb index 942a44ea..91b39114 100644 --- a/test/middleware/request_validation_open_api_3_test.rb +++ b/test/middleware/request_validation_open_api_3_test.rb @@ -419,7 +419,7 @@ def app assert_equal env['committee.query_hash']['integer'], 42 #assert_equal env['rack.request.query_hash'][:integer], 42 # this isn't hash indifferent hash because we use rack.request.query_hash [204, {}, []] - end, schema: open_api_3_schema, parameter_overwite_by_rails_rule: false) + end, schema: open_api_3_schema, parameter_overwrite_by_rails_rule: false) header "Content-Type", "application/json" post '/overwrite_same_parameter?integer=42' @@ -434,7 +434,7 @@ def app assert_equal env['committee.request_body_hash'][:integer], 21 assert_equal env['committee.query_hash']['integer'], 42 [204, {}, []] - end, schema: open_api_3_schema, parameter_overwite_by_rails_rule: false) + end, schema: open_api_3_schema, parameter_overwrite_by_rails_rule: false) params = {integer: 21} @@ -454,7 +454,7 @@ def app assert_equal env['committee.query_hash']['integer'], 84 # we can't use query_parameter :( #assert_equal env['rack.request.query_hash'][:integer], 21 # this isn't hash indifferent hash because we use rack.request.query_hash [204, {}, []] - end, schema: open_api_3_schema, parameter_overwite_by_rails_rule: false) + end, schema: open_api_3_schema, parameter_overwrite_by_rails_rule: false) params = {integer: 21}