allows custom format

- adds spec for default value

Author: LeFnord

lib/grape-swagger/doc_methods/parse_params.rb

@@ -20,7 +20,7 @@ def call(param, settings, route, definitions)
 
           # optional properties
           document_description(settings)
-          document_type_and_format(data_type)
+          document_type_and_format(settings, data_type)
           document_array_param(value_type, definitions) if value_type[:is_array]
           document_default_value(settings)
           document_range_values(settings)
@@ -51,13 +51,14 @@ def document_default_value(settings)
           @parsed_param[:default] = settings[:default] if settings[:default].present?
         end
 
-        def document_type_and_format(data_type)
+        def document_type_and_format(settings, data_type)
           if DataType.primitive?(data_type)
             data = DataType.mapping(data_type)
             @parsed_param[:type], @parsed_param[:format] = data
           else
             @parsed_param[:type] = data_type
           end
+          @parsed_param[:format] = settings[:format] if settings[:format].present?
         end
 
         def document_array_param(value_type, definitions)

spec/issues/532_allow_custom_format_spec.rb

@@ -0,0 +1,36 @@
+require 'spec_helper'
+
+describe '#542 array of type in post params' do
+  let(:app) do
+    Class.new(Grape::API) do
+      namespace :issue_532 do
+        params do
+          requires :logs, type: String, documentation: { format: 'log' }
+          optional :phone_number, type: Integer, documentation: { format: 'phone_number' }
+        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_532']['post']['parameters'] }
+
+  specify do
+    expect(parameters).to eql(
+      [
+        { 'in' => 'formData', 'name' => 'logs', 'type' => 'string', 'format' => 'log', 'required' => true },
+        { 'in' => 'formData', 'name' => 'phone_number', 'type' => 'integer', 'format' => 'phone_number', 'required' => false }
+      ]
+    )
+  end
+end

spec/issues/537_enum_values_spec.rb

@@ -5,7 +5,9 @@
     Class.new(Grape::API) do
       namespace :issue_539 do
         class Spec < Grape::Entity
-          expose :test_property, documentation: { values: [:foo, :bar] }
+          expose :enum_property, documentation: { values: [:foo, :bar] }
+          expose :enum_property_default, documentation: { values: %w(a b c), default: 'c' }
+          expose :own_format, documentation: { format: 'log' }
         end
 
         desc 'create account',
@@ -23,10 +25,23 @@ class Spec < Grape::Entity
     JSON.parse(last_response.body)
   end
 
-  let(:property) { subject['definitions']['Spec']['properties']['test_property'] }
-
+  let(:property) { subject['definitions']['Spec']['properties']['enum_property'] }
   specify do
     expect(property).to include 'enum'
     expect(property['enum']).to eql %w(foo bar)
   end
+
+  let(:property_default) { subject['definitions']['Spec']['properties']['enum_property_default'] }
+  specify do
+    expect(property_default).to include 'enum'
+    expect(property_default['enum']).to eql %w(a b c)
+    expect(property_default).to include 'default'
+    expect(property_default['default']).to eql 'c'
+  end
+
+  let(:own_format) { subject['definitions']['Spec']['properties']['own_format'] }
+  specify do
+    expect(own_format).to include 'format'
+    expect(own_format['format']).to eql 'log'
+  end
 end