pry 0.9.10 → 0.9.11

data/spec/commands/show_doc_spec.rb

@@ -0,0 +1,378 @@
+require 'helper'
+require "fixtures/show_source_doc_examples"
+
+if !PryTestHelpers.mri18_and_no_real_source_location?
+  describe "show-doc" do
+    before do
+      @o = Object.new
+
+      # sample doc
+      def @o.sample_method
+        :sample
+      end
+    end
+
+    it 'should output a method\'s documentation' do
+      pry_eval(binding, "show-doc @o.sample_method").should =~ /sample doc/
+    end
+
+    it 'should output a method\'s documentation with line numbers' do
+      pry_eval(binding, "show-doc @o.sample_method -l").should =~ /\d: sample doc/
+    end
+
+    it 'should output a method\'s documentation with line numbers (base one)' do
+      pry_eval(binding, "show-doc @o.sample_method -b").should =~ /1: sample doc/
+    end
+
+    it 'should output a method\'s documentation if inside method without needing to use method name' do
+      # sample comment
+      def @o.sample
+        pry_eval(binding, 'show-doc').should =~ /sample comment/
+      end
+      @o.sample
+    end
+
+    it "should be able to find super methods" do
+      b = Class.new{
+        # daddy initialize!
+        def initialize(*args) ;end
+      }
+
+      c = Class.new(b){
+        # classy initialize!
+        def initialize(*args); end
+      }
+
+      d = Class.new(c){
+        # grungy initialize??
+        def initialize(*args, &block); end
+      }
+
+      o = d.new
+
+      # instancey initialize!
+      def o.initialize; end
+
+      t = pry_tester(binding)
+
+      t.eval("show-doc o.initialize").should =~ /instancey initialize/
+      t.eval("show-doc --super o.initialize").should =~ /grungy initialize/
+      t.eval("show-doc o.initialize -ss").should =~ /classy initialize/
+      t.eval("show-doc o.initialize -sss").should =~ /daddy initialize/
+    end
+
+    describe "rdoc highlighting" do
+      it "should syntax highlight code in rdoc" do
+        c = Class.new{
+          # This can initialize your class:
+          #
+          #   a = c.new :foo
+          #
+          # @param foo
+          def initialize(foo); end
+        }
+
+        begin
+          t = pry_tester(binding)
+          t.eval("show-doc c#initialize").should =~ /c.new :foo/
+          Pry.config.color = true
+          # I don't want the test to rely on which colour codes are there, just to
+          # assert that "something" is being colourized.
+          t.eval("show-doc c#initialize").should.not =~ /c.new :foo/
+        ensure
+          Pry.config.color = false
+        end
+      end
+
+      it "should syntax highlight `code` in rdoc" do
+        c = Class.new{
+          # After initializing your class with `c.new(:foo)`, go have fun!
+          #
+          # @param foo
+          def initialize(foo); end
+        }
+
+        begin
+          t = pry_tester(binding)
+          t.eval("show-doc c#initialize").should =~ /c.new\(:foo\)/
+          Pry.config.color = true
+          # I don't want the test to rely on which colour codes are there, just to
+          # assert that "something" is being colourized.
+          t.eval("show-doc c#initialize").should.not =~ /c.new\(:foo\)/
+        ensure
+          Pry.config.color = false
+        end
+
+      end
+
+      it "should not syntax highlight `` inside code" do
+        c = Class.new{
+          # Convert aligned output (from many shell commands) into nested arrays:
+          #
+          #   a = decolumnize `ls -l $HOME`
+          #
+          # @param output
+          def decolumnize(output); end
+        }
+
+        begin
+          t = pry_tester(binding)
+          Pry.config.color = true
+          t.eval("show-doc c#decolumnize").should =~ /ls -l \$HOME/
+          t.eval("show-doc c#decolumnize").should.not =~ /`ls -l \$HOME`/
+        ensure
+          Pry.config.color = false
+        end
+      end
+    end
+
+    describe "on sourcable objects" do
+      it "should show documentation for object" do
+        # this is a documentation
+        hello = proc { puts 'hello world!' }
+        mock_pry(binding, "show-doc hello").should =~ /this is a documentation/
+      end
+    end
+
+    describe "on modules" do
+      before do
+        # god this is boring1
+        class ShowSourceTestClass
+          def alpha
+          end
+        end
+
+        # god this is boring2
+        module ShowSourceTestModule
+          def alpha
+          end
+        end
+
+        # god this is boring3
+        ShowSourceTestClassWeirdSyntax = Class.new do
+          def beta
+          end
+        end
+
+        # god this is boring4
+        ShowSourceTestModuleWeirdSyntax = Module.new do
+          def beta
+          end
+        end
+      end
+
+      after do
+        Object.remove_const :ShowSourceTestClass
+        Object.remove_const :ShowSourceTestClassWeirdSyntax
+        Object.remove_const :ShowSourceTestModule
+        Object.remove_const :ShowSourceTestModuleWeirdSyntax
+      end
+
+      describe "basic functionality, should show docs for top-level module definitions" do
+        it 'should show docs for a class' do
+          pry_eval("show-doc ShowSourceTestClass").should =~
+            /god this is boring1/
+        end
+
+        it 'should show docs for a module' do
+          pry_eval("show-doc ShowSourceTestModule").should =~
+            /god this is boring2/
+        end
+
+        it 'should show docs for a class when Const = Class.new syntax is used' do
+          pry_eval("show-doc ShowSourceTestClassWeirdSyntax").should =~
+            /god this is boring3/
+        end
+
+        it 'should show docs for a module when Const = Module.new syntax is used' do
+          pry_eval("show-doc ShowSourceTestModuleWeirdSyntax").should =~
+            /god this is boring4/
+        end
+      end
+
+      if !Pry::Helpers::BaseHelpers.mri_18?
+        describe "in REPL" do
+          it 'should find class defined in repl' do
+            t = pry_tester
+            t.eval <<-RUBY
+              # hello tobina
+              class TobinaMyDog
+                def woof
+                end
+              end
+            RUBY
+            t.eval('show-doc TobinaMyDog').should =~ /hello tobina/
+            Object.remove_const :TobinaMyDog
+          end
+        end
+      end
+
+      it 'should lookup module name with respect to current context' do
+        constant_scope(:AlphaClass, :BetaClass) do
+          # top-level beta
+          class BetaClass
+            def alpha
+            end
+          end
+
+          class AlphaClass
+            # nested beta
+            class BetaClass
+              def beta
+              end
+            end
+          end
+
+          pry_eval(AlphaClass, "show-doc BetaClass").should =~ /nested beta/
+        end
+      end
+
+      it 'should look up nested modules' do
+        constant_scope(:AlphaClass) do
+          class AlphaClass
+            # nested beta
+            class BetaClass
+              def beta
+              end
+            end
+          end
+
+          pry_eval("show-doc AlphaClass::BetaClass").should =~
+            /nested beta/
+        end
+      end
+
+      describe "show-doc -a" do
+        it 'should show the docs for all monkeypatches defined in different files' do
+          # local monkeypatch
+          class TestClassForShowSource
+            def beta
+            end
+          end
+
+          result = pry_eval("show-doc TestClassForShowSource -a")
+          result.should =~ /used by/
+          result.should =~ /local monkeypatch/
+        end
+      end
+
+      describe "when no class/module arg is given" do
+        before do
+          module TestHost
+
+            # hello there froggy
+            module M
+              def d; end
+              def e; end
+            end
+          end
+        end
+
+        after do
+          Object.remove_const(:TestHost)
+        end
+
+        it 'should return doc for current module' do
+          pry_eval(TestHost::M, "show-doc").should =~ /hello there froggy/
+        end
+      end
+
+      # FIXME: THis is nto a good spec anyway, because i dont think it
+      # SHOULD skip!
+      # describe "should skip over broken modules" do
+      #   before do
+      #     module TestHost
+      #       # hello
+      #       module M
+      #         binding.eval("def a; end", "dummy.rb", 1)
+      #         binding.eval("def b; end", "dummy.rb", 2)
+      #         binding.eval("def c; end", "dummy.rb", 3)
+      #       end
+
+      #       # goodbye
+      #       module M
+      #         def d; end
+      #         def e; end
+      #       end
+      #     end
+      #   end
+
+      #   after do
+      #     Object.remove_const(:TestHost)
+      #   end
+
+      #   it 'should return doc for first valid module' do
+      #     result = pry_eval("show-doc TestHost::M")
+      #     result.should =~ /goodbye/
+      #     result.should.not =~ /hello/
+      #   end
+      # end
+    end
+
+    describe "on commands" do
+      # mostly copied & modified from test_help.rb
+      before do
+        @oldset = Pry.config.commands
+        @set = Pry.config.commands = Pry::CommandSet.new do
+          import Pry::Commands
+        end
+      end
+
+      after do
+        Pry.config.commands = @oldset
+      end
+
+      it 'should display help for a specific command' do
+        pry_eval('show-doc ls').should =~ /Usage: ls/
+      end
+
+      it 'should display help for a regex command with a "listing"' do
+        @set.command /bar(.*)/, "Test listing", :listing => "foo" do; end
+        pry_eval('show-doc foo').should =~ /Test listing/
+      end
+
+      it 'should display help for a command with a spaces in its name' do
+        @set.command "command with spaces", "description of a command with spaces" do; end
+        pry_eval('show-doc command with spaces').should =~ /description of a command with spaces/
+      end
+
+      describe "class commands" do
+        before do
+          # pretty pink pincers
+          class LobsterLady < Pry::ClassCommand
+            match "lobster-lady"
+            description "nada."
+            def process
+              "lobster"
+            end
+          end
+
+          Pry.commands.add_command(LobsterLady)
+        end
+
+        after do
+          Object.remove_const(:LobsterLady)
+        end
+
+        it 'should display "help" when looking up by command name' do
+          pry_eval('show-doc lobster-lady').should =~ /nada/
+          Pry.commands.delete("lobster-lady")
+        end
+
+        it 'should display actual preceding comment for a class command, when class is used (rather than command name) when looking up' do
+          pry_eval('show-doc LobsterLady').should =~ /pretty pink pincers/
+          Pry.commands.delete("lobster-lady")
+        end
+      end
+    end
+
+    describe "should set _file_ and _dir_" do
+      it 'should set _file_ and _dir_ to file containing method source' do
+        t = pry_tester
+        t.process_command "show-doc TestClassForShowSource#alpha"
+        t.pry.last_file.should =~ /show_source_doc_examples/
+        t.pry.last_dir.should =~ /fixtures/
+      end
+    end
+  end
+end

data/spec/commands/show_input_spec.rb

@@ -0,0 +1,17 @@
+require 'helper'
+
+describe "show-input" do
+  before do
+    @t = pry_tester
+  end
+
+  it 'should correctly show the current lines in the input buffer' do
+    eval_str = unindent(<<-STR)
+      def hello
+        puts :bing
+    STR
+
+    @t.process_command 'show-input', eval_str
+    @t.last_output.should =~ /\A\d+: def hello\n\d+:   puts :bing/
+  end
+end

data/spec/commands/show_source_spec.rb

@@ -0,0 +1,597 @@
+require 'helper'
+require "fixtures/show_source_doc_examples"
+
+if !PryTestHelpers.mri18_and_no_real_source_location?
+  describe "show-source" do
+    before do
+      @o = Object.new
+      def @o.sample_method
+        :sample
+      end
+
+      Object.const_set(:Test, Module.new)
+    end
+
+    after do
+      Pad.clear
+    end
+
+    it "should output a method's source" do
+      pry_eval(binding, 'show-source @o.sample_method').should =~ /def @o.sample/
+    end
+
+    it "should output help" do
+      pry_eval('show-source -h').should =~ /Usage:\s+show-source/
+    end
+
+    it "should output a method's source with line numbers" do
+      pry_eval(binding, 'show-source -l @o.sample_method').should =~ /\d+: def @o.sample/
+    end
+
+    it "should output a method's source with line numbers starting at 1" do
+      pry_eval(binding, 'show-source -b @o.sample_method').should =~ /1: def @o.sample/
+    end
+
+    it "should output a method's source if inside method and no name given" do
+      def @o.sample
+        pry_eval(binding, 'show-source').should =~ /def @o.sample/
+      end
+      @o.sample
+    end
+
+    it "should output a method's source inside method using the -l switch" do
+      def @o.sample
+        pry_eval(binding, 'show-source -l').should =~ /def @o.sample/
+      end
+      @o.sample
+    end
+
+    it "should find methods even if there are spaces in the arguments" do
+      def @o.foo(*bars)
+        "Mr flibble"
+        self
+      end
+
+      out = pry_eval(binding, "show-source @o.foo('bar', 'baz bam').foo")
+      out.should =~ /Mr flibble/
+    end
+
+    it "should find methods even if the object overrides method method" do
+      c = Class.new{
+        def method;
+          98
+        end
+      }
+
+      pry_eval(binding, "show-source c.new.method").should =~ /98/
+    end
+
+    it "should not show the source when a non-extant method is requested" do
+      c = Class.new{ def method; 98; end }
+      mock_pry(binding, "show-source c#wrongmethod").should =~ /Couldn't locate/
+    end
+
+    it "should find instance_methods if the class overrides instance_method" do
+      c = Class.new{
+        def method;
+          98
+        end
+
+        def self.instance_method; 789; end
+      }
+
+      pry_eval(binding, "show-source c#method").should =~ /98/
+    end
+
+    it "should find instance methods with self#moo" do
+      c = Class.new{ def moo; "ve over!"; end }
+
+      pry_eval(binding, "cd c", "show-source self#moo").should =~ /ve over/
+    end
+
+    it "should not find instance methods with self.moo" do
+      c = Class.new{ def moo; "ve over!"; end }
+
+      proc {
+        pry_eval(binding, 'cd c', 'show-source self.moo')
+      }.should.raise(Pry::CommandError).message.should =~ /Couldn't locate/
+    end
+
+    it "should find normal methods with self.moo" do
+      c = Class.new{ def self.moo; "ve over!"; end }
+
+      pry_eval(binding, 'cd c', 'show-source self.moo').should =~ /ve over/
+    end
+
+    it "should not find normal methods with self#moo" do
+      c = Class.new{ def self.moo; "ve over!"; end }
+
+      proc {
+        pry_eval(binding, 'cd c', 'show-source self#moo')
+      }.should.raise(Pry::CommandError).message.should =~ /Couldn't locate/
+    end
+
+    it "should find normal methods (i.e non-instance methods) by default" do
+      c = Class.new{ def self.moo; "ve over!"; end }
+
+      pry_eval(binding, "cd c", "show-source moo").should =~ /ve over/
+    end
+
+    it "should find instance methods if no normal methods available" do
+      c = Class.new{ def moo; "ve over!"; end }
+
+      pry_eval(binding, "cd c", "show-source moo").should =~ /ve over/
+    end
+
+    it "should find super methods" do
+      class Foo
+        def foo(*bars)
+          :super_wibble
+        end
+      end
+      o = Foo.new
+      Object.remove_const(:Foo)
+      def o.foo(*bars)
+        :wibble
+      end
+
+      pry_eval(binding, "show-source --super o.foo").
+          should =~ /:super_wibble/
+    end
+
+    it "should raise a CommandError when super method doesn't exist" do
+      def @o.foo(*bars); end
+
+      proc {
+        pry_eval(binding, "show-source --super @o.foo")
+      }.should.raise(Pry::CommandError).message.should =~ /No superclass found/
+    end
+
+    # dynamically defined method source retrieval is only supported in
+    # 1.9 - where Method#source_location is native
+    if RUBY_VERSION =~ /1.9/
+      it "should output the source of a method defined inside Pry" do
+        out = pry_eval("def dyn_method\n:test\nend", 'show-source dyn_method')
+        out.should =~ /def dyn_method/
+        Object.remove_method :dyn_method
+      end
+
+      it 'should output source for an instance method defined inside pry' do
+        pry_tester.tap do |t|
+          t.eval "class Test::A\n  def yo\n  end\nend"
+          t.eval('show-source Test::A#yo').should =~ /def yo/
+        end
+      end
+
+      it 'should output source for a repl method defined using define_method' do
+        pry_tester.tap do |t|
+          t.eval "class Test::A\n  define_method(:yup) {}\nend"
+          t.eval('show-source Test::A#yup').should =~ /define_method\(:yup\)/
+        end
+      end
+
+      it "should output the source of a command defined inside Pry" do
+        command_definition = %{
+          Pry.commands.command "hubba-hubba" do
+            puts "that's what she said!"
+          end
+        }
+        out = pry_eval(command_definition, 'show-source hubba-hubba')
+        out.should =~ /what she said/
+        Pry.commands.delete "hubba-hubba"
+      end
+    end
+
+    describe "on sourcable objects" do
+      if RUBY_VERSION =~ /1.9/
+        it "should output source defined inside pry" do
+          pry_tester.tap do |t|
+            t.eval "hello = proc { puts 'hello world!' }"
+            t.eval("show-source hello").should =~ /proc \{ puts/
+          end
+        end
+      end
+
+      it "should output source for procs/lambdas stored in variables" do
+        hello = proc { puts 'hello world!' }
+        pry_eval(binding, 'show-source hello').should =~ /proc \{ puts/
+      end
+
+      it "should output source for procs/lambdas stored in constants" do
+        HELLO = proc { puts 'hello world!' }
+        pry_eval(binding, "show-source HELLO").should =~ /proc \{ puts/
+        Object.remove_const(:HELLO)
+      end
+
+      it "should output source for method objects" do
+        def @o.hi; puts 'hi world'; end
+        meth = @o.method(:hi)
+        pry_eval(binding, "show-source meth").should =~ /puts 'hi world'/
+      end
+
+      describe "on variables that shadow methods" do
+        before do
+          @t = pry_tester.eval unindent(<<-EOS)
+            class ::TestHost
+              def hello
+                hello = proc { ' smile ' }
+                pry_tester(binding)
+              end
+            end
+            ::TestHost.new.hello
+          EOS
+        end
+
+        after do
+          Object.remove_const(:TestHost)
+        end
+
+          it "source of variable should take precedence over method that is being shadowed" do
+            source = @t.eval('show-source hello')
+            source.should.not =~ /def hello/
+            source.should =~ /proc \{ ' smile ' \}/
+          end
+
+        it "source of method being shadowed should take precedence over variable
+            if given self.meth_name syntax" do
+          @t.eval('show-source self.hello').should =~ /def hello/
+        end
+      end
+
+    end
+
+    describe "on variable or constant" do
+      before do
+        class TestHost
+          def hello
+            "hi there"
+          end
+        end
+      end
+
+      after do
+        Object.remove_const(:TestHost)
+      end
+
+      it "should output source of its class if variable doesn't respond to source_location" do
+        test_host = TestHost.new
+        pry_eval(binding, 'show-source test_host').
+            should =~ /class TestHost\n.*def hello/
+      end
+
+      it "should output source of its class if constant doesn't respond to source_location" do
+        TEST_HOST = TestHost.new
+        pry_eval(binding, 'show-source TEST_HOST').
+            should =~ /class TestHost\n.*def hello/
+        Object.remove_const(:TEST_HOST)
+      end
+    end
+
+    describe "on modules" do
+      before do
+        class ShowSourceTestSuperClass
+          def alpha
+          end
+        end
+
+        class ShowSourceTestClass<ShowSourceTestSuperClass
+          def alpha
+          end
+        end
+
+        module ShowSourceTestSuperModule
+          def alpha
+          end
+        end
+
+        module ShowSourceTestModule
+          include ShowSourceTestSuperModule
+          def alpha
+          end
+        end
+
+        ShowSourceTestClassWeirdSyntax = Class.new do
+          def beta
+          end
+        end
+
+        ShowSourceTestModuleWeirdSyntax = Module.new do
+          def beta
+          end
+        end
+      end
+
+      after do
+        Object.remove_const :ShowSourceTestSuperClass
+        Object.remove_const :ShowSourceTestClass
+        Object.remove_const :ShowSourceTestClassWeirdSyntax
+        Object.remove_const :ShowSourceTestSuperModule
+        Object.remove_const :ShowSourceTestModule
+        Object.remove_const :ShowSourceTestModuleWeirdSyntax
+      end
+
+      describe "basic functionality, should find top-level module definitions" do
+        it 'should show source for a class' do
+          pry_eval('show-source ShowSourceTestClass').
+              should =~ /class ShowSourceTestClass.*?def alpha/m
+        end
+
+        it 'should show source for a super class' do
+          pry_eval('show-source -s ShowSourceTestClass').
+              should =~ /class ShowSourceTestSuperClass.*?def alpha/m
+        end
+
+        it 'should show source for a module' do
+          pry_eval('show-source ShowSourceTestModule').
+              should =~ /module ShowSourceTestModule/
+        end
+
+        it 'should show source for an ancestor module' do
+          pry_eval('show-source -s ShowSourceTestModule').
+              should =~ /module ShowSourceTestSuperModule/
+        end
+
+        it 'should show source for a class when Const = Class.new syntax is used' do
+          pry_eval('show-source ShowSourceTestClassWeirdSyntax').
+              should =~ /ShowSourceTestClassWeirdSyntax = Class.new/
+        end
+
+        it 'should show source for a super class when Const = Class.new syntax is used' do
+          pry_eval('show-source -s ShowSourceTestClassWeirdSyntax').
+              should =~ /class Object/
+        end
+
+        it 'should show source for a module when Const = Module.new syntax is used' do
+          pry_eval('show-source ShowSourceTestModuleWeirdSyntax').
+              should =~ /ShowSourceTestModuleWeirdSyntax = Module.new/
+        end
+      end
+
+      if !Pry::Helpers::BaseHelpers.mri_18?
+        before do
+          pry_eval unindent(<<-EOS)
+            class Dog
+              def woof
+              end
+            end
+
+            class TobinaMyDog < Dog
+              def woof
+              end
+            end
+          EOS
+        end
+
+        after do
+          Object.remove_const :Dog
+          Object.remove_const :TobinaMyDog
+        end
+
+        describe "in REPL" do
+          it 'should find class defined in repl' do
+            pry_eval('show-source TobinaMyDog').should =~ /class TobinaMyDog/
+          end
+          it 'should find superclass defined in repl' do
+            pry_eval('show-source -s TobinaMyDog').should =~ /class Dog/
+          end
+        end
+      end
+
+        it 'should lookup module name with respect to current context' do
+
+        constant_scope(:AlphaClass, :BetaClass) do
+          class BetaClass
+            def alpha
+            end
+          end
+
+          class AlphaClass
+            class BetaClass
+              def beta
+              end
+            end
+          end
+
+          pry_eval(AlphaClass, 'show-source BetaClass').should =~ /def beta/
+        end
+      end
+
+      it 'should lookup nested modules' do
+        constant_scope(:AlphaClass) do
+          class AlphaClass
+            class BetaClass
+              def beta
+              end
+            end
+          end
+
+          pry_eval('show-source AlphaClass::BetaClass').should =~ /class Beta/
+        end
+      end
+
+      # note that pry assumes a class is only monkey-patched at most
+      # ONCE per file, so will not find multiple monkeypatches in the
+      # SAME file.
+      describe "show-source -a" do
+        it 'should show the source for all monkeypatches defined in different files' do
+          class TestClassForShowSource
+            def beta
+            end
+          end
+
+          result = pry_eval('show-source TestClassForShowSource -a')
+          result.should =~ /def alpha/
+          result.should =~ /def beta/
+        end
+
+        it 'should show the source for a class_eval-based monkeypatch' do
+          TestClassForShowSourceClassEval.class_eval do
+            def class_eval_method
+            end
+          end
+
+          result = pry_eval('show-source TestClassForShowSourceClassEval -a')
+          result.should =~ /def class_eval_method/
+        end
+
+        it 'should ignore -a when object is not a module' do
+          TestClassForShowSourceClassEval.class_eval do
+            def class_eval_method
+              :bing
+            end
+          end
+
+          result = pry_eval('show-source TestClassForShowSourceClassEval#class_eval_method -a')
+          result.should =~ /bing/
+        end
+
+        it 'should show the source for an instance_eval-based monkeypatch' do
+          TestClassForShowSourceInstanceEval.instance_eval do
+            def instance_eval_method
+            end
+          end
+
+          result = pry_eval('show-source TestClassForShowSourceInstanceEval -a')
+          result.should =~ /def instance_eval_method/
+        end
+      end
+
+      describe "when show-source is invoked without a method or class argument" do
+        before do
+          module TestHost
+            class M
+              def alpha; end
+              def beta; end
+            end
+
+            module C
+            end
+
+            module D
+              def self.invoked_in_method
+                pry_eval(binding, 'show-source')
+              end
+            end
+          end
+        end
+
+        after do
+          Object.remove_const(:TestHost)
+        end
+
+        describe "inside a module" do
+          it 'should display module source by default' do
+            out = pry_eval(TestHost::M, 'show-source')
+            out.should =~ /class M/
+            out.should =~ /def alpha/
+            out.should =~ /def beta/
+          end
+
+          it 'should be unable to find module source if no methods defined' do
+            proc {
+              pry_eval(TestHost::C, 'show-source')
+            }.should.raise(Pry::CommandError).
+              message.should =~ /Cannot find a definition for/
+          end
+
+          it 'should display method code (rather than class) if Pry started inside method binding' do
+            out = TestHost::D.invoked_in_method
+            out.should =~ /invoked_in_method/
+            out.should.not =~ /module D/
+          end
+
+          it 'should display class source when inside instance' do
+            out = pry_eval(TestHost::M.new, 'show-source')
+            out.should =~ /class M/
+            out.should =~ /def alpha/
+            out.should =~ /def beta/
+          end
+
+          it 'should allow options to be passed' do
+            out = pry_eval(TestHost::M, 'show-source -b')
+            out.should =~ /\d:\s*class M/
+            out.should =~ /\d:\s*def alpha/
+            out.should =~ /\d:\s*def beta/
+          end
+
+           describe "should skip over broken modules" do
+            before do
+              module BabyDuck
+
+                module Muesli
+                  binding.eval("def a; end", "dummy.rb", 1)
+                  binding.eval("def b; end", "dummy.rb", 2)
+                  binding.eval("def c; end", "dummy.rb", 3)
+                end
+
+                module Muesli
+                  def d; end
+                  def e; end
+                end
+              end
+            end
+
+            after do
+              Object.remove_const(:BabyDuck)
+            end
+
+            # TODO: !!! This is a bad spec, should not be expected behaviour?!?!
+            #
+            # it 'should return source for first valid module' do
+            #   out = pry_eval('show-source BabyDuck::Muesli')
+            #   out.should =~ /def d; end/
+            #   out.should.not =~ /def a; end/
+            # end
+
+          end
+        end
+      end
+    end
+
+    describe "on commands" do
+      before do
+        @oldset = Pry.config.commands
+        @set = Pry.config.commands = Pry::CommandSet.new do
+          import Pry::Commands
+        end
+      end
+
+      after do
+        Pry.config.commands = @oldset
+      end
+
+      it 'should show source for an ordinary command' do
+        @set.command "foo", :body_of_foo do; end
+
+        pry_eval('show-source foo').should =~ /:body_of_foo/
+      end
+
+      it "should output source of commands using special characters" do
+        @set.command "!%$", "I gots the yellow fever" do; end
+
+        pry_eval('show-source !%$').should =~ /yellow fever/
+      end
+
+      it 'should show source for a command with spaces in its name' do
+        @set.command "foo bar", :body_of_foo_bar do; end
+
+        pry_eval('show-source foo bar').should =~ /:body_of_foo_bar/
+      end
+
+      it 'should show source for a command by listing name' do
+        @set.command /foo(.*)/, :body_of_foo_bar_regex, :listing => "bar" do; end
+
+        pry_eval('show-source bar').should =~ /:body_of_foo_bar_regex/
+      end
+    end
+
+    describe "should set _file_ and _dir_" do
+      it 'should set _file_ and _dir_ to file containing method source' do
+        t = pry_tester
+        t.process_command "show-source TestClassForShowSource#alpha"
+        t.pry.last_file.should =~ /show_source_doc_examples/
+        t.pry.last_dir.should =~ /fixtures/
+      end
+    end
+  end
+end