pry-exception_explorer 0.1.1pre1 → 0.1.1pre3

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/Gemfile

@@ -0,0 +1,2 @@
+source :rubygems
+gemspec

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

@@ -1,76 +1,91 @@
-$:.unshift 'lib'
-
-dlext = Config::CONFIG['DLEXT']
-direc = File.dirname(__FILE__)
-
-PROJECT_NAME = "pry-exception_explorer"
-
-require 'rake/clean'
-require 'rake/gempackagetask'
-require "#{PROJECT_NAME}/version"
-
-CLOBBER.include("**/*~", "**/*#*", "**/*.log")
-CLEAN.include("**/*#*", "**/*#*.*", "**/*_flymake*.*", "**/*_flymake",
-              "**/*.rbc", "**/.#*.*")
-
-def apply_spec_defaults(s)
-  s.name = PROJECT_NAME
-  s.summary = "FIX ME"
-  s.version = PryExceptionExplorer::VERSION
-  s.date = Time.now.strftime '%Y-%m-%d'
-  s.author = "John Mair (banisterfiend)"
-  s.email = 'jrmair@gmail.com'
-  s.description = s.summary
-  s.require_path = 'lib'
-  s.homepage = "https://github.com/banister/pry-exception_explorer"
-    s.add_dependency('pry-stack_explorer')
-  s.files = Dir["lib/**/*.rb", "test/*.rb", "CHANGELOG", "README.md", "Rakefile"]
-end
-
-desc "run pry with plugin enabled"
-task :pry do
-  exec("pry -I#{direc}/lib/ -r #{direc}/lib/#{PROJECT_NAME}")
-end
-
-desc "reinstall gem"
-task :reinstall => :gems do
+$:.unshift 'lib'
+
+dlext = RbConfig::CONFIG['DLEXT']
+direc = File.dirname(__FILE__)
+
+PROJECT_NAME = "pry-exception_explorer"
+
+require 'rake/clean'
+require 'rake/gempackagetask'
+require "#{PROJECT_NAME}/version"
+
+CLOBBER.include("**/*~", "**/*#*", "**/*.log")
+CLEAN.include("**/*#*", "**/*#*.*", "**/*_flymake*.*", "**/*_flymake",
+              "**/*.rbc", "**/.#*.*")
+
+def apply_spec_defaults(s)
+  s.name = PROJECT_NAME
+  s.summary = "Enter the context of exceptions"
+  s.version = PryExceptionExplorer::VERSION
+  s.date = Time.now.strftime '%Y-%m-%d'
+  s.author = "John Mair (banisterfiend)"
+  s.email = 'jrmair@gmail.com'
+  s.description = s.summary
+  s.require_path = 'lib'
+  s.homepage = "https://github.com/banister/pry-exception_explorer"
+  s.add_dependency('pry-stack_explorer')
+  s.add_development_dependency("bacon","~>1.1.0")
+  s.add_development_dependency('rake', '~> 0.9')
+
+  s.files = `git ls-files`.split("\n")
+  s.test_files = `git ls-files -- test/*`.split("\n")
+end
+
+desc "run pry with plugin enabled"
+task :pry do
+  exec("pry -I#{direc}/lib/ -r #{direc}/lib/#{PROJECT_NAME}")
+end
+
+desc "reinstall gem"
+task :reinstall => :gems do
+  sh "rm -rf ~/.pry-exception_explorer"
   sh "gem uninstall pry-exception_explorer" rescue nil
-  sh "gem install #{direc}/pkg/pry-exception_explorer-#{PryExceptionExplorer::VERSION}.gem"
+  sh "gem install #{direc}/pkg/pry-exception_explorer-#{PryExceptionExplorer::VERSION}.gem"
+end
+
+desc "run tests"
+task :test do
+  sh "bacon -Itest -rubygems -a"
+end
+
+desc "Build gemspec"
+task :gemspec => "ruby:gemspec"
+
+namespace :ruby do
+  spec = Gem::Specification.new do |s|
+    apply_spec_defaults(s)
+    s.platform = Gem::Platform::RUBY
+  end
+
+  Rake::GemPackageTask.new(spec) do |pkg|
+    pkg.need_zip = false
+    pkg.need_tar = false
+  end
+
+  desc  "Generate gemspec file"
+  task :gemspec do
+    File.open("#{spec.name}.gemspec", "w") do |f|
+      f << spec.to_ruby
+    end
+  end
+end
+
+desc "shorthand for :gems task"
+task :gem => :gems
+
+desc "build all platform gems at once"
+task :gems => [:clean, :rmgems, "ruby:gem"]
+
+desc "remove all platform gems"
+task :rmgems => ["ruby:clobber_package"]
+
+desc "build and push latest gems"
+task :pushgems => :gems do
+  chdir("#{File.dirname(__FILE__)}/pkg") do
+    Dir["*.gem"].each do |gemfile|
+      sh "gem push #{gemfile}"
+    end
+  end
 end
-
-desc "run tests"
-task :test do
-  sh "bacon -Itest -rubygems -a"
-end
-
-namespace :ruby do
-  spec = Gem::Specification.new do |s|
-    apply_spec_defaults(s)
-    s.platform = Gem::Platform::RUBY
-  end
-
-  Rake::GemPackageTask.new(spec) do |pkg|
-    pkg.need_zip = false
-    pkg.need_tar = false
-  end
-end
-
-desc "shorthand for :gems task"
-task :gem => :gems
-
-desc "build all platform gems at once"
-task :gems => [:clean, :rmgems, "ruby:gem"]
-
-desc "remove all platform gems"
-task :rmgems => ["ruby:clobber_package"]
-
-desc "build and push latest gems"
-task :pushgems => :gems do
-  chdir("#{File.dirname(__FILE__)}/pkg") do
-    Dir["*.gem"].each do |gemfile|
-      sh "gem push #{gemfile}"
-    end
-  end
-end
-
-
+
+

data/lib/pry-exception_explorer/commands.rb

@@ -0,0 +1,42 @@
+require 'pry-stack_explorer'
+
+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
+      PryStackExplorer.create_and_push_frame_manager(ex.exception_call_stack, _pry_)
+      PryStackExplorer.frame_manager(_pry_).user[:exception] = ex
+      PryStackExplorer.frame_manager(_pry_).refresh_frame
+    elsif ex
+      output.puts "Current exception can't be entered! (perhaps a C exception)"
+    else
+      output.puts "No exception to enter!"
+    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!"
+    end
+  end
+
+  command "continue-exception", "Attempt to continue the current exception." 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_)
+    elsif fm && fm.user[:exception] && fm.user[:exception].continuation
+      PryStackExplorer.pop_frame_manager(_pry_)
+      fm.user[:exception].continue
+    else
+      output.puts "No exception to continue!"
+    end
+  end
+
+end

data/lib/pry-exception_explorer/exception_wrap.rb

@@ -1,9 +1,14 @@
-require 'pry_stack_explorer'
+require 'pry-exception_explorer'
 
 Pry.config.hooks.delete_hook(:when_started, :save_caller_bindings)
 
+# default is to capture all exceptions that bubble to the top
+PryExceptionExplorer.intercept { true }
+
 module PryExceptionExplorer
+
   def self.wrap
+    self.wrap_active = true
     yield
   rescue Exception => ex
     Pry.config.hooks.add_hook(:when_started, :setup_exception_context) do |binding_stack, _pry_|
@@ -12,6 +17,13 @@ module PryExceptionExplorer
       PryStackExplorer.frame_manager(_pry_).user[:exception] = ex
     end
 
-    pry
+    if ex.should_capture?
+      pry
+    else
+      raise ex
+    end
+  ensure
+    self.wrap_active = false
   end
 end
+

data/lib/pry-exception_explorer/lazy_frame.rb

@@ -0,0 +1,32 @@
+module PryExceptionExplorer
+  class LazyFrame
+
+    # we need to jump over a few irrelevant frames to begin with
+    START_FRAME_OFFSET = 5
+    
+    def initialize(frame, frame_counter = 0)
+      @frame         = frame
+      @frame_counter = frame_counter
+    end
+
+    def raw_frame
+      @frame
+    end
+
+    def klass
+      @frame.eval("self.class")
+    end
+
+    def self
+      @frame.eval("self")
+    end
+
+    def method_name
+      @frame.eval("__method__")
+    end
+
+    def prev
+      LazyFrame.new(binding.of_caller(@frame_counter + START_FRAME_OFFSET), @frame_counter + 1)
+    end
+  end
+end

data/lib/pry-exception_explorer/version.rb

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

data/lib/pry-exception_explorer.rb

@@ -3,22 +3,78 @@
 
 require 'pry-stack_explorer'
 require "pry-exception_explorer/version"
+require "pry-exception_explorer/lazy_frame"
+require "pry-exception_explorer/commands"
 require "pry"
 
 if RUBY_VERSION =~ /1.9/
   require 'continuation'
 end
 
+module PryExceptionExplorer
+  CONTINUE_INLINE_EXCEPTION = Object.new
+
+  class << self
+    def wrap_active=(v)
+      Thread.current[:__pry_exception_explorer_wrap__] = v
+    end
+
+    def wrap_active
+      !!Thread.current[:__pry_exception_explorer_wrap__]
+    end
+
+    alias_method :wrap_active?, :wrap_active
+  end
+
+  self.wrap_active = false
+
+  def self.should_capture_exception?(ex)
+    true
+  end
+
+  def self.intercept(&block)
+    Thread.current[:__pry_exception_explorer_intercept_block__] = block
+  end
+
+  def self.intercept_block
+    Thread.current[:__pry_exception_explorer_intercept_block__]
+  end
+
+  def self.should_capture_exception?(ex, frame)
+    if intercept_block
+      intercept_block.call(LazyFrame.new(frame), ex)
+    else
+      false
+    end
+  end
+
+  def self.enter_exception_inline(ex)
+    _pry_ = Pry.new
+
+    Pry.initial_session_setup
+    PryStackExplorer.create_and_push_frame_manager(ex.exception_call_stack, _pry_)
+    PryStackExplorer.frame_manager(_pry_).user[:exception]        = ex
+    PryStackExplorer.frame_manager(_pry_).user[:inline_exception] = true
+    _pry_.repl(ex.exception_call_stack.first)
+
+  ensure
+    PryStackExplorer.clear_frame_managers(_pry_)
+  end
+end
+
 class Exception
   NoContinuation = Class.new(StandardError)
 
   attr_accessor :continuation
   attr_accessor :exception_call_stack
+  attr_accessor :should_capture
 
   def continue
     raise NoContinuation unless continuation.respond_to?(:call)
     continuation.call
   end
+
+  alias_method :should_capture?, :should_capture
 end
 
 class Object
@@ -30,51 +86,31 @@ class Object
 
     ex = exception.exception(string)
     ex.set_backtrace(array)
-    ex.exception_call_stack = binding.callers.tap(&:shift)
-
-    callcc do |cc|
-      ex.continuation = cc
-      super(ex)
-    end
-  end
-end
 
-PryExceptionExplorer::Commands = Pry::CommandSet.new do
+    if PryExceptionExplorer.should_capture_exception?(ex, binding.of_caller(1))
+      ex.exception_call_stack = binding.callers.tap(&:shift)
+      ex.should_capture       = true
 
-  command "enter-exception", "Enter the context of the last exception" do
-    ex = _pry_.last_exception
-    if ex && ex.exception_call_stack
-      PryStackExplorer.create_and_push_frame_manager(ex.exception_call_stack, _pry_)
-      PryStackExplorer.frame_manager(_pry_).user[:exception] = ex
-      PryStackExplorer.frame_manager(_pry_).refresh_frame
-    elsif ex
-      output.puts "Current exception can't be entered! (perhaps a C exception)"
-    else
-      output.puts "No exception to enter!"
+      if !PryExceptionExplorer.wrap_active?
+        retval = PryExceptionExplorer.enter_exception_inline(ex)
+      end
     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!"
+    if retval != PryExceptionExplorer::CONTINUE_INLINE_EXCEPTION
+      callcc do |cc|
+        ex.continuation = cc
+        super(ex)
+      end
     end
   end
+end
 
-  command "continue-exception", "Attempt to continue the current exception." do
-    fm = PryStackExplorer.frame_manager(_pry_)
 
-    if fm && fm.user[:exception] && fm.user[:exception].continuation
-      PryStackExplorer.pop_frame_manager(_pry_)
-      fm.user[:exception].continue
-    else
-      output.puts "No exception to continue!"
-    end
-  end
+# Let exceptions get caught by Pry REPL loop (i.e dont catch
+# immediately at point of 'raise')
+PryExceptionExplorer.wrap_active = true
 
-end
+# default is to capture all exceptions that bubble to the top
+PryExceptionExplorer.intercept { true }
 
 Pry.config.commands.import PryExceptionExplorer::Commands

data/pry-exception_explorer.gemspec

@@ -0,0 +1,36 @@
+# -*- encoding: utf-8 -*-
+
+Gem::Specification.new do |s|
+  s.name = "pry-exception_explorer"
+  s.version = "0.1.1pre3"
+
+  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.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.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"]
+
+  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<pry-stack_explorer>, [">= 0"])
+      s.add_development_dependency(%q<bacon>, ["~> 1.1.0"])
+      s.add_development_dependency(%q<rake>, ["~> 0.9"])
+    else
+      s.add_dependency(%q<pry-stack_explorer>, [">= 0"])
+      s.add_dependency(%q<bacon>, ["~> 1.1.0"])
+      s.add_dependency(%q<rake>, ["~> 0.9"])
+    end
+  else
+    s.add_dependency(%q<pry-stack_explorer>, [">= 0"])
+    s.add_dependency(%q<bacon>, ["~> 1.1.0"])
+    s.add_dependency(%q<rake>, ["~> 0.9"])
+  end
+end

data/test/helper.rb

@@ -0,0 +1,101 @@
+require 'rubygems'
+
+unless Object.const_defined? 'PryExceptionExplorer'
+  $:.unshift File.expand_path '../../lib', __FILE__
+  require 'pry-exception_explorer'
+end
+
+require 'bacon'
+
+puts "Testing pry-exception_explorer version #{PryExceptionExplorer::VERSION}..."
+puts "Ruby version: #{RUBY_VERSION}"
+
+EE = PryExceptionExplorer
+
+class Ratty
+  def ratty
+    Weasel.new.weasel
+  end
+end
+
+class Weasel
+  def weasel
+    Toad.new.toad
+  end
+end
+
+class Toad
+  def toad
+    raise "toad hall"
+  end
+end
+
+class << Pry
+  alias_method :orig_reset_defaults, :reset_defaults
+  def reset_defaults
+    orig_reset_defaults
+
+    Pry.color = false
+    Pry.pager = false
+    Pry.config.should_load_rc      = false
+    Pry.config.plugins.enabled     = false
+    Pry.config.history.should_load = false
+    Pry.config.history.should_save = false
+    Pry.config.auto_indent         = false
+    Pry.config.hooks               = Pry::Hooks.new
+    Pry.config.collision_warning   = false
+  end
+end
+
+Pry.reset_defaults
+
+class InputTester
+  def initialize(*actions)
+    if actions.last.is_a?(Hash) && actions.last.keys == [:history]
+      @hist = actions.pop[:history]
+    end
+    @orig_actions = actions.dup
+    @actions = actions
+  end
+
+  def readline(*)
+    @actions.shift.tap{ |line| @hist << line if @hist }
+  end
+
+  def rewind
+    @actions = @orig_actions.dup
+  end
+end
+
+# Set I/O streams.
+#
+# Out defaults to an anonymous StringIO.
+#
+def redirect_pry_io(new_in, new_out = StringIO.new)
+  old_in = Pry.input
+  old_out = Pry.output
+
+  Pry.input = new_in
+  Pry.output = new_out
+  begin
+    yield
+  ensure
+    Pry.input = old_in
+    Pry.output = old_out
+  end
+end
+
+def mock_pry(*args)
+
+  binding = args.first.is_a?(Binding) ? args.shift : binding()
+
+  input = InputTester.new(*args)
+  output = StringIO.new
+
+  redirect_pry_io(input, output) do
+    binding.pry
+  end
+
+  output.string
+end
+

data/test/test_exception_explorer.rb

@@ -0,0 +1,159 @@
+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

data/test/test_exceptions_in_pry.rb

@@ -0,0 +1,29 @@
+require 'helper'
+
+describe PryExceptionExplorer do
+
+  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.
+        # 2. when exceptions are intercepted 'inline' (i.e dropped
+        #    into pry directly from `raise` itself)
+        # 3. exceptions are caught by pry and entered into by using
+        #    the 'enter-exception' command
+        # The case of 1. and 3. are actually very similar, but in
+        # 3. the exception never bubbles to the top as it's caught by
+        # pry instead; also in 3. a pry session is not started
+        # automatically, the user must explicitly type
+        # `enter-exception` to start the session.
+        #
+        # This test is for type 3.
+        mock_pry("Ratty.new.ratty", "enter-exception", "show-stack", "exit").should =~ /toad.*?weasel.*?ratty/m
+      end
+    end
+  end
+end

data/test/test_wrapped_exceptions.rb

@@ -0,0 +1,160 @@
+require 'helper'
+
+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
+  end
+
+  after do
+    Pry.config.hooks.clear(:when_started)
+  end
+
+  describe "PryExceptionExplorer.wrap" do
+
+    it 'should default to capturing ALL exceptions' do
+      PryExceptionExplorer.wrap do
+        raise CaughtException, "catch me if u can"
+      end.should == :caught
+    end
+
+    it 'should NOT capture rescued exceptions' do
+      o = Object.new
+      def o.evil_fish
+        Ratty.new.ratty
+      rescue Exception
+      end
+
+      PryExceptionExplorer.intercept { true }
+
+      PryExceptionExplorer.wrap do
+        o.evil_fish
+      end.should.not == :caught
+    end
+
+    it 'should have the full callstack attached to exception' do
+      PryExceptionExplorer.intercept { |frame, ex| frame.method_name == :toad }
+
+      PryExceptionExplorer.wrap do
+        begin
+          Ratty.new.ratty
+        rescue Exception => ex
+          ex.exception_call_stack[0..2].map { |b| b.eval("__method__") }.should == [:toad, :weasel, :ratty]
+        end
+      end
+    end
+
+    it 'should NOT have callstack attached if exception not matched' do
+      PryExceptionExplorer.intercept { |frame, ex| false }
+
+      begin
+        PryExceptionExplorer.wrap do
+          raise UncaughtException, "Catch me if you can't.."
+        end
+      rescue UncaughtException => ex
+        ex.exception_call_stack.should == nil
+      end
+    end
+
+    describe "PryExceptionExplorer.intercept with wrapped exceptions" do
+      describe "klass" do
+        describe "first frame" do
+          it 'should catch a matched exception based on klass' do
+            PryExceptionExplorer.intercept { |frame, ex| frame.klass == Toad }
+
+            PryExceptionExplorer.wrap do
+              Ratty.new.ratty
+            end.should == :caught
+          end
+
+          it 'should NOT catch an unmatched exception' do
+            PryExceptionExplorer.intercept { |frame, ex| frame.klass == Weasel }
+
+            begin
+              PryExceptionExplorer.wrap do
+                raise UncaughtException, "Catch me if you can't.."
+              end
+            rescue Exception => ex
+              ex.is_a?(UncaughtException).should == true
+            end
+          end
+        end
+
+        describe "third frame" do
+          it 'should catch a matched exception' do
+            PryExceptionExplorer.intercept { |frame, ex| frame.prev.prev.klass == Ratty }
+
+            PryExceptionExplorer.wrap do
+              Ratty.new.ratty
+            end.should == :caught
+          end
+
+          it 'should NOT catch an unmatched exception' do
+            PryExceptionExplorer.intercept { |frame, ex| frame.prev.prev.klass == Weasel }
+
+            begin
+              PryExceptionExplorer.wrap do
+                raise UncaughtException, "Catch me if you can't.."
+              end
+            rescue Exception => ex
+              ex.is_a?(UncaughtException).should == true
+            end
+          end
+        end
+      end
+
+      describe "method_name" do
+        describe "first frame" do
+          it 'should catch a matched exception' do
+            PryExceptionExplorer.intercept { |frame, ex| frame.method_name == :toad }
+
+            PryExceptionExplorer.wrap do
+              Ratty.new.ratty
+            end.should == :caught
+          end
+
+          it 'should NOT catch an unmatched exception' do
+            PryExceptionExplorer.intercept { |frame, ex| frame.method_name == :weasel }
+
+            begin
+              PryExceptionExplorer.wrap do
+                raise UncaughtException, "Catch me if you can't.."
+              end
+            rescue Exception => ex
+              ex.is_a?(UncaughtException).should == true
+            end
+          end
+        end
+
+        describe "third frame" do
+          it 'should catch a matched exception' do
+            PryExceptionExplorer.intercept { |frame, ex| frame.prev.prev.method_name == :ratty }
+
+            PryExceptionExplorer.wrap do
+              Ratty.new.ratty
+            end.should == :caught
+          end
+
+          it 'should NOT catch an unmatched exception' do
+            PryExceptionExplorer.intercept { |frame, ex| frame.prev.prev.method_name == :weasel }
+
+            begin
+              PryExceptionExplorer.wrap do
+                raise UncaughtException, "Catch me if you can't.."
+              end
+            rescue Exception => ex
+              ex.is_a?(UncaughtException).should == true
+            end
+          end
+        end
+
+      end
+    end
+  end
+end

metadata

@@ -2,7 +2,7 @@
 name: pry-exception_explorer
 version: !ruby/object:Gem::Version 
   prerelease: 5
-  version: 0.1.1pre1
+  version: 0.1.1pre3
 platform: ruby
 authors: 
 - John Mair (banisterfiend)
@@ -10,7 +10,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-12-17 00:00:00 Z
+date: 2012-01-05 00:00:00 Z
 dependencies: 
 - !ruby/object:Gem::Dependency 
   name: pry-stack_explorer
@@ -23,7 +23,29 @@ dependencies:
         version: "0"
   type: :runtime
   version_requirements: *id001
-description: FIX ME
+- !ruby/object:Gem::Dependency 
+  name: bacon
+  prerelease: false
+  requirement: &id002 !ruby/object:Gem::Requirement 
+    none: false
+    requirements: 
+    - - ~>
+      - !ruby/object:Gem::Version 
+        version: 1.1.0
+  type: :development
+  version_requirements: *id002
+- !ruby/object:Gem::Dependency 
+  name: rake
+  prerelease: false
+  requirement: &id003 !ruby/object:Gem::Requirement 
+    none: false
+    requirements: 
+    - - ~>
+      - !ruby/object:Gem::Version 
+        version: "0.9"
+  type: :development
+  version_requirements: *id003
+description: Enter the context of exceptions
 email: jrmair@gmail.com
 executables: []
 
@@ -32,15 +54,27 @@ extensions: []
 extra_rdoc_files: []
 
 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
-- lib/pry-exception_explorer.rb
-- test/test.rb
-- CHANGELOG
-- README.md
-- Rakefile
+- pry-exception_explorer.gemspec
+- test/helper.rb
+- test/test_exception_explorer.rb
+- test/test_exceptions_in_pry.rb
+- test/test_wrapped_exceptions.rb
 homepage: https://github.com/banister/pry-exception_explorer
 licenses: []
 
@@ -67,6 +101,9 @@ rubyforge_project:
 rubygems_version: 1.8.11
 signing_key: 
 specification_version: 3
-summary: FIX ME
-test_files: []
-
+summary: Enter the context of exceptions
+test_files: 
+- test/helper.rb
+- test/test_exception_explorer.rb
+- test/test_exceptions_in_pry.rb
+- test/test_wrapped_exceptions.rb

data/test/test.rb

@@ -1,12 +0,0 @@
-direc = File.dirname(__FILE__)
-
-require 'rubygems'
-require "#{direc}/../lib/pry-exception_explorer"
-require 'bacon'
-
-puts "Testing pry-exception_explorer version #{PryExceptionExplorer::VERSION}..." 
-puts "Ruby version: #{RUBY_VERSION}"
-
-describe PryExceptionExplorer do
-end
-