matthew-method_lister 0.2.3 → 0.3.0

data/lib/method_lister/find_result.rb

@@ -3,8 +3,8 @@ module MethodLister
     attr_reader :object
     VISIBILITIES = [:public, :protected, :private]
     
-    def initialize(options={})
-      @object = options[:object] || raise(ArgumentError, "No :object given.")
+    def initialize(object, options={})
+      @object = object
       @methods = Hash.new
       VISIBILITIES.each do |visibility|
         @methods[visibility] = options[visibility] || Array.new

data/lib/method_lister/finder.rb

@@ -45,16 +45,16 @@ module MethodLister
     end
     
     def record_methods_for(klass_or_module)
-      record(
-        :object    => klass_or_module,
+      record_result(
+        klass_or_module,
         :public    => klass_or_module.public_instance_methods(false),
         :protected => klass_or_module.protected_instance_methods(false),
         :private   => klass_or_module.private_instance_methods(false)
       )
     end
     
-    def record(result_options)
-      @results << FindResult.new(result_options) 
+    def record_result(*args)
+      @results << FindResult.new(*args) 
     end
     
     def modules_for(obj_type, klass_or_module)
@@ -90,7 +90,7 @@ module MethodLister
         ancestor.private_instance_methods(false)
       end.flatten.uniq
       
-      record(:object    => object, 
+      record_result(object, 
              :public    => singleton_methods & public_methods,
              :protected => singleton_methods & protected_methods,
              :private   => private_methods - ancestor_privates)

data/spec/find_result_spec.rb

@@ -9,24 +9,23 @@ describe MethodLister::FindResult do
     @all       = @public + @protected + @private
 
     @object         = Object.new
-    @empty_result   = MethodLister::FindResult.new(:object => @object)
-    @only_public    = MethodLister::FindResult.new(:object => @object, 
+    @empty_result   = MethodLister::FindResult.new(@object)
+    @only_public    = MethodLister::FindResult.new(@object, 
                                                    :public => @public)
-    @only_protected = MethodLister::FindResult.new(:object => @object, 
+    @only_protected = MethodLister::FindResult.new(@object, 
                                                    :protected => @protected)
-    @only_private   = MethodLister::FindResult.new(:object => @object,
+    @only_private   = MethodLister::FindResult.new(@object,
                                                    :private => @private)
-    @mixed_result   = MethodLister::FindResult.new(:object => @object,
+    @mixed_result   = MethodLister::FindResult.new(@object,
                                                    :public => @public,
                                                    :protected => @protected,
                                                    :private => @private)
   end
   
   describe "#object" do
-    it "requires at least an associated object" do
-      lambda do
-        MethodLister::FindResult.new(:public => @public)
-      end.should raise_error(ArgumentError)
+    it "works even if object is nil" do
+      result = MethodLister::FindResult.new(nil, :public => @public)
+      result.object.should be_nil
     end
     
     it "knows its associated object" do
@@ -136,7 +135,7 @@ describe MethodLister::FindResult do
   describe "#==" do
     it "is equal if both have the same methods and object" do
       @mixed_result.should == MethodLister::FindResult.new(
-        :object    => @mixed_result.object,
+        @mixed_result.object,
         :public    => @mixed_result.methods(:public),
         :protected => @mixed_result.methods(:protected),
         :private   => @mixed_result.methods(:private)
@@ -145,7 +144,7 @@ describe MethodLister::FindResult do
     
     it "is not equal if both have same methods but different objects" do
       @mixed_result.should_not == MethodLister::FindResult.new(
-        :object    => Object.new,
+        Object.new,
         :public    => @mixed_result.methods(:public),
         :protected => @mixed_result.methods(:protected),
         :private   => @mixed_result.methods(:private)
@@ -154,7 +153,7 @@ describe MethodLister::FindResult do
 
     it "is not equal if both have the same object but different methods" do
       @mixed_result.should_not == MethodLister::FindResult.new(
-        :object    => @mixed_result.object,
+        @mixed_result.object,
         :public    => ["something_else"]
       )
     end

data/spec/helpers/object_mother/find_result.rb

@@ -1,3 +1,3 @@
 def result(object, options)
-  MethodLister::FindResult.new(options.merge(:object => object))
+  MethodLister::FindResult.new(object, options)
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: matthew-method_lister
 version: !ruby/object:Gem::Version 
-  version: 0.2.3
+  version: 0.3.0
 platform: ruby
 authors: 
 - Matthew O'Connor
@@ -9,7 +9,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2008-10-27 00:00:00 -07:00
+date: 2009-05-16 00:00:00 -07:00
 default_executable: 
 dependencies: []