added test_command_helpers.rb, fixed spelling error in comments in command_set.rb

lib/pry/command_set.rb

@@ -87,7 +87,7 @@ class Pry
     end
 
     # Removes some commands from the set
-    # @param [Arary<String>] names name of the commands to remove
+    # @param [Array<String>] names name of the commands to remove
     def delete(*names)
       names.each { |name| commands.delete name }
     end

test/test_command_helpers.rb

@@ -0,0 +1,77 @@
+require 'helper'
+
+describe Pry::Helpers::CommandHelpers do
+  before do
+    @helper = Pry::Helpers::CommandHelpers
+  end
+
+  describe "get_method_object" do
+    it 'should look up instance methods if no methods available and no options provided' do
+      klass = Class.new { def hello; end }
+      meth = @helper.get_method_object(:hello, Pry.binding_for(klass), {})
+      meth.should == klass.instance_method(:hello)
+    end
+
+    it 'should look up methods if no instance methods available and no options provided' do
+      klass = Class.new { def self.hello; end }
+      meth = @helper.get_method_object(:hello, Pry.binding_for(klass), {})
+      meth.should == klass.method(:hello)
+    end
+
+    it 'should look up instance methods first even if methods available and no options provided' do
+      klass = Class.new { def hello; end; def self.hello; end  }
+      meth = @helper.get_method_object(:hello, Pry.binding_for(klass), {})
+      meth.should == klass.instance_method(:hello)
+    end
+
+    it 'should look up instance methods if "instance-methods"  option provided' do
+      klass = Class.new { def hello; end; def self.hello; end  }
+      meth = @helper.get_method_object(:hello, Pry.binding_for(klass), {"instance-methods" => true})
+      meth.should == klass.instance_method(:hello)
+    end
+
+    it 'should look up methods if :methods  option provided' do
+      klass = Class.new { def hello; end; def self.hello; end  }
+      meth = @helper.get_method_object(:hello, Pry.binding_for(klass), {:methods => true})
+      meth.should == klass.method(:hello)
+    end
+
+    it 'should look up instance methods using the Class#method syntax' do
+      klass = Class.new { def hello; end; def self.hello; end  }
+      meth = @helper.get_method_object("klass#hello", Pry.binding_for(binding), {})
+      meth.should == klass.instance_method(:hello)
+    end
+
+    it 'should look up methods using the object.method syntax' do
+      klass = Class.new { def hello; end; def self.hello; end  }
+      meth = @helper.get_method_object("klass.hello", Pry.binding_for(binding), {})
+      meth.should == klass.method(:hello)
+    end
+
+    it 'should NOT look up instance methods using the Class#method syntax if no instance methods defined' do
+      klass = Class.new { def self.hello; end  }
+      meth = @helper.get_method_object("klass#hello", Pry.binding_for(binding), {})
+      meth.should == nil
+    end
+
+    it 'should NOT look up methods using the object.method syntax if no methods defined' do
+      klass = Class.new { def hello; end  }
+      meth = @helper.get_method_object("klass.hello", Pry.binding_for(binding), {})
+      meth.should == nil
+    end
+
+    it 'should look up methods using klass.new.method syntax' do
+      klass = Class.new { def hello; :hello; end }
+      meth = @helper.get_method_object("klass.new.hello", Pry.binding_for(binding), {})
+      meth.name.to_sym.should == :hello
+    end
+
+    it 'should look up instance methods using klass.meth#method syntax' do
+      klass = Class.new { def self.meth; Class.new; end }
+      meth = @helper.get_method_object("klass.meth#initialize", Pry.binding_for(binding), {})
+      meth.name.to_sym.should == :initialize
+    end
+  end
+end
+
+