fixes #539 and #542; not all of 530 was commited

CHANGELOG.md

@@ -9,6 +9,12 @@
 * Your contribution here.
 
 
+### 0.25.2 (November 30, 2016)
+
+#### Fixes
+
+* [#544](https://github.com/ruby-grape/grape-swagger/pull/544): Fixes #539 and #542; not all of 530 was commited - [@LeFnord](https://github.com/LeFnord).
+
 ### 0.25.1 (November 29, 2016)
 
 #### Features

lib/grape-swagger/doc_methods/move_params.rb

@@ -28,6 +28,8 @@ def should_correct_array?(param)
 
         def correct_array_param(param)
           param.first[:schema] = { type: param.first.delete(:type), items: param.first.delete(:items) }
+
+          param
         end
 
         def parent_definition_of_params(params, route)

spec/issues/539_array_post_body.rb → spec/issues/539_array_post_body_spec.rb

@@ -2,10 +2,10 @@
 require 'grape-entity'
 require 'grape-swagger-entity'
 
-describe '#427 nested entity given as string' do
+describe '#539 post params given as array' do
   let(:app) do
     Class.new(Grape::API) do
-      namespace :issue_427 do
+      namespace :issue_539 do
         class Element < Grape::Entity
           expose :id
           expose :description
@@ -35,7 +35,7 @@ class ArrayOfElements < Grape::Entity
     JSON.parse(last_response.body)
   end
 
-  let(:parameters) { subject['paths']['/issue_427']['post']['parameters'] }
+  let(:parameters) { subject['paths']['/issue_539']['post']['parameters'] }
   let(:definitions) { subject['definitions'] }
 
   specify do
@@ -48,7 +48,9 @@ class ArrayOfElements < Grape::Entity
         }
       ]
     )
+  end
 
+  specify do
     expect(definitions).to eql(
       'Element' => {
         'type' => 'object',

spec/issues/542_array_of_type_in_post_body_spec.rb

@@ -0,0 +1,46 @@
+require 'spec_helper'
+require 'grape-entity'
+require 'grape-swagger-entity'
+
+describe '#542 array of type in post params' do
+  let(:app) do
+    Class.new(Grape::API) do
+      namespace :issue_542 do
+        params do
+          requires :logs, type: Array[String], documentation: { param_type: 'body' }
+        end
+
+        post do
+          present params
+        end
+      end
+
+      add_swagger_documentation format: :json
+    end
+  end
+
+  subject do
+    get '/swagger_doc'
+    JSON.parse(last_response.body)
+  end
+
+  let(:parameters) { subject['paths']['/issue_542']['post']['parameters'] }
+
+  specify do
+    expect(parameters).to eql(
+      [
+        {
+          'in' => 'body',
+          'name' => 'logs',
+          'required' => true,
+          'schema' => {
+            'type' => 'array',
+            'items' => {
+              'type' => 'string'
+            }
+          }
+        }
+      ]
+    )
+  end
+end