Merge pull request #162 from trans0082/feature/coerce_form_params_rec…

…usrsive

Change to coerce form params recursively

lib/committee/middleware/request_validation.rb

@@ -46,6 +46,7 @@ def handle(request)
         allow_query_params: @allow_query_params,
         coerce_form_params: @coerce_form_params,
         optimistic_json:    @optimistic_json,
+        coerce_recursive:   @coerce_recursive,
         schema:             link ? link.schema : nil
       ).call
 

lib/committee/request_unpacker.rb

@@ -7,6 +7,7 @@ def initialize(request, options={})
       @allow_query_params = options[:allow_query_params]
       @coerce_form_params = options[:coerce_form_params]
       @optimistic_json    = options[:optimistic_json]
+      @coerce_recursive   = options[:coerce_recursive]
       @schema             = options[:schema]
     end
 
@@ -31,7 +32,7 @@ def call
         p = @request.POST
 
         if @coerce_form_params && @schema
-          Committee::StringParamsCoercer.new(p, @schema).call!
+          Committee::StringParamsCoercer.new(p, @schema, coerce_recursive: @coerce_recursive).call!
         end
 
         p

test/request_unpacker_test.rb

@@ -113,6 +113,30 @@
     end
   end
 
+  it "coerces form params with coerce_form_params, coerce_recursive and a schema" do
+    %w[application/x-www-form-urlencoded multipart/form-data].each do |content_type|
+      schema = JsonSchema::Schema.new
+      schema.properties = { "x" => JsonSchema::Schema.new }
+      schema.properties["x"].type = ["object"]
+      schema.properties["x"].properties = { "y" => JsonSchema::Schema.new }
+      schema.properties["x"].properties["y"].type = ["integer"]
+
+      env = {
+        "CONTENT_TYPE" => content_type,
+        "rack.input"   => StringIO.new("x[y]=1"),
+      }
+      request = Rack::Request.new(env)
+      params, _ = Committee::RequestUnpacker.new(
+        request,
+        allow_form_params: true,
+        coerce_form_params: true,
+        coerce_recursive: true,
+        schema: schema
+      ).call
+      assert_equal({ "x" => { "y" => 1 } }, params)
+    end
+  end
+
   it "unpacks form & query params with allow_form_params and allow_query_params" do
     %w[application/x-www-form-urlencoded multipart/form-data].each do |content_type|
       env = {