pry-exception_explorer 0.1.1pre3 → 0.1.1pre5

data/Rakefile

@@ -43,9 +43,21 @@ task :reinstall => :gems do
   sh "gem install #{direc}/pkg/pry-exception_explorer-#{PryExceptionExplorer::VERSION}.gem"
 end
 
+desc "Run example"
+task :example do
+  sh "ruby -I#{direc}/lib/ #{direc}/examples/example.rb "
+end
+
+task :default => :test
+
+desc "Show version"
+task :version do
+  puts "PryExceptionExplorer version: #{PryExceptionExplorer::VERSION}"
+end
+
 desc "run tests"
 task :test do
-  sh "bacon -Itest -rubygems -a"
+  sh "bacon -Itest -rubygems -a -q"
 end
 
 desc "Build gemspec"

data/examples/example.rb

@@ -0,0 +1,30 @@
+unless Object.const_defined? :PryExceptionExplorer
+  $:.unshift File.expand_path '../../lib', __FILE__
+  require 'pry'
+end
+
+require 'pry-exception_explorer'
+require 'pry-exception_explorer/exception_wrap'
+
+def alpha
+  name = "john"
+  beta
+  puts name
+end
+
+def beta
+  x = 20
+  gamma
+  puts x
+end
+
+def gamma
+  raise "oh my, an exception"
+end
+
+
+PryExceptionExplorer.intercept(Exception)
+
+PryExceptionExplorer.wrap do
+  alpha
+end

data/lib/pry-exception_explorer/commands.rb

@@ -5,6 +5,7 @@ PryExceptionExplorer::Commands = Pry::CommandSet.new do
   command "enter-exception", "Enter the context of the last exception" do
     ex = _pry_.last_exception
     if ex && ex.exception_call_stack
+      _pry_.backtrace = ex.backtrace
       PryStackExplorer.create_and_push_frame_manager(ex.exception_call_stack, _pry_)
       PryStackExplorer.frame_manager(_pry_).user[:exception] = ex
       PryStackExplorer.frame_manager(_pry_).refresh_frame
@@ -15,13 +16,42 @@ PryExceptionExplorer::Commands = Pry::CommandSet.new do
     end
   end
 
-  command "exit-exception", "Leave the context of the current exception." do
-    fm = PryStackExplorer.frame_manager(_pry_)
-    if fm && fm.user[:exception]
-      PryStackExplorer.pop_frame_manager(_pry_)
-      PryStackExplorer.frame_manager(_pry_).refresh_frame
-    else
-      output.puts "You are not in an exception!"
+  command_class "exit-exception", "Leave the context of the current exception." do
+    banner <<-BANNER
+      Usage: exit-exception
+      Exit active exception and return to containing context.
+    BANNER
+
+    def process
+      if !in_exception?
+        raise Pry::CommandError, "You are not in an exception!"
+      elsif !prior_context_exists?
+        run "exit-all"
+      else
+        popped_fm = PryStackExplorer.pop_frame_manager(_pry_)
+        if frame_manager
+          frame_manager.refresh_frame
+        else
+          _pry_.binding_stack[-1] = popped_fm.prior_binding
+        end
+      end
+    end
+
+    private
+    def frame_manager
+      PryStackExplorer.frame_manager(_pry_)
+    end
+
+    def frame_managers
+      PryStackExplorer.frame_managers(_pry_)
+    end
+
+    def prior_context_exists?
+      frame_managers.count > 1 || frame_manager.prior_binding
+    end
+
+    def in_exception?
+      frame_manager && frame_manager.user[:exception]
     end
   end
 
@@ -29,8 +59,8 @@ PryExceptionExplorer::Commands = Pry::CommandSet.new do
     fm = PryStackExplorer.frame_manager(_pry_)
 
     if fm && fm.user[:exception] && fm.user[:inline_exception]
-       _pry_.run_command "exit-all PryExceptionExplorer::CONTINUE_INLINE_EXCEPTION"
       PryStackExplorer.pop_frame_manager(_pry_)
+      _pry_.run_command "exit-all PryExceptionExplorer::CONTINUE_INLINE_EXCEPTION"
     elsif fm && fm.user[:exception] && fm.user[:exception].continuation
       PryStackExplorer.pop_frame_manager(_pry_)
       fm.user[:exception].continue

data/lib/pry-exception_explorer/exception_wrap.rb

@@ -1,6 +1,6 @@
 require 'pry-exception_explorer'
 
-Pry.config.hooks.delete_hook(:when_started, :save_caller_bindings)
+#Pry.config.hooks.delete_hook(:when_started, :save_caller_bindings)
 
 # default is to capture all exceptions that bubble to the top
 PryExceptionExplorer.intercept { true }
@@ -8,22 +8,26 @@ PryExceptionExplorer.intercept { true }
 module PryExceptionExplorer
 
   def self.wrap
+    old_enabled, old_wrap_active = enabled, wrap_active
+    self.enabled     = true
     self.wrap_active = true
     yield
   rescue Exception => ex
-    Pry.config.hooks.add_hook(:when_started, :setup_exception_context) do |binding_stack, _pry_|
-      binding_stack.replace([ex.exception_call_stack.first])
-      PryStackExplorer.create_and_push_frame_manager(ex.exception_call_stack, _pry_)
-      PryStackExplorer.frame_manager(_pry_).user[:exception] = ex
-    end
-
     if ex.should_capture?
-      pry
+      hooks = Pry.config.hooks.dup.add_hook(:before_session, :set_exception_flag) do |_, _, _pry_|
+        PryStackExplorer.frame_manager(_pry_).user[:exception] = ex
+
+        _pry_.last_exception = ex
+        _pry_.backtrace = ex.backtrace
+      end
+
+      Pry.start self, :call_stack => ex.exception_call_stack, :hooks => hooks
     else
       raise ex
     end
   ensure
-    self.wrap_active = false
+    self.enabled     = old_enabled
+    self.wrap_active = old_wrap_active
   end
 end
 

data/lib/pry-exception_explorer/version.rb

@@ -1,3 +1,3 @@
 module PryExceptionExplorer
-  VERSION = "0.1.1pre3"
+  VERSION = "0.1.1pre5"
 end

data/lib/pry-exception_explorer.rb

@@ -15,6 +15,10 @@ module PryExceptionExplorer
   CONTINUE_INLINE_EXCEPTION = Object.new
 
   class << self
+
+    # @return [Boolean] Whether `PryStackExplorer` is enabled.
+    attr_accessor :enabled
+
     def wrap_active=(v)
       Thread.current[:__pry_exception_explorer_wrap__] = v
     end
@@ -24,6 +28,7 @@ module PryExceptionExplorer
     end
 
     alias_method :wrap_active?, :wrap_active
+    alias_method :enabled?, :enabled
   end
 
   self.wrap_active = false
@@ -32,7 +37,11 @@ module PryExceptionExplorer
     true
   end
 
-  def self.intercept(&block)
+  def self.intercept(*exceptions, &block)
+    if !exceptions.empty?
+      block = proc { |_, ex| exceptions.any? { |v| v === ex } }
+    end
+
     Thread.current[:__pry_exception_explorer_intercept_block__] = block
   end
 
@@ -79,6 +88,7 @@ end
 
 class Object
   def raise(exception = RuntimeError, string = nil, array = caller)
+
     if exception.is_a?(String)
       string = exception
       exception = RuntimeError
@@ -87,6 +97,11 @@ class Object
     ex = exception.exception(string)
     ex.set_backtrace(array)
 
+    # revert to normal exception behaviour if EE not enabled.
+    if !PryExceptionExplorer.enabled?
+      return super(ex)
+    end
+
     if PryExceptionExplorer.should_capture_exception?(ex, binding.of_caller(1))
       ex.exception_call_stack = binding.callers.tap(&:shift)
       ex.should_capture       = true
@@ -114,3 +129,10 @@ PryExceptionExplorer.wrap_active = true
 PryExceptionExplorer.intercept { true }
 
 Pry.config.commands.import PryExceptionExplorer::Commands
+
+# disable by default
+PryExceptionExplorer.enabled = false
+
+Pry.config.hooks.add_hook(:when_started, :try_enable_exception_explorer) do
+  PryExceptionExplorer.enabled = true if Pry.cli
+end

data/pry-exception_explorer.gemspec

@@ -2,19 +2,19 @@
 
 Gem::Specification.new do |s|
   s.name = "pry-exception_explorer"
-  s.version = "0.1.1pre3"
+  s.version = "0.1.1pre5"
 
   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-12"
   s.description = "Enter the context of exceptions"
   s.email = "jrmair@gmail.com"
-  s.files = [".gemtest", ".gitignore", ".travis.yml", ".yardopts", "CHANGELOG", "Gemfile", "LICENSE", "README.md", "Rakefile", "lib/pry-exception_explorer.rb", "lib/pry-exception_explorer/cli.rb", "lib/pry-exception_explorer/commands.rb", "lib/pry-exception_explorer/exception_wrap.rb", "lib/pry-exception_explorer/lazy_frame.rb", "lib/pry-exception_explorer/shim_builder.rb", "lib/pry-exception_explorer/version.rb", "pry-exception_explorer.gemspec", "test/helper.rb", "test/test_exception_explorer.rb", "test/test_exceptions_in_pry.rb", "test/test_wrapped_exceptions.rb"]
+  s.files = [".gemtest", ".gitignore", ".travis.yml", ".yardopts", "CHANGELOG", "Gemfile", "LICENSE", "README.md", "Rakefile", "examples/example.rb", "lib/pry-exception_explorer.rb", "lib/pry-exception_explorer/cli.rb", "lib/pry-exception_explorer/commands.rb", "lib/pry-exception_explorer/exception_wrap.rb", "lib/pry-exception_explorer/lazy_frame.rb", "lib/pry-exception_explorer/shim_builder.rb", "lib/pry-exception_explorer/version.rb", "pry-exception_explorer.gemspec", "test/helper.rb", "test/test_exceptions_in_pry.rb", "test/test_inline_exceptions.rb", "test/test_wrapped_exceptions.rb"]
   s.homepage = "https://github.com/banister/pry-exception_explorer"
   s.require_paths = ["lib"]
   s.rubygems_version = "1.8.11"
   s.summary = "Enter the context of exceptions"
-  s.test_files = ["test/helper.rb", "test/test_exception_explorer.rb", "test/test_exceptions_in_pry.rb", "test/test_wrapped_exceptions.rb"]
+  s.test_files = ["test/helper.rb", "test/test_exceptions_in_pry.rb", "test/test_inline_exceptions.rb", "test/test_wrapped_exceptions.rb"]
 
   if s.respond_to? :specification_version then
     s.specification_version = 3

data/test/helper.rb

@@ -6,10 +6,17 @@ unless Object.const_defined? 'PryExceptionExplorer'
 end
 
 require 'bacon'
+require 'ostruct'
 
 puts "Testing pry-exception_explorer version #{PryExceptionExplorer::VERSION}..."
 puts "Ruby version: #{RUBY_VERSION}"
 
+class OpenStruct
+  def clear
+    @table.clear
+  end
+end
+
 EE = PryExceptionExplorer
 
 class Ratty
@@ -47,6 +54,9 @@ class << Pry
   end
 end
 
+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
@@ -99,3 +109,4 @@ def mock_pry(*args)
   output.string
 end
 
+PryExceptionExplorer.enabled = true

data/test/test_exceptions_in_pry.rb

@@ -1,13 +1,21 @@
 require 'helper'
 
+O = OpenStruct.new
+
 describe PryExceptionExplorer do
 
+  before do
+    PryExceptionExplorer.intercept { true }
+    PryExceptionExplorer.wrap_active = true
+  end
+
+  after do
+    O.clear
+  end
+
   describe "Exceptions caught by Pry" do
     describe "enter-exception" do
       it  "should be able to enter an exception caught by pry" do
-
-        PryExceptionExplorer.intercept { true }
-
         # there are 3 types of situations where exception_explorer is invoked:
         # 1. when 'wrap' is used, i.e only exceptions that bubble to
         #    the top are intercepted.
@@ -22,8 +30,67 @@ describe PryExceptionExplorer do
         # `enter-exception` to start the session.
         #
         # This test is for type 3.
+        redirect_pry_io(InputTester.new("Ratty.new.ratty",
+                                        "enter-exception",
+                                        "O.method_name = __method__",
+                                        "exit", StringIO.new)) do
+          Pry.start
+        end
+
+        O.method_name.should == :toad
+      end
+
+      it "should have access to exception's caller" do
         mock_pry("Ratty.new.ratty", "enter-exception", "show-stack", "exit").should =~ /toad.*?weasel.*?ratty/m
       end
+
+      describe "exit-exception" do
+        it 'should display error message when exit-exception used outside of exception context' do
+          mock_pry("exit-exception").should =~ /You are not in an exception!/
+        end
+
+       it  "should exit a nested exception and correctly pop FrameManagers" do
+          redirect_pry_io(InputTester.new("Ratty.new.ratty",
+                                          "enter-exception",
+                                          "raise 'yo'",
+                                          "enter-exception",
+                                          "O.first_pry = _pry_",
+                                          "O.first_count = PryStackExplorer.frame_managers(_pry_).count",
+                                          "exit-exception",
+                                          "O.second_pry = _pry_",
+                                          "O.second_count = PryStackExplorer.frame_managers(_pry_).count",
+                                          "exit-exception",
+                                          "exit-all", StringIO.new)) do
+            Pry.start(binding)
+          end
+
+          O.first_pry.should == O.second_pry
+          O.first_count.should == 2
+          O.second_count.should == 1
+          PryStackExplorer.frame_managers(O.first_pry).count.should == 0
+        end
+
+        it  "should exit an exception and return to initial context" do
+          redirect_pry_io(InputTester.new("Ratty.new.ratty",
+                                          "O.initial_self = self",
+                                          "enter-exception",
+                                          "O.exception_self = self",
+                                          "exit-exception",
+                                          "O.return_self = self",
+                                          "exit-all", StringIO.new)) do
+            Pry.start(binding)
+          end
+
+          O.initial_self.should == self
+          O.initial_self.should == O.return_self
+
+          # actual exception context is Toad, as call chain is:
+          # Ratty -> Weasel -> Toad (raise is here)
+          O.exception_self.is_a?(Toad).should == true
+        end
+      end
     end
   end
 end
+
+Object.send(:remove_const, :O)

data/test/test_inline_exceptions.rb

@@ -0,0 +1,262 @@
+require 'helper'
+require 'ostruct'
+
+# globally accessible state
+O = OpenStruct.new
+
+prev_wrap_state = PryExceptionExplorer.wrap_active
+PryExceptionExplorer.wrap_active = false
+
+prev_intercept_state = PryExceptionExplorer.intercept_block
+
+describe PryExceptionExplorer do
+
+  before do
+    PryExceptionExplorer.wrap_active = false
+    O.exception_intercepted = false
+
+    # Ensure that when an exception is intercepted (a pry session
+    # started) that this is registered by setting state on `O`
+    Pry.config.input = StringIO.new("O.exception_intercepted = true\ncontinue-exception")
+    Pry.config.output = StringIO.new
+  end
+
+  after do
+    Pry.config.input.rewind
+    O.clear
+  end
+
+  describe "PryExceptionExplorer.intercept" do
+    describe "special case exception-only syntax" do
+
+      describe "single exception" do
+        it 'should intercept provided exceptions when given parameters (and no block)' do
+          my_error = Class.new(StandardError)
+          EE.intercept(my_error)
+          raise my_error
+          O.exception_intercepted.should == true
+        end
+
+        it 'should NOT intercept provided exceptions when not matched' do
+          my_error = Class.new(StandardError)
+          EE.intercept(my_error)
+
+          begin
+            raise RuntimeError
+          rescue => ex
+            ex.is_a?(RuntimeError).should == true
+          end
+        end
+      end
+
+      describe "multiple exceptions" do
+        it 'should intercept provided exceptions when given parameters (and no block)' do
+          errors = Array.new(3) { Class.new(StandardError) }
+          EE.intercept(*errors)
+
+          errors.each do |my_error|
+            raise my_error
+            O.exception_intercepted.should == true
+            O.exception_intercepted  = false
+            Pry.config.input.rewind
+          end
+        end
+
+        it 'should NOT intercept provided exceptions when not matched' do
+          errors = Array.new(3) { Class.new(StandardError) }
+
+          EE.intercept(*errors)
+
+          errors.each do |my_error|
+            begin
+              raise RuntimeError
+            rescue => ex
+              ex.is_a?(RuntimeError).should == true
+            end
+          end
+        end
+      end
+
+    end
+
+    describe "class" do
+      describe "first frame" do
+        it  "should intercept exception based on first frame's class" do
+          EE.intercept { |frame, ex| frame.klass == Toad }
+          Ratty.new.ratty
+          O.exception_intercepted.should == true
+        end
+
+        it  "should NOT intercept exception if class doesn't match" do
+          EE.intercept { |frame, ex| frame.klass == Ratty }
+          begin
+            Ratty.new.ratty
+          rescue Exception => ex
+            ex.is_a?(RuntimeError).should == true
+          end
+          O.exception_intercepted.should == false
+        end
+      end
+
+      describe "second frame" do
+        it  "should intercept exception based on second frame's method name" do
+          EE.intercept { |frame, ex| frame.prev.klass == Weasel }
+          Ratty.new.ratty
+          O.exception_intercepted.should == true
+        end
+
+        it  "should NOT intercept exception if method name doesn't match" do
+          EE.intercept { |frame, ex| frame.prev.klass == Toad }
+          begin
+            Ratty.new.ratty
+          rescue Exception => ex
+            ex.is_a?(RuntimeError).should == true
+          end
+          O.exception_intercepted.should == false
+        end
+      end
+
+      describe "third frame" do
+        it  "should intercept exception based on third frame's method name" do
+          EE.intercept { |frame, ex| frame.prev.prev.klass == Ratty }
+          Ratty.new.ratty
+          O.exception_intercepted.should == true
+        end
+
+        it  "should NOT intercept exception if method name doesn't match" do
+          EE.intercept { |frame, ex| frame.prev.prev.klass == Toad }
+          begin
+            Ratty.new.ratty
+          rescue Exception => ex
+            ex.is_a?(RuntimeError).should == true
+          end
+          O.exception_intercepted.should == false
+        end
+      end
+
+    end
+
+    describe "method_name" do
+      describe "first frame" do
+        it  "should intercept exception based on first frame's method name" do
+          EE.intercept { |frame, ex| frame.method_name == :toad }
+          Ratty.new.ratty
+          O.exception_intercepted.should == true
+        end
+
+        it  "should NOT intercept exception if method name doesn't match" do
+          EE.intercept { |frame, ex| frame.method_name == :ratty }
+          begin
+            Ratty.new.ratty
+          rescue Exception => ex
+            ex.is_a?(RuntimeError).should == true
+          end
+          O.exception_intercepted.should == false
+        end
+      end
+
+      describe "second frame" do
+        it  "should intercept exception based on second frame's method name" do
+          EE.intercept { |frame, ex| frame.prev.method_name == :weasel }
+          Ratty.new.ratty
+          O.exception_intercepted.should == true
+        end
+
+        it  "should NOT intercept exception if method name doesn't match" do
+          EE.intercept { |frame, ex| frame.prev.method_name == :toad }
+          begin
+            Ratty.new.ratty
+          rescue Exception => ex
+            ex.is_a?(RuntimeError).should == true
+          end
+          O.exception_intercepted.should == false
+        end
+      end
+
+      describe "third frame" do
+        it  "should intercept exception based on third frame's method name" do
+          EE.intercept { |frame, ex| frame.prev.prev.method_name == :ratty }
+          Ratty.new.ratty
+          O.exception_intercepted.should == true
+        end
+
+        it  "should NOT intercept exception if method name doesn't match" do
+          EE.intercept { |frame, ex| frame.prev.prev.method_name == :toad }
+          begin
+            Ratty.new.ratty
+          rescue Exception => ex
+            ex.is_a?(RuntimeError).should == true
+          end
+          O.exception_intercepted.should == false
+        end
+      end
+
+    end
+
+
+  end
+
+  describe "call-stack management" do
+    it 'should pop the call-stack after session ends (continue)' do
+      EE.intercept { |frame, ex| frame.prev.prev.method_name == :ratty }
+
+      redirect_pry_io(InputTester.new(
+                                      "O.stack_count = PryStackExplorer.frame_managers(_pry_).count",
+                                      "O._pry_ = _pry_",
+                                      "continue-exception"), StringIO.new) do
+        Ratty.new.ratty
+      end
+      O.stack_count.should == 1
+      PryStackExplorer.frame_managers(O._pry_).count.should == 0
+    end
+
+    it 'should pop the call-stack after session ends (exit)' do
+      EE.intercept { |frame, ex| frame.prev.prev.method_name == :ratty }
+
+      redirect_pry_io(InputTester.new(
+                                      "O.stack_count = PryStackExplorer.frame_managers(_pry_).count",
+                                      "O._pry_ = _pry_",
+                                      "exit"), StringIO.new) do
+        begin
+          Ratty.new.ratty
+        rescue
+        end
+      end
+      O.stack_count.should == 1
+      PryStackExplorer.frame_managers(O._pry_).count.should == 0
+    end
+
+    describe "nested exceptions" do
+      it 'Each successive exception interception should be managed by its own pry instance and have its own call-stack' do
+        EE.intercept { |frame, ex| frame.prev.prev.method_name == :ratty }
+
+        redirect_pry_io(InputTester.new(
+                                        "O.first_stack_count = PryStackExplorer.frame_managers(_pry_).count",
+                                        "O._pry_ = _pry_",
+                                        "EE.intercept(ArgumentError)",
+                                        "raise ArgumentError",
+                                        "O._pry_2 = _pry_",
+                                        "O.second_stack_count = PryStackExplorer.frame_managers(_pry_).count",
+                                        "continue-exception",
+                                        "continue-exception"), StringIO.new) do
+          Ratty.new.ratty
+        end
+
+        O._pry_.should.not == O._pry_2
+        O.first_stack_count.should == 1
+        O.second_stack_count.should == 1
+        PryStackExplorer.frame_managers(O._pry_).count.should == 0
+        PryStackExplorer.frame_managers(O._pry_2).count.should == 0
+      end
+
+    end
+
+  end
+
+end
+
+# restore to default
+PryExceptionExplorer.wrap_active = prev_wrap_state
+PryExceptionExplorer.intercept &prev_intercept_state
+
+Object.send(:remove_const, :O)

data/test/test_wrapped_exceptions.rb

@@ -5,19 +5,42 @@ require 'pry-exception_explorer/exception_wrap'
 CaughtException   = Class.new(StandardError)
 UncaughtException = Class.new(StandardError)
 
+
 describe PryExceptionExplorer do
 
+
   before do
     Pry.config.input = StringIO.new("exit :caught\n")
     Pry.config.output = StringIO.new
+    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.clear(:when_started)
+    Pry.config.hooks.delete_hook(:when_started, :save_caller_bindings)
+    Pry.config.hooks.delete_hook(:after_session, :delete_frame_manager)
   end
 
   describe "PryExceptionExplorer.wrap" do
 
+    # use of exit-exception inside a wrapped exception is weird
+    # (because exit-exception is really designed for pry exceptions)
+    # but when we do receive one, we should exit out of pry
+    # altogether.
+    # This test is weird as we can't use lambda { }.should.not.raise, as we override
+    # 'raise' method ourself, which kills bacon's functionality here.
+    it 'should exit out of Pry session when using exit-exception' do
+      PryExceptionExplorer.intercept { true }
+
+      x = :no_exception_raised
+      redirect_pry_io(InputTester.new("exit-exception"), $stdout) do
+        PryExceptionExplorer.wrap do
+          Ratty.new.ratty
+        end
+      end
+      x.should == :no_exception_raised
+    end
+
     it 'should default to capturing ALL exceptions' do
       PryExceptionExplorer.wrap do
         raise CaughtException, "catch me if u can"

metadata

@@ -2,7 +2,7 @@
 name: pry-exception_explorer
 version: !ruby/object:Gem::Version 
   prerelease: 5
-  version: 0.1.1pre3
+  version: 0.1.1pre5
 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-12 00:00:00 Z
 dependencies: 
 - !ruby/object:Gem::Dependency 
   name: pry-stack_explorer
@@ -63,6 +63,7 @@ files:
 - LICENSE
 - README.md
 - Rakefile
+- examples/example.rb
 - lib/pry-exception_explorer.rb
 - lib/pry-exception_explorer/cli.rb
 - lib/pry-exception_explorer/commands.rb
@@ -72,8 +73,8 @@ files:
 - lib/pry-exception_explorer/version.rb
 - pry-exception_explorer.gemspec
 - test/helper.rb
-- test/test_exception_explorer.rb
 - test/test_exceptions_in_pry.rb
+- test/test_inline_exceptions.rb
 - test/test_wrapped_exceptions.rb
 homepage: https://github.com/banister/pry-exception_explorer
 licenses: []
@@ -104,6 +105,6 @@ specification_version: 3
 summary: Enter the context of exceptions
 test_files: 
 - test/helper.rb
-- test/test_exception_explorer.rb
 - test/test_exceptions_in_pry.rb
+- test/test_inline_exceptions.rb
 - test/test_wrapped_exceptions.rb

data/test/test_exception_explorer.rb

@@ -1,159 +0,0 @@
-require 'helper'
-
-
-# override enter_exception_inline so we can use it for testing purposes
-EE.instance_eval do
-  alias original_enter_exception_inline enter_exception_inline
-end
-
-def EE.exception_intercepted?
-  @exception_intercepted
-end
-
-EE.instance_eval do
-  @exception_intercepted = false
-end
-
-def EE.enter_exception_inline(ex)
-  @exception_intercepted = true
-  EE::CONTINUE_INLINE_EXCEPTION
-end
-
-prev_wrap_state = PryExceptionExplorer.wrap_active
-PryExceptionExplorer.wrap_active = false
-
-describe PryExceptionExplorer do
-
-  before do
-    PryExceptionExplorer.wrap_active = false
-  end
-
-  after do
-    EE.instance_eval do
-      @exception_intercepted = false
-    end
-  end
-
-  describe "PryExceptionExplorer.intercept" do
-    describe "class" do
-      describe "first frame" do
-        it  "should intercept exception based on first frame's method name" do
-          EE.intercept { |frame, ex| frame.klass == Toad }
-          Ratty.new.ratty
-          EE.exception_intercepted?.should == true
-        end
-
-        it  "should NOT intercept exception if method name doesn't match" do
-          EE.intercept { |frame, ex| frame.klass == Ratty }
-          begin
-            Ratty.new.ratty
-          rescue Exception => ex
-            ex.is_a?(RuntimeError).should == true
-          end
-          EE.exception_intercepted?.should == false
-        end
-      end
-
-      describe "second frame" do
-        it  "should intercept exception based on second frame's method name" do
-          EE.intercept { |frame, ex| frame.prev.klass == Weasel }
-          Ratty.new.ratty
-          EE.exception_intercepted?.should == true
-        end
-
-        it  "should NOT intercept exception if method name doesn't match" do
-          EE.intercept { |frame, ex| frame.prev.klass == Toad }
-          begin
-            Ratty.new.ratty
-          rescue Exception => ex
-            ex.is_a?(RuntimeError).should == true
-          end
-          EE.exception_intercepted?.should == false
-        end
-      end
-
-      describe "third frame" do
-        it  "should intercept exception based on third frame's method name" do
-          EE.intercept { |frame, ex| frame.prev.prev.klass == Ratty }
-          Ratty.new.ratty
-          EE.exception_intercepted?.should == true
-        end
-
-        it  "should NOT intercept exception if method name doesn't match" do
-          EE.intercept { |frame, ex| frame.prev.prev.klass == Toad }
-          begin
-            Ratty.new.ratty
-          rescue Exception => ex
-            ex.is_a?(RuntimeError).should == true
-          end
-          EE.exception_intercepted?.should == false
-        end
-      end
-
-    end
-
-    describe "method_name" do
-      describe "first frame" do
-        it  "should intercept exception based on first frame's method name" do
-          EE.intercept { |frame, ex| frame.method_name == :toad }
-          Ratty.new.ratty
-          EE.exception_intercepted?.should == true
-        end
-
-        it  "should NOT intercept exception if method name doesn't match" do
-          EE.intercept { |frame, ex| frame.method_name == :ratty }
-          begin
-            Ratty.new.ratty
-          rescue Exception => ex
-            ex.is_a?(RuntimeError).should == true
-          end
-          EE.exception_intercepted?.should == false
-        end
-      end
-
-      describe "second frame" do
-        it  "should intercept exception based on second frame's method name" do
-          EE.intercept { |frame, ex| frame.prev.method_name == :weasel }
-          Ratty.new.ratty
-          EE.exception_intercepted?.should == true
-        end
-
-        it  "should NOT intercept exception if method name doesn't match" do
-          EE.intercept { |frame, ex| frame.prev.method_name == :toad }
-          begin
-            Ratty.new.ratty
-          rescue Exception => ex
-            ex.is_a?(RuntimeError).should == true
-          end
-          EE.exception_intercepted?.should == false
-        end
-      end
-
-      describe "third frame" do
-        it  "should intercept exception based on third frame's method name" do
-          EE.intercept { |frame, ex| frame.prev.prev.method_name == :ratty }
-          Ratty.new.ratty
-          EE.exception_intercepted?.should == true
-        end
-
-        it  "should NOT intercept exception if method name doesn't match" do
-          EE.intercept { |frame, ex| frame.prev.prev.method_name == :toad }
-          begin
-            Ratty.new.ratty
-          rescue Exception => ex
-            ex.is_a?(RuntimeError).should == true
-          end
-          EE.exception_intercepted?.should == false
-        end
-      end
-
-    end
-
-  end
-end
-
-EE.instance_eval do
-  alias enter_exception_inline original_enter_exception_inline
-end
-
-PryExceptionExplorer.wrap_active = prev_wrap_state