grape-route-helpers 2.0.0 → 2.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 83b55766476f52f66e8ca85e84cbb3982cc30ffc
-  data.tar.gz: 4e95656ec76aec7d9c61c09ed5f8d3ecf9d82992
+  metadata.gz: 2bdf855209f4731d0d522b8e0b8a6df50913e437
+  data.tar.gz: 410ff06bb630ce4f87fe41a97ff9c7c610236235
 SHA512:
-  metadata.gz: b0e423a706e7c000da1c40203dfa482e38582189521917dfa271ba5f024c9e3c9efc67d66dbaeff1651e2b15ab7655534d99ae5bc4035f00cf294478844df0b5
-  data.tar.gz: 87182c670bca69856919540aba66c97d965d2c17c2da71cc94763561bc881e4ded309a9cf5d13e2846c875d62d512ebf0f657950351786a8c807acfb8b78a219
+  metadata.gz: 731a34ae06496c10f3fcd1e55bc6fdbc155f3c08559cab2843e35c3e7efe7006d90ac4192eb724299a8e8f120654ca0c3c7102d9b90012b9941311c0f801ae1b
+  data.tar.gz: e7f775f3fb66a7f8b7bd58ff124b5973a9d6a3df5dca087edc1d407535d4451bc36800a49e2eff90e78e536a4389dbf0af76dbcfa1eb7c7d3c0ae6047ae5832b

data/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## December 17 2016
+
+* Bump to 2.1.0
+* Fix bug that caused POST routes to be ignored if more than one was defined
+* Many thanks to @njd5475 for the bug report and pull request that helped me write this
+
 ## April 28 2016
 
 * Release 2.0.0

data/Gemfile.lock

@@ -1,9 +1,9 @@
 PATH
   remote: .
   specs:
-    grape-route-helpers (2.0.0)
+    grape-route-helpers (2.1.0)
       activesupport
-      grape
+      grape (>= 0.16.0)
       rake
 
 GEM

data/grape-route-helpers.gemspec

@@ -11,7 +11,7 @@ Gem::Specification.new do |gem|
   gem.files       = `git ls-files`.split($INPUT_RECORD_SEPARATOR)
   gem.homepage    = 'https://github.com/reprah/grape-route-helpers'
 
-  gem.add_runtime_dependency 'grape', '~> 0.16', '>= 0.16.0'
+  gem.add_runtime_dependency 'grape', '>= 0.16.0'
   gem.add_runtime_dependency 'activesupport'
   gem.add_runtime_dependency 'rake'
 

data/lib/grape-route-helpers/all_routes.rb

@@ -10,7 +10,7 @@ module GrapeRouteHelpers
 
     def all_routes
       routes = subclasses.flat_map { |s| s.send(:prepare_routes) }
-      routes.uniq(&:options)
+      routes.uniq { |r| r.options.merge(path: r.path) }
     end
   end
 end

data/lib/grape-route-helpers/version.rb

@@ -1,4 +1,4 @@
 # Gem version
 module GrapeRouteHelpers
-  VERSION = '2.0.0'
+  VERSION = '2.1.0'
 end

data/spec/grape_route_helpers/all_routes_spec.rb

@@ -12,7 +12,7 @@ describe GrapeRouteHelpers::AllRoutes do
 
         # A route is unique if no other route shares the same set of options
         all_route_options = Grape::API.all_routes.map do |r|
-          r.instance_variable_get(:@options)
+          r.instance_variable_get(:@options).merge(path: r.path)
         end
 
         duplicates = all_route_options.select do |o|
@@ -20,7 +20,15 @@ describe GrapeRouteHelpers::AllRoutes do
         end
 
         expect(duplicates).to be_empty
-        expect(all_route_options.size).to eq(7)
+      end
+    end
+
+    context 'when there are multiple POST routes with the same namespace in the same API' do
+      it 'returns all POST routes' do
+        expected_routes = Spec::Support::MultiplePostsAPI.routes.map(&:path)
+
+        all_routes = Grape::API.all_routes
+        expect(all_routes.map(&:path)).to include(*expected_routes)
       end
     end
   end

data/spec/grape_route_helpers/decorated_route_spec.rb

@@ -60,6 +60,14 @@ describe GrapeRouteHelpers::DecoratedRoute do
       end
     end
 
+    context 'when an API has multiple POST routes in a resource' do
+      let(:api) { Spec::Support::MultiplePostsAPI }
+
+      it 'it creates a helper for each POST route' do
+        expect(routes.size).to eq(2)
+      end
+    end
+
     context 'when an API has multiple versions' do
       let(:api) { Spec::Support::APIWithMultipleVersions }
 

data/spec/support/api.rb

@@ -54,5 +54,16 @@ module Spec
         'pong'
       end
     end
+
+    # API with multiple POST routes
+    class MultiplePostsAPI < Grape::API
+      resource :hamlet do
+        post 'to_be' do
+        end
+
+        post 'or_not_to_be' do
+        end
+      end
+    end
   end
 end

metadata

@@ -1,22 +1,19 @@
 --- !ruby/object:Gem::Specification
 name: grape-route-helpers
 version: !ruby/object:Gem::Version
-  version: 2.0.0
+  version: 2.1.0
 platform: ruby
 authors:
 - Harper Henn
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-04-29 00:00:00.000000000 Z
+date: 2017-07-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: grape
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '0.16'
     - - ">="
       - !ruby/object:Gem::Version
         version: 0.16.0
@@ -24,9 +21,6 @@ dependencies:
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '0.16'
     - - ">="
       - !ruby/object:Gem::Version
         version: 0.16.0