pry 0.9.8pre7 → 0.9.8pre8

data/test/test_default_commands/test_shell.rb

@@ -1,6 +1,196 @@
 require 'helper'
 
 describe "Pry::DefaultCommands::Shell" do
+  describe "save-file" do
+    before do
+      @tf = Tempfile.new(["pry", ".py"])
+      @path = @tf.path
+    end
+
+    after do
+      @tf.close(true)
+    end
+
+    describe "-f" do
+      it 'should save a file to a file' do
+        f = Tempfile.new(["pry", ".py"])
+        path = f.path
+        f.write ":cute_horse"
+
+        redirect_pry_io(InputTester.new("save-file -f #{path} #{@path}",
+                                        "exit-all")) do
+          Pry.start(@o)
+        end
+        File.read(@path).should == File.read(path)
+
+        f.close(true)
+      end
+    end
+
+    describe "-i" do
+      it 'should save input expressions to a file (single expression)' do
+        redirect_pry_io(InputTester.new(":horse_nostrils",
+                                        "save-file -i 1 #{@path}",
+                                        "exit-all")) do
+          Pry.start(@o)
+        end
+        File.read(@path).should == ":horse_nostrils\n"
+      end
+
+      it 'should save input expressions to a file (range)' do
+        redirect_pry_io(InputTester.new(":horse_nostrils",
+                                        ":sucking_up_all_the_oxygen",
+                                        "save-file -i 1..2 #{@path}",
+                                        "exit-all")) do
+          Pry.start(@o)
+        end
+        File.read(@path).should == ":horse_nostrils\n:sucking_up_all_the_oxygen\n"
+      end
+    end
+
+    describe "-m" do
+      before do
+        @o = Object.new
+        def @o.baby
+          :baby
+        end
+        def @o.bang
+          :bang
+        end
+      end
+
+      describe "single method" do
+        it 'should save a method to a file' do
+          redirect_pry_io(InputTester.new("save-file #{@path} -m baby",
+                                          "exit-all")) do
+            Pry.start(@o)
+          end
+          File.read(@path).should == Pry::Method.from_obj(@o, :baby).source
+        end
+
+        it 'should save a method to a file truncated by --lines' do
+          redirect_pry_io(InputTester.new("save-file #{@path} -m baby --lines 2..4",
+                                          "exit-all")) do
+            Pry.start(@o)
+          end
+
+          # must add 1 as first line of method is 1
+          File.read(@path).should == Pry::Method.from_obj(@o, :baby).source.lines.to_a[1..5].join
+        end
+      end
+
+      describe "multiple method" do
+        it 'should save multiple methods to a file' do
+          redirect_pry_io(InputTester.new("save-file #{@path} -m baby -m bang",
+                                          "exit-all")) do
+            Pry.start(@o)
+          end
+          File.read(@path).should == Pry::Method.from_obj(@o, :baby).source +
+            Pry::Method.from_obj(@o, :bang).source
+        end
+
+        it 'should save multiple methods to a file trucated by --lines' do
+          redirect_pry_io(InputTester.new("save-file #{@path} -m baby -m bang --lines 2..-2",
+                                          "exit-all")) do
+            Pry.start(@o)
+          end
+
+          # must add 1 as first line of method is 1
+          File.read(@path).should == (Pry::Method.from_obj(@o, :baby).source +
+            Pry::Method.from_obj(@o, :bang).source).lines.to_a[1..-2].join
+        end
+
+        it 'should save multiple methods to a file trucated by --lines 1 (single parameter, not range)' do
+          redirect_pry_io(InputTester.new("save-file #{@path} -m baby -m bang --lines 1",
+                                          "exit-all")) do
+            Pry.start(@o)
+          end
+
+          # must add 1 as first line of method is 1
+          File.read(@path).should == (Pry::Method.from_obj(@o, :baby).source +
+            Pry::Method.from_obj(@o, :bang).source).lines.to_a[0]
+        end
+
+      end
+
+    end
+
+    describe "overwrite by default (no --append)" do
+      it 'should overwrite specified file with new input' do
+        redirect_pry_io(InputTester.new(":horse_nostrils",
+                                        "save-file -i 1 #{@path}",
+                                        "exit-all")) do
+          Pry.start(@o)
+        end
+
+        redirect_pry_io(InputTester.new(":sucking_up_all_the_oxygen",
+                                        "save-file -i 1 #{@path}",
+                                        "exit-all")) do
+          Pry.start(@o)
+        end
+
+        File.read(@path).should == ":sucking_up_all_the_oxygen\n"
+      end
+
+    end
+
+    describe "--append" do
+      it 'should append to end of specified file' do
+        redirect_pry_io(InputTester.new(":horse_nostrils",
+                                        "save-file -i 1 #{@path}",
+                                        "exit-all")) do
+          Pry.start(@o)
+        end
+
+        redirect_pry_io(InputTester.new(":sucking_up_all_the_oxygen",
+                                        "save-file -i 1 #{@path} -a",
+                                        "exit-all")) do
+          Pry.start(@o)
+        end
+
+        File.read(@path).should == ":horse_nostrils\n:sucking_up_all_the_oxygen\n"
+      end
+    end
+
+    describe "-c" do
+      it 'should save a command to a file' do
+        redirect_pry_io(InputTester.new("save-file #{@path} -c show-method",
+                                        "exit-all")) do
+          Pry.start(@o)
+        end
+        cmd = Pry::Method.new(Pry.commands.find_command("show-method").block)
+        File.read(@path).should == Pry::Code.from_method(cmd).to_s
+      end
+    end
+
+    describe "combined options" do
+      before do
+        @o = Object.new
+        def @o.baby
+          :baby
+        end
+      end
+
+      it 'should save input cache and a method to a file (in that order)' do
+        redirect_pry_io(InputTester.new(":horse_nostrils",
+                                        "save-file -i 1 -m baby #{@path}",
+                                        "exit-all")) do
+          Pry.start(@o)
+        end
+        File.read(@path).should == ":horse_nostrils\n" + Pry::Method.from_obj(@o, :baby).source
+      end
+
+      it 'should select a portion to save using --lines' do
+        redirect_pry_io(InputTester.new(":horse_nostrils",
+                                        "save-file -i 1 -m baby #{@path} --lines 2..-2",
+                                        "exit-all")) do
+          Pry.start(@o)
+        end
+        File.read(@path).should == (":horse_nostrils\n" + Pry::Method.from_obj(@o, :baby).source).lines.to_a[1..-2].join
+      end
+    end
+  end
+
   describe "cat" do
 
     describe "on receiving a file that does not exist" do

data/test/test_hooks.rb

@@ -152,7 +152,7 @@ describe Pry::Hooks do
       hooks_dup.get_hook(:test_hook, :testing).should == @hooks.get_hook(:test_hook, :testing)
     end
 
-     it 'adding a new event to dupped instance should not affect original' do
+    it 'adding a new event to dupped instance should not affect original' do
       @hooks.add_hook(:test_hook, :testing) { :none_such }
       hooks_dup = @hooks.dup
 
@@ -161,7 +161,7 @@ describe Pry::Hooks do
       hooks_dup.get_hook(:other_test_hook, :testing).should.not == @hooks.get_hook(:other_test_hook, :testing)
     end
 
-     it 'adding a new hook to dupped instance should not affect original' do
+    it 'adding a new hook to dupped instance should not affect original' do
       @hooks.add_hook(:test_hook, :testing) { :none_such }
       hooks_dup = @hooks.dup
 
@@ -415,6 +415,76 @@ describe Pry::Hooks do
         end
       end
     end
+  end
 
+  describe "anonymous hooks" do
+    it 'should allow adding of hook without a name' do
+      @hooks.add_hook(:test_hook, nil) {}
+      @hooks.hook_count(:test_hook).should == 1
+    end
+
+    it 'should only allow one anonymous hook to exist' do
+      @hooks.add_hook(:test_hook, nil) {  }
+      @hooks.add_hook(:test_hook, nil) {  }
+      @hooks.hook_count(:test_hook).should == 1
+    end
+
+    it 'should execute most recently added anonymous hook' do
+      x = nil
+      y = nil
+      @hooks.add_hook(:test_hook, nil) { y = 1 }
+      @hooks.add_hook(:test_hook, nil) { x = 2 }
+      @hooks.exec_hook(:test_hook)
+      y.should == nil
+      x.should == 2
+    end
   end
+
+  describe "deprecated hash-based API" do
+    after do
+      Pry.config.hooks.clear_all if Pry.config.hooks
+    end
+
+    describe "Pry.config.hooks" do
+      it 'should allow a hash-assignment' do
+        Pry.config.hooks = { :before_session => proc { :hello } }
+        Pry.config.hooks.get_hook(:before_session, nil).call.should == :hello
+      end
+
+      describe "Pry.config.hooks[]" do
+        it 'should return the only anonymous hook' do
+          Pry.config.hooks = { :before_session => proc { :hello } }
+          Pry.config.hooks[:before_session].call.should == :hello
+        end
+
+        it 'should add an anonymous hook when using Pry.config.hooks[]=' do
+          Pry.config.hooks[:before_session] = proc { :bing }
+          Pry.config.hooks.hook_count(:before_session).should == 1
+        end
+
+        it 'should add overwrite previous anonymous hooks with new one when calling Pry.config.hooks[]= multiple times' do
+          x = nil
+          Pry.config.hooks[:before_session] = proc { x = 1 }
+          Pry.config.hooks[:before_session] = proc { x = 2 }
+
+          Pry.config.hooks.exec_hook(:before_session)
+          Pry.config.hooks.hook_count(:before_session).should == 1
+          x.should == 2
+        end
+      end
+    end
+
+    describe "Pry.start" do
+      it 'should accept a hash for :hooks parameter' do
+
+        redirect_pry_io(InputTester.new("exit-all"), out=StringIO.new) do
+          Pry.start binding, :hooks => { :before_session => proc { |output, _, _| output.puts 'hello friend' } }
+        end
+
+        out.string.should =~ /hello friend/
+      end
+
+    end
+  end
+
 end

metadata

@@ -1,92 +1,95 @@
---- !ruby/object:Gem::Specification
+--- !ruby/object:Gem::Specification 
 name: pry
-version: !ruby/object:Gem::Version
-  version: 0.9.8pre7
+version: !ruby/object:Gem::Version 
   prerelease: 5
+  version: 0.9.8pre8
 platform: ruby
-authors:
+authors: 
 - John Mair (banisterfiend)
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-01-20 00:00:00.000000000Z
-dependencies:
-- !ruby/object:Gem::Dependency
+
+date: 2012-01-25 00:00:00 Z
+dependencies: 
+- !ruby/object:Gem::Dependency 
   name: coderay
-  requirement: &70177446226220 !ruby/object:Gem::Requirement
+  prerelease: false
+  requirement: &id001 !ruby/object:Gem::Requirement 
     none: false
-    requirements:
+    requirements: 
     - - ~>
-      - !ruby/object:Gem::Version
+      - !ruby/object:Gem::Version 
         version: 1.0.5
   type: :runtime
-  prerelease: false
-  version_requirements: *70177446226220
-- !ruby/object:Gem::Dependency
+  version_requirements: *id001
+- !ruby/object:Gem::Dependency 
   name: slop
-  requirement: &70177446227200 !ruby/object:Gem::Requirement
+  prerelease: false
+  requirement: &id002 !ruby/object:Gem::Requirement 
     none: false
-    requirements:
-    - - ! '>='
-      - !ruby/object:Gem::Version
+    requirements: 
+    - - ">="
+      - !ruby/object:Gem::Version 
         version: 2.4.3
     - - <
-      - !ruby/object:Gem::Version
-        version: '3'
+      - !ruby/object:Gem::Version 
+        version: "3"
   type: :runtime
-  prerelease: false
-  version_requirements: *70177446227200
-- !ruby/object:Gem::Dependency
+  version_requirements: *id002
+- !ruby/object:Gem::Dependency 
   name: method_source
-  requirement: &70177446782420 !ruby/object:Gem::Requirement
+  prerelease: false
+  requirement: &id003 !ruby/object:Gem::Requirement 
     none: false
-    requirements:
+    requirements: 
     - - ~>
-      - !ruby/object:Gem::Version
-        version: '0.7'
+      - !ruby/object:Gem::Version 
+        version: "0.7"
   type: :runtime
-  prerelease: false
-  version_requirements: *70177446782420
-- !ruby/object:Gem::Dependency
+  version_requirements: *id003
+- !ruby/object:Gem::Dependency 
   name: bacon
-  requirement: &70177446839420 !ruby/object:Gem::Requirement
+  prerelease: false
+  requirement: &id004 !ruby/object:Gem::Requirement 
     none: false
-    requirements:
+    requirements: 
     - - ~>
-      - !ruby/object:Gem::Version
-        version: '1.1'
+      - !ruby/object:Gem::Version 
+        version: "1.1"
   type: :development
-  prerelease: false
-  version_requirements: *70177446839420
-- !ruby/object:Gem::Dependency
+  version_requirements: *id004
+- !ruby/object:Gem::Dependency 
   name: open4
-  requirement: &70177446844900 !ruby/object:Gem::Requirement
+  prerelease: false
+  requirement: &id005 !ruby/object:Gem::Requirement 
     none: false
-    requirements:
+    requirements: 
     - - ~>
-      - !ruby/object:Gem::Version
-        version: '1.3'
+      - !ruby/object:Gem::Version 
+        version: "1.3"
   type: :development
-  prerelease: false
-  version_requirements: *70177446844900
-- !ruby/object:Gem::Dependency
+  version_requirements: *id005
+- !ruby/object:Gem::Dependency 
   name: rake
-  requirement: &70177446853320 !ruby/object:Gem::Requirement
+  prerelease: false
+  requirement: &id006 !ruby/object:Gem::Requirement 
     none: false
-    requirements:
+    requirements: 
     - - ~>
-      - !ruby/object:Gem::Version
-        version: '0.9'
+      - !ruby/object:Gem::Version 
+        version: "0.9"
   type: :development
-  prerelease: false
-  version_requirements: *70177446853320
+  version_requirements: *id006
 description: An IRB alternative and runtime developer console
 email: jrmair@gmail.com
-executables:
+executables: 
 - pry
 extensions: []
+
 extra_rdoc_files: []
-files:
+
+files: 
 - .document
 - .gemtest
 - .gitignore
@@ -187,29 +190,32 @@ files:
 - wiki/Home.md
 homepage: http://pry.github.com
 licenses: []
+
 post_install_message: 
 rdoc_options: []
-require_paths:
+
+require_paths: 
 - lib
-required_ruby_version: !ruby/object:Gem::Requirement
+required_ruby_version: !ruby/object:Gem::Requirement 
   none: false
-  requirements:
-  - - ! '>='
-    - !ruby/object:Gem::Version
-      version: '0'
-required_rubygems_version: !ruby/object:Gem::Requirement
+  requirements: 
+  - - ">="
+    - !ruby/object:Gem::Version 
+      version: "0"
+required_rubygems_version: !ruby/object:Gem::Requirement 
   none: false
-  requirements:
-  - - ! '>'
-    - !ruby/object:Gem::Version
+  requirements: 
+  - - ">"
+    - !ruby/object:Gem::Version 
       version: 1.3.1
 requirements: []
+
 rubyforge_project: 
-rubygems_version: 1.8.6
+rubygems_version: 1.8.11
 signing_key: 
 specification_version: 3
 summary: An IRB alternative and runtime developer console
-test_files:
+test_files: 
 - test/helper.rb
 - test/test_cli.rb
 - test/test_code.rb