committee 5.5.0 → 5.5.2

data/test/middleware/base_test.rb

@@ -10,12 +10,8 @@ describe Committee::Middleware::Base do
   end
 
   it "accepts just a schema object" do
-    @app = new_rack_app(
-      schema: hyper_schema
-    )
-    params = {
-      "name" => "cloudnasium"
-    }
+    @app = new_rack_app(schema: hyper_schema)
+    params = { "name" => "cloudnasium" }
     header "Content-Type", "application/json"
     post "/apps", JSON.generate(params)
     assert_equal 200, last_response.status
@@ -23,69 +19,50 @@ describe Committee::Middleware::Base do
 
   it "accepts just a OpenAPI3 schema object" do
     @app = new_rack_app(schema: open_api_3_schema)
-    params = {
-        "name" => "cloudnasium"
-    }
+    params = { "name" => "cloudnasium" }
     header "Content-Type", "application/json"
     post "/apps", JSON.generate(params)
     assert_equal 200, last_response.status
   end
 
   it "doesn't accept a schema string" do
-    @app = new_rack_app(
-      schema: JSON.dump(hyper_schema_data)
-    )
-    params = {
-      "name" => "cloudnasium"
-    }
+    @app = new_rack_app(schema: JSON.dump(hyper_schema_data))
+    params = { "name" => "cloudnasium" }
     header "Content-Type", "application/json"
 
     e = assert_raises(ArgumentError) do
       post "/apps", JSON.generate(params)
     end
 
-    assert_equal "Committee: schema expected to be an instance " +
-                     "of Committee::Drivers::Schema.", e.message
+    assert_equal "Committee: schema expected to be an instance " + "of Committee::Drivers::Schema.", e.message
   end
 
   it "doesn't accept a schema hash" do
-    @app = new_rack_app(
-      schema: hyper_schema_data
-    )
-    params = {
-      "name" => "cloudnasium"
-    }
+    @app = new_rack_app(schema: hyper_schema_data)
+    params = { "name" => "cloudnasium" }
     header "Content-Type", "application/json"
 
     e = assert_raises(ArgumentError) do
       post "/apps", JSON.generate(params)
     end
 
-    assert_equal "Committee: schema expected to be an instance " +
-                     "of Committee::Drivers::Schema.", e.message
+    assert_equal "Committee: schema expected to be an instance " + "of Committee::Drivers::Schema.", e.message
   end
 
   it "doesn't accept a JsonSchema::Schema object" do
-    @app = new_rack_app(
-      schema: JsonSchema.parse!(hyper_schema_data)
-    )
-    params = {
-      "name" => "cloudnasium"
-    }
+    @app = new_rack_app(schema: JsonSchema.parse!(hyper_schema_data))
+    params = { "name" => "cloudnasium" }
     header "Content-Type", "application/json"
 
     e = assert_raises(ArgumentError) do
       post "/apps", JSON.generate(params)
     end
 
-    assert_equal "Committee: schema expected to be an instance " +
-                     "of Committee::Drivers::Schema.", e.message
+    assert_equal "Committee: schema expected to be an instance " + "of Committee::Drivers::Schema.", e.message
   end
 
   it "doesn't accept other schema types" do
-    @app = new_rack_app(
-      schema: 7,
-    )
+    @app = new_rack_app(schema: 7,)
     e = assert_raises(ArgumentError) do
       post "/apps"
     end

data/test/middleware/request_validation_open_api_3_test.rb

@@ -11,9 +11,7 @@ describe Committee::Middleware::RequestValidation do
 
   it "OpenAPI3 pass through a valid request" do
     @app = new_rack_app(schema: open_api_3_schema)
-    params = {
-        "string_post_1" => "cloudnasium"
-    }
+    params = { "string_post_1" => "cloudnasium" }
     header "Content-Type", "application/json"
     post "/characters", JSON.generate(params)
 
@@ -22,12 +20,12 @@ describe Committee::Middleware::RequestValidation do
 
   it "not parameter request" do
     check_parameter_string = lambda { |_|
-      [200, {integer: 1}, []]
+      [200, { integer: 1 }, []]
     }
 
     @app = new_rack_app_with_lambda(check_parameter_string, schema: open_api_3_schema)
 
-    put "/validate_no_parameter", {no_schema: 'no'}
+    put "/validate_no_parameter", { no_schema: 'no' }
   end
 
   it "passes given a datetime and with coerce_date_times enabled on GET endpoint" do
@@ -150,9 +148,7 @@ describe Committee::Middleware::RequestValidation do
 
   it "passes given an invalid datetime string with coerce_date_times enabled" do
     @app = new_rack_app(schema: open_api_3_schema, coerce_date_times: true)
-    params = {
-        "datetime_string" => "invalid_datetime_format"
-    }
+    params = { "datetime_string" => "invalid_datetime_format" }
     get "/string_params_coercer", params
 
     assert_equal 400, last_response.status
@@ -160,14 +156,7 @@ describe Committee::Middleware::RequestValidation do
   end
 
   it "passes a nested object with recursive option" do
-    params = {
-        "nested_array" => [
-            {
-                "update_time" => "2016-04-01T16:00:00.000+09:00",
-                "per_page" => "1",
-            }
-        ],
-    }
+    params = { "nested_array" => [{ "update_time" => "2016-04-01T16:00:00.000+09:00", "per_page" => "1", }], }
 
     check_parameter = lambda { |env|
       hash = env["committee.query_hash"]
@@ -248,9 +237,7 @@ describe Committee::Middleware::RequestValidation do
       [200, {}, []]
     }
 
-    @app = new_rack_app_with_lambda(check_parameter,
-                                    coerce_date_times: true,
-                                    schema: open_api_3_schema)
+    @app = new_rack_app_with_lambda(check_parameter, coerce_date_times: true, schema: open_api_3_schema)
 
     header "Content-Type", "application/json"
     post "/string_params_coercer", JSON.generate(params)
@@ -260,9 +247,7 @@ describe Committee::Middleware::RequestValidation do
   it "OpenAPI3 detects an invalid request" do
     @app = new_rack_app(schema: open_api_3_schema, strict: true)
     header "Content-Type", "application/json"
-    params = {
-        "string_post_1" => 1
-    }
+    params = { "string_post_1" => 1 }
     post "/characters", JSON.generate(params)
     assert_equal 400, last_response.status
     assert_match(/expected string, but received Integer:/i, last_response.body)
@@ -278,9 +263,7 @@ describe Committee::Middleware::RequestValidation do
 
   it "take a prefix" do
     @app = new_rack_app(prefix: "/v1", schema: open_api_3_schema)
-    params = {
-        "string_post_1" => "cloudnasium"
-    }
+    params = { "string_post_1" => "cloudnasium" }
     header "Content-Type", "application/json"
     post "/v1/characters", JSON.generate(params)
     assert_equal 200, last_response.status
@@ -288,9 +271,7 @@ describe Committee::Middleware::RequestValidation do
 
   it "take a prefix with invalid data" do
     @app = new_rack_app(prefix: "/v1", schema: open_api_3_schema)
-    params = {
-        "string_post_1" => 1
-    }
+    params = { "string_post_1" => 1 }
     header "Content-Type", "application/json"
     post "/v1/characters", JSON.generate(params)
     assert_equal 400, last_response.status
@@ -299,9 +280,7 @@ describe Committee::Middleware::RequestValidation do
 
   it "ignores paths outside the prefix" do
     @app = new_rack_app(prefix: "/v1", schema: open_api_3_schema)
-    params = {
-        "string_post_1" => 1
-    }
+    params = { "string_post_1" => 1 }
     header "Content-Type", "application/json"
     post "/characters", JSON.generate(params)
     assert_equal 200, last_response.status
@@ -309,9 +288,7 @@ describe Committee::Middleware::RequestValidation do
 
   it "don't check prefix with no option" do
     @app = new_rack_app(schema: open_api_3_schema)
-    params = {
-        "string_post_1" => 1
-    }
+    params = { "string_post_1" => 1 }
     header "Content-Type", "application/json"
     post "/v1/characters", JSON.generate(params)
     assert_equal 200, last_response.status
@@ -361,14 +338,14 @@ describe Committee::Middleware::RequestValidation do
   it "optionally coerces query params" do
     @app = new_rack_app(coerce_query_params: true, schema: open_api_3_schema)
     header "Content-Type", "application/json"
-    get "/string_params_coercer", {"integer_1" => "1"}
+    get "/string_params_coercer", { "integer_1" => "1" }
     assert_equal 200, last_response.status
   end
 
   it "still raises an error if query param coercion is not possible" do
     @app = new_rack_app(coerce_query_params: false, schema: open_api_3_schema)
     header "Content-Type", "application/json"
-    get "/string_params_coercer", {"integer_1" => "1"}
+    get "/string_params_coercer", { "integer_1" => "1" }
 
     assert_equal 400, last_response.status
     assert_match(/expected integer, but received String:/i, last_response.body)
@@ -376,7 +353,7 @@ describe Committee::Middleware::RequestValidation do
 
   it "passes through a valid request for OpenAPI3" do
     check_parameter = lambda { |env|
-      assert_equal 3, env['committee.query_hash']['limit'] #5.0.x-
+      assert_equal 3, env['committee.query_hash']['limit'] # 5.0.x-
       [200, {}, []]
     }
 
@@ -417,7 +394,7 @@ describe Committee::Middleware::RequestValidation do
         assert_equal env['committee.params']['integer'], 42
         assert_equal env['committee.params'][:integer], 42
         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
+        # 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_overwrite_by_rails_rule: false)
 
@@ -436,7 +413,7 @@ describe Committee::Middleware::RequestValidation do
         [204, {}, []]
       end, schema: open_api_3_schema, parameter_overwrite_by_rails_rule: false)
 
-      params = {integer: 21}
+      params = { integer: 21 }
 
       header "Content-Type", "application/json"
       post '/overwrite_same_parameter?integer=42', JSON.generate(params)
@@ -452,11 +429,11 @@ describe Committee::Middleware::RequestValidation do
         assert_equal env['committee.request_body_hash']['integer'], 21
         assert_equal env['committee.request_body_hash'][:integer], 21
         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
+        # 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_overwrite_by_rails_rule: false)
 
-      params = {integer: 21}
+      params = { integer: 21 }
 
       header "Content-Type", "application/json"
       post '/overwrite_same_parameter/84?integer=42', JSON.generate(params)
@@ -475,7 +452,7 @@ describe Committee::Middleware::RequestValidation do
         [204, {}, []]
       end, schema: open_api_3_schema)
 
-      params = {integer: 21}
+      params = { integer: 21 }
 
       header "Content-Type", "application/json"
       post '/overwrite_same_parameter', JSON.generate(params)
@@ -492,7 +469,7 @@ describe Committee::Middleware::RequestValidation do
         [204, {}, []]
       end, schema: open_api_3_schema)
 
-      params = {integer: 21}
+      params = { integer: 21 }
 
       header "Content-Type", "application/json"
       post '/overwrite_same_parameter?integer=42', JSON.generate(params)
@@ -511,7 +488,7 @@ describe Committee::Middleware::RequestValidation do
         [204, {}, []]
       end, schema: open_api_3_schema)
 
-      params = {integer: 21}
+      params = { integer: 21 }
 
       header "Content-Type", "application/json"
       post '/overwrite_same_parameter/84?integer=42', JSON.generate(params)
@@ -541,7 +518,7 @@ describe Committee::Middleware::RequestValidation do
     }
 
     assert_equal 'Committee OpenAPI3 not support trace method', e.message
-  end  
+  end
 
   describe 'check header' do
     [
@@ -580,8 +557,8 @@ describe Committee::Middleware::RequestValidation do
   describe ':accept_request_filter' do
     [
       { description: 'when not specified, includes everything', accept_request_filter: nil, expected: { status: 400 } },
-      { description: 'when predicate matches, performs validation', accept_request_filter: -> (request) { request.path.start_with?('/v1/c') }, expected: { status: 400 } },
-      { description: 'when predicate does not match, skips validation', accept_request_filter: -> (request) { request.path.start_with?('/v1/x') }, expected: { status: 200 } },
+      { description: 'when predicate matches, performs validation', accept_request_filter: ->(request) { request.path.start_with?('/v1/c') }, expected: { status: 400 } },
+      { description: 'when predicate does not match, skips validation', accept_request_filter: ->(request) { request.path.start_with?('/v1/x') }, expected: { status: 200 } },
     ].each do |h|
       description = h[:description]
       accept_request_filter = h[:accept_request_filter]

data/test/middleware/request_validation_test.rb

@@ -55,9 +55,7 @@ describe Committee::Middleware::RequestValidation do
 
   it "passes through a valid request" do
     @app = new_rack_app(schema: hyper_schema)
-    params = {
-      "name" => "cloudnasium"
-    }
+    params = { "name" => "cloudnasium" }
     header "Content-Type", "application/json"
     post "/apps", JSON.generate(params)
     assert_equal 200, last_response.status
@@ -67,9 +65,7 @@ describe Committee::Middleware::RequestValidation do
     called_error = false
     pr = ->(_e) { called_error = true }
     @app = new_rack_app(schema: hyper_schema, error_handler: pr)
-    params = {
-      "name" => "cloudnasium"
-    }
+    params = { "name" => "cloudnasium" }
     header "Content-Type", "application/json"
     post "/apps", JSON.generate(params)
     assert !called_error
@@ -79,9 +75,7 @@ describe Committee::Middleware::RequestValidation do
     called_error = false
     pr = ->(_e, _env) { called_error = true }
     @app = new_rack_app(schema: hyper_schema, error_handler: pr)
-    params = {
-      "name" => "cloudnasium"
-    }
+    params = { "name" => "cloudnasium" }
     header "Content-Type", "application/json"
     post "/apps", JSON.generate(params)
     assert !called_error
@@ -89,10 +83,7 @@ describe Committee::Middleware::RequestValidation do
 
   it "passes given a datetime and with coerce_date_times enabled on GET endpoint" do
     key_name = "update_time"
-    params = {
-        key_name => "2016-04-01T16:00:00.000+09:00",
-        "query" => "cloudnasium"
-    }
+    params = { key_name => "2016-04-01T16:00:00.000+09:00", "query" => "cloudnasium" }
 
     check_parameter = lambda { |env|
       assert_equal DateTime, env['rack.request.query_hash'][key_name].class
@@ -165,10 +156,7 @@ describe Committee::Middleware::RequestValidation do
 
   it "passes given a nested datetime and with coerce_recursive=true and coerce_date_times=true on POST endpoint" do
     key_name = "update_time"
-    params = {
-        key_name => "2016-04-01T16:00:00.000+09:00",
-        "array_property" => ARRAY_PROPERTY
-    }
+    params = { key_name => "2016-04-01T16:00:00.000+09:00", "array_property" => ARRAY_PROPERTY }
 
     check_parameter = lambda { |env|
       hash = env['committee.params']
@@ -191,10 +179,7 @@ describe Committee::Middleware::RequestValidation do
 
   it "passes given a nested datetime and with coerce_recursive=true and coerce_date_times=true on POST endpoint" do
     key_name = "update_time"
-    params = {
-        key_name => "2016-04-01T16:00:00.000+09:00",
-        "array_property" => ARRAY_PROPERTY
-    }
+    params = { key_name => "2016-04-01T16:00:00.000+09:00", "array_property" => ARRAY_PROPERTY }
 
     check_parameter = lambda { |env|
       hash = env['committee.params']
@@ -217,10 +202,7 @@ describe Committee::Middleware::RequestValidation do
 
   it "passes given a nested datetime and with coerce_recursive=false and coerce_date_times=true on POST endpoint" do
     key_name = "update_time"
-    params = {
-        key_name => "2016-04-01T16:00:00.000+09:00",
-        "array_property" => ARRAY_PROPERTY
-    }
+    params = { key_name => "2016-04-01T16:00:00.000+09:00", "array_property" => ARRAY_PROPERTY }
 
     check_parameter = lambda { |env|
       hash = env['committee.params']
@@ -243,10 +225,7 @@ describe Committee::Middleware::RequestValidation do
 
   it "passes given a nested datetime and with coerce_recursive=false and coerce_date_times=false on POST endpoint" do
     key_name = "update_time"
-    params = {
-        key_name => "2016-04-01T16:00:00.000+09:00",
-        "array_property" => ARRAY_PROPERTY
-    }
+    params = { key_name => "2016-04-01T16:00:00.000+09:00", "array_property" => ARRAY_PROPERTY }
 
     check_parameter = lambda { |env|
       hash = env['committee.params']
@@ -269,9 +248,7 @@ describe Committee::Middleware::RequestValidation do
 
   it "passes given an invalid datetime string with coerce_date_times enabled" do
     @app = new_rack_app(coerce_date_times: true, schema: hyper_schema)
-    params = {
-        "update_time" => "invalid_datetime_format"
-    }
+    params = { "update_time" => "invalid_datetime_format" }
     header "Content-Type", "application/json"
     post "/apps", JSON.generate(params)
     assert_equal 400, last_response.status
@@ -288,9 +265,7 @@ describe Committee::Middleware::RequestValidation do
   it "detects an invalid request" do
     @app = new_rack_app(schema: hyper_schema)
     header "Content-Type", "application/json"
-    params = {
-      "name" => 1
-    }
+    params = { "name" => 1 }
     post "/apps", JSON.generate(params)
     assert_equal 400, last_response.status
     assert_match(/invalid request/i, last_response.body)
@@ -299,9 +274,7 @@ describe Committee::Middleware::RequestValidation do
   it "ignores errors when ignore_error: true" do
     @app = new_rack_app(schema: hyper_schema, ignore_error: true)
     header "Content-Type", "application/json"
-    params = {
-      "name" => 1
-    }
+    params = { "name" => 1 }
     post "/apps", JSON.generate(params)
     assert_equal 200, last_response.status
   end
@@ -311,9 +284,7 @@ describe Committee::Middleware::RequestValidation do
     pr = ->(e, _env) { called_err = e }
     @app = new_rack_app(schema: hyper_schema, error_handler: pr)
     header "Content-Type", "application/json"
-    params = {
-      "name" => 1
-    }
+    params = { "name" => 1 }
     post "/apps", JSON.generate(params)
     assert_kind_of Committee::InvalidRequest, called_err
   end
@@ -337,9 +308,7 @@ describe Committee::Middleware::RequestValidation do
 
   it "takes a prefix" do
     @app = new_rack_app(prefix: "/v1", schema: hyper_schema)
-    params = {
-      "name" => "cloudnasium"
-    }
+    params = { "name" => "cloudnasium" }
     header "Content-Type", "application/json"
     post "/v1/apps", JSON.generate(params)
     assert_equal 200, last_response.status
@@ -374,9 +343,7 @@ describe Committee::Middleware::RequestValidation do
 
   it "optionally skip content_type check" do
     @app = new_rack_app(check_content_type: false, schema: hyper_schema)
-    params = {
-      "name" => "cloudnasium"
-    }
+    params = { "name" => "cloudnasium" }
     header "Content-Type", "text/html"
     post "/apps", JSON.generate(params)
     assert_equal 200, last_response.status
@@ -385,14 +352,14 @@ describe Committee::Middleware::RequestValidation do
   it "optionally coerces query params" do
     @app = new_rack_app(coerce_query_params: true, schema: hyper_schema)
     header "Content-Type", "application/json"
-    get "/search/apps", {"per_page" => "10", "query" => "cloudnasium"}
+    get "/search/apps", { "per_page" => "10", "query" => "cloudnasium" }
     assert_equal 200, last_response.status
   end
 
   it "still raises an error if query param coercion is not possible" do
     @app = new_rack_app(coerce_query_params: true, schema: hyper_schema)
     header "Content-Type", "application/json"
-    get "/search/apps", {"per_page" => "foo", "query" => "cloudnasium"}
+    get "/search/apps", { "per_page" => "foo", "query" => "cloudnasium" }
     assert_equal 400, last_response.status
     assert_match(/invalid request/i, last_response.body)
   end
@@ -470,7 +437,7 @@ describe Committee::Middleware::RequestValidation do
         coerce_recursive: true
     }
 
-    @app = new_rack_app({schema: hyper_schema}.merge(options))
+    @app = new_rack_app({ schema: hyper_schema }.merge(options))
     header "Content-Type", "application/x-www-form-urlencoded"
     post "/unknown"
     assert_equal 200, last_response.status
@@ -479,8 +446,8 @@ describe Committee::Middleware::RequestValidation do
   describe ':accept_request_filter' do
     [
       { description: 'when not specified, includes everything', accept_request_filter: nil, expected: { status: 400 } },
-      { description: 'when predicate matches, performs validation', accept_request_filter: -> (request) { request.path.start_with?('/v1/a') }, expected: { status: 400 } },
-      { description: 'when predicate does not match, skips validation', accept_request_filter: -> (request) { request.path.start_with?('/v1/x') }, expected: { status: 200 } },
+      { description: 'when predicate matches, performs validation', accept_request_filter: ->(request) { request.path.start_with?('/v1/a') }, expected: { status: 400 } },
+      { description: 'when predicate does not match, skips validation', accept_request_filter: ->(request) { request.path.start_with?('/v1/x') }, expected: { status: 200 } },
     ].each do |h|
       description = h[:description]
       accept_request_filter = h[:accept_request_filter]
@@ -503,7 +470,6 @@ describe Committee::Middleware::RequestValidation do
     }, options)
   end
 
-
   def new_rack_app_with_lambda(check_lambda, options = {})
     Rack::Builder.new {
       use Committee::Middleware::RequestValidation, options

data/test/middleware/response_validation_open_api_3_test.rb

@@ -5,7 +5,7 @@ require "test_helper"
 describe Committee::Middleware::ResponseValidation do
   include Rack::Test::Methods
 
-  CHARACTERS_RESPONSE = {"Otonokizaka" => ["Honoka.Kousaka"]}
+  CHARACTERS_RESPONSE = { "Otonokizaka" => ["Honoka.Kousaka"] }
 
   def app
     @app
@@ -17,6 +17,22 @@ describe Committee::Middleware::ResponseValidation do
     assert_equal 200, last_response.status
   end
 
+  it "passes through a valid response with content-type (lower-case)" do
+    status = 200
+    headers = { "content-type" => "application/json" }
+    response = JSON.generate(CHARACTERS_RESPONSE)
+
+    @app = Rack::Builder.new {
+      use Committee::Middleware::ResponseValidation, { schema: open_api_3_schema }
+      run lambda { |_|
+        [status, headers, [response]]
+      }
+    }
+
+    get "/characters"
+    assert_equal 200, last_response.status
+  end
+
   it "passes through a invalid json" do
     @app = new_response_rack("not_json", {}, schema: open_api_3_schema)
 
@@ -35,7 +51,7 @@ describe Committee::Middleware::ResponseValidation do
   end
 
   it "passes through a invalid json with parse_response_by_content_type option" do
-    @app = new_response_rack("csv response", { "Content-Type" => "test/csv"}, schema: open_api_3_schema, parse_response_by_content_type: true)
+    @app = new_response_rack("csv response", { "Content-Type" => "test/csv" }, schema: open_api_3_schema, parse_response_by_content_type: true)
 
     get "/csv"
 
@@ -59,13 +75,13 @@ describe Committee::Middleware::ResponseValidation do
   end
 
   it "passes through a 204 (no content) response" do
-    @app = new_response_rack("", {}, {schema: open_api_3_schema}, {status: 204})
+    @app = new_response_rack("", {}, { schema: open_api_3_schema }, { status: 204 })
     post "/validate"
     assert_equal 204, last_response.status
   end
 
   it "passes through a 304 (not modified) response" do
-    @app = new_response_rack("", {}, {schema: open_api_3_schema}, {status: 304})
+    @app = new_response_rack("", {}, { schema: open_api_3_schema }, { status: 304 })
     post "/validate"
     assert_equal 304, last_response.status
   end
@@ -93,10 +109,10 @@ describe Committee::Middleware::ResponseValidation do
   end
 
   it "not parameter request" do
-    @app = new_response_rack({integer: '1'}.to_json, {}, schema: open_api_3_schema, raise: true)
+    @app = new_response_rack({ integer: '1' }.to_json, {}, schema: open_api_3_schema, raise: true)
 
     assert_raises(Committee::InvalidResponse) do
-      patch "/validate_no_parameter", {no_schema: 'no'}
+      patch "/validate_no_parameter", { no_schema: 'no' }
     end
   end
 
@@ -178,7 +194,6 @@ describe Committee::Middleware::ResponseValidation do
                                raise: true,
                                validate_success_only: false)
 
-
       e = assert_raises(Committee::InvalidResponse) do
         get "/characters"
       end
@@ -194,7 +209,6 @@ describe Committee::Middleware::ResponseValidation do
                                raise: true,
                                validate_success_only: true)
 
-
       get "/characters"
 
       assert_equal 400, last_response.status
@@ -204,8 +218,8 @@ describe Committee::Middleware::ResponseValidation do
   describe ':accept_request_filter' do
     [
       { description: 'when not specified, includes everything', accept_request_filter: nil, expected: { status: 500 } },
-      { description: 'when predicate matches, performs validation', accept_request_filter: -> (request) { request.path.start_with?('/v1/c') }, expected: { status: 500 } },
-      { description: 'when predicate does not match, skips validation', accept_request_filter: -> (request) { request.path.start_with?('/v1/x') }, expected: { status: 200 } },
+      { description: 'when predicate matches, performs validation', accept_request_filter: ->(request) { request.path.start_with?('/v1/c') }, expected: { status: 500 } },
+      { description: 'when predicate does not match, skips validation', accept_request_filter: ->(request) { request.path.start_with?('/v1/x') }, expected: { status: 200 } },
     ].each do |h|
       description = h[:description]
       accept_request_filter = h[:accept_request_filter]
@@ -223,9 +237,9 @@ describe Committee::Middleware::ResponseValidation do
 
   it 'does not suppress application error' do
     @app = Rack::Builder.new {
-      use Committee::Middleware::ResponseValidation, {schema: open_api_3_schema, raise: true}
+      use Committee::Middleware::ResponseValidation, { schema: open_api_3_schema, raise: true }
       run lambda { |_|
-        JSON.load('-') # invalid json  
+        JSON.load('-') # invalid json
       }
     }
 
@@ -234,14 +248,14 @@ describe Committee::Middleware::ResponseValidation do
     end
   end
 
-  it "strict and invalid status" do    
-    @app = new_response_rack(JSON.generate(CHARACTERS_RESPONSE), {}, {schema: open_api_3_schema, strict: true}, {status: 201})
+  it "strict and invalid status" do
+    @app = new_response_rack(JSON.generate(CHARACTERS_RESPONSE), {}, { schema: open_api_3_schema, strict: true }, { status: 201 })
     get "/characters"
     assert_equal 500, last_response.status
   end
 
   it "strict and invalid status with raise" do
-    @app = new_response_rack(JSON.generate(CHARACTERS_RESPONSE), {}, {schema: open_api_3_schema, strict: true, raise: true}, {status: 201})
+    @app = new_response_rack(JSON.generate(CHARACTERS_RESPONSE), {}, { schema: open_api_3_schema, strict: true, raise: true }, { status: 201 })
 
     assert_raises(Committee::InvalidResponse) do
       get "/characters"
@@ -249,11 +263,7 @@ describe Committee::Middleware::ResponseValidation do
   end
 
   it "strict and invalid content type" do
-    @app = new_response_rack("abc", 
-      {}, 
-      {schema: open_api_3_schema, strict: true},
-      {content_type: 'application/text'}
-    )
+    @app = new_response_rack("abc", {}, { schema: open_api_3_schema, strict: true }, { content_type: 'application/text' })
     get "/characters"
     assert_equal 500, last_response.status
   end
@@ -261,8 +271,8 @@ describe Committee::Middleware::ResponseValidation do
   it "strict and invalid content type with raise" do
     @app = new_response_rack("abc",
       {},
-      {schema: open_api_3_schema, strict: true, raise: true},
-      {content_type: 'application/text'}
+      { schema: open_api_3_schema, strict: true, raise: true },
+      { content_type: 'application/text' }
     )
 
     assert_raises(Committee::InvalidResponse) do
@@ -275,9 +285,7 @@ describe Committee::Middleware::ResponseValidation do
   def new_response_rack(response, headers = {}, options = {}, rack_options = {})
     status = rack_options[:status] || 200
     content_type = rack_options[:content_type] || "application/json"
-    headers = {
-      "Content-Type" => content_type
-    }.merge(headers)
+    headers = { "Content-Type" => content_type }.merge(headers)
     Rack::Builder.new {
       use Committee::Middleware::ResponseValidation, options
       run lambda { |_|