traceroute 0.8.0 → 0.8.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 291c384ad195a4230680d6d2a39d0f408e60d16eb0b51427d9d2fda091c162fb
-  data.tar.gz: 8f23318498f820271d61e9d841d8970e3dbff7de0a9d61708a77bd5243f48f2e
+  metadata.gz: 6846d384263474eece3eb673268a34b514b0ec7b9f096073e2b45f3af8817bc6
+  data.tar.gz: 2776ad5760ade33db8534647b0dfdb2ca7df609bd3feeeb13c10fedbd6a8e755
 SHA512:
-  metadata.gz: f165fa1c116887d9993c06da4a29872e4f72d816724314878e24d619bc893e074897b36b64ff819656cce2fca14b492664723e3dc7e4d3a28da2e1d39f19b7ab
-  data.tar.gz: '069cf2f92bd6bd7468e64c1b2bbe6f5431b855dc757613ee1aa0b87322fd02d57cbe90e7c813a2e3135e3b7d3237c65a6572e6a72a2b0ecfd721af603ed17ce6'
+  metadata.gz: 2088865d44a331d44da1033107db6aac13c8c0b782d715cb0661be0f0966b6addc35782b6d6c140e24ffa594e904007a43017d5b92e712bc8d8ac43b8a7e86e5
+  data.tar.gz: 46354a4b08c79d8d9af15bf40f7297141f00d4af438b06712cbc73e2190a1fd55d3ceb56ab2c66368e19781c4b559e819d7bef49feb6a647c80b8eb91c789adb

data/README.rdoc

@@ -10,9 +10,9 @@ This Rake task investigates your Rails application's routes definition, then sho
 
 == Supported versions
 
-* Ruby 1.8.7, 1.9.2, 1.9.3, 2.0, 2.1, 2.2, 2.3, 2.4, 2.5, 2.6 (trunk)
+* Ruby 1.8.7, 1.9.2, 1.9.3, 2.0, 2.1, 2.2, 2.3, 2.4, 2.5, 2.6, 2.7, 2.8 (trunk)
 
-* Rails 3.0.x, 3.1.x, 3.2.x, 4.0.x, 4.1.x, 5.0.x, 5.1.x
+* Rails 3.0.x, 3.1.x, 3.2.x, 4.0.x, 4.1.x, 5.0.x, 5.1.x, 5.2.x, 6.0, 6.1 (master)
 
 
 == Install

data/lib/tasks/traceroute.rake

@@ -5,12 +5,8 @@ task :traceroute => :environment do
   traceroute = Traceroute.new Rails.application
   traceroute.load_everything!
 
-  defined_action_methods = traceroute.defined_action_methods
-
-  routed_actions = traceroute.routed_actions
-
   unless ENV['UNREACHABLE_ACTION_METHODS_ONLY']
-    unused_routes = routed_actions - defined_action_methods
+    unused_routes = traceroute.unused_routes
     puts "Unused routes (#{unused_routes.count}):"
     unused_routes.each {|route| puts "  #{route}"}
   end
@@ -18,7 +14,7 @@ task :traceroute => :environment do
   puts unless (ENV['UNREACHABLE_ACTION_METHODS_ONLY'] || ENV['UNUSED_ROUTES_ONLY'])
 
   unless ENV['UNUSED_ROUTES_ONLY']
-    unreachable_action_methods = defined_action_methods - routed_actions
+    unreachable_action_methods = traceroute.unreachable_action_methods
     puts "Unreachable action methods (#{unreachable_action_methods.count}):"
     unreachable_action_methods.each {|action| puts "  #{action}"}
   end

data/lib/traceroute.rb

@@ -42,8 +42,16 @@ class Traceroute
     Rails::Engine.subclasses.each(&:eager_load!)
   end
 
+  def unused_routes
+    routed_actions - defined_action_methods
+  end
+
+  def unreachable_action_methods
+    defined_action_methods - routed_actions
+  end
+
   def defined_action_methods
-    [ActionController::Base, (ActionController::API if defined?(ActionController::API))].compact.map do |klass|
+    @defined_action_methods ||= [ActionController::Base, (ActionController::API if defined?(ActionController::API))].compact.map do |klass|
       klass.descendants.map do |controller|
         controller.action_methods.reject {|a| (a =~ /\A(_conditional)?_callback_/) || (a == '_layout_from_proc')}.map do |action|
           "#{controller.controller_path}##{action}"
@@ -53,7 +61,7 @@ class Traceroute
   end
 
   def routed_actions
-    routes.map do |r|
+    @routed_actions ||= routes.map do |r|
       if r.requirements[:controller].present? && r.requirements[:action].present?
         "#{r.requirements[:controller]}##{r.requirements[:action]}"
       elsif (String === r.path) && (WILDCARD_ROUTES =~ r.path)
@@ -72,7 +80,7 @@ class Traceroute
 
   def collect_routes(routes)
     routes = routes.each_with_object([]) do |r, tmp_routes|
-      next if (ActionDispatch::Routing::Mapper::Constraints === r.app) && (ActionDispatch::Routing::PathRedirect === r.app.app)
+      next if (ActionDispatch::Routing::Mapper::Constraints === r.app) && (%w[ActionDispatch::Routing::PathRedirect ActionDispatch::Routing::Redirect].include?(r.app.app.class.name))
 
       if r.app.is_a?(ActionDispatch::Routing::Mapper::Constraints) && r.app.app.respond_to?(:routes)
         engine_routes = r.app.app.routes

data/test/traceroute_test.rb

@@ -22,6 +22,8 @@ module TracerouteTest
       DummyApp::Application.routes.draw do
         resources :users, :only => [:index, :show, :new, :create]
 
+        get '/', to: redirect('/users'), constraints: lambda { true }
+
         namespace :admin do
           resources :shops, :only => :index
         end

data/traceroute.gemspec

@@ -3,7 +3,7 @@ $:.push File.expand_path("../lib", __FILE__)
 
 Gem::Specification.new do |s|
   s.name        = "traceroute"
-  s.version     = '0.8.0'
+  s.version     = '0.8.1'
   s.platform    = Gem::Platform::RUBY
   s.authors     = ['Akira Matsuda']
   s.email       = ['ronnie@dio.jp']

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: traceroute
 version: !ruby/object:Gem::Version
-  version: 0.8.0
+  version: 0.8.1
 platform: ruby
 authors:
 - Akira Matsuda
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-10-31 00:00:00.000000000 Z
+date: 2020-04-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -66,8 +66,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.7.6
+rubygems_version: 3.1.2
 signing_key: 
 specification_version: 4
 summary: A Rake task that helps you find the dead routes and actions for your Rails