grep_routes 0.1.0 → 0.1.1

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    grep_routes (0.0.6)
+    grep_routes (0.1.0)
 
 GEM
   remote: http://rubygems.org/

data/grep_routes.gemspec

@@ -2,7 +2,7 @@
 $:.push File.expand_path("../lib", __FILE__)
 Gem::Specification.new do |s|
   s.name        = "grep_routes"
-  s.version     = "0.1.0"
+  s.version     = "0.1.1"
   s.authors     = ["Tyler Montgomery"]
   s.email       = ["tyler.a.montgomery@gmail.com"]
   s.homepage    = "http://github.com/ubermajestix/grep_routes"

data/lib/grep_routes.rb

@@ -9,6 +9,7 @@ end
 require 'active_support'
 require 'active_support/core_ext/hash/reverse_merge'
 require 'active_support/core_ext/enumerable'
+require 'active_support/inflector'
 require 'grep_routes_mock_rack_app'
 
 # See if the user has the 3.2 or 3.1 version of actionpack for action_dispatch.
@@ -30,14 +31,20 @@ class GrepRoutes
   
   attr_reader :path_to_routes_file
   attr_reader :route_file
-  attr_reader :class_name
+  attr_reader :rails_class
   attr_reader :pattern
+  attr_reader :classes
+  attr_reader :classes_with_methods
+  attr_reader :eval_failures
   
   def initialize(path_to_routes_file)
     @path_to_routes_file = path_to_routes_file
-    @route_file = File.read(path_to_routes_file)
-    @class_name = route_file.match(/(\w+)::Application\.routes\.draw/)[1]
+    @route_file = ""
+    File.open(path_to_routes_file){|f| f.each_line{|l| @route_file << l unless l.strip[0]=='#' }}
+    @rails_class = route_file.match(/(\w+)::Application\.routes\.draw/)[1]
+    @eval_failures = 0
     self.init_rails_class
+    self.init_other_classes
   end
   
   # To make this fast we don't load your rails app or any of your gems.
@@ -56,8 +63,57 @@ class GrepRoutes
   end
   
   def rails_app
-    Object.const_set(class_name,Module.new) unless Object.const_defined?(class_name)
-    Object.const_get(class_name,Module.new)
+    Object.const_set(rails_class,Module.new) unless Object.const_defined?(rails_class)
+    Object.const_get(rails_class,Module.new)
+  end
+  
+  def init_other_classes
+    find_classes
+    find_classes_with_methods
+    classes.each do |class_name|
+      define_objects(class_name)
+    end
+    # This is causing problems: 'Stack level too deep' during eval, the retry will
+    # catch any undefined methods and define them on the right class
+    # classes_with_methods.each do |class_and_method|
+    #     # define_methods(*class_and_method)
+    #   end
+  end
+  
+  def find_classes
+    @classes = route_file.scan(/([A-Z][a-zA-Z0-9::]*)/).flatten.reject{|c| c.match(rails_class) || c.match("Rails") }
+  end
+  
+  def find_classes_with_methods
+    match = route_file.scan(/([A-Z][a-zA-Z0-9::]*)\.([_a-z]+[_a-z0-9_<>=~@\[\]]*[=!\?]?)/)
+    match.reject!{|m| m.first.match(rails_class) || m.first.match("Rails") }
+    # parse out the first method if there's a chain
+    # TODO deal with method chains like `app.rack.server`
+    match.each{|m| m[1] = m.last.split(".").first}
+    @classes_with_methods = match
+  end
+  
+  def define_objects(class_name)
+    objects = class_name.split("::")
+    rack_class = objects.pop
+    last_object = Object
+    objects.each do |obj|
+       last_object.const_set(obj, Module.new) unless last_object.const_defined?(obj)
+       last_object = last_object.const_get(obj)
+    end
+    make_rackapp(last_object, rack_class)
+  end
+  
+  # TODO there has got to be a better way to do this!
+  def define_methods(class_name, method_name)
+    class_name.constantize.class_eval do
+      method_to_eval = <<-method_to_eval
+      def self.#{method_name}
+        GrepRoutesMockRackApp.new
+      end
+      method_to_eval
+      eval(method_to_eval)
+    end
   end
   
   def make_rackapp(mod, obj)
@@ -66,44 +122,25 @@ class GrepRoutes
   
   # This evals the routes file. After this method is called the RouteSet will 
   # have all of our routes inside it.
-  
   def eval_routes
-    # no_method_retries = 3
     begin
       eval(route_file)
     # If a method is not defined on a class, we define it and try again
     rescue NoMethodError => e
+      @eval_failures += 1
       match = e.message.match(/undefined method `(.+)' for (.+):Class/)
       undefined_method = match[1]
-      # TODO there has got to be a better way to do this!
-      obj = Object.const_get(match[2], Class.new).class_eval do
-        method_to_eval = <<-method_to_eval
-        def self.#{undefined_method}
-          GrepRoutesMockRackApp.new
-        end
-        method_to_eval
-        eval(method_to_eval)
-      end
+      define_methods(match[2], undefined_method)
       retry
     # If a class is not defined we define it and try again
     rescue NameError => e
+      @eval_failures += 1
       class_name = e.message.match(/uninitialized constant (.+)$/)[1].gsub("GrepRoutes::", '')
       define_objects(class_name)
       retry
     end
   end
   
-  def define_objects(class_name)
-    objects = class_name.split("::")
-    rack_class = objects.pop
-    last_object = Object
-    objects.each do |obj|
-       last_object.const_set(obj, Module.new) unless last_object.const_defined?(obj)
-       last_object = last_object.const_get(obj)
-    end
-    make_rackapp(last_object, rack_class)
-  end
-  
   # A shortcut to the RouteSet we defined in init_rails_class.
   def route_set
     rails_app.const_get('Application', Class.new).routes

data/spec/finding_classes_spec.rb

@@ -0,0 +1,21 @@
+require 'spec_helper'
+
+describe 'finding classes in routes file' do
+  subject do
+    # The initialization loades up classes and classes_with_methods
+    GrepRoutes.new('./spec/fixtures/routes_with_two_classes.rb')
+  end
+  
+  it "should find two classes excluding the Rails application class" do
+    subject.classes.length.must_equal 2
+  end
+  
+  it "should find one class that has a method defined on it" do
+    subject.classes_with_methods.length.must_equal 1 
+  end
+  
+  it "curiously has to define methods on GrepRoutesMockRackApp" do
+    subject.eval_routes
+    subject.eval_failures.must_equal 1
+  end
+end

data/spec/fixtures/routes_with_two_classes.rb

@@ -0,0 +1,5 @@
+SomeRailsApp::Application.routes.draw do
+  resources :user
+  mount Someapp, at: '/some/app'
+  mount Another::App.server!, at: '/another'
+end

data/spec/with_rack_mounts_spec.rb

@@ -24,4 +24,8 @@ describe GrepRoutes do
     subject.routes.map{|r| r[:path]}.must_include "/server/new"
   end
   
+  it "should not have any eval failures and be fast" do
+    subject.eval_failures.must_equal 0
+  end
+  
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: grep_routes
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.1.1
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-02-25 00:00:00.000000000 Z
+date: 2012-02-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
-  requirement: &70270111676060 !ruby/object:Gem::Requirement
+  requirement: &70366093883200 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -21,10 +21,10 @@ dependencies:
         version: '3.1'
   type: :development
   prerelease: false
-  version_requirements: *70270111676060
+  version_requirements: *70366093883200
 - !ruby/object:Gem::Dependency
   name: actionpack
-  requirement: &70270111675140 !ruby/object:Gem::Requirement
+  requirement: &70366093882280 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -32,10 +32,10 @@ dependencies:
         version: '3.1'
   type: :development
   prerelease: false
-  version_requirements: *70270111675140
+  version_requirements: *70366093882280
 - !ruby/object:Gem::Dependency
   name: minitest
-  requirement: &70270111674580 !ruby/object:Gem::Requirement
+  requirement: &70366093881740 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -43,7 +43,7 @@ dependencies:
         version: 2.11.2
   type: :development
   prerelease: false
-  version_requirements: *70270111674580
+  version_requirements: *70366093881740
 description: Greppin in ur routes
 email:
 - tyler.a.montgomery@gmail.com
@@ -62,8 +62,10 @@ files:
 - lib/grep_routes.rb
 - lib/grep_routes_mock_rack_app.rb
 - spec/basic_routes_spec.rb
+- spec/finding_classes_spec.rb
 - spec/fixtures/routes.rb
 - spec/fixtures/routes_with_rack_mounts.rb
+- spec/fixtures/routes_with_two_classes.rb
 - spec/spec_helper.rb
 - spec/with_rack_mounts_spec.rb
 homepage: http://github.com/ubermajestix/grep_routes
@@ -92,7 +94,9 @@ specification_version: 3
 summary: Fast Routes for Rails
 test_files:
 - spec/basic_routes_spec.rb
+- spec/finding_classes_spec.rb
 - spec/fixtures/routes.rb
 - spec/fixtures/routes_with_rack_mounts.rb
+- spec/fixtures/routes_with_two_classes.rb
 - spec/spec_helper.rb
 - spec/with_rack_mounts_spec.rb