method_info 0.0.0 → 0.0.1

data/.gitignore

@@ -19,3 +19,4 @@ rdoc
 pkg
 
 ## PROJECT::SPECIFIC
+method_info.gemspec

data/README.rdoc

@@ -4,8 +4,12 @@ Provides info about methods that can be called on an object and where they are d
 
 Usage:
 
-require 'method_info'
-"abc".method_info
+>> require 'method_info'
+=> true
+>> "abc".method_info.ancestors
+=> [String, Enumerable, Comparable, Object, MethodInfoMethod, Kernel]
+>> "abc".method_info.method_owner(:==)
+=> String 
 
 == Note on Patches/Pull Requests
  

data/VERSION

@@ -1 +1 @@
-0.0.0
+0.0.1

data/lib/method_info.rb

@@ -1,4 +1,3 @@
-
 module MethodInfoMethod
   def method_info
     MethodInfo.new(self)
@@ -25,18 +24,57 @@ class MethodInfo
     @ancestors += @object.class.ancestors
   end
 
+  # Returns the class or module where method is defined
   def method_owner(method)
     @object.method(method).owner
   end
 
+  # Returns the same value as :method_owner, but also check if the
+  # method is handled by a method_missing method in the chain. If it
+  # is :method_missing is returned, otherwise the error raised is
+  # reraised. This requires an invocation of the method which could
+  # cause side effects. Hence this method is considered to be
+  # dangerous.
   def method_owner!(method)
     method_owner(method)
   rescue NameError => e
     begin
-      @object.send(method)
-      method_owner(:method_missing)
+      @object.clone.send(method)
+      :method_missing
     rescue NoMethodError
       raise e
     end
   end
+
+  def method_map
+    @method_map = Hash.new
+    current_ancestors = ancestors
+    @method_map['__ancestors'] = current_ancestors
+    current_ancestors.each do |ancestor|
+      @method_map[ancestor] = []
+    end
+
+    @object.methods.each do |method|
+      @method_map[method_owner(method)] << method
+    end
+    @method_map
+  end
+
+  def to_s
+    map = method_map
+
+    result = ""
+    map['__ancestors'].each do |ancestor|
+      break if ancestor == Object
+      next if map[ancestor].empty?
+      result +=
+        "=== #{ancestor} ===\n" +
+        map[ancestor].sort.join(", ") +
+        "\n"
+    end
+    index_of_object = map['__ancestors'].index(Object)
+    result +=
+      "=== #{map['__ancestors'][index_of_object..-1].join(", ")} ==="
+    result
+  end
 end

data/spec/method_info_spec.rb

@@ -50,6 +50,13 @@ describe MethodInfo do
       def duplicate_instance_method
         :abstract_duplicate_instance_method
       end
+
+      def method_missing(method)
+        if method == :missing_method_handled_at_abstract
+          return :missing_method_handled_at_abstract
+        end
+        super
+      end
     end
 
     class ConcreteMethodOwnerDummy < AbstractMethodOwnerDummy
@@ -83,6 +90,10 @@ describe MethodInfo do
       lambda { ConcreteMethodOwnerDummy.new.method_info.method_owner(:poof) }.should raise_error(NameError)
     end
 
+    it "raises an error if the method is handled by :method_missing" do
+      lambda { ConcreteMethodOwnerDummy.new.method_info.method_owner(:missing_method_handled_at_concrete) }.should raise_error(NameError)
+    end
+
     describe "method_owner!" do
       it "is the class of the object for an instance_method" do
         ConcreteMethodOwnerDummy.new.method_info.method_owner!(:concrete_instance_method).should ==
@@ -98,9 +109,47 @@ describe MethodInfo do
         lambda { ConcreteMethodOwnerDummy.new.method_info.method_owner!(:poof) }.should raise_error(NameError)
       end
 
-      it "is the first method missing method if there is one that handles the method" do
+      it "is :method_missing if the concrete class handles the method" do
         ConcreteMethodOwnerDummy.new.method_info.method_owner!(:missing_method_handled_at_concrete).should ==
-          "ConcreteMethodOwnerDummy::method_missing"
+          :method_missing
+      end
+
+      it "is :method_missing if the abstract class handles the method" do
+        ConcreteMethodOwnerDummy.new.method_info.method_owner!(:missing_method_handled_at_abstract).should ==
+          :method_missing
+      end
+
+      it "is :method_missing if the object has a method_missing singleton method that handles the method" do
+        monkey = Object.new
+        def monkey.method_missing(method)
+          if method == :missing_method_handled_at_singleton_method
+            return :missing_method_handled_at_singleton_method
+          end
+          super
+        end
+        monkey.method_info.method_owner!(:missing_method_handled_at_singleton_method).should ==
+          :method_missing
+      end
+
+      it "does not modify an object whose method_missing has side effects" do
+        monkey = Object.new
+        monkey.instance_eval { @hair = "brown" }
+        def monkey.method_missing(method)
+          @hair = "blue"
+        end
+        monkey.method_info.method_owner!(:unknown_method)
+        monkey.instance_eval('@hair').should == "brown"
+      end
+
+      # Undesirable behaviour, but I don't think there is an easy way around it
+      it "will not protect an object's objects from method_missing side effects" do
+        monkey = Object.new
+        monkey.instance_eval { @limbs = [:arms, :legs] }
+        def monkey.method_missing(method)
+          @limbs.shift
+        end
+        monkey.method_info.method_owner!(:unknown_method)
+        monkey.instance_eval('@limbs').should == [:legs]
       end
 
       it "raises an error if the object does not respond to the method" do

data/spec/spec_helper.rb

@@ -3,7 +3,6 @@ $LOAD_PATH.unshift(File.join(File.dirname(__FILE__), '..', 'lib'))
 require 'method_info'
 require 'spec'
 require 'spec/autorun'
-require 'ruby-debug'
 
 Spec::Runner.configure do |config|
   

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: method_info
 version: !ruby/object:Gem::Version 
-  version: 0.0.0
+  version: 0.0.1
 platform: ruby
 authors: 
 - Tom ten Thij