pry-stack_explorer 0.2.8pre4 → 0.2.8pre5

data/.gitignore

@@ -0,0 +1,7 @@
+Makefile
+*.so
+*.o
+*.def
+doc/
+pkg/
+.yardoc/

data/.travis.yml

@@ -0,0 +1,15 @@
+script:
+  rake test --trace
+
+rvm:
+  - 1.9.2
+  - 1.9.3
+
+notifications:
+  irc: "irc.freenode.org#pry"
+  recipients:
+    - jrmair@gmail.com
+
+branches:
+  only:
+    - master

data/.yardopts

@@ -0,0 +1 @@
+--markup markdown

data/LICENSE

@@ -0,0 +1,25 @@
+License
+-------
+
+(The MIT License) 
+
+Copyright (c) 2011 John Mair (banisterfiend)
+
+Permission is hereby granted, free of charge, to any person obtaining
+a copy of this software and associated documentation files (the
+'Software'), to deal in the Software without restriction, including
+without limitation the rights to use, copy, modify, merge, publish,
+distribute, sublicense, and/or sell copies of the Software, and to
+permit persons to whom the Software is furnished to do so, subject to
+the following conditions:
+
+The above copyright notice and this permission notice shall be
+included in all copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED 'AS IS', WITHOUT WARRANTY OF ANY KIND,
+EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
+MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.
+IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY
+CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT,
+TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE
+SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.

data/Rakefile

@@ -23,9 +23,12 @@ def apply_spec_defaults(s)
   s.description = s.summary
   s.require_path = 'lib'
   s.add_dependency("binding_of_caller","~>0.6.1")
+#  s.add_dependency("pry","0.9.8.0pre1")
   s.add_development_dependency("bacon","~>1.1.0")
+  s.add_development_dependency('rake', '~> 0.9')
   s.homepage = "https://github.com/banister"
-  s.files = Dir["lib/**/*.rb", "test/*.rb", "CHANGELOG", "README.md", "Rakefile"]
+  s.files = `git ls-files`.split("\n")
+  s.test_files = `git ls-files -- test/*`.split("\n")
 end
 
 desc "run pry with plugin enabled"
@@ -43,9 +46,12 @@ task :version do
   puts "PryStackExplorer version: #{PryStackExplorer::VERSION}"
 end
 
+desc "run tests"
+task :default => :test
+
 desc "run tests"
 task :test do
-  sh "bacon -Itest -rubygems -a"
+  sh "bacon -Itest -rubygems -a -q"
 end
 
 namespace :ruby do

data/examples/example.rb

@@ -0,0 +1,45 @@
+unless Object.const_defined? :PryStackExplorer
+  $:.unshift File.expand_path '../../lib', __FILE__
+  require 'pry'
+end
+
+require 'pry-stack_explorer'
+
+def alphabet(y)
+  x = 20
+  b
+end
+
+def b
+  x = 30
+  proc {
+    c
+  }.call
+end
+
+def c
+  u = 50
+  J.pry
+end
+
+# hello
+def beta
+  gamma
+end
+
+def gamma
+  zeta
+end
+
+def zeta
+  vitamin = 100
+  binding.pry
+end
+#
+
+proc {
+  class J
+    alphabet(22)
+  end
+}.call
+

data/lib/pry-stack_explorer/commands.rb

@@ -77,7 +77,7 @@ module PryStackExplorer
       def frame_info(b, verbose = false)
         meth = b.eval('__method__')
         b_self = b.eval('self')
-        meth_obj = Pry::Method.new(b_self.method(meth)) if meth
+        meth_obj = Pry::Method.from_binding(b) if meth
 
         type = b.frame_type ? "[#{b.frame_type}]" : ""
         desc = b.frame_description ? "#{b.frame_description} in " : "#{PryStackExplorer.frame_manager(_pry_).frame_info_for(b)} in "
@@ -90,18 +90,22 @@ module PryStackExplorer
       end
 
       def se_signature_with_owner(meth_obj)
-        args = meth_obj.parameters.inject([]) do |arr, (type, name)|
-          name ||= (type == :block ? 'block' : "arg#{arr.size + 1}")
-          arr << case type
-                 when :req   then name.to_s
-                 when :opt   then "#{name}=?"
-                 when :rest  then "*#{name}"
-                 when :block then "&#{name}"
-                 else '?'
-                 end
+        if !meth_obj.undefined?
+          args = meth_obj.parameters.inject([]) do |arr, (type, name)|
+            name ||= (type == :block ? 'block' : "arg#{arr.size + 1}")
+            arr << case type
+                   when :req   then name.to_s
+                   when :opt   then "#{name}=?"
+                   when :rest  then "*#{name}"
+                   when :block then "&#{name}"
+                   else '?'
+                   end
+          end
+          "#{meth_obj.name_with_owner}(#{args.join(', ')})"
+        else
+          "#{meth_obj.name_with_owner}(UNKNOWN) (undefined method)"
         end
 
-        "#{meth_obj.name_with_owner}(#{args.join(', ')})"
       end
     end
 

data/lib/pry-stack_explorer/version.rb

@@ -1,3 +1,3 @@
 module PryStackExplorer
-  VERSION = "0.2.8pre4"
+  VERSION = "0.2.8pre5"
 end

data/pry-stack_explorer.gemspec

@@ -0,0 +1,36 @@
+# -*- encoding: utf-8 -*-
+
+Gem::Specification.new do |s|
+  s.name = "pry-stack_explorer"
+  s.version = "0.2.8pre5"
+
+  s.required_rubygems_version = Gem::Requirement.new("> 1.3.1") if s.respond_to? :required_rubygems_version=
+  s.authors = ["John Mair (banisterfiend)"]
+  s.date = "2011-12-21"
+  s.description = "Walk the stack in a Pry session"
+  s.email = "jrmair@gmail.com"
+  s.files = [".gemtest", ".gitignore", ".travis.yml", ".yardopts", "CHANGELOG", "LICENSE", "README.md", "Rakefile", "examples/example.rb", "lib/pry-stack_explorer.rb", "lib/pry-stack_explorer/commands.rb", "lib/pry-stack_explorer/frame_manager.rb", "lib/pry-stack_explorer/version.rb", "pry-stack_explorer.gemspec", "test/helper.rb", "test/test_frame_manager.rb", "test/test_stack_explorer.rb", "tester.rb"]
+  s.homepage = "https://github.com/banister"
+  s.require_paths = ["lib"]
+  s.rubygems_version = "1.8.10"
+  s.summary = "Walk the stack in a Pry session"
+  s.test_files = ["test/helper.rb", "test/test_frame_manager.rb", "test/test_stack_explorer.rb"]
+
+  if s.respond_to? :specification_version then
+    s.specification_version = 3
+
+    if Gem::Version.new(Gem::VERSION) >= Gem::Version.new('1.2.0') then
+      s.add_runtime_dependency(%q<binding_of_caller>, ["~> 0.6.1"])
+      s.add_development_dependency(%q<bacon>, ["~> 1.1.0"])
+      s.add_development_dependency(%q<rake>, ["~> 0.9"])
+    else
+      s.add_dependency(%q<binding_of_caller>, ["~> 0.6.1"])
+      s.add_dependency(%q<bacon>, ["~> 1.1.0"])
+      s.add_dependency(%q<rake>, ["~> 0.9"])
+    end
+  else
+    s.add_dependency(%q<binding_of_caller>, ["~> 0.6.1"])
+    s.add_dependency(%q<bacon>, ["~> 1.1.0"])
+    s.add_dependency(%q<rake>, ["~> 0.9"])
+  end
+end

data/test/test_frame_manager.rb

@@ -0,0 +1,65 @@
+require 'helper'
+
+Pry.config.output = StringIO.new
+
+describe PryStackExplorer::FrameManager do
+
+  before do
+    @pry_instance = Pry.new
+    @bindings = [binding, binding, binding, binding]
+    @bindings.each_with_index { |v, i| v.eval("x = #{i}") }
+    @pry_instance.binding_stack.push @bindings.last
+    @frame_manager = PE::FrameManager.new(@bindings, @pry_instance)
+  end
+
+  describe "creation" do
+    it "should make bindings accessible via 'bindings' method" do
+      @frame_manager.bindings.should == @bindings
+    end
+
+    it "should set binding_index to 0" do
+      @frame_manager.binding_index.should == 0
+    end
+
+    it "should set current_frame to first frame" do
+      @frame_manager.current_frame.should == @bindings.first
+    end
+  end
+
+  describe "FrameManager#change_frame_to" do
+    it 'should change the frame to the given one' do
+      @frame_manager.change_frame_to(1)
+
+      @frame_manager.binding_index.should == 1
+      @frame_manager.current_frame.should == @bindings[1]
+      @pry_instance.binding_stack.last.should == @frame_manager.current_frame
+    end
+
+    it 'should accept negative indices when specifying frame' do
+      @frame_manager.change_frame_to(-1)
+
+      # negative index is converted to a positive one inside change_frame_to
+      @frame_manager.binding_index.should == @bindings.size - 1
+
+      @frame_manager.current_frame.should == @bindings[-1]
+      @pry_instance.binding_stack.last.should == @frame_manager.current_frame
+    end
+  end
+
+  describe "FrameManager#refresh_frame" do
+    it 'should change the Pry frame to the active one in the FrameManager' do
+      @frame_manager.binding_index = 2
+      @frame_manager.refresh_frame
+
+      @pry_instance.binding_stack.last.should == @frame_manager.current_frame
+    end
+  end
+
+  describe "FrameManager is Enumerable" do
+    it 'should perform an Enumerable#map on the frames' do
+      @frame_manager.map { |v| v.eval("x") }.should == (0..(@bindings.size - 1)).to_a
+    end
+  end
+
+end
+

data/test/test_stack_explorer.rb

@@ -11,29 +11,95 @@ describe PryStackExplorer do
     PE.clear_frame_managers(@pry_instance)
   end
 
-  it  "should create and push one new FrameManager" do
-    PE.create_and_push_frame_manager(@bindings, @pry_instance)
-    PE.frame_manager(@pry_instance).is_a?(PE::FrameManager).should == true
-    PE.all_frame_managers(@pry_instance).count.should == 1
+  describe "PryStackExplorer.create_and_push_frame_manager" do
+    it  "should create and push one new FrameManager" do
+      PE.create_and_push_frame_manager(@bindings, @pry_instance)
+      PE.frame_manager(@pry_instance).is_a?(PE::FrameManager).should == true
+      PE.all_frame_managers(@pry_instance).count.should == 1
+    end
+
+    it  "should create and push multiple FrameManagers" do
+      PE.create_and_push_frame_manager(@bindings, @pry_instance)
+      PE.create_and_push_frame_manager(@bindings, @pry_instance)
+      PE.all_frame_managers(@pry_instance).count.should == 2
+    end
+
+    it 'should push FrameManagers to stacks based on Pry instance' do
+      p2 = Pry.new
+      bindings = [binding, binding]
+      PE.create_and_push_frame_manager(@bindings, @pry_instance)
+      PE.create_and_push_frame_manager(bindings, p2)
+      PE.all_frame_managers(@pry_instance).count.should == 1
+      PE.all_frame_managers(p2).count.should == 1
+    end
   end
 
-  it  "should have the correct bindings" do
-    PE.create_and_push_frame_manager(@bindings, @pry_instance)
-    PE.frame_manager(@pry_instance).bindings.should == @bindings
+  describe "PryStackExplorer.frame_manager" do
+    it  "should have the correct bindings" do
+      PE.create_and_push_frame_manager(@bindings, @pry_instance)
+      PE.frame_manager(@pry_instance).bindings.should == @bindings
+    end
+
+    it "should return the last pushed FrameManager" do
+      bindings = [binding, binding]
+      PE.create_and_push_frame_manager(@bindings, @pry_instance)
+      PE.create_and_push_frame_manager(bindings, @pry_instance)
+      PE.frame_manager(@pry_instance).bindings.should == bindings
+    end
+
+    it "should return the correct FrameManager for the given Pry instance" do
+      bindings = [binding, binding]
+      p2 = Pry.new
+      PE.create_and_push_frame_manager(@bindings, @pry_instance)
+      PE.create_and_push_frame_manager(bindings, p2)
+      PE.frame_manager(@pry_instance).bindings.should == @bindings
+      PE.frame_manager(p2).bindings.should == bindings
+    end
   end
 
-  it "should pop a FrameManager" do
-    PE.create_and_push_frame_manager(@bindings, @pry_instance)
-    PE.create_and_push_frame_manager(@bindings, @pry_instance)
-    PE.pop_frame_manager(@pry_instance)
-    PE.all_frame_managers(@pry_instance).count.should == 1
+  describe "PryStackExplorer.pop_frame_manager" do
+    it "should remove FrameManager from stack" do
+      PE.create_and_push_frame_manager(@bindings, @pry_instance)
+      PE.create_and_push_frame_manager(@bindings, @pry_instance)
+      PE.pop_frame_manager(@pry_instance)
+      PE.all_frame_managers(@pry_instance).count.should == 1
+    end
+
+    it "should return the most recently added FrameManager" do
+      bindings = [binding, binding]
+      PE.create_and_push_frame_manager(@bindings, @pry_instance)
+      PE.create_and_push_frame_manager(bindings, @pry_instance)
+      PE.pop_frame_manager(@pry_instance).bindings.should == bindings
+    end
+
+    it "should remove FrameManager from the appropriate stack based on Pry instance" do
+      p2 = Pry.new
+      bindings = [binding, binding]
+      PE.create_and_push_frame_manager(@bindings, @pry_instance)
+      PE.create_and_push_frame_manager(bindings, p2)
+      PE.pop_frame_manager(@pry_instance)
+      PE.all_frame_managers(@pry_instance).count.should == 0
+      PE.all_frame_managers(p2).count.should == 1
+    end
   end
 
-  it "should clear all FrameManagers for a Pry instance" do
-    PE.create_and_push_frame_manager(@bindings, @pry_instance)
-    PE.create_and_push_frame_manager(@bindings, @pry_instance)
-    PE.clear_frame_managers(@pry_instance)
-    PE.all_frame_managers(@pry_instance).count.should == 0
+  describe "PryStackExplorer.clear_frame_managers" do
+    it "should clear all FrameManagers for a Pry instance" do
+      PE.create_and_push_frame_manager(@bindings, @pry_instance)
+      PE.create_and_push_frame_manager(@bindings, @pry_instance)
+      PE.clear_frame_managers(@pry_instance)
+      PE.all_frame_managers(@pry_instance).count.should == 0
+    end
+
+    it "should clear all FrameManagers for a Pry instance" do
+      p2 = Pry.new
+      bindings = [binding, binding]
+      PE.create_and_push_frame_manager(@bindings, @pry_instance)
+      PE.create_and_push_frame_manager(bindings, p2)
+      PE.clear_frame_managers(@pry_instance)
+      PE.all_frame_managers(p2).count.should == 1
+      PE.all_frame_managers(@pry_instance).count.should == 0
+    end
   end
 end
 

data/tester.rb

@@ -0,0 +1,27 @@
+def a
+  x = 20
+  b
+end
+
+def b
+  x = 30
+  c
+end
+
+def c
+  u = 50
+  binding.pry
+  puts "hi"
+  puts "bye"
+  v = 20
+  puts v
+  k
+end
+
+def k
+  puts "lovely girl"
+  james = "hello"
+  puts james
+end
+
+a

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: pry-stack_explorer
 version: !ruby/object:Gem::Version
-  version: 0.2.8pre4
+  version: 0.2.8pre5
   prerelease: 5
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2011-12-20 00:00:00.000000000 Z
+date: 2011-12-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: binding_of_caller
-  requirement: &70241510740720 !ruby/object:Gem::Requirement
+  requirement: &70302726218440 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -21,10 +21,10 @@ dependencies:
         version: 0.6.1
   type: :runtime
   prerelease: false
-  version_requirements: *70241510740720
+  version_requirements: *70302726218440
 - !ruby/object:Gem::Dependency
   name: bacon
-  requirement: &70241510738600 !ruby/object:Gem::Requirement
+  requirement: &70302726217820 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -32,22 +32,42 @@ dependencies:
         version: 1.1.0
   type: :development
   prerelease: false
-  version_requirements: *70241510738600
+  version_requirements: *70302726217820
+- !ruby/object:Gem::Dependency
+  name: rake
+  requirement: &70302726217160 !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '0.9'
+  type: :development
+  prerelease: false
+  version_requirements: *70302726217160
 description: Walk the stack in a Pry session
 email: jrmair@gmail.com
 executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- .gemtest
+- .gitignore
+- .travis.yml
+- .yardopts
+- CHANGELOG
+- LICENSE
+- README.md
+- Rakefile
+- examples/example.rb
+- lib/pry-stack_explorer.rb
 - lib/pry-stack_explorer/commands.rb
 - lib/pry-stack_explorer/frame_manager.rb
 - lib/pry-stack_explorer/version.rb
-- lib/pry-stack_explorer.rb
+- pry-stack_explorer.gemspec
 - test/helper.rb
+- test/test_frame_manager.rb
 - test/test_stack_explorer.rb
-- CHANGELOG
-- README.md
-- Rakefile
+- tester.rb
 homepage: https://github.com/banister
 licenses: []
 post_install_message: 
@@ -72,4 +92,7 @@ rubygems_version: 1.8.10
 signing_key: 
 specification_version: 3
 summary: Walk the stack in a Pry session
-test_files: []
+test_files:
+- test/helper.rb
+- test/test_frame_manager.rb
+- test/test_stack_explorer.rb