fix for #47 - upload button doesnt work on rails;

CHANGELOG.md

@@ -1,5 +1,6 @@
 ### Next Release
 
+* [#139](https://github.com/tim-vandecasteele/grape-swagger/pull/139): Fix for not functioning upload button when using rails; by [@timgluz](https://github.com/timgluz).
 * [#136](https://github.com/tim-vandecasteele/grape-swagger/pull/136), [#94](https://github.com/tim-vandecasteele/grape-swagger/pull/94): Recurse combination of namespaces when using mounted apps - [@renier](https://github.com/renier).
 * [#100](https://github.com/tim-vandecasteele/grape-swagger/pull/100): Added ability to specify a nickname for an endpoint - [@lhorne](https://github.com/lhorne).
 * [#94](https://github.com/tim-vandecasteele/grape-swagger/pull/94): Added support for namespace descriptions - [@renier](https://github.com/renier).

lib/grape-swagger.rb

@@ -224,9 +224,11 @@ def as_markdown(description)
             end
 
             def parse_params(params, path, method)
+              accepted_file_classes = ['Rack::Multipart::UploadedFile', 'Hash'] # 1.st for Rack, 2nd for Rails
+
               params ||= []
               params.map do |param, value|
-                value[:type] = 'File' if value.is_a?(Hash) && value[:type] == 'Rack::Multipart::UploadedFile'
+                value[:type] = 'File' if value.is_a?(Hash) && accepted_file_classes.include?(value[:type])
                 items = {}
 
                 raw_data_type = value.is_a?(Hash) ? (value[:type] || 'string').to_s : 'string'

spec/grape-swagger_helper_spec.rb

@@ -46,6 +46,44 @@ class HelperTestAPI < Grape::API
       ]
     end
 
+    it 'parses file param' do
+      params = {
+        rack: {
+          type: 'Rack::Multipart::UploadedFile',
+          desc: 'rack file',
+          datafile: 'content',
+          required: true
+        },
+        rails: {
+          type: 'Hash',
+          desc: 'rails file',
+          datafile: 'content',
+          required: true
+        }
+      }
+      path = '/coolness'
+      method = 'POST'
+      expect(subject.parse_params(params, path, method)).to eq [
+        {
+          paramType: 'body',
+          name: :rack,
+          description: 'rack file',
+          type: 'File',
+          required: true,
+          allowMultiple: false
+        },
+        {
+          paramType: 'body',
+          name: :rails,
+          description: 'rails file',
+          type: 'File',
+          required: true,
+          allowMultiple: false
+        }
+      ]
+
+    end
+
     context 'custom type' do
       before :all do
         class CustomType

test/api.rb

@@ -45,6 +45,17 @@ class Api < Grape::API
     get do
       @@splines.values
     end
+
+    # TEST api for testing uploading
+    # curl --form file=@splines.png http://localhost:9292/splines/upload
+    desc 'Update image'
+    post 'upload' do
+      filename = params[:file][:filename]
+      content_type 'application/octet-stream'
+      env['api.format'] = :binary # there's no formatter for :binary, data will be returned "as is"
+      header 'Content-Disposition', "attachment; filename*=UTF-8''#{URI.escape(filename)}"
+      params[:file][:tempfile].read
+    end
   end
 
   add_swagger_documentation