pry 0.9.12pre2 → 0.9.12pre3

data/lib/pry/commands/show_info.rb

@@ -126,6 +126,12 @@ class Pry
       h << "@ line #{line_num}:\n"
       h << text.bold(code_object.module? ? "Module" : "Class")
       h << " #{text.bold('name:')} #{code_object.nonblank_name}"
+
+      if code_object.number_of_candidates > 1
+        h << (text.bold("\nNumber of monkeypatches: ") + code_object.number_of_candidates.to_s)
+        h << ". Use the `-a` option to display all available monkeypatches"
+      end
+      h
     end
 
     def method_sections(code_object)

data/lib/pry/completion.rb

@@ -71,10 +71,16 @@ class Pry
         # get new target for 1/2 and find candidates for 3
         path, input = build_path(input)
 
-        # we silence warnings here or ruby 1.8 cries about "multiple values for block 0 for 1"
+        # We silence warnings here or Ruby 1.8 cries about "multiple values for
+        # block 0 for 1".
         Helpers::BaseHelpers.silence_warnings do
           unless path.call.empty?
-            target, _ = Pry::Helpers::BaseHelpers.context_from_object_path(path.call, pry)
+            target = begin
+              ctx = Helpers::BaseHelpers.context_from_object_path(path.call, pry)
+              ctx.first
+            rescue Pry::CommandError
+              []
+            end
             target = target.last
           end
         end

data/lib/pry/module_candidate.rb

@@ -23,9 +23,10 @@ class Pry
       # Methods to delegate to associated `Pry::WrappedModule
       # instance`.
       private_delegates = [:lines_for_file, :method_candidates,
-                           :yard_docs?, :number_of_candidates]
+                           :yard_docs?]
 
-      public_delegates = [:wrapped, :module?, :class?, :name, :nonblank_name]
+      public_delegates = [:wrapped, :module?, :class?, :name, :nonblank_name,
+                          :number_of_candidates]
 
       def_delegators :@wrapper, *(private_delegates + public_delegates)
       private *private_delegates

data/lib/pry/pager.rb

@@ -27,8 +27,7 @@ class Pry::Pager
   end
 
   def self.page_size
-    rows = Pry::Terminal.screen_size
-    @page_size ||= (rows && rows.first || 27)
+    @page_size ||= Pry::Terminal.height!
   end
 
   def initialize(text)

data/lib/pry/plugins.rb

@@ -50,6 +50,8 @@ class Pry
         rescue LoadError => e
           warn "Found plugin #{gem_name}, but could not require '#{gem_name}'"
           warn e
+        rescue => e
+          warn "require '#{gem_name}' # Failed, saying: #{e}"
         end
 
         self.active = true

data/lib/pry/terminal.rb

@@ -13,15 +13,20 @@ class Pry::Terminal
     end
 
     # Return a screen size or a default if that fails.
-    def size! default = [25, 80]
+    def size! default = [27, 80]
       screen_size || default
     end
 
     # Return a screen width or the default if that fails.
-    def width! default = 80
+    def width!
       size![1]
     end
 
+    # Return a screen height or the default if that fails.
+    def height!
+      size![0]
+    end
+
     def actual_screen_size
       # The best way, if possible (requires non-jruby ≥1.9 or io-console gem)
       screen_size_according_to_io_console or

data/lib/pry/version.rb

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

data/spec/Procfile

@@ -0,0 +1,3 @@
+# Run this with:
+# gem install foreman && foreman start -f spec/Procfile
+pryhere: sh -c '(cd ..; rake pry)'

data/spec/commands/show_doc_spec.rb

@@ -200,286 +200,311 @@ if !PryTestHelpers.mri18_and_no_real_source_location?
                 def woof
                 end
               end
-            RUBY
-            t.eval('show-doc TobinaMyDog').should =~ /hello tobina/
-            Object.remove_const :TobinaMyDog
+              RUBY
+              t.eval('show-doc TobinaMyDog').should =~ /hello tobina/
+              Object.remove_const :TobinaMyDog
+            end
           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
+        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
-          end
 
-          class AlphaClass
-            # nested beta
-            class BetaClass
-              def beta
+            class AlphaClass
+              # nested beta
+              class BetaClass
+                def beta
+                end
               end
             end
-          end
 
-          pry_eval(AlphaClass, "show-doc BetaClass").should =~ /nested beta/
+            pry_eval(AlphaClass, "show-doc BetaClass").should =~ /nested beta/
+          end
         end
-      end
 
-      it 'should look up nested modules' do
-        constant_scope(:AlphaClass) do
-          class AlphaClass
-            # nested beta
-            class BetaClass
-              def beta
+        it 'should look up nested modules' do
+          constant_scope(:AlphaClass) do
+            class AlphaClass
+              # nested beta
+              class BetaClass
+                def beta
+                end
               end
             end
-          end
 
-          pry_eval("show-doc AlphaClass::BetaClass").should =~
-            /nested beta/
+            pry_eval("show-doc AlphaClass::BetaClass").should =~
+              /nested beta/
+          end
         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
+        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
-          end
 
-          result = pry_eval("show-doc TestClassForShowSource -a")
-          result.should =~ /used by/
-          result.should =~ /local monkeypatch/
-        end
-      end
+            result = pry_eval("show-doc TestClassForShowSource -a")
+            result.should =~ /used by/
+            result.should =~ /local monkeypatch/
+          end
 
-      describe "when no class/module arg is given" do
-        before do
-          module TestHost
+          describe "messages relating to -a" do
+            it 'indicates all available monkeypatches can be shown with -a when (when -a not used and more than one candidate exists for class)' do
+              class TestClassForShowSource
+                def beta
+                end
+              end
 
-            # hello there froggy
-            module M
-              def d; end
-              def e; end
+              result = pry_eval('show-doc TestClassForShowSource')
+              result.should =~ /available monkeypatches/
             end
-          end
-        end
 
-        after do
-          Object.remove_const(:TestHost)
-        end
+            it 'shouldnt say anything about monkeypatches when only one candidate exists for selected class' do
+              class Aarrrrrghh
+                def o;end
+              end
 
-        it 'should return doc for current module' do
-          pry_eval(TestHost::M, "show-doc").should =~ /hello there froggy/
+              result = pry_eval('show-doc Aarrrrrghh')
+              result.should.not =~ /available monkeypatches/
+              Object.remove_const(:Aarrrrrghh)
+            end
+          end
         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
+        describe "when no class/module arg is given" do
+          before do
+            module TestHost
 
-            # goodbye
-            module M
-              def d; end
-              def e; end
+              # hello there froggy
+              module M
+                def d; end
+                def e; end
+              end
             end
           end
-        end
 
-        after do
-          Object.remove_const(:TestHost)
-        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/
+          it 'should return doc for current module' do
+            pry_eval(TestHost::M, "show-doc").should =~ /hello there froggy/
+          end
         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
+        # 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
 
-      it 'should display help for a specific command' do
-        pry_eval('show-doc ls').should =~ /Usage: ls/
-      end
+              # goodbye
+              module M
+                def d; end
+                def e; end
+              end
+            end
+          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
+          after do
+            Object.remove_const(:TestHost)
+          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/
+          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 "class commands" do
+      describe "on commands" do
+        # mostly copied & modified from test_help.rb
         before do
-          # pretty pink pincers
-          class LobsterLady < Pry::ClassCommand
-            match "lobster-lady"
-            description "nada."
-            def process
-              "lobster"
-            end
+          @oldset = Pry.config.commands
+          @set = Pry.config.commands = Pry::CommandSet.new do
+            import Pry::Commands
           end
-
-          Pry.commands.add_command(LobsterLady)
         end
 
         after do
-          Object.remove_const(:LobsterLady)
+          Pry.config.commands = @oldset
         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")
+        it 'should display help for a specific command' do
+          pry_eval('show-doc ls').should =~ /Usage: ls/
         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")
+        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
-      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
 
-    describe "can't find class docs" do
-      describe "for classes" do
-        before do
-          module Jesus
-            class Brian; 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
 
-            # doink-doc
-            class Jingle
-              def a; :doink; end
+        describe "class commands" do
+          before do
+            # pretty pink pincers
+            class LobsterLady < Pry::ClassCommand
+              match "lobster-lady"
+              description "nada."
+              def process
+                "lobster"
+              end
             end
 
-            class Jangle < Jingle; end
-            class Bangle < Jangle; end
+            Pry.commands.add_command(LobsterLady)
           end
-        end
 
-        after do
-          Object.remove_const(:Jesus)
-        end
+          after do
+            Object.remove_const(:LobsterLady)
+          end
 
-        it 'shows superclass doc' do
-          t = pry_tester
-          t.process_command "show-doc Jesus::Jangle"
-          t.last_output.should =~ /doink-doc/
-        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 '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/
+          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
 
-        it 'shows warning when reverting to superclass docs' do
+      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 Jesus::Jangle"
-          t.last_output.should =~ /Warning.*?Cannot find.*?Jesus::Jangle.*Showing.*Jesus::Jingle instead/
+          t.process_command "show-doc TestClassForShowSource#alpha"
+          t.pry.last_file.should =~ /show_source_doc_examples/
+          t.pry.last_dir.should =~ /fixtures/
         end
+      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
+      unless Pry::Helpers::BaseHelpers.rbx?
+        describe "can't find class docs" do
+          describe "for classes" do
+            before do
+              module Jesus
+                class Brian; 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
+                # 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
 
-            # alpha-doc
-            module Alpha
-              def alpha; :alpha; 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
 
-            module Zeta; 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
 
-            module Beta
-              include Alpha
+            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
 
-            module Gamma
-              include Beta
+            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
-        end
 
-        after do
-          Object.remove_const(:Jesus)
-        end
+          describe "for modules" do
+            before do
+              module Jesus
 
-        it 'shows included module doc' do
-          t = pry_tester
-          t.process_command "show-doc Jesus::Beta"
-          t.last_output.should =~ /alpha-doc/
-        end
+                # alpha-doc
+                module Alpha
+                  def alpha; :alpha; end
+                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
+                module Zeta; 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
+                module Beta
+                  include Alpha
+                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
+                module Gamma
+                  include Beta
+                end
+              end
+            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/
+            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
   end
-end

data/spec/commands/show_source_spec.rb

@@ -453,6 +453,28 @@ if !PryTestHelpers.mri18_and_no_real_source_location?
             result = pry_eval('show-source TestClassForShowSourceInstanceEval -a')
             result.should =~ /def instance_eval_method/
           end
+
+          describe "messages relating to -a" do
+            it 'indicates all available monkeypatches can be shown with -a when (when -a not used and more than one candidate exists for class)' do
+              class TestClassForShowSource
+                def beta
+                end
+              end
+
+              result = pry_eval('show-source TestClassForShowSource')
+              result.should =~ /available monkeypatches/
+            end
+
+            it 'shouldnt say anything about monkeypatches when only one candidate exists for selected class' do
+              class Aarrrrrghh
+                def o;end
+              end
+
+              result = pry_eval('show-source Aarrrrrghh')
+              result.should.not =~ /available monkeypatches/
+              Object.remove_const(:Aarrrrrghh)
+            end
+          end
         end
 
         describe "when show-source is invoked without a method or class argument" do
@@ -641,116 +663,118 @@ if !PryTestHelpers.mri18_and_no_real_source_location?
       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
+    unless Pry::Helpers::BaseHelpers.rbx?
+      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 Brian; end
+              class Jingle
+                def a; :doink; end
+              end
 
-            class Jangle < Jingle; include Pig; end
-            class Bangle < Jangle; end
+              class Jangle < Jingle; include Pig; end
+              class Bangle < Jangle; end
+            end
           end
-        end
 
-        after do
-          Object.remove_const(:Jesus)
-        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 '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 '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 '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 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 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/
+          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
-      end
 
-      describe "for modules" do
-        before do
-          module Jesus
-            module Alpha
-              def alpha; :alpha; end
-            end
+        describe "for modules" do
+          before do
+            module Jesus
+              module Alpha
+                def alpha; :alpha; end
+              end
 
-            module Zeta; end
+              module Zeta; end
 
-            module Beta
-              include Alpha
-            end
+              module Beta
+                include Alpha
+              end
 
-            module Gamma
-              include Beta
+              module Gamma
+                include Beta
+              end
             end
           end
-        end
 
-        after do
-          Object.remove_const(:Jesus)
-        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 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 '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 '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 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/
+          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

data/spec/completion_spec.rb

@@ -230,4 +230,10 @@ describe Pry::InputCompleter do
     pry = Pry.new
     new_completer(binding, pry).call("pry.").should.not.include nil
   end
+
+  it "does not raise when complete file paths" do
+    should.not.raise(Pry::CommandError) {
+      new_completer(binding, Pry.new).call("cat lib/p")
+    }
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: pry
 version: !ruby/object:Gem::Version
-  version: 0.9.12pre2
+  version: 0.9.12pre3
   prerelease: 6
 platform: ruby
 authors:
@@ -11,7 +11,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-02-07 00:00:00.000000000 Z
+date: 2013-02-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: coderay
@@ -168,7 +168,6 @@ extensions: []
 extra_rdoc_files: []
 files:
 - .document
-- .gemtest
 - .gitignore
 - .travis.yml
 - .yardopts
@@ -280,6 +279,7 @@ files:
 - man/pry.1.html
 - man/pry.1.ronn
 - pry.gemspec
+- spec/Procfile
 - spec/cli_spec.rb
 - spec/code_object_spec.rb
 - spec/code_spec.rb
@@ -368,6 +368,7 @@ signing_key:
 specification_version: 3
 summary: An IRB alternative and runtime developer console
 test_files:
+- spec/Procfile
 - spec/cli_spec.rb
 - spec/code_object_spec.rb
 - spec/code_spec.rb