Spec tests

Author: dmoss18

lib/grape-swagger/endpoint.rb

@@ -371,15 +371,15 @@ def get_path_params(stackable_values)
       return params unless stackable_values.is_a? Grape::Util::StackableValues
       stackable_values&.new_values[:namespace]&.each do |namespace|
         space = namespace.space.to_s.gsub(':', '')
-        params[space] = (namespace.options || {}).merge(required: true)
+        params[space] = namespace.options || {}
       end
       inherited_params = get_path_params(stackable_values.inherited_values)
       inherited_params.merge(params)
     end
 
     def default_type(params)
       default_param_type = { required: true, type: 'Integer' }
-      params.each { |param| param[-1] = param.last == '' ? default_param_type : param.last }
+      params.each { |param| param[-1] = param.last.empty? ? default_param_type : param.last }
     end
 
     def expose_params(value)

spec/issues/833_route_param_options_spec.rb

@@ -0,0 +1,37 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe '#833 route_param type is included in documentation' do
+  let(:app) do
+    Class.new(Grape::API) do
+      resource :accounts do
+        route_param :account_number, type: String do
+          resource :records do
+            route_param :id do
+              get do
+                { message: 'hello world' }
+              end
+            end
+          end
+        end
+      end
+
+      add_swagger_documentation
+    end
+  end
+  let(:parameters) { subject['paths']['/accounts/{account_number}/records/{id}']['get']['parameters'] }
+
+  subject do
+    get '/swagger_doc'
+    JSON.parse(last_response.body)
+  end
+
+  specify do
+    account_number_param = parameters.find { |param| param['name'] == 'account_number' }
+    expect(account_number_param['type']).to eq 'string'
+    id_param = parameters.find { |param| param['name'] == 'id' }
+    # Default is still integer
+    expect(id_param['type']).to eq 'integer'
+  end
+end