pry 0.9.12pre1 → 0.9.12pre2

data/lib/pry/commands/show_info.rb

@@ -18,7 +18,8 @@ class Pry
 
     def process
       code_object = Pry::CodeObject.lookup(obj_name, _pry_, :super => opts[:super])
-      cannot_locate_source_error if !code_object
+      raise CommandError, no_definition_message if !code_object
+      @original_code_object = code_object
 
       if show_all_modules?(code_object)
         # show all monkey patches for a module
@@ -26,7 +27,6 @@ class Pry
         result = content_and_headers_for_all_module_candidates(code_object)
       else
         # show a specific code object
-
         co = code_object_with_accessible_source(code_object)
         result = content_and_header_for_code_object(co)
       end
@@ -44,14 +44,24 @@ class Pry
     # @return [Pry::WrappedModule, Pry::Method, Pry::Command]
     def code_object_with_accessible_source(code_object)
       if code_object.is_a?(WrappedModule)
-        code_object.candidates.find(&:source).tap do |candidate|
-          cannot_locate_source_error if !candidate
+        candidate = code_object.candidates.find(&:source)
+        if candidate
+          return candidate
+        else
+          raise CommandError, no_definition_message if !valid_superclass?(code_object)
+
+          @used_super = true
+          code_object_with_accessible_source(code_object.super)
         end
       else
         code_object
       end
     end
 
+    def valid_superclass?(code_object)
+      code_object.super && code_object.super.wrapped != Object
+    end
+
     def content_and_header_for_code_object(code_object)
       header(code_object) + content_for(code_object)
     end
@@ -73,8 +83,8 @@ class Pry
       result
     end
 
-    def cannot_locate_source_error
-      raise CommandError, "Couldn't locate a definition for #{obj_name}!"
+    def no_definition_message
+      "Couldn't locate a definition for #{obj_name}!"
     end
 
     # Generate a header (meta-data information) for all the code
@@ -82,9 +92,24 @@ class Pry
     def header(code_object)
       file_name, line_num = file_and_line_for(code_object)
       h = "\n#{Pry::Helpers::Text.bold('From:')} #{file_name} "
-      h << method_header(code_object, line_num) if code_object.real_method_object?
+      h << code_object_header(code_object, line_num)
       h << "\n#{Pry::Helpers::Text.bold('Number of lines:')} " <<
         "#{content_for(code_object).lines.count}\n\n"
+      h << Helpers::Text.bold('** Warning:') + " Cannot find code for #{@original_code_object.nonblank_name}. Showing superclass #{code_object.nonblank_name} instead. **\n\n" if @used_super
+      h
+    end
+
+    def code_object_header(code_object, line_num)
+      if code_object.real_method_object?
+        method_header(code_object, line_num)
+
+        # It sucks we have to test for both Pry::WrappedModule and WrappedModule::Candidate,
+        # probably indicates a deep refactor needs to happen in those classes.
+      elsif code_object.is_a?(Pry::WrappedModule) || code_object.is_a?(Pry::WrappedModule::Candidate)
+        module_header(code_object, line_num)
+      else
+        ""
+      end
     end
 
     def method_header(code_object, line_num)
@@ -96,6 +121,13 @@ class Pry
       h
     end
 
+    def module_header(code_object, line_num)
+      h = ""
+      h << "@ line #{line_num}:\n"
+      h << text.bold(code_object.module? ? "Module" : "Class")
+      h << " #{text.bold('name:')} #{code_object.nonblank_name}"
+    end
+
     def method_sections(code_object)
       {
         :owner => "\n#{text.bold("Owner:")} #{code_object.owner || "N/A"}\n",

data/lib/pry/module_candidate.rb

@@ -20,12 +20,16 @@ class Pry
       attr_reader :line
       alias_method :source_line, :line
 
-      # Methods to delegate to associated `Pry::WrappedModule instance`.
-      to_delegate = [:lines_for_file, :method_candidates, :name, :wrapped,
-                     :yard_docs?, :number_of_candidates]
+      # Methods to delegate to associated `Pry::WrappedModule
+      # instance`.
+      private_delegates = [:lines_for_file, :method_candidates,
+                           :yard_docs?, :number_of_candidates]
 
-      def_delegators :@wrapper, *to_delegate
-      private(*to_delegate)
+      public_delegates = [:wrapped, :module?, :class?, :name, :nonblank_name]
+
+      def_delegators :@wrapper, *(private_delegates + public_delegates)
+      private *private_delegates
+      public *public_delegates
 
       # @raise [Pry::CommandError] If `rank` is out of bounds.
       # @param [Pry::WrappedModule] wrapper The associated

data/lib/pry/version.rb

@@ -1,3 +1,3 @@
 class Pry
-  VERSION = "0.9.12pre1"
+  VERSION = "0.9.12pre2"
 end

data/lib/pry/wrapped_module.rb

@@ -123,6 +123,18 @@ class Pry
       wrapped != wrapped.ancestors.first
     end
 
+    # Is this strictly a module? (does not match classes)
+    # @return [Boolean]
+    def module?
+      wrapped.instance_of?(Module)
+    end
+
+    # Is this strictly a class?
+    # @return [Boolean]
+    def class?
+      wrapped.instance_of?(Class)
+    end
+
     # Get the instance associated with this singleton class.
     #
     # @raise ArgumentError: tried to get instance of non singleton class

data/spec/commands/show_doc_spec.rb

@@ -374,5 +374,112 @@ if !PryTestHelpers.mri18_and_no_real_source_location?
         t.pry.last_dir.should =~ /fixtures/
       end
     end
+
+    describe "can't find class docs" do
+      describe "for classes" do
+        before do
+          module Jesus
+            class Brian; end
+
+            # doink-doc
+            class Jingle
+              def a; :doink; end
+            end
+
+            class Jangle < Jingle; end
+            class Bangle < Jangle; end
+          end
+        end
+
+        after do
+          Object.remove_const(:Jesus)
+        end
+
+        it 'shows superclass doc' do
+          t = pry_tester
+          t.process_command "show-doc Jesus::Jangle"
+          t.last_output.should =~ /doink-doc/
+        end
+
+        it 'errors when class has no superclass to show' do
+          t = pry_tester
+          lambda { t.process_command "show-doc Jesus::Brian" }.should.raise(Pry::CommandError).message.
+            should =~ /Couldn't locate/
+        end
+
+        it 'shows warning when reverting to superclass docs' do
+          t = pry_tester
+          t.process_command "show-doc Jesus::Jangle"
+          t.last_output.should =~ /Warning.*?Cannot find.*?Jesus::Jangle.*Showing.*Jesus::Jingle instead/
+        end
+
+        it 'shows nth level superclass docs (when no intermediary superclasses have code either)' do
+          t = pry_tester
+          t.process_command "show-doc Jesus::Bangle"
+          t.last_output.should =~ /doink-doc/
+        end
+
+        it 'shows correct warning when reverting to nth level superclass' do
+          t = pry_tester
+          t.process_command "show-doc Jesus::Bangle"
+          t.last_output.should =~ /Warning.*?Cannot find.*?Jesus::Bangle.*Showing.*Jesus::Jingle instead/
+        end
+      end
+describe "for modules" do
+        before do
+          module Jesus
+
+            # alpha-doc
+            module Alpha
+              def alpha; :alpha; end
+            end
+
+            module Zeta; end
+
+            module Beta
+              include Alpha
+            end
+
+            module Gamma
+              include Beta
+            end
+          end
+        end
+
+        after do
+          Object.remove_const(:Jesus)
+        end
+
+        it 'shows included module doc' do
+          t = pry_tester
+          t.process_command "show-doc Jesus::Beta"
+          t.last_output.should =~ /alpha-doc/
+        end
+
+        it 'shows warning when reverting to included module doc' do
+          t = pry_tester
+          t.process_command "show-doc Jesus::Beta"
+          t.last_output.should =~ /Warning.*?Cannot find.*?Jesus::Beta.*Showing.*Jesus::Alpha instead/
+        end
+
+        it 'errors when module has no included module to show' do
+          t = pry_tester
+          lambda { t.process_command "show-source Jesus::Zeta" }.should.raise(Pry::CommandError).message.
+            should =~ /Couldn't locate/
+        end
+
+        it 'shows nth level included module doc (when no intermediary modules have code either)' do
+          t = pry_tester
+          t.process_command "show-doc Jesus::Gamma"
+          t.last_output.should =~ /alpha-doc/
+        end
+
+        it 'shows correct warning when reverting to nth level included module' do
+          t = pry_tester
+          t.process_command "show-source Jesus::Gamma"
+          t.last_output.should =~ /Warning.*?Cannot find.*?Jesus::Gamma.*Showing.*Jesus::Alpha instead/
+        end
+      end
+    end
   end
 end

data/spec/commands/show_source_spec.rb

@@ -136,7 +136,7 @@ if !PryTestHelpers.mri18_and_no_real_source_location?
       end
 
       pry_eval(binding, "show-source --super o.foo").
-          should =~ /:super_wibble/
+        should =~ /:super_wibble/
     end
 
     it "should raise a CommandError when super method doesn't exist" do
@@ -219,12 +219,12 @@ if !PryTestHelpers.mri18_and_no_real_source_location?
               end
             end
             ::TestHost.new.hello
-          EOS
-        end
+            EOS
+          end
 
-        after do
-          Object.remove_const(:TestHost)
-        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')
@@ -232,124 +232,123 @@ if !PryTestHelpers.mri18_and_no_real_source_location?
             source.should =~ /proc \{ ' smile ' \}/
           end
 
-        it "source of method being shadowed should take precedence over variable
+          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/
+            @t.eval('show-source self.hello').should =~ /def hello/
+          end
         end
       end
 
-    end
-
-    describe "on variable or constant" do
-      before do
-        class TestHost
-          def hello
-            "hi there"
+      describe "on variable or constant" do
+        before do
+          class TestHost
+            def hello
+              "hi there"
+            end
           end
         end
-      end
 
-      after do
-        Object.remove_const(:TestHost)
-      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').
+        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
+        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').
+        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)
+          Object.remove_const(:TEST_HOST)
+        end
       end
-    end
 
-    describe "on modules" do
-      before do
-        class ShowSourceTestSuperClass
-          def alpha
+      describe "on modules" do
+        before do
+          class ShowSourceTestSuperClass
+            def alpha
+            end
           end
-        end
 
-        class ShowSourceTestClass<ShowSourceTestSuperClass
-          def alpha
+          class ShowSourceTestClass<ShowSourceTestSuperClass
+            def alpha
+            end
           end
-        end
 
-        module ShowSourceTestSuperModule
-          def alpha
+          module ShowSourceTestSuperModule
+            def alpha
+            end
           end
-        end
 
-        module ShowSourceTestModule
-          include ShowSourceTestSuperModule
-          def alpha
+          module ShowSourceTestModule
+            include ShowSourceTestSuperModule
+            def alpha
+            end
           end
-        end
 
-        ShowSourceTestClassWeirdSyntax = Class.new do
-          def beta
+          ShowSourceTestClassWeirdSyntax = Class.new do
+            def beta
+            end
           end
-        end
 
-        ShowSourceTestModuleWeirdSyntax = Module.new do
-          def beta
+          ShowSourceTestModuleWeirdSyntax = Module.new do
+            def beta
+            end
           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
+        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').
+        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
+          end
 
-        it 'should show source for a super class' do
-          pry_eval('show-source -s ShowSourceTestClass').
+          it 'should show source for a super class' do
+            pry_eval('show-source -s ShowSourceTestClass').
               should =~ /class ShowSourceTestSuperClass.*?def alpha/m
-        end
+          end
 
-        it 'should show source for a module' do
-          pry_eval('show-source ShowSourceTestModule').
+          it 'should show source for a module' do
+            pry_eval('show-source ShowSourceTestModule').
               should =~ /module ShowSourceTestModule/
-        end
+          end
 
-        it 'should show source for an ancestor module' do
-          pry_eval('show-source -s ShowSourceTestModule').
+          it 'should show source for an ancestor module' do
+            pry_eval('show-source -s ShowSourceTestModule').
               should =~ /module ShowSourceTestSuperModule/
-        end
+          end
 
-        it 'should show source for a class when Const = Class.new syntax is used' do
-          pry_eval('show-source ShowSourceTestClassWeirdSyntax').
+          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
+          end
 
-        it 'should show source for a super class when Const = Class.new syntax is used' do
-          pry_eval('show-source -s ShowSourceTestClassWeirdSyntax').
+          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
+          end
 
-        it 'should show source for a module when Const = Module.new syntax is used' do
-          pry_eval('show-source ShowSourceTestModuleWeirdSyntax').
+          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
-      end
 
-      if !Pry::Helpers::BaseHelpers.mri_18?
-        before do
-          pry_eval unindent(<<-EOS)
+        if !Pry::Helpers::BaseHelpers.mri_18?
+          before do
+            pry_eval unindent(<<-EOS)
             class Dog
               def woof
               end
@@ -359,240 +358,240 @@ if !PryTestHelpers.mri18_and_no_real_source_location?
               def woof
               end
             end
-          EOS
-        end
-
-        after do
-          Object.remove_const :Dog
-          Object.remove_const :TobinaMyDog
-        end
+            EOS
+          end
 
-        describe "in REPL" do
-          it 'should find class defined in repl' do
-            pry_eval('show-source TobinaMyDog').should =~ /class TobinaMyDog/
+          after do
+            Object.remove_const :Dog
+            Object.remove_const :TobinaMyDog
           end
-          it 'should find superclass defined in repl' do
-            pry_eval('show-source -s TobinaMyDog').should =~ /class Dog/
+
+          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
-      end
 
         it 'should lookup module name with respect to current context' do
 
-        constant_scope(:AlphaClass, :BetaClass) do
-          class BetaClass
-            def alpha
+          constant_scope(:AlphaClass, :BetaClass) do
+            class BetaClass
+              def alpha
+              end
             end
-          end
 
-          class AlphaClass
-            class BetaClass
-              def beta
+            class AlphaClass
+              class BetaClass
+                def beta
+                end
               end
             end
-          end
 
-          pry_eval(AlphaClass, 'show-source BetaClass').should =~ /def beta/
+            pry_eval(AlphaClass, 'show-source BetaClass').should =~ /def beta/
+          end
         end
-      end
 
-      it 'should lookup nested modules' do
-        constant_scope(:AlphaClass) do
-          class AlphaClass
-            class BetaClass
-              def beta
+        it 'should lookup nested modules' do
+          constant_scope(:AlphaClass) do
+            class AlphaClass
+              class BetaClass
+                def beta
+                end
               end
             end
-          end
 
-          pry_eval('show-source AlphaClass::BetaClass').should =~ /class Beta/
+            pry_eval('show-source AlphaClass::BetaClass').should =~ /class Beta/
+          end
         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
+        # 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
-          end
 
-          result = pry_eval('show-source TestClassForShowSource -a')
-          result.should =~ /def alpha/
-          result.should =~ /def beta/
-        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
+          it 'should show the source for a class_eval-based monkeypatch' do
+            TestClassForShowSourceClassEval.class_eval do
+              def class_eval_method
+              end
             end
-          end
 
-          result = pry_eval('show-source TestClassForShowSourceClassEval -a')
-          result.should =~ /def class_eval_method/
-        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
+          it 'should ignore -a when object is not a module' do
+            TestClassForShowSourceClassEval.class_eval do
+              def class_eval_method
+                :bing
+              end
             end
-          end
 
-          result = pry_eval('show-source TestClassForShowSourceClassEval#class_eval_method -a')
-          result.should =~ /bing/
-        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
+          it 'should show the source for an instance_eval-based monkeypatch' do
+            TestClassForShowSourceInstanceEval.instance_eval do
+              def instance_eval_method
+              end
             end
-          end
 
-          result = pry_eval('show-source TestClassForShowSourceInstanceEval -a')
-          result.should =~ /def instance_eval_method/
+            result = pry_eval('show-source TestClassForShowSourceInstanceEval -a')
+            result.should =~ /def instance_eval_method/
+          end
         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
+        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 C
+              end
 
-            module D
-              def self.invoked_in_method
-                pry_eval(binding, 'show-source')
+              module D
+                def self.invoked_in_method
+                  pry_eval(binding, 'show-source')
+                end
               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/
+          after do
+            Object.remove_const(:TestHost)
           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 =~ /Couldn't locate/
-          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 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 be unable to find module source if no methods defined' do
+              proc {
+                pry_eval(TestHost::C, 'show-source')
+              }.should.raise(Pry::CommandError).
+                message.should =~ /Couldn't locate/
+            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 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 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
+            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
 
-           describe "should skip over broken modules" do
-            before do
-              module BabyDuck
+            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
 
-                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
+            describe "should skip over broken modules" do
+              before do
+                module BabyDuck
 
-                module Muesli
-                  def d; end
-                  def e; end
+                  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
-            end
 
-            after do
-              Object.remove_const(:BabyDuck)
-            end
+              after do
+                Object.remove_const(:BabyDuck)
+              end
 
-            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/
+              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
-    end
 
-    describe "on commands" do
-      before do
-        @oldset = Pry.config.commands
-        @set = Pry.config.commands = Pry::CommandSet.new do
-          import Pry::Commands
+      describe "on commands" do
+        before do
+          @oldset = Pry.config.commands
+          @set = Pry.config.commands = Pry::CommandSet.new do
+            import Pry::Commands
+          end
         end
-      end
 
-      after do
-        Pry.config.commands = @oldset
-      end
+        after do
+          Pry.config.commands = @oldset
+        end
 
-      describe "block commands" do
-        it 'should show source for an ordinary command' do
-          @set.command "foo", :body_of_foo do; end
+        describe "block commands" do
+          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
+            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
+          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
+            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
+          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
+            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
+          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/
+            pry_eval('show-source bar').should =~ /:body_of_foo_bar_regex/
+          end
         end
-      end
 
-      describe "create_command commands" do
-        it 'should show source for a command' do
-          @set.create_command "foo", "babble" do
-            def process() :body_of_foo end
+        describe "create_command commands" do
+          it 'should show source for a command' do
+            @set.create_command "foo", "babble" do
+              def process() :body_of_foo end
+            end
+            pry_eval('show-source foo').should =~ /:body_of_foo/
           end
-          pry_eval('show-source foo').should =~ /:body_of_foo/
-        end
 
-        it 'should show source for a command defined inside pry' do
-          pry_eval %{
+          it 'should show source for a command defined inside pry' do
+            pry_eval %{
             _pry_.commands.create_command "foo", "babble" do
               def process() :body_of_foo end
             end
@@ -641,5 +640,119 @@ if !PryTestHelpers.mri18_and_no_real_source_location?
         t.pry.last_dir.should =~ /fixtures/
       end
     end
+
+    describe "can't find class/module code" do
+      describe "for classes" do
+        before do
+          module Jesus
+            module Pig
+              def lillybing; :lillybing; end
+            end
+
+            class Brian; end
+            class Jingle
+              def a; :doink; end
+            end
+
+            class Jangle < Jingle; include Pig; end
+            class Bangle < Jangle; end
+          end
+        end
+
+        after do
+          Object.remove_const(:Jesus)
+        end
+
+        it 'shows superclass code' do
+          t = pry_tester
+          t.process_command "show-source Jesus::Jangle"
+          t.last_output.should =~ /doink/
+        end
+
+        it 'ignores included modules' do
+          t = pry_tester
+          t.process_command "show-source Jesus::Jangle"
+          t.last_output.should.not =~ /lillybing/
+        end
+
+        it 'errors when class has no superclass to show' do
+          t = pry_tester
+          lambda { t.process_command "show-source Jesus::Brian" }.should.raise(Pry::CommandError).message.
+            should =~ /Couldn't locate/
+        end
+
+        it 'shows warning when reverting to superclass code' do
+          t = pry_tester
+          t.process_command "show-source Jesus::Jangle"
+          t.last_output.should =~ /Warning.*?Cannot find.*?Jesus::Jangle.*Showing.*Jesus::Jingle instead/
+        end
+
+        it 'shows nth level superclass code (when no intermediary superclasses have code either)' do
+          t = pry_tester
+          t.process_command "show-source Jesus::Bangle"
+          t.last_output.should =~ /doink/
+        end
+
+        it 'shows correct warning when reverting to nth level superclass' do
+          t = pry_tester
+          t.process_command "show-source Jesus::Bangle"
+          t.last_output.should =~ /Warning.*?Cannot find.*?Jesus::Bangle.*Showing.*Jesus::Jingle instead/
+        end
+      end
+
+      describe "for modules" do
+        before do
+          module Jesus
+            module Alpha
+              def alpha; :alpha; end
+            end
+
+            module Zeta; end
+
+            module Beta
+              include Alpha
+            end
+
+            module Gamma
+              include Beta
+            end
+          end
+        end
+
+        after do
+          Object.remove_const(:Jesus)
+        end
+
+        it 'shows included module code' do
+          t = pry_tester
+          t.process_command "show-source Jesus::Beta"
+          t.last_output.should =~ /alpha/
+        end
+
+        it 'shows warning when reverting to included module code' do
+          t = pry_tester
+          t.process_command "show-source Jesus::Beta"
+          t.last_output.should =~ /Warning.*?Cannot find.*?Jesus::Beta.*Showing.*Jesus::Alpha instead/
+        end
+
+        it 'errors when module has no included module to show' do
+          t = pry_tester
+          lambda { t.process_command "show-source Jesus::Zeta" }.should.raise(Pry::CommandError).message.
+            should =~ /Couldn't locate/
+        end
+
+        it 'shows nth level included module code (when no intermediary modules have code either)' do
+          t = pry_tester
+          t.process_command "show-source Jesus::Gamma"
+          t.last_output.should =~ /alpha/
+        end
+
+        it 'shows correct warning when reverting to nth level included module' do
+          t = pry_tester
+          t.process_command "show-source Jesus::Gamma"
+          t.last_output.should =~ /Warning.*?Cannot find.*?Jesus::Gamma.*Showing.*Jesus::Alpha instead/
+        end
+      end
+    end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: pry
 version: !ruby/object:Gem::Version
-  version: 0.9.12pre1
+  version: 0.9.12pre2
   prerelease: 6
 platform: ruby
 authors:
@@ -11,7 +11,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-02-05 00:00:00.000000000 Z
+date: 2013-02-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: coderay