evalhook 0.5.4 → 0.5.5

data/CHANGELOG

@@ -1,3 +1,9 @@
+0.5.5 Fixed handling of method_missing
+
+0.5.4 Fixed release issues
+
+0.5.3 Added license to gemspec
+
 0.5.1 Fixed compatibility issues with sexp_processor, ruby_parser and ruby2ruby
 
 0.5.0	Implemented code packets (see documentation)

data/Rakefile

@@ -6,7 +6,7 @@ require 'rake/testtask'
 
 spec = Gem::Specification.new do |s|
   s.name = 'evalhook'
-  s.version = '0.5.4'
+  s.version = '0.5.5'
   s.author = 'Dario Seminara'
   s.email = 'robertodarioseminara@gmail.com'
   s.platform = Gem::Platform::RUBY

data/lib/evalhook.rb

@@ -139,8 +139,15 @@ module EvalHook
 
     def hooked_method(receiver, mname, klass = nil) #:nodoc:
       m = nil
+      is_method_missing = false
+
       unless klass
-        m = receiver.method(mname)
+        m = begin
+          receiver.method(mname)
+        rescue
+          is_method_missing = true
+          receiver.public_method(:method_missing)
+        end
         klass = m.owner
       else
         m = klass.instance_method(mname).bind(receiver)
@@ -160,6 +167,10 @@ module EvalHook
         end
       end
 
+      if is_method_missing
+        orig_m = m
+        m = lambda{|*x| orig_m.call(mname,*x) }
+      end
       m
     end
 
@@ -168,7 +179,14 @@ module EvalHook
       if local_vars.include? mname.to_s
         HookedCallValue.new( _binding.eval(mname.to_s) )
       else
-        m = receiver.method(mname)
+        is_method_missing = false
+        m = begin 
+          receiver.method(mname)
+        rescue
+          is_method_missing = true
+          receiver.method(:method_missing)
+        end
+
         klass = m.owner
         ret = handle_method(klass, receiver, mname )
 
@@ -184,6 +202,10 @@ module EvalHook
           end
         end
 
+        if is_method_missing
+          orig_m = m
+          m = lambda{|*x| orig_m.call(mname,*x) }
+        end
         m
       end
     end
@@ -285,7 +307,15 @@ module EvalHook
     end
 
     def private_method_check(recv, mname)
-      recv.public_method(mname) rescue NameError raise NoMethodError
+      begin
+        recv.public_method(mname) 
+      rescue NameError 
+        begin
+          recv.public_method(:method_missing) 
+        rescue NameError
+          raise NoMethodError
+        end
+      end
       recv
     end
 

data/spec/call/call_spec.rb

@@ -3,7 +3,7 @@ require "evalhook"
 
 describe EvalHook::HookHandler, "hook handler hooks" do
 
-  it "should raise NoMethodError when tring to call private method" do
+  it "should raise NoMethodError when trying to call private method" do
     hh = EvalHook::HookHandler.new
     lambda {
       hh.evalhook("{}.system('ls -l')", binding)
@@ -11,7 +11,7 @@ describe EvalHook::HookHandler, "hook handler hooks" do
   end
 
 
-  it "should raise NoMethodError when tring to call private method" do
+  it "should raise NoMethodError when trying to call private method" do
     hh = EvalHook::HookHandler.new
     lambda {
       hh.evalhook("
@@ -29,4 +29,39 @@ describe EvalHook::HookHandler, "hook handler hooks" do
     }.should_not raise_error
   end
 
+  class XDefinedOutside55
+    def method_missing(name)
+      name
+    end
+  end
+  XDefinedOutside55_ins = XDefinedOutside55.new
+
+  context "when there is method_missing defined" do
+    it "shouldn't raise NoMethodError when trying to call a public method NOT defined" do
+      hh = EvalHook::HookHandler.new
+      lambda {
+        hh.evalhook("
+          XDefinedOutside55_ins.foo
+          ", binding).should be == :foo
+      }.should_not raise_error
+    end
+  end
+
+  class XDefinedOutside56
+    def method_missing(name, param1)
+      name
+    end
+  end
+  XDefinedOutside56_ins = XDefinedOutside56.new
+  
+  context "when there is method_missing defined (two arguments)" do
+    it "shouldn't raise NoMethodError when trying to call a public method NOT defined" do
+      hh = EvalHook::HookHandler.new
+      lambda {
+        hh.evalhook("
+          XDefinedOutside56_ins.foo(1)
+          ", binding).should be == :foo
+      }.should_not raise_error
+    end
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: evalhook
 version: !ruby/object:Gem::Version
-  version: 0.5.4
+  version: 0.5.5
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-09-05 00:00:00.000000000 Z
+date: 2013-09-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: partialruby
-  requirement: &81627980 !ruby/object:Gem::Requirement
+  requirement: &71965870 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,10 +21,10 @@ dependencies:
         version: 0.2.3
   type: :runtime
   prerelease: false
-  version_requirements: *81627980
+  version_requirements: *71965870
 - !ruby/object:Gem::Dependency
   name: ruby_parser
-  requirement: &81627690 !ruby/object:Gem::Requirement
+  requirement: &71965480 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -32,7 +32,7 @@ dependencies:
         version: '2.0'
   type: :runtime
   prerelease: false
-  version_requirements: *81627690
+  version_requirements: *71965480
 description: 
 email: robertodarioseminara@gmail.com
 executables: []