routes_revealer 2.2.0 → 2.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 38bf504deba5b0ef19a4ac783011b2e72c7ea704
-  data.tar.gz: 12b551cd56974368652737c7722edfa1bfb757d8
+  metadata.gz: ffa27ef9ffe75cd1d8a596b0b01d7f248df72b24
+  data.tar.gz: 51f4f34474c2fa44d9e1b92a684a21e140fe0444
 SHA512:
-  metadata.gz: 6489103ca721d33b27c636d9f9296e6a4808b71412ee3e3e497c0e485d666f0570e7c45a97c32e6593b4f119d2bca7410cf9c5f552a77c11993aa10aa1eff56b
-  data.tar.gz: 46b80c06d9be4df50bf5a042785316cbfb2d1dc0a1bd0b8fe52ca540fdfc7e1c0a95c7eb5b053a32c57ccbd933ffa611bc14ef7940f5619d9fdf30bcc33c71c5
+  metadata.gz: 8f7f2c87169a1b1e13d57de1b71cd46529baec427a936d57970cd8c7015e5610fe0899a5a4d78c1cba3f297c9a562e77745705b70ef3becce3fdb9fa67a45f0b
+  data.tar.gz: cf702ce31597e75615a72023ea496c508bfb37b20b27749c40ffd0c921ba5dfd1efc2ef6e5864d5a1cf1e35f95239c657c2efde4dbf5ffa1a578858bf780f417

data/app/controllers/routes_revealer/routes_controller.rb

@@ -46,8 +46,10 @@ module RoutesRevealer
     def map_routes(routes, prepend='')
       prepend = '' if prepend == '/'
       route_hash_array(routes).map do |route_hash|
-        if route_hash[:reqs].include?("::Engine")
+        if route_hash[:reqs].is_a?(String) && route_hash[:reqs].include?("::Engine")
           map_routes(Module.const_get(route_hash[:reqs]).routes.routes, route_hash[:path])
+        elsif route_hash[:reqs].is_a? Class
+          map_routes(route_hash[:reqs].routes.routes, route_hash[:path])
         else
           "#{prepend}#{route_hash[:path]}" unless ignore_route?(route_hash[:path])
         end

data/lib/routes_revealer/version.rb

@@ -9,5 +9,5 @@
 # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 # See the License for the specific language governing permissions and limitations under the License.
 module RoutesRevealer
-  VERSION = '2.2.0'
+  VERSION = '2.2.1'
 end

data/spec/controllers/routes_controller_spec.rb

@@ -10,10 +10,19 @@
 # See the License for the specific language governing permissions and limitations under the License.
 require 'spec_helper'
 
+module Some
+  module Other
+    module Thing
+
+    end
+  end
+end
+
 describe RoutesRevealer::RoutesController do
   describe '#index' do
     let(:this_engine_string) { 'This::Engine' }
     let(:that_engine_string) { 'That::Engine' }
+    let(:some_other_engine) { Some::Other::Thing }
 
     let(:route1) { double('route1', path: '/1', verb: 'GET', reqs: 'controller#1') }
     let(:route2) { double('route2', path: '/2', verb: 'POST', reqs: 'controller#2') }
@@ -21,6 +30,7 @@ describe RoutesRevealer::RoutesController do
     let(:route4) { double('route4', path: '/4', verb: 'DELETE', reqs: 'controller#4') }
     let(:route5) { double('route5', path: '/', verb: 'GET', reqs: this_engine_string) }
     let(:route6) { double('route6', path: '/that', verb: 'GET', reqs: that_engine_string) }
+    let(:some_other_route) { double('other_route', path: '/other_route/1', verb: 'GET', reqs: some_other_engine) }
     let(:bad_route7) { double('bad_route7', path: '/routes', verb: 'HEAD', reqs: 'controller#route7') }
     let(:bad_route8) { double('bad_route7', path: '/rails', verb: 'TRACE', reqs: 'controller#route8') }
 
@@ -29,7 +39,8 @@ describe RoutesRevealer::RoutesController do
     let(:that_route1) { double('that_route1', path: '/1', verb: 'SHOOTS', reqs: 'that_controller#1') }
     let(:that_route2) { double('that_route2', path: '/2', verb: 'WALNUTS', reqs: 'that_controller#2') }
 
-    let(:routes) { [route1, route2, route3, route4, route5, route6, bad_route7, bad_route8] }
+
+    let(:routes) { [route1, route2, route3, route4, route5, route6, some_other_route, bad_route7, bad_route8] }
     let(:this_routes) { [this_route1, this_route2] }
     let(:that_routes) { [that_route1, that_route2] }
 
@@ -51,6 +62,7 @@ describe RoutesRevealer::RoutesController do
       allow(route_wrapper).to receive(:new).with(route4).and_return(route4)
       allow(route_wrapper).to receive(:new).with(route5).and_return(route5)
       allow(route_wrapper).to receive(:new).with(route6).and_return(route6)
+      allow(route_wrapper).to receive(:new).with(some_other_route).and_return(some_other_route)
       allow(route_wrapper).to receive(:new).with(bad_route7).and_return(bad_route7)
       allow(route_wrapper).to receive(:new).with(bad_route8).and_return(bad_route8)
 
@@ -65,35 +77,36 @@ describe RoutesRevealer::RoutesController do
     end
 
     it { expect(response.status).to eq 200 }
-    it { expect(JSON.parse(response.body).length).to eq 10 }
+    it { expect(JSON.parse(response.body).length).to eq 11 }
     it { expect(JSON.parse(response.body)[0]).to eq '/1' }
     it { expect(JSON.parse(response.body)[1]).to eq '/2' }
     it { expect(JSON.parse(response.body)[2]).to eq '/3' }
     it { expect(JSON.parse(response.body)[3]).to eq '/4' }
     it { expect(JSON.parse(response.body)[4]).to eq '/assets' }
-    it { expect(JSON.parse(response.body)[5]).to eq '/t1' }
-    it { expect(JSON.parse(response.body)[6]).to eq '/t2' }
-    it { expect(JSON.parse(response.body)[7]).to eq '/that/1' }
-    it { expect(JSON.parse(response.body)[8]).to eq '/that/2' }
-    it { expect(JSON.parse(response.body)[9]).to eq '/zzz/public.txt' }
+    it { expect(JSON.parse(response.body)[5]).to eq '/other_route/1' }
+    it { expect(JSON.parse(response.body)[6]).to eq '/t1' }
+    it { expect(JSON.parse(response.body)[7]).to eq '/t2' }
+    it { expect(JSON.parse(response.body)[8]).to eq '/that/1' }
+    it { expect(JSON.parse(response.body)[9]).to eq '/that/2' }
+    it { expect(JSON.parse(response.body)[10]).to eq '/zzz/public.txt' }
 
     context 'Rails app includes a public folder' do
       let(:public_folder) do
         ['public/robots.txt','public/humans.txt','public/dir/something.txt', 'public/assets/boom.png']
       end
       it 'adds the contents of the public folder' do
-        expect(JSON.parse(response.body).length).to eq 12
+        expect(JSON.parse(response.body).length).to eq 13
       end
       it { expect(JSON.parse(response.body)[5]).to eq '/dir/something.txt' }
       it { expect(JSON.parse(response.body)[6]).to eq '/humans.txt' }
-      it { expect(JSON.parse(response.body)[7]).to eq '/robots.txt' }
+      it { expect(JSON.parse(response.body)[8]).to eq '/robots.txt' }
     end
 
     context 'Rails app has an empty / non existant public folder' do
       let(:public_folder) { [] }
 
       it 'adds no additional entries' do
-        expect(JSON.parse(response.body).length).to eq 9
+        expect(JSON.parse(response.body).length).to eq 10
       end
     end
   end

metadata

@@ -1,145 +1,145 @@
 --- !ruby/object:Gem::Specification
 name: routes_revealer
 version: !ruby/object:Gem::Version
-  version: 2.2.0
+  version: 2.2.1
 platform: ruby
 authors:
 - The CareerBuilder.com Consumer Development teams
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-11-20 00:00:00.000000000 Z
+date: 2016-10-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>'
+    - - ">"
       - !ruby/object:Gem::Version
         version: 4.0.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>'
+    - - ">"
       - !ruby/object:Gem::Version
         version: 4.0.0
 - !ruby/object:Gem::Dependency
   name: sqlite3
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: pry
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: rspec-rails
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: binding_of_caller
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: better_errors
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '1.1'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '1.1'
 - !ruby/object:Gem::Dependency
   name: codeclimate-test-reporter
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: webmock
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '1.21'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '1.21'
 - !ruby/object:Gem::Dependency
   name: simplecov
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '0.7'
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: 0.7.1
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '0.7'
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: 0.7.1
 description: The /routes route will list all of the consuming application's routes
@@ -171,22 +171,22 @@ require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
-  - - '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.8
+rubygems_version: 2.5.1
 signing_key: 
 specification_version: 4
 summary: Provides the route /routes which will list the applications routes
 test_files:
 - spec/config/initializers/response_injections_spec.rb
-- spec/controllers/routes_controller_spec.rb
-- spec/spec_helper.rb
 - spec/support/helpers/controller_helpers.rb
+- spec/spec_helper.rb
+- spec/controllers/routes_controller_spec.rb