Fix bug with coercion

Author: ffloyd

lib/grape/validations/types/build_coercer.rb

@@ -26,7 +26,15 @@ def self.build_coercer(type, method = nil)
         converter_options = {
           nullify_blank: true
         }
-        conversion_type = type
+        conversion_type = if method == JSON
+                            # becase we want just parsed JSON content:
+                            # if type is Array and data is `"{}"`
+                            # result will be [] because Virtus converts Hashes
+                            # to arrays
+                            Object
+                          else
+                            type
+                          end
 
         # Use a special coercer for multiply-typed parameters.
         if Types.multiple?(type)

spec/grape/validations/validators/coerce_spec.rb

@@ -382,7 +382,7 @@ class User
         expect(JSON.parse(last_response.body)).to eq([0, 0, 0, 0])
       end
 
-      it 'uses parse where available', focus: true do
+      it 'uses parse where available' do
         subject.params do
           requires :ints, type: Array, coerce_with: JSON do
             requires :i, type: Integer
@@ -398,10 +398,9 @@ class User
         expect(last_response.status).to eq(400)
         expect(last_response.body).to eq('ints is invalid')
 
-        # TODO: there is a bug in coercer. '{"i":1,"j":"2"}' transformed into [["i", 1], ["j", "2"]] WTF?
         get '/ints', ints: '{"i":1,"j":"2"}'
         expect(last_response.status).to eq(400)
-        expect(last_response.body).to eq('ints[0][i] is missing, ints[0][i] is invalid, ints[0][j] is missing')
+        expect(last_response.body).to eq('ints is invalid')
 
         get '/ints', ints: '[{"i":"1","j":"2"}]'
         expect(last_response.status).to eq(200)

spec/spec_helper.rb

@@ -25,8 +25,5 @@
   config.include Rack::Test::Methods
   config.raise_errors_for_deprecations!
 
-  config.filter_run :focus
-  config.run_all_when_everything_filtered = true
-
   config.before(:each) { Grape::Util::InheritableSetting.reset_global! }
 end