pry-stack_explorer 0.2.8pre8 → 0.2.8pre9

data/README.md

@@ -5,6 +5,10 @@ pry-stack_explorer
 
 _Walk the stack in a Pry session_
 
+**Please note, pry-stack_explorer has not been officially released yet and the prerelease gem that is available is NOT compatible with the current Pry gem (0.9.7.4).
+The prerelease gem only exists for testing purposes among Pry core committers. This gem will be fully available soon, when the next Pry version (0.9.8) is released.**
+
+
 pry-stack_explorer is a plugin for the [Pry](http://pry.github.com)
 REPL that enables the user to navigate the call-stack.
 

data/lib/pry-stack_explorer.rb

@@ -26,7 +26,7 @@ module PryStackExplorer
   # @param [Pry] _pry_ The Pry instance associated with the frame
   #   managers
   # @return [Array] The stack of Pry::FrameManager objections
-  def self.all_frame_managers(_pry_)
+  def self.frame_managers(_pry_)
     init_frame_hash
     Thread.current[:__pry_frame_managers__][_pry_]
   end
@@ -45,6 +45,8 @@ module PryStackExplorer
     Thread.current[:__pry_frame_managers__][_pry_].clear
   end
 
+  class << self; alias_method :delete_frame_managers, :clear_frame_managers; end
+
   # @return [PryStackExplorer::FrameManager] The currently active frame manager
   def self.frame_manager(_pry_)
     init_frame_hash
@@ -67,7 +69,9 @@ Pry.config.hooks.add_hook(:after_session, :delete_frame_manager) do |_, _, _pry_
 end
 
 Pry.config.hooks.add_hook(:when_started, :save_caller_bindings) do |binding_stack, options, _pry_|
-  options[:call_stack] = true unless options.has_key?(:call_stack)
+  options[:call_stack]    = true unless options.has_key?(:call_stack)
+  options[:initial_frame] = 0 unless options.has_key?(:initial_frame)
+  initial_frame = options[:initial_frame]
 
   next if !options[:call_stack]
 
@@ -94,8 +98,9 @@ Pry.config.hooks.add_hook(:when_started, :save_caller_bindings) do |binding_stac
     end
   end
 
-  binding_stack.replace([bindings.first])
+  binding_stack.replace [bindings[initial_frame]]
   PryStackExplorer.create_and_push_frame_manager(bindings, _pry_)
+  PryStackExplorer.frame_manager(_pry_).set_binding_index_safely(initial_frame)
 end
 
 # Import the StackExplorer commands

data/lib/pry-stack_explorer/commands.rb

@@ -21,7 +21,7 @@ module PryStackExplorer
       else
         binding_index = PryStackExplorer.frame_manager(_pry_).binding_index
         if binding_index - inc < 0
-          output.puts "Warning: At bottom of stack, cannot go further!"
+          raise Pry::CommandError, "At bottom of stack, cannot go further!"
         else
           PryStackExplorer.frame_manager(_pry_).change_frame_to binding_index - inc
         end

data/lib/pry-stack_explorer/frame_manager.rb

@@ -41,8 +41,8 @@ module PryStackExplorer
 
     # Ensure the Pry instance's active binding is the frame manager's
     # active binding.
-    def refresh_frame
-      change_frame_to binding_index
+    def refresh_frame(run_whereami=true)
+      change_frame_to binding_index, run_whereami
     end
 
     # @return [Binding] The currently active frame
@@ -50,24 +50,36 @@ module PryStackExplorer
       bindings[binding_index]
     end
 
-    # Change active frame to the one indexed by `index`.
-    # Note that indexing base is `0`
-    # @param [Fixnum] index The index of the frame.
-    def change_frame_to(index)
-
+    # Set the binding index (aka frame index), but raising an Exception when invalid
+    # index received. Also converts negative indices to their positive counterparts.
+    # @param [Fixnum] index The index.
+    def set_binding_index_safely(index)
       if index > bindings.size - 1
-        @pry.output.puts "Warning: At top of stack, cannot go further!"
+        raise Pry::CommandError, "At top of stack, cannot go further!"
       elsif index < -bindings.size
-        @pry.output.puts "Warning: At bottom of stack, cannot go further!"
+        raise Pry::CommandError, "At bottom of stack, cannot go further!"
       else
         # wrap around negative indices
         index = (bindings.size - 1) + index + 1 if index < 0
 
         self.binding_index = index
-        @pry.binding_stack[-1] = bindings[binding_index]
+      end
+    end
 
-        @pry.run_command "whereami"
+    # Change active frame to the one indexed by `index`.
+    # Note that indexing base is `0`
+    # @param [Fixnum] index The index of the frame.
+    def change_frame_to(index, run_whereami=true)
+
+      set_binding_index_safely(index)
+
+      if @pry.binding_stack.empty?
+        @pry.binding_stack.replace [bindings[binding_index]]
+      else
+        @pry.binding_stack[-1] = bindings[binding_index]
       end
+
+      @pry.run_command "whereami" if run_whereami
     end
 
   end

data/lib/pry-stack_explorer/version.rb

@@ -1,3 +1,3 @@
 module PryStackExplorer
-  VERSION = "0.2.8pre8"
+  VERSION = "0.2.8pre9"
 end

data/pry-stack_explorer.gemspec

@@ -2,20 +2,20 @@
 
 Gem::Specification.new do |s|
   s.name = "pry-stack_explorer"
-  s.version = "0.2.8pre8"
+  s.version = "0.2.8pre9"
 
   s.required_rubygems_version = Gem::Requirement.new("> 1.3.1") if s.respond_to? :required_rubygems_version=
   s.authors = ["John Mair (banisterfiend)"]
-  s.date = "2012-01-05"
+  s.date = "2012-01-08"
   s.description = "Walk the stack in a Pry session"
   s.email = "jrmair@gmail.com"
-  s.files = [".gemtest", ".gitignore", ".travis.yml", ".yardopts", "CHANGELOG", "Gemfile", "LICENSE", "README.md", "Rakefile", "examples/example.rb", "examples/example2.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.files = [".gemtest", ".gitignore", ".travis.yml", ".yardopts", "CHANGELOG", "Gemfile", "LICENSE", "README.md", "Rakefile", "examples/example.rb", "examples/example2.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_commands.rb", "test/test_frame_manager.rb", "test/test_stack_explorer.rb", "tester.rb"]
   s.homepage = "https://github.com/banister"
   s.require_paths = ["lib"]
   s.required_ruby_version = Gem::Requirement.new(">= 1.9.2")
   s.rubygems_version = "1.8.11"
   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"]
+  s.test_files = ["test/helper.rb", "test/test_commands.rb", "test/test_frame_manager.rb", "test/test_stack_explorer.rb"]
 
   if s.respond_to? :specification_version then
     s.specification_version = 3

data/test/helper.rb

@@ -29,10 +29,9 @@ class << Pry
   end
 end
 
-#Pry.config.hooks.clear(:before_session)
-
 AfterSessionHook    = Pry.config.hooks.get_hook(:after_session, :delete_frame_manager)
 WhenStartedHook = Pry.config.hooks.get_hook(:when_started, :save_caller_bindings)
+
 Pry.reset_defaults
 
 class InputTester

data/test/test_commands.rb

@@ -0,0 +1,130 @@
+require 'helper'
+
+describe PryStackExplorer::StackCommands do
+
+  before do
+    Pry.config.hooks.add_hook(:when_started, :save_caller_bindings, &WhenStartedHook)
+    Pry.config.hooks.add_hook(:after_session, :delete_frame_manager, &AfterSessionHook)
+
+    @o = Object.new
+    class << @o; attr_accessor :first_method, :second_method, :third_method; end
+    def @o.bing() bong end
+    def @o.bong() bang end
+    def @o.bang() Pry.start(binding) end
+  end
+
+  after do
+    Pry.config.hooks.delete_hook(:when_started, :save_caller_bindings)
+    Pry.config.hooks.delete_hook(:after_session, :delete_frame_manager)
+  end
+
+  describe "up" do
+    it 'should move up the call stack one frame at a time' do
+      redirect_pry_io(InputTester.new("@first_method = __method__",
+                                      "up",
+                                      "@second_method = __method__",
+                                      "up",
+                                      "@third_method = __method__",
+                                      "exit-all"), out=StringIO.new) do
+        @o.bing
+      end
+
+      @o.first_method.should  == :bang
+      @o.second_method.should == :bong
+      @o.third_method.should  == :bing
+    end
+
+    it 'should move up the call stack two frames at a time' do
+      redirect_pry_io(InputTester.new("@first_method = __method__",
+                                      "up 2",
+                                      "@second_method = __method__",
+                                      "exit-all"), out=StringIO.new) do
+        @o.bing
+      end
+
+      @o.first_method.should  == :bang
+      @o.second_method.should == :bing
+    end
+  end
+
+  describe "down" do
+    it 'should move down the call stack one frame at a time' do
+      def @o.bang() Pry.start(binding, :initial_frame => 1) end
+
+      redirect_pry_io(InputTester.new("@first_method = __method__",
+                                      "down",
+                                      "@second_method = __method__",
+                                      "exit-all"), out=StringIO.new) do
+        @o.bing
+      end
+
+      @o.first_method.should  == :bong
+      @o.second_method.should == :bang
+    end
+
+    it 'should move down the call stack two frames at a time' do
+      def @o.bang() Pry.start(binding, :initial_frame => 2) end
+
+      redirect_pry_io(InputTester.new("@first_method = __method__",
+                                      "down 2",
+                                      "@second_method = __method__",
+                                      "exit-all"), out=StringIO.new) do
+        @o.bing
+      end
+
+      @o.first_method.should  == :bing
+      @o.second_method.should == :bang
+    end
+  end
+
+  describe "frame" do
+    it 'should move to the given frame in the call stack' do
+      redirect_pry_io(InputTester.new("frame 2",
+                                      "@first_method = __method__",
+                                      "exit-all"), out=StringIO.new) do
+        @o.bing
+      end
+
+      @o.first_method.should == :bing
+    end
+
+    describe "negative indices" do
+      it 'should work with negative frame numbers' do
+        o = Object.new
+        class << o; attr_accessor :frame; end
+        def o.alpha() binding end
+        def o.beta()  binding end
+        def o.gamma() binding end
+
+        call_stack   = [o.alpha, o.beta, o.gamma]
+        method_names = call_stack.map { |v| v.eval('__method__') }.reverse
+        (1..3).each_with_index do |v, idx|
+          redirect_pry_io(InputTester.new("frame -#{v}",
+                                          "@frame = __method__",
+                                          "exit-all"), out=StringIO.new) do
+            Pry.start(o, :call_stack => call_stack)
+          end
+          o.frame.should == method_names[idx]
+        end
+      end
+
+      it 'should convert negative indices to their positive counterparts' do
+        o = Object.new
+        class << o; attr_accessor :frame_number; end
+        def o.alpha() binding end
+        def o.beta()  binding end
+        def o.gamma() binding end
+
+        call_stack   = [o.alpha, o.beta, o.gamma]
+        (1..3).each_with_index do |v, idx|
+          redirect_pry_io(InputTester.new("frame -#{v}",
+                                          "@frame_number = PryStackExplorer.frame_manager(_pry_).binding_index",
+                                          "exit-all"), out=StringIO.new) do
+            Pry.start(o, :call_stack => call_stack)
+          end
+          o.frame_number.should == call_stack.size - v
+        end
+      end
+    end
+  end
+end

data/test/test_stack_explorer.rb

@@ -1,168 +1,210 @@
-require 'helper'
-
-describe PryStackExplorer do
-
-  describe "Pry.start" do
-    before do
-      Pry.config.hooks.add_hook(:when_started, :save_caller_bindings, &WhenStartedHook)
-      Pry.config.hooks.add_hook(:after_session, :delete_frame_manager, &AfterSessionHook)
-    end
-
-    after do
-      Pry.config.hooks.delete_hook(:when_started, :save_caller_bindings)
-      Pry.config.hooks.delete_hook(:after_session, :delete_frame_manager)
-    end
-
-    it 'should invoke a session with the call stack set' do
-      o = Object.new
-      def o.bing() bong end
-      def o.bong() bang end
-      def o.bang() Pry.start(binding) end
-
-      redirect_pry_io(StringIO.new("show-stack\nexit\n"), out=StringIO.new) do
-        o.bing
-      end
-
-      out.string.should =~ /bang.*?bong.*?bing/m
-    end
-
-    it 'should set no call stack when :call_stack => false' do
-      o = Object.new
-      def o.bing() bong end
-      def o.bong() bang end
-      def o.bang() Pry.start(binding, :call_stack => false) end
-
-      redirect_pry_io(StringIO.new("show-stack\nexit\n"), out=StringIO.new) do
-        o.bing
-      end
-
-      out.string.should =~ /No caller stack/
-    end
-
-    it 'should set custom call stack when :call_stack => [b1, b2]' do
-      o = Object.new
-      def o.alpha() binding end
-      def o.beta() binding end
-      def o.gamma() binding end
-
-      redirect_pry_io(StringIO.new("show-stack\nexit\n"), out=StringIO.new) do
-        Pry.start(binding, :call_stack => [o.beta, o.gamma, o.alpha])
-      end
-
-      out.string.should =~ /beta.*?gamma.*?alpha/m
-    end
-
-    it 'should raise if custom call stack does not contain bindings or is empty' do
-      redirect_pry_io(StringIO.new("show-stack\nexit\n"), out=StringIO.new) do
-        lambda { Pry.start(binding, :call_stack => [1, 2, 3]) }.should.raise ArgumentError
-      end
-
-      redirect_pry_io(StringIO.new("show-stack\nexit\n"), out=StringIO.new) do
-        lambda { Pry.start(binding, :call_stack => []) }.should.raise ArgumentError
-      end
-    end
-  end
-
-  describe "unit tests for PryStackExplorer class methods" do
-    before do
-      @pry_instance = Pry.new
-      @bindings = [binding, binding]
-    end
-
-    after do
-      PE.clear_frame_managers(@pry_instance)
-    end
-
-    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
-
-    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
-
-    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
-
-    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
-end
+require 'helper'
+
+describe PryStackExplorer do
+
+  describe "Pry.start" do
+    before do
+      Pry.config.hooks.add_hook(:when_started, :save_caller_bindings, &WhenStartedHook)
+      Pry.config.hooks.add_hook(:after_session, :delete_frame_manager, &AfterSessionHook)
+
+      @o = Object.new
+      class << @o; attr_reader :frame; end
+      def @o.bing() bong end
+      def @o.bong() bang end
+      def @o.bang() Pry.start(binding) end
+    end
+
+    after do
+      Pry.config.hooks.delete_hook(:when_started, :save_caller_bindings)
+      Pry.config.hooks.delete_hook(:after_session, :delete_frame_manager)
+    end
+
+    describe ":initial_frame option" do
+      it 'should default to first frame when no option provided' do
+        redirect_pry_io(StringIO.new("@frame = __method__\nexit\n"), out=StringIO.new) do
+          @o.bing
+        end
+
+        @o.frame.should == :bang
+      end
+
+      it 'should begin session at specified frame' do
+        o = Object.new
+        class << o; attr_reader :frame; end
+        def o.bing() bong end
+        def o.bong() bang end
+        def o.bang() Pry.start(binding, :initial_frame => 1) end
+
+        redirect_pry_io(StringIO.new("@frame = __method__\nexit\n"), out=StringIO.new) do
+          o.bing
+        end
+
+        o.frame.should == :bong
+      end
+
+      it 'should begin session at specified frame when using :call_stack' do
+        o = Object.new
+        class << o; attr_accessor :frame; end
+        def o.alpha() binding end
+        def o.beta() binding end
+        def o.gamma() binding end
+
+        redirect_pry_io(StringIO.new("@frame = __method__\nexit\n"), out=StringIO.new) do
+          Pry.start(binding, :call_stack => [o.gamma, o.beta, o.alpha], :initial_frame => 1)
+        end
+
+        o.frame.should == :beta
+      end
+
+    end
+
+    describe ":call_stack option" do
+      it 'should invoke a session with the call stack set' do
+        redirect_pry_io(StringIO.new("show-stack\nexit\n"), out=StringIO.new) do
+          @o.bing
+        end
+
+        out.string.should =~ /bang.*?bong.*?bing/m
+      end
+
+      it 'should set no call stack when :call_stack => false' do
+        o = Object.new
+        def o.bing() bong end
+        def o.bong() bang end
+        def o.bang() Pry.start(binding, :call_stack => false) end
+
+        redirect_pry_io(StringIO.new("show-stack\nexit\n"), out=StringIO.new) do
+          o.bing
+        end
+
+        out.string.should =~ /No caller stack/
+      end
+
+      it 'should set custom call stack when :call_stack => [b1, b2]' do
+        o = Object.new
+        def o.alpha() binding end
+        def o.beta() binding end
+        def o.gamma() binding end
+
+        redirect_pry_io(StringIO.new("show-stack\nexit\n"), out=StringIO.new) do
+          Pry.start(binding, :call_stack => [o.beta, o.gamma, o.alpha])
+        end
+
+        out.string.should =~ /beta.*?gamma.*?alpha/m
+      end
+
+      it 'should raise if custom call stack does not contain bindings or is empty' do
+        redirect_pry_io(StringIO.new("show-stack\nexit\n"), out=StringIO.new) do
+          lambda { Pry.start(binding, :call_stack => [1, 2, 3]) }.should.raise ArgumentError
+        end
+
+        redirect_pry_io(StringIO.new("show-stack\nexit\n"), out=StringIO.new) do
+          lambda { Pry.start(binding, :call_stack => []) }.should.raise ArgumentError
+        end
+      end
+    end
+  end
+
+  describe "unit tests for PryStackExplorer class methods" do
+    before do
+      @pry_instance = Pry.new
+      @bindings = [binding, binding]
+    end
+
+    after do
+      PE.clear_frame_managers(@pry_instance)
+    end
+
+    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.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.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.frame_managers(@pry_instance).count.should == 1
+        PE.frame_managers(p2).count.should == 1
+      end
+    end
+
+    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
+
+    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.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.frame_managers(@pry_instance).count.should == 0
+        PE.frame_managers(p2).count.should == 1
+      end
+    end
+
+    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.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.frame_managers(p2).count.should == 1
+        PE.frame_managers(@pry_instance).count.should == 0
+      end
+    end
+  end
+end

metadata

@@ -2,7 +2,7 @@
 name: pry-stack_explorer
 version: !ruby/object:Gem::Version 
   prerelease: 5
-  version: 0.2.8pre8
+  version: 0.2.8pre9
 platform: ruby
 authors: 
 - John Mair (banisterfiend)
@@ -10,7 +10,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2012-01-05 00:00:00 Z
+date: 2012-01-08 00:00:00 Z
 dependencies: 
 - !ruby/object:Gem::Dependency 
   name: binding_of_caller
@@ -82,6 +82,7 @@ files:
 - lib/pry-stack_explorer/version.rb
 - pry-stack_explorer.gemspec
 - test/helper.rb
+- test/test_commands.rb
 - test/test_frame_manager.rb
 - test/test_stack_explorer.rb
 - tester.rb
@@ -114,5 +115,6 @@ specification_version: 3
 summary: Walk the stack in a Pry session
 test_files: 
 - test/helper.rb
+- test/test_commands.rb
 - test/test_frame_manager.rb
 - test/test_stack_explorer.rb