Do not shadow helpers with same method but more params

lib/grape-route-helpers/all_routes.rb

@@ -5,7 +5,9 @@ module GrapeRouteHelpers
   module AllRoutes
     def decorated_routes
       # memoize so that construction of decorated routes happens once
-      @decorated_routes ||= all_routes.map { |r| DecoratedRoute.new(r) }
+      @decorated_routes ||= all_routes
+                            .map { |r| DecoratedRoute.new(r) }
+                            .sort_by { |r| -r.dynamic_path_segments.count }
     end
 
     def all_routes

spec/grape_route_helpers/named_route_matcher_spec.rb

@@ -148,6 +148,14 @@
         index_path = api_v1_cats_path
         expect(index_path).to eq('/api/v1/cats.json')
       end
+
+      it 'does not get shadowed by another route with less segments' do
+        show_path = api_v1_cats_owners_path('id' => 1)
+        expect(show_path).to eq('/api/v1/cats/1/owners.json')
+
+        show_path = api_v1_cats_owners_path('id' => 1, 'owner_id' => 1)
+        expect(show_path).to eq('/api/v1/cats/1/owners/1.json')
+      end
     end
 
     context 'when query params are passed in' do

spec/support/api.rb

@@ -24,6 +24,14 @@ class API < Grape::API
             'cat'
           end
         end
+
+        get ':id/owners' do
+          %w(owner1 owner2)
+        end
+
+        get ':id/owners/:owner_id' do
+          'owner'
+        end
       end
 
       route :any, '*path' do