pry 0.9.10pre1-i386-mingw32 → 0.9.11-i386-mingw32

data/spec/commands/exit_all_spec.rb

@@ -0,0 +1,34 @@
+require 'helper'
+
+describe "exit-all" do
+  it 'should break out of the repl loop of Pry instance and return nil' do
+    pry_tester(0).simulate_repl do |t|
+      t.eval 'exit-all'
+    end.should == nil
+  end
+
+  it 'should break out of the repl loop of Pry instance wth a user specified value' do
+    pry_tester(0).simulate_repl do |t|
+      t.eval "exit-all 'message'"
+    end.should == 'message'
+  end
+
+  it 'should break of the repl loop even if multiple bindings still on stack' do
+    pry_tester(0).simulate_repl do |t|
+      t.eval 'cd 1', 'cd 2', "exit-all 'message'"
+    end.should == 'message'
+  end
+
+  it 'binding_stack should be empty after breaking out of the repl loop' do
+    t = pry_tester(0) do
+      def binding_stack
+        @pry.binding_stack
+      end
+    end
+
+    t.simulate_repl do |t|
+      t.eval 'cd 1', 'cd 2', 'exit-all'
+    end
+    t.binding_stack.empty?.should == true
+  end
+end

data/spec/commands/exit_program_spec.rb

@@ -0,0 +1,19 @@
+require 'helper'
+
+describe "exit-program" do
+  it 'should raise SystemExit' do
+    proc {
+      pry_eval('exit-program')
+    }.should.raise SystemExit
+  end
+
+  it 'should exit the program with the provided value' do
+    begin
+      pry_eval 'exit-program 66'
+    rescue SystemExit => e
+      e.status.should == 66
+    else
+      raise "Failed to raise SystemExit"
+    end
+  end
+end

data/spec/commands/exit_spec.rb

@@ -0,0 +1,34 @@
+require 'helper'
+
+describe "exit" do
+  it 'should pop a binding with exit' do
+    pry_tester(:outer).simulate_repl do |t|
+      t.eval 'cd :inner'
+      t.eval('self').should == :inner
+      t.eval 'exit'
+      t.eval('self').should == :outer
+      t.eval 'exit-all'
+    end
+  end
+
+  it 'should break out of the repl loop of Pry instance when binding_stack has only one binding with exit' do
+    pry_tester(0).simulate_repl do |t|
+      t.eval 'exit'
+    end.should == nil
+  end
+
+  it 'should break out of the repl loop of Pry instance when binding_stack has only one binding with exit, and return user-given value' do
+    pry_tester(0).simulate_repl do |t|
+      t.eval 'exit :john'
+    end.should == :john
+  end
+
+  it 'should break out the repl loop of Pry instance even after an exception in user-given value' do
+    pry_tester(0).simulate_repl do |t|
+      proc {
+        t.eval 'exit = 42'
+      }.should.raise(SyntaxError)
+      t.eval 'exit'
+    end.should == nil
+  end
+end

data/{test/test_default_commands/test_find_method.rb → spec/commands/find_method_spec.rb}

@@ -4,7 +4,7 @@ require 'helper'
 # for C methods actually runs the methods - and since it runs ALL
 # methods (in an attempt to find a match) it runs 'exit' and aborts
 # the test, causing a failure. We should fix this in the future by
-# blacklisting certain methods for 1.8 MRI (such as exit, fork, and so on) 
+# blacklisting certain methods for 1.8 MRI (such as exit, fork, and so on)
 unless Pry::Helpers::BaseHelpers.mri_18?
   MyKlass = Class.new do
     def hello
@@ -13,26 +13,36 @@ unless Pry::Helpers::BaseHelpers.mri_18?
     def goodbye
       "jenny"
     end
+    def tea_tim?
+      "timothy"
+    end
+    def tea_time?
+      "polly"
+    end
   end
 
-  describe "find-command" do
+  describe "find-method" do
     describe "find matching methods by name regex (-n option)" do
       it "should find a method by regex" do
-        mock_pry("find-method hell MyKlass").should =~ /MyKlass.*?hello/m
+        pry_eval("find-method hell MyKlass").should =~
+          /MyKlass.*?hello/m
       end
 
       it "should NOT match a method that does not match the regex" do
-        mock_pry("find-method hell MyKlass").should.not =~ /MyKlass.*?goodbye/m
+        pry_eval("find-method hell MyKlass").should.not =~
+          /MyKlass.*?goodbye/m
       end
     end
 
     describe "find matching methods by content regex (-c option)" do
       it "should find a method by regex" do
-        mock_pry("find-method -c timothy MyKlass").should =~ /MyKlass.*?hello/m
+        pry_eval("find-method -c timothy MyKlass").should =~
+          /MyKlass.*?hello/m
       end
 
       it "should NOT match a method that does not match the regex" do
-        mock_pry("find-method timothy MyKlass").should.not =~ /MyKlass.*?goodbye/m
+        pry_eval("find-method timothy MyKlass").should.not =~
+          /MyKlass.*?goodbye/m
       end
     end
 
@@ -42,7 +52,17 @@ unless Pry::Helpers::BaseHelpers.mri_18?
         raise "mooo"
       end
 
-      mock_pry("find-method -c timothy MyKlass").should =~ /MyKlass.*?hello/m
+      pry_eval("find-method -c timothy MyKlass").should =~
+        /MyKlass.*?hello/m
+    end
+
+    it "should escape regexes correctly" do
+      good = /tea_time\?/
+      bad  = /tea_tim\?/
+      pry_eval('find-method tea_time? MyKlass').should =~ good
+      pry_eval('find-method tea_time? MyKlass').should =~ good
+      pry_eval('find-method tea_time\? MyKlass').should.not =~ bad
+      pry_eval('find-method tea_time\? MyKlass').should =~ good
     end
   end
 

data/spec/commands/gem_list_spec.rb

@@ -0,0 +1,26 @@
+require 'helper'
+
+describe "gem-list" do
+  # fixing bug for 1.8 compat
+  it 'should not raise when invoked' do
+    proc {
+      pry_eval(self, 'gem-list')
+    }.should.not.raise
+  end
+
+  it 'should work arglessly' do
+    list = pry_eval('gem-list')
+    list.should =~ /slop \(/
+    list.should =~ /bacon \(/
+  end
+
+  it 'should find arg' do
+    prylist = pry_eval('gem-list slop')
+    prylist.should =~ /slop \(/
+    prylist.should.not =~ /bacon/
+  end
+
+  it 'should return non-results as silence' do
+    pry_eval('gem-list aoeuoueouaou').should.empty?
+  end
+end

data/spec/commands/gist_spec.rb

@@ -0,0 +1,75 @@
+# These tests are out of date.
+# THey need to be updated for the new 'gist' API, but im too sleepy to
+# do that now.
+
+
+# require 'helper'
+
+# describe 'gist' do
+#   before do
+#     Pad.jist_calls = {}
+#   end
+
+#   # In absence of normal mocking, just monkeysmash these with no undoing after.
+#   module Jist
+#     class << self
+#       def login!; Pad.jist_calls[:login!] = true end
+#       def gist(*args)
+#         Pad.jist_calls[:gist_args] = args
+#         {'html_url' => 'http://gist.blahblah'}
+#       end
+#       def copy(content); Pad.jist_calls[:copy_args] = content end
+#     end
+#   end
+
+#   module Pry::Gist
+#     # a) The actual require fails for jruby for some odd reason.
+#     # b) 100% of jist should be stubbed by the above, so this ensures that.
+#     def self.require_jist; 'nope' end
+#   end
+
+#   it 'nominally logs in' do
+#     pry_eval 'gist --login'
+#     Pad.jist_calls[:login!].should.not.be.nil
+#   end
+
+#   EXAMPLE_REPL_METHOD = <<-EOT
+#   # docdoc
+#   def my_method
+#     # line 1
+#     'line 2'
+#     line 3
+#     Line.four
+#   end
+#   EOT
+
+#   RANDOM_COUPLE_OF_LINES = %w(a=1 b=2)
+#   run_case = proc do |sym|
+#     actual_command = Pry::Gist.example_code(sym)
+#     pry_eval EXAMPLE_REPL_METHOD, RANDOM_COUPLE_OF_LINES, actual_command
+#   end
+
+#   it 'deduces filenames' do
+#     Pry::Gist::INVOCATIONS.keys.each do |e|
+#       run_case.call(e)
+#       if Pad.jist_calls[:gist_args]
+#         text, args = Pad.jist_calls[:gist_args]
+#         args[:filename].should.not == '(pry)'
+#       end
+#       Pad.jist_calls[:copy_args].should.not.be.nil
+#     end
+#   end
+
+#   it 'equates aliae' do
+#     run_case.call(:clipit).should == run_case.call(:cliponly)
+#     run_case.call(:jist).should   == run_case.call(:class)
+#   end
+
+#   it 'has a reasonable --help' do
+#     help = pry_eval('gist --help')
+#     Pry::Gist::INVOCATIONS.keys.each do |e|
+#       help.should.include? Pry::Gist.example_code(e)
+#       help.should.include? Pry::Gist.example_description(e)
+#     end
+#   end
+# end

data/{test/test_default_commands/test_help.rb → spec/commands/help_spec.rb}

@@ -1,11 +1,10 @@
 require 'helper'
 
-describe "'help' command" do
+describe "help" do
   before do
     @oldset = Pry.config.commands
     @set = Pry.config.commands = Pry::CommandSet.new do
-      import Pry::DefaultCommands::Help
-      import Pry::DefaultCommands::Ls
+      import Pry::Commands
     end
   end
 
@@ -14,17 +13,17 @@ describe "'help' command" do
   end
 
   it 'should display help for a specific command' do
-    mock_pry('help ls').should =~ /Usage: ls/
+    pry_eval('help 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
-    mock_pry('help foo').should =~ /Test listing/
+    pry_eval('help 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
-    mock_pry('help "command with spaces"').should =~ /description of a command with spaces/
+    @set.command "cmd with spaces", "desc of a cmd with spaces" do; end
+    pry_eval('help "cmd with spaces"').should =~ /desc of a cmd with spaces/
   end
 
   it 'should display help for all commands with a description' do
@@ -33,7 +32,7 @@ describe "'help' command" do
     @set.command "c" do;end
     @set.command "d", "" do;end
 
-    output = mock_pry('help')
+    output = pry_eval('help')
     output.should =~ /Test listing/
     output.should =~ /description for b/
     output.should =~ /No description/
@@ -45,7 +44,7 @@ describe "'help' command" do
     @set.command 'maa', "Mooerizes" do; end
     @set.command 'baa', "Booerizes" do; end
 
-    doc = mock_pry('help')
+    doc = pry_eval('help')
 
     order = [doc.index("baa"),
              doc.index("faa"),

data/spec/commands/hist_spec.rb

@@ -0,0 +1,181 @@
+require 'helper'
+
+describe "hist" do
+  before do
+    Pry.history.clear
+    @hist = Pry.history
+
+    @str_output = StringIO.new
+    @t = pry_tester do
+      # For looking at what hist pushes into the input stack. The
+      # implementation of this helper will definitely have to change at some
+      # point.
+      def next_input
+        @pry.input.string
+      end
+    end
+  end
+
+  it 'should display the correct history' do
+    @hist.push "hello"
+    @hist.push "world"
+
+    @t.eval('hist').should =~ /hello\n.*world/
+  end
+
+  it 'should replay history correctly (single item)' do
+    o = Object.new
+    @hist.push "@x = 10"
+    @hist.push "@y = 20"
+    @hist.push "@z = 30"
+
+    @t.context = o
+    @t.eval 'hist --replay -1'
+
+    @t.next_input.should == "@z = 30\n"
+  end
+
+  it 'should replay a range of history correctly (range of items)' do
+    o = Object.new
+    @hist.push "@x = 10"
+    @hist.push "@y = 20"
+
+    @t.context = o
+    @t.eval 'hist --replay 0..2'
+
+    @t.next_input.should == "@x = 10\n@y = 20\n"
+  end
+
+  # this is to prevent a regression where input redirection is
+  # replaced by just appending to `eval_string`
+  it 'should replay a range of history correctly (range of commands)' do
+    o = Object.new
+    @hist.push "cd 1"
+    @hist.push "cd 2"
+    redirect_pry_io(InputTester.new("hist --replay 0..2", "Pad.stack = _pry_.binding_stack.dup", "exit-all")) do
+      o.pry
+    end
+    o = Pad.stack[-2..-1].map { |v| v.eval('self') }
+    o.should == [1, 2]
+    Pad.clear
+  end
+
+  it 'should grep for correct lines in history' do
+    @hist.push "abby"
+    @hist.push "box"
+    @hist.push "button"
+    @hist.push "pepper"
+    @hist.push "orange"
+    @hist.push "grape"
+    @hist.push "def blah 1"
+    @hist.push "def boink 2"
+    @hist.push "place holder"
+
+    @t.eval('hist --grep o').should =~ /\d:.*?box\n\d:.*?button\n\d:.*?orange/
+
+    # test more than one word in a regex match (def blah)
+    @t.eval('hist --grep def blah').should =~ /def blah 1/
+
+    # test more than one word with leading white space in a regex match (def boink)
+    @t.eval('hist --grep      def boink').should =~ /def boink 2/
+  end
+
+  it 'should return last N lines in history with --tail switch' do
+    ("a".."z").each do |v|
+      @hist.push v
+    end
+
+    out = @t.eval 'hist --tail 3'
+    out.each_line.count.should == 3
+    out.should =~ /x\n\d+:.*y\n\d+:.*z/
+  end
+
+  it 'should apply --tail after --grep' do
+    @hist.push "print 1"
+    @hist.push "print 2"
+    @hist.push "puts  3"
+    @hist.push "print 4"
+    @hist.push "puts  5"
+
+    out = @t.eval 'hist --tail 2 --grep print'
+    out.each_line.count.should == 2
+    out.should =~ /\d:.*?print 2\n\d:.*?print 4/
+  end
+
+  it 'should apply --head after --grep' do
+    @hist.push "puts  1"
+    @hist.push "print 2"
+    @hist.push "puts  3"
+    @hist.push "print 4"
+    @hist.push "print 5"
+
+    out = @t.eval 'hist --head 2 --grep print'
+    out.each_line.count.should == 2
+    out.should =~ /\d:.*?print 2\n\d:.*?print 4/
+  end
+
+  # strangeness in this test is due to bug in Readline::HISTORY not
+  # always registering first line of input
+  it 'should return first N lines in history with --head switch' do
+    ("a".."z").each do |v|
+      @hist.push v
+    end
+
+    out = @t.eval 'hist --head 4'
+    out.each_line.count.should == 4
+    out.should =~ /a\n\d+:.*b\n\d+:.*c/
+  end
+
+  # strangeness in this test is due to bug in Readline::HISTORY not
+  # always registering first line of input
+  it 'should show lines between lines A and B with the --show switch' do
+    ("a".."z").each do |v|
+      @hist.push v
+    end
+
+    out = @t.eval 'hist --show 1..4'
+    out.each_line.count.should == 4
+    out.should =~ /b\n\d+:.*c\n\d+:.*d/
+  end
+
+  it "should store a call with `--replay` flag" do
+    redirect_pry_io(InputTester.new(":banzai", "hist --replay 1",
+                                    "hist", "exit-all"), @str_output) do
+      Pry.start
+    end
+
+    @str_output.string.should =~ /hist --replay 1/
+  end
+
+  it "should not contain lines produced by `--replay` flag" do
+    redirect_pry_io(InputTester.new(":banzai", ":geronimo", ":huzzah",
+                                    "hist --replay 1..3", "hist",
+                                    "exit-all"), @str_output) do
+      Pry.start
+    end
+
+    @str_output.string.each_line.to_a.reject { |line| line.start_with?("=>") }.size.should == 4
+    @str_output.string.each_line.to_a.last.should =~ /hist --replay 1\.\.3/
+    @str_output.string.each_line.to_a[-2].should =~ /:huzzah/
+  end
+
+  it "should raise CommandError when index of `--replay` points out to another `hist --replay`" do
+    redirect_pry_io(InputTester.new(":banzai", "hist --replay 1",
+                                    "hist --replay 2", "exit-all"), @str_output) do
+      Pry.start
+    end
+
+    @str_output.string.should =~ /Replay index 2 points out to another replay call: `hist --replay 1`/
+  end
+
+  it "should disallow execution of `--replay <i>` when CommandError raised" do
+    redirect_pry_io(InputTester.new("a = 0", "a += 1", "hist --replay 2",
+                                    "hist --replay 3", "'a is ' + a.to_s",
+                                    "hist", "exit-all"), @str_output) do
+      Pry.start
+    end
+
+    @str_output.string.each_line.to_a.reject { |line| line !~ /\A\d/ }.size.should == 5
+    @str_output.string.should =~ /a is 2/
+  end
+end