pry-rescue 1.4.2 → 1.5.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 5c100b1d2a77a25c90d941e40641bdf84382df35
-  data.tar.gz: 9bf84fe4f040c636b6e1c456ef28f5378d88bdc5
+SHA256:
+  metadata.gz: 785059388e26fc08a04afc6d370b89efdc4506f3882738444e2ab0b99c229f20
+  data.tar.gz: 11ab228ba3b614eef01ba6687c97a36c76411245fdc6bcb03aaaaf9cd2a3c59c
 SHA512:
-  metadata.gz: d7b8e77d58f7fa9e0f7d7170b38904833027ead2ace1216ad86283679c7a76225202a4a4cb686be9cac3de884d60458de7650d00f9b2240b80eaeb8bc5af0ebf
-  data.tar.gz: 1b161c794e9fa6749e9921fe7c02468ddf67147c96252c62f03c018f2f35299865b9fd3a9a9e7bef5783b56a04709977cef1e8764dfb92e3c600eea736229afb
+  metadata.gz: 4ebeff09115b1ad7937b09e42ced421309b24d9674f7edab5d4f9c6acb504778de1506732220f687f81ebccdb0b16b1c34e49bed0c294b980a6eb3701e6d9d1a
+  data.tar.gz: 5c4ecf866a9d06f07447bab62cb51046e40cecbc8ad8cac796c7e40e573ce6c8ceda1a390e6d457a9eef146d9e2c8be79336025515c59188690d8567bc244f47

data/.rspec

@@ -0,0 +1 @@
+--require spec_helper

data/.travis.yml

@@ -1,7 +1,10 @@
 language: ruby
 cache: bundler
 rvm:
-  - 1.9.3
-  - 2.0.0
-  - 2.1.0
-  - rbx
+  - 2.5
+  - 2.6
+  - 2.7
+  - ruby-head
+before_install:
+  - "gem update --system"
+  - "gem update bundler"

data/CHANGELOG.md

@@ -0,0 +1,11 @@
+## v1.5.2 (24 June 2020)
+* RSpec: Fix Rescue opening after transactional tests are rolled back.
+(issue #99 - PR #118) (@joallard)
+
+* bin/rescue: Use realpaths (issue #109 - PR #110)
+
+  *(Damien Robert)*
+
+## v1.5.1 (22 May 2020)
+* Make Binding#source_location polyfill. (Removes deprecation warnings
+    for Ruby 2.6+)

data/README.md

@@ -109,6 +109,12 @@ Unfortunately using `edit -c` to edit `_spec.rb` files does not yet reload the
 code in a way that the `try-again` command can understand. You can still use
 `try-again` if you edit code that is not in spec files.
 
+If you want pry-rescue to *always* be enabled when you run tests, simply add this line to your `test_helper.rb`:
+
+```ruby
+require 'pry-rescue/rspec'
+```
+
 ### Minitest
 
 Add the following to your `test_helper.rb` or to the top of your test file.

data/Rakefile

@@ -1,3 +1,4 @@
+require "bundler/gem_tasks"
 require 'rspec/core/rake_task'
 
 task :default => :test

data/bin/rescue

@@ -34,18 +34,26 @@ when '-i'
 when /\A-/
   puts USAGE
   exit
-when 'rails'
-  ENV['PRY_RESCUE_RAILS'] = 'true'
-  exec(*ARGV)
-when /^re?spec|rake$/
-  ENV['SPEC_OPTS'] = "#{ENV['SPEC_OPTS']} -r pry-rescue/rspec"
-  exec(*ARGV)
+else
+  case File.basename(ARGV[0] || "")
+  when 'rails'
+    ENV['PRY_RESCUE_RAILS'] = 'true'
+    exec(*ARGV)
+  when 'rake'
+    require File.realpath(File.expand_path('../../lib/pry-rescue.rb', __FILE__))
+    PryRescue.load_rake ARGV[1]
+    exit
+  when /^re?spec$/
+    ENV['SPEC_OPTS'] = "#{ENV['SPEC_OPTS']} -r pry-rescue/rspec"
+    exec(*ARGV)
+  end
 end
 
 if script = ARGV.shift
   $0 = File.expand_path(script)
+
   if File.exists? script
-    require File.expand_path('../../lib/pry-rescue.rb', __FILE__)
+    require File.realpath(File.expand_path('../../lib/pry-rescue.rb', __FILE__))
     PryRescue.load $0, ensure_repl
   else
     $stderr.puts "Error: #{script.inspect} not found."

data/lib/pry-rescue.rb

@@ -6,13 +6,14 @@ require File.expand_path('../pry-rescue/core_ext', __FILE__)
 require File.expand_path('../pry-rescue/commands', __FILE__)
 require File.expand_path('../pry-rescue/rack', __FILE__)
 require File.expand_path('../pry-rescue/peek.rb', __FILE__)
+require File.expand_path('../pry-rescue/source_location.rb', __FILE__)
 
 if ENV['PRY_RESCUE_RAILS']
   require File.expand_path('../pry-rescue/rails', __FILE__)
 end
 case ENV['PRY_PEEK']
 when nil
-  PryRescue.peek_on_signal('QUIT') unless Pry::Helpers::BaseHelpers.windows?
+  PryRescue.peek_on_signal('QUIT') unless Pry::Helpers::Platform.windows?
 when ''
   # explicitly disable QUIT.
 else
@@ -79,6 +80,14 @@ class PryRescue
       end
     end
 
+    def load_rake(task)
+      require 'rake'
+      Pry::rescue do
+        load "#{Dir.pwd}/Rakefile"
+        Rake::Task[task].invoke
+      end
+    end
+
     # Is the user currently inside pry rescue?
     # @return [Boolean]
     def in_exception_context?
@@ -95,7 +104,7 @@ class PryRescue
     # @param [Exception] e  The raised exception
     def phantom_load_raise?(e)
       bindings = e.instance_variable_get(:@rescue_bindings)
-      bindings.any? && bindings.first.eval("__FILE__") == __FILE__
+      bindings.any? && SourceLocation.call(bindings.first)[0] == __FILE__
     end
 
     # When using pry-stack-explorer we want to start the rescue session outside of gems
@@ -105,7 +114,7 @@ class PryRescue
     # @return [Fixnum]  The offset of the first binding of user code
     def initial_frame(bindings)
       bindings.each_with_index do |binding, i|
-        return i if user_path?(binding.eval("__FILE__"))
+        return i if user_path?(SourceLocation.call(binding)[0])
       end
 
       0
@@ -162,7 +171,7 @@ class PryRescue
     def without_bindings_below_raise(bindings)
       return bindings if bindings.size <= 1
       bindings.drop_while do |b|
-        b.eval("__FILE__") == File.expand_path("../pry-rescue/core_ext.rb", __FILE__)
+        SourceLocation.call(b)[0] == File.expand_path("../pry-rescue/core_ext.rb", __FILE__)
       end.drop_while do |b|
         Interception == b.eval("self")
       end

data/lib/pry-rescue/commands.rb

@@ -47,6 +47,8 @@ Pry::Commands.create_command "cd-cause", "Move to the exception that caused this
     ex = ex.instance_variable_get(:@rescue_cause) if rescued == ex
     raise Pry::CommandError, "No previous exception to cd-cause into" if ex.nil? || ex == rescued
 
+    ex = ex.cause if ex.respond_to?(:cause) && !ex.cause.nil?
+
     Pry.rescued ex
   end
 end

data/lib/pry-rescue/rspec.rb

@@ -1,5 +1,6 @@
 require 'pry-rescue'
-require 'rspec'
+require 'pry-stack_explorer'
+require 'rspec' unless defined?(RSpec)
 
 class PryRescue
   class RSpec
@@ -7,24 +8,23 @@ class PryRescue
     # Run an Rspec example within Pry::rescue{ }.
     #
     # Takes care to ensure that `try-again` will work.
+    #
+    # `example` is a RSpec::Core::Example::Procsy
     def self.run(example)
       Pry::rescue do
         begin
           before
 
-          example.binding.eval('@exception = nil; @example && @example.instance_variable_set(:@exception, nil)')
-          example.binding.eval('example.instance_variable_set(:@exception, nil) if defined?(example)')
-          example.binding.eval('@example && @example.example_group_instance.instance_variable_set(:@__memoized, {})')
-          example.binding.eval('example.example_group_instance.instance_variable_set(:@__memoized, {}) if defined?(example)')
+          example.example.instance_variable_set(:@exception, nil)
+          example.example_group_instance.instance_variable_set(:@__init_memoized, true)
+
           example.run
-          e = example.binding.eval('@exception || @example && @example.instance_variable_get(:@exception)')
-          e ||= example.binding.eval('example.instance_variable_get(:@exception) if defined?(example)')
-          if e
-            Pry::rescued(e)
-          end
+
+          # Rescued will be called in :after hook, which is ran before the second
+          # :around leg
 
         ensure
-          after
+          after_outside
         end
       end
     end
@@ -33,7 +33,12 @@ class PryRescue
       monkeypatch_capybara if defined?(Capybara)
     end
 
-    def self.after
+    def self.after(example)
+      e = example.exception
+      Pry::rescued(e) if e
+    end
+
+    def self.after_outside
       after_filters.each(&:call)
     end
 
@@ -62,4 +67,8 @@ RSpec.configure do |c|
   c.around(:each) do |example|
     PryRescue::RSpec.run example
   end
+
+  c.after(:each) do |example|
+    PryRescue::RSpec.after(example)
+  end
 end

data/lib/pry-rescue/source_location.rb

@@ -0,0 +1,13 @@
+class PryRescue
+  module SourceLocation
+    DEPRECATION_TIME = Time.new(2021,4,1)
+
+    WithRuby2_5 = ->(b){ [b.eval("__FILE__"), b.eval("__LINE__")] }
+    WithRuby2_6 = ->(b){ b.source_location }
+
+    define_singleton_method(
+      :call,
+      (RUBY_VERSION < "2.6.0") ? WithRuby2_5 : WithRuby2_6
+    )
+  end
+end

data/pry-rescue.gemspec

@@ -1,23 +1,23 @@
 Gem::Specification.new do |s|
   s.name          = 'pry-rescue'
-  s.version       = '1.4.2'
+  s.version       = '1.5.2'
   s.summary       = 'Open a pry session on any unhandled exceptions'
   s.description   = 'Allows you to wrap code in Pry::rescue{ } to open a pry session at any unhandled exceptions'
   s.homepage      = 'https://github.com/ConradIrwin/pry-rescue'
   s.email         = ['conrad.irwin@gmail.com', 'jrmair@gmail.com', 'chris@ill-logic.com']
   s.authors       = ['Conrad Irwin', 'banisterfiend', 'epitron']
   s.files         = `git ls-files`.split("\n")
+  s.license       = 'MIT'
   s.require_paths = ['lib']
   s.executables   = s.files.grep(%r{^bin/}).map{|f| File.basename f}
 
-  s.add_dependency 'pry'
+  s.add_dependency 'pry', '>= 0.12.0'
   s.add_dependency 'interception', '>= 0.5'
 
   s.add_development_dependency 'pry-stack_explorer' # upgrade to regular dep?
 
   s.add_development_dependency 'rake'
   s.add_development_dependency 'rspec'
-  s.add_development_dependency 'yard'
   s.add_development_dependency 'redcarpet'
   s.add_development_dependency 'capybara'
 end

data/spec/commands_spec.rb

@@ -1,21 +1,22 @@
 require './spec/spec_helper'
+require './spec/fixtures/template_error'
 
 describe "pry-rescue commands" do
   describe "try-again" do
     it "should throw try_again" do
-      PryRescue.should_receive(:in_exception_context?).and_return{ true }
+      expect(PryRescue).to receive(:in_exception_context?).and_return(true)
 
-      lambda{
-        Pry.new.process_command "try-again", '', TOPLEVEL_BINDING
-      }.should throw_symbol :try_again
+      expect {
+        Pry.new.process_command("try-again")
+      }.to throw_symbol(:try_again)
     end
 
     it "should raise a CommandError if not in Pry::rescue" do
-      PryRescue.should_receive(:in_exception_context?).and_return{ false }
+      expect(PryRescue).to receive(:in_exception_context?).and_return(false)
 
-      lambda{
-        Pry.new.process_command "try-again", '', TOPLEVEL_BINDING
-      }.should raise_error Pry::CommandError
+      expect {
+        Pry.new.process_command "try-again"
+      }.to raise_error Pry::CommandError
     end
   end
 
@@ -28,11 +29,11 @@ describe "pry-rescue commands" do
         b2 = binding
       end
 
-      Pry.should_receive(:rescued).once.with{ |raised|
-        raised.should == e1
-      }
+      allow(Pry).to receive(:rescued).once do |raised|
+        expect(raised).to eq e1
+      end
 
-      Pry.new.process_command 'cd-cause e1', '', binding
+      Pry.new.tap{ |p| p.push_binding(binding) }.process_command 'cd-cause e1'
     end
 
     it "should enter the context of _ex_ if no exception is given" do
@@ -47,15 +48,39 @@ describe "pry-rescue commands" do
         end
       end
 
-      Pry.should_receive(:rescued).once.with{ |raised|
-        raised.should == _ex_
-      }
+      allow(Pry).to receive(:rescued).once do |raised|
+        expect(raised).to eq _ex_
+      end
 
-      Pry.new.process_command 'cd-cause', '', b2
+      Pry.new.tap{ |p| p.push_binding(b2) }.process_command 'cd-cause'
     end
   end
 
   describe "cd-cause" do
+    context 're-raising with custom Exception' do
+      it "should not loop through same context" do
+        _ex_ = nil
+        e1 = nil
+        Pry::rescue do
+          begin
+            begin
+              _b1 = binding
+              raise "original"
+            rescue => e1
+              _b2 = binding
+              raise TemplateError.new(e1)
+            end
+          rescue => e2
+            _ex_ = e2
+          end
+        end
+
+        expect(PryRescue).to receive(:enter_exception_context).once.with(e1)
+
+        Pry.new.tap{ |p| p.push_binding(binding) }.process_command 'cd-cause'
+      end
+    end
+
     it "should enter the next exception's context" do
       _ex_ = nil
       e1 = nil
@@ -73,9 +98,10 @@ describe "pry-rescue commands" do
         end
       end
 
-      PryRescue.should_receive(:enter_exception_context).once.with(e1)
+      expect(PryRescue).to receive(:enter_exception_context).once.with(e1)
 
-      Pry.new.process_command 'cd-cause', '', binding
+      Pry.new.tap{ |p| p.push_binding(binding) }.process_command 'cd-cause'
+      # PryTester.new(binding).process_command 'cd-cause'
     end
 
     it "should raise a CommandError if no previous commands" do
@@ -88,9 +114,9 @@ describe "pry-rescue commands" do
         _ex_ = e1
       end
 
-      lambda{
-        Pry.new.process_command 'cd-cause', '', binding
-      }.should raise_error Pry::CommandError, /No previous exception/
+      expect {
+        Pry.new.tap{ |p| p.push_binding(binding) }.process_command 'cd-cause'
+      }.to raise_error Pry::CommandError, /No previous exception/
     end
 
     it "should raise a CommandError on a re-raise" do
@@ -107,15 +133,15 @@ describe "pry-rescue commands" do
       end
       _rescued_ = _ex_
 
-      lambda{
-        Pry.new.process_command 'cd-cause', '', binding
-      }.should raise_error Pry::CommandError, /No previous exception/
+      expect {
+        Pry.new.tap{ |p| p.push_binding(binding) }.process_command 'cd-cause'
+      }.to raise_error Pry::CommandError, /No previous exception/
     end
 
     it "should raise a CommandError if not in Pry::rescue" do
-      lambda{
-        Pry.new.process_command 'cd-cause', '', binding
-      }.should raise_error Pry::CommandError, /No previous exception/
+      expect {
+        Pry.new.tap{ |p| p.push_binding(binding) }.process_command 'cd-cause'
+      }.to raise_error Pry::CommandError, /No previous exception/
     end
   end
 end

data/spec/core_ext_spec.rb

@@ -2,25 +2,25 @@ require File.expand_path('../../lib/pry-rescue.rb', __FILE__)
 
 describe 'Pry.rescue' do
   it 'should call PryRescue.enter_exception_context' do
-    lambda{
-      PryRescue.should_receive(:enter_exception_context).once
+    expect(lambda {
+      expect(PryRescue).to receive(:enter_exception_context).once
       Pry::rescue{ raise "foobar" }
-    }.should raise_error(/foobar/)
+    }).to raise_error(/foobar/)
   end
 
   it "should retry on try-again" do
     @called = 0
-    PryRescue.should_receive(:enter_exception_context).once{ throw :try_again }
+    expect(PryRescue).to receive(:enter_exception_context).once{ throw :try_again }
     Pry::rescue do
       @called += 1
       raise "foobar" if @called == 1
     end
-    @called.should == 2
+    expect(@called).to be(2)
   end
 
   it "should try-again from innermost block" do
     @outer = @inner = 0
-    PryRescue.should_receive(:enter_exception_context).once{ throw :try_again }
+    expect(PryRescue).to receive(:enter_exception_context).once{ throw :try_again }
     Pry::rescue do
       @outer += 1
       Pry::rescue do
@@ -29,13 +29,13 @@ describe 'Pry.rescue' do
       end
     end
 
-    @outer.should == 1
-    @inner.should == 2
+    expect(@outer).to be(1)
+    expect(@inner).to be(2)
   end
 
   it "should enter the first occurence of an exception that is re-raised" do
-    PryRescue.should_receive(:enter_exception_context).once{ |raised| raised.size.should == 1 }
-    lambda do
+    expect(PryRescue).to receive(:enter_exception_context).once{ |raised| raised.size.should == 1 }
+    expect(lambda do
       Pry::rescue do
         begin
           raise "first_occurance"
@@ -43,26 +43,26 @@ describe 'Pry.rescue' do
           raise
         end
       end
-    end.should raise_error(/first_occurance/)
+    end).to raise_error(/first_occurance/)
   end
 
   it "should not catch SystemExit" do
-    PryRescue.should_not_receive(:enter_exception_context)
+    expect(PryRescue).to_not receive(:enter_exception_context)
 
-    lambda do
+    expect(lambda do
       Pry::rescue do
         exit
       end
-    end.should raise_error SystemExit
+    end).to raise_error SystemExit
   end
 
   it 'should not catch Ctrl+C' do
-    PryRescue.should_not_receive(:enter_exception_context)
-    lambda do
+    expect(PryRescue).to_not receive(:enter_exception_context)
+    expect(lambda do
       Pry::rescue do
         raise Interrupt, "ctrl+c (fake)"
       end
-    end.should raise_error Interrupt
+    end).to raise_error Interrupt
   end
 end
 
@@ -72,15 +72,15 @@ describe "Pry.rescued" do
     begin
       raise "foo"
     rescue => e
-      Pry.should_receive(:warn)
+      expect(Pry).to receive(:warn)
       Pry.rescued(e)
     end
   end
 
   it "should raise an error if used on an exception not raised" do
     Pry::rescue do
-      Pry.should_receive(:warn) do |message|
-        message.should =~ /^WARNING: Tried to inspect exception outside of Pry::rescue/
+      expect(Pry).to receive(:warn) do |message|
+        expect(message).to match(/^WARNING: Tried to inspect exception outside of Pry::rescue/)
       end
       Pry.rescued(RuntimeError.new("foo").exception)
     end
@@ -91,7 +91,7 @@ describe "Pry.rescued" do
       begin
         raise "foo"
       rescue => e
-        PryRescue.should_receive(:enter_exception_context).once
+        expect(PryRescue).to receive(:enter_exception_context).once
         Pry::rescued(e)
       end
     end

data/spec/fixtures/template_error.rb

@@ -0,0 +1,13 @@
+# This Exception Class behaves same as ActionView::Template::Error
+# https://github.com/rails/rails/blob/master/actionview/lib/action_view/template/error.rb#L68
+#
+# It encapsulates an Exception and stores the latest Exception raised in its
+# `cause` attribute
+class TemplateError < StandardError
+  attr_reader :cause
+
+  def initialize(cause)
+    @cause = $!
+  end
+end
+

data/spec/peek_spec.rb

@@ -6,9 +6,9 @@ describe "PryRescue.peek!" do
     Pry.config.output = StringIO.new
     foo = 5
 
-    lambda do
+    expect(lambda do
       PryRescue.peek!
-    end.should change{ foo }.from(5).to(6)
+    end).to change{ foo }.from(5).to(6)
   end
 
   # this will fail, or not?
@@ -22,8 +22,8 @@ describe "PryRescue.peek!" do
 
     foo = 5
 
-    lambda do
+    expect(lambda do
       PryRescue.peek!
-    end.should change{ foo }.from(5).to(6)
+    end).to change{ foo }.from(5).to(6)
   end
 end

data/spec/pry_rescue_spec.rb

@@ -2,113 +2,128 @@ require File.expand_path('../../lib/pry-rescue.rb', __FILE__)
 require 'uri'
 
 describe "PryRescue.load" do
+  before :all do
+    if !binding.respond_to?(:source_location)
+      Binding.define_method :source_location do
+        PryRescue::SourceLocation.call(self)
+      end
+    end
+  end
+
   if defined?(PryStackExplorer)
     it "should open at the correct point" do
-      PryRescue.should_receive(:pry).once{ |opts|
-        opts[:call_stack].first.eval("__FILE__").should end_with('spec/fixtures/simple.rb')
+      expect(PryRescue).to receive(:pry).once { |opts|
+        expect(opts[:call_stack].first.source_location[0]).to end_with('spec/fixtures/simple.rb')
       }
-      lambda{
+      expect(lambda {
         PryRescue.load("spec/fixtures/simple.rb")
-      }.should raise_error(/simple-exception/)
+      }).to raise_error(/simple-exception/)
     end
 
     it "should open above the standard library" do
-      PryRescue.should_receive(:pry).once do |opts|
-        opts[:call_stack][opts[:initial_frame]].eval("__FILE__").should end_with('spec/fixtures/uri.rb')
+      expect(PryRescue).to receive(:pry).once do |opts|
+        expect(opts[:call_stack][opts[:initial_frame]].source_location[0]).to end_with('spec/fixtures/uri.rb')
       end
-      lambda{
+      expect(lambda{
         PryRescue.load("spec/fixtures/uri.rb")
-      }.should raise_error(URI::InvalidURIError)
+      }).to raise_error(URI::InvalidURIError)
     end
 
     it "should keep the standard library on the binding stack" do
-      PryRescue.should_receive(:pry).once do |opts|
-        opts[:call_stack].first.eval("__FILE__").should start_with(RbConfig::CONFIG['libdir'])
+      expect(PryRescue).to receive(:pry).once do |opts|
+        expect(opts[:call_stack].first.source_location[0]).to start_with(RbConfig::CONFIG['libdir'])
       end
-      lambda{
+      expect(lambda{
         PryRescue.load("spec/fixtures/uri.rb")
-      }.should raise_error(URI::InvalidURIError)
+      }).to raise_error(URI::InvalidURIError)
     end
 
     it "should open above gems" do
-      PryRescue.should_receive(:pry).once do |opts|
-        opts[:call_stack][opts[:initial_frame]].eval("__FILE__").should end_with('spec/fixtures/coderay.rb')
+      expect(PryRescue).to receive(:pry).once do |opts|
+        expect(opts[:call_stack][opts[:initial_frame]].source_location[0]).to end_with('spec/fixtures/coderay.rb')
       end
-      lambda{
+      expect(lambda{
         PryRescue.load("spec/fixtures/coderay.rb")
-      }.should raise_error(ArgumentError)
+      }).to raise_error(ArgumentError)
     end
 
     it "should open above gems" do
-      PryRescue.should_receive(:pry).once do |opts|
+      expect(PryRescue).to receive(:pry).once do |opts|
         coderay_path = Gem::Specification.respond_to?(:detect) ?
                          Gem::Specification.detect{|x| x.name == 'coderay' }.full_gem_path :
                          Gem.all_load_paths.grep(/coderay/).last
 
-        opts[:call_stack].first.eval("__FILE__").should start_with(coderay_path)
+        expect(opts[:call_stack].first.source_location[0]).to start_with(coderay_path)
       end
-      lambda{
+      expect(lambda{
         PryRescue.load("spec/fixtures/coderay.rb")
-      }.should raise_error(ArgumentError)
+      }).to raise_error(ArgumentError)
     end
 
     it "should skip pwd, even if it is a gem (but not vendor stuff)" do
-      Gem::Specification.stub :any? do true end
-      PryRescue.send(:user_path?, Dir.pwd + '/asdf.rb').should be_true
-      PryRescue.send(:user_path?, Dir.pwd + '/vendor/asdf.rb').should be_false
+      # Gem::Specification.stub :any? do true end
+      allow(Gem::Specification).to receive(:any?).and_return(true)
+
+      expect(
+        PryRescue.send(:user_path?, Dir.pwd + '/asdf.rb')
+      ).to be true
+
+      expect(
+        PryRescue.send(:user_path?, Dir.pwd + '/vendor/asdf.rb')
+      ).to be false
     end
 
     it "should filter out duplicate stack frames" do
-      PryRescue.should_receive(:pry).once do |opts|
-        opts[:call_stack][0].eval("__LINE__").should == 4
-        opts[:call_stack][1].eval("__LINE__").should == 12
+      expect(PryRescue).to receive(:pry).once do |opts|
+        expect(opts[:call_stack][0].source_location[1]).to be(4)
+        expect(opts[:call_stack][1].source_location[1]).to be(12)
       end
-      lambda{
+      expect(lambda{
         PryRescue.load("spec/fixtures/super.rb")
-      }.should raise_error(/super-exception/)
+      }).to raise_error(/super-exception/)
     end
 
     it "should calculate correct initial frame even when duplicates are present" do
-      PryRescue.should_receive(:pry).once do |opts|
-        opts[:call_stack][0].eval("__FILE__").should end_with('fake.rb')
-        opts[:call_stack][opts[:initial_frame]].eval("__FILE__").should end_with('spec/fixtures/initial.rb')
+      expect(PryRescue).to receive(:pry).once do |opts|
+        expect(opts[:call_stack][0].source_location[0]).to end_with('fake.rb')
+        expect(opts[:call_stack][opts[:initial_frame]].source_location[0]).to end_with('spec/fixtures/initial.rb')
       end
-      lambda{
+      expect(lambda{
         PryRescue.load("spec/fixtures/initial.rb")
-      }.should raise_error(/no :baz please/)
+      }).to raise_error(/no :baz please/)
     end
 
     it "should skip over reraises from within gems" do
-      PryRescue.should_receive(:pry).once do |opts|
-        opts[:call_stack][0].eval("__FILE__").should end_with('spec/fixtures/reraise.rb')
+      expect(PryRescue).to receive(:pry).once do |opts|
+        expect(opts[:call_stack][0].source_location[0]).to end_with('spec/fixtures/reraise.rb')
       end
-      lambda{
+      expect(lambda{
         PryRescue.load("spec/fixtures/reraise.rb")
-      }.should raise_error(/reraise-exception/)
+      }).to raise_error(/reraise-exception/)
     end
 
     it "should not skip over independent raises within gems" do
-      PryRescue.should_receive(:pry).once do |opts|
-        opts[:call_stack][0].eval("__FILE__").should end_with('fake.rb')
+      expect(PryRescue).to receive(:pry).once do |opts|
+        expect(opts[:call_stack][0].source_location[0]).to end_with('fake.rb')
       end
-      lambda{
+      expect(lambda{
         PryRescue.load("spec/fixtures/raiseother.rb")
-      }.should raise_error(/raiseother_exception/)
+      }).to raise_error(/raiseother_exception/)
     end
 
     it "should output a warning if the exception was not raised" do
-      PryRescue.should_not_receive(:enter_exception_context)
-      Pry.should_receive(:warn).once
+      expect(PryRescue).to_not receive(:enter_exception_context)
+      expect(Pry).to receive(:warn).once
       Pry.rescued(RuntimeError.new("foo"))
     end
   else
     it "should open at the correct point" do
-      Pry.should_receive(:start).once{ |binding, h|
-        binding.eval("__FILE__").should end_with('spec/fixtures/simple.rb')
+      expect(Pry).to receive(:start).once { |binding, h|
+        expect(binding.source_location[0]).to end_with('spec/fixtures/simple.rb')
       }
-      lambda{
+      expect(lambda{
         PryRescue.load("spec/fixtures/simple.rb")
-      }.should raise_error(/simple-exception/)
+      }).to raise_error(/simple-exception/)
     end
   end
 end

data/spec/source_location_spec.rb

@@ -0,0 +1,19 @@
+describe PryRescue::SourceLocation do
+  describe ".call" do
+    subject{ described_class.call(binding) }
+
+    it "matches [file, line]" do
+      is_expected.to match([__FILE__, be_between(2,30)])
+    end
+  end
+
+  it "will be removed when Ruby 2.5 is EOL" do
+    time = Time.now
+
+    if time >= described_class::DEPRECATION_TIME
+      expect(
+        defined?(PryRescue::SourceLocation)
+      ).to be false
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -1,6 +1,3 @@
 require 'rspec'
-require 'rspec/autorun'
-
-require 'pry/test/helper'
 
 require './lib/pry-rescue'

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: pry-rescue
 version: !ruby/object:Gem::Version
-  version: 1.4.2
+  version: 1.5.2
 platform: ruby
 authors:
 - Conrad Irwin
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-05-06 00:00:00.000000000 Z
+date: 2020-06-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: pry
@@ -18,14 +18,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 0.12.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 0.12.0
 - !ruby/object:Gem::Dependency
   name: interception
   requirement: !ruby/object:Gem::Requirement
@@ -82,20 +82,6 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-- !ruby/object:Gem::Dependency
-  name: yard
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
 - !ruby/object:Gem::Dependency
   name: redcarpet
   requirement: !ruby/object:Gem::Requirement
@@ -137,7 +123,9 @@ extensions: []
 extra_rdoc_files: []
 files:
 - ".gitignore"
+- ".rspec"
 - ".travis.yml"
+- CHANGELOG.md
 - Gemfile
 - LICENSE.MIT
 - README.md
@@ -168,6 +156,7 @@ files:
 - lib/pry-rescue/rack.rb
 - lib/pry-rescue/rails.rb
 - lib/pry-rescue/rspec.rb
+- lib/pry-rescue/source_location.rb
 - lib/pry/rescue.rb
 - pry-rescue.gemspec
 - spec/commands_spec.rb
@@ -178,12 +167,15 @@ files:
 - spec/fixtures/reraise.rb
 - spec/fixtures/simple.rb
 - spec/fixtures/super.rb
+- spec/fixtures/template_error.rb
 - spec/fixtures/uri.rb
 - spec/peek_spec.rb
 - spec/pry_rescue_spec.rb
+- spec/source_location_spec.rb
 - spec/spec_helper.rb
 homepage: https://github.com/ConradIrwin/pry-rescue
-licenses: []
+licenses:
+- MIT
 metadata: {}
 post_install_message: 
 rdoc_options: []
@@ -200,10 +192,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.2.2
+rubygems_version: 3.1.2
 signing_key: 
 specification_version: 4
 summary: Open a pry session on any unhandled exceptions
 test_files: []
-has_rdoc: