Fix multiple version definitions for path versioning

CHANGELOG.md

@@ -15,6 +15,7 @@
 * [#1365](https://github.com/ruby-grape/grape/pull/1365): Fix finding exception handler in error middleware - [@ktimothy](https://github.com/ktimothy).
 * [#1380](https://github.com/ruby-grape/grape/pull/1380): Fix `allow_blank: false` for `Time` attributes with valid values causes `NoMethodError` - [@ipkes](https://github.com/ipkes).
 * [#1384](https://github.com/ruby-grape/grape/pull/1384): Fix parameter validation with an empty optional nested `Array` - [@ipkes](https://github.com/ipkes).
+* [#1414](https://github.com/ruby-grape/grape/pull/1414): Fix multiple version definitions for path versioning - [@304](https://github.com/304).
 
 0.16.2 (4/12/2016)
 ==================

lib/grape/dsl/routing.rb

@@ -30,20 +30,21 @@ def version(*args, &block)
           if args.any?
             options = args.extract_options!
             options = options.reverse_merge(using: :path)
+            requested_versions = args.flatten
 
             raise Grape::Exceptions::MissingVendorOption.new if options[:using] == :header && !options.key?(:vendor)
 
-            @versions = versions | args
+            @versions = versions | requested_versions
 
             if block_given?
               within_namespace do
-                namespace_inheritable(:version, args)
+                namespace_inheritable(:version, requested_versions)
                 namespace_inheritable(:version_options, options)
 
                 instance_eval(&block)
               end
             else
-              namespace_inheritable(:version, args)
+              namespace_inheritable(:version, requested_versions)
               namespace_inheritable(:version_options, options)
             end
           end

spec/grape/api_spec.rb

@@ -302,16 +302,27 @@ def subject.enable_root_route!
 
       describe 'path versioned APIs' do
         before do
-          subject.version 'v1', using: :path
+          subject.version version, using: :path
           subject.enable_root_route!
         end
 
-        it 'without a format' do
-          versioned_get '/', 'v1', using: :path
+        context 'when a single version provided' do
+          let(:version) { 'v1' }
+
+          it 'without a format' do
+            versioned_get '/', 'v1', using: :path
+          end
+
+          it 'with a format' do
+            get '/v1/.json'
+          end
         end
 
-        it 'with a format' do
-          get '/v1/.json'
+        context 'when array of versions provided' do
+          let(:version) { %w(v1 v2) }
+
+          it { versioned_get '/', 'v1', using: :path }
+          it { versioned_get '/', 'v2', using: :path }
         end
       end