pry-rescue 1.4.5 → 1.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: f8498adfc62776fab874402fd91ff59f7a0b3811
-  data.tar.gz: 2efe1214b7acb9361f5c84754bad36aebfc769bd
+SHA256:
+  metadata.gz: 9858e248a006df546dbc0e2e4a5b0055e9f0e2276bce651729af051d88741ec7
+  data.tar.gz: dbc440ce345d54877bd894a8dfe89c9d497e4f144ffcc5dadf702bb89887c7cc
 SHA512:
-  metadata.gz: a60814bf969deba350559387f5a11c79fa6b03e86bb90656cee06f1ec26dd52e3db99ea815dbe033cb10fd65dcce59a932959c838a2b1b70a40d238057f2be54
-  data.tar.gz: 94c9e761195dcd0c7eb01c17af7670b0eea83fa9d59e36d4b24a1612f4b49807f6f12f0463072c4f5a7838d5ff256f54fb2d4629fb896d8edc9cfc37ea8e1121
+  metadata.gz: cccaff14cd38dfe24cfa8a85d7121b90f526e214cb6accf474e1237429a372d871eb5189e696ea18b4eb34371f236ebfeee0559202d1301be11b0bf8d06ebdd9
+  data.tar.gz: 5236813e00b7a5198159d181c1b660c4aad9b4f8c1b7c50a9c69d39753d5641d8f8efe24176a51f709b91d1f28301baae577d930c5cfe393d93ebbc8f958db89

data/.rspec

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

data/.travis.yml

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

data/CHANGELOG.md

@@ -0,0 +1,14 @@
+## v1.6.0 (9 January 2024)
+* Fix ruby 3.2 compatibility
+
+## 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

@@ -39,7 +39,11 @@ else
   when 'rails'
     ENV['PRY_RESCUE_RAILS'] = 'true'
     exec(*ARGV)
-  when /^re?spec|rake$/
+  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
@@ -47,8 +51,9 @@ end
 
 if script = ARGV.shift
   $0 = File.expand_path(script)
-  if File.exists? script
-    require File.expand_path('../../lib/pry-rescue.rb', __FILE__)
+
+  if File.exist? script
+    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/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,4 +1,5 @@
 require 'pry-rescue'
+require 'pry-stack_explorer'
 require 'rspec' unless defined?(RSpec)
 
 class PryRescue
@@ -7,29 +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)')
-          if example.example_group_instance.respond_to?(:__init_memoized, true)
-            example.binding.eval('@example && @example.example_group_instance.instance_variable_set(:@__init_memoized, true)')
-            example.binding.eval('example.example_group_instance.instance_variable_set(:@__init_memoized, true) if defined?(example)')
-          else
-            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)')
-          end
+          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
@@ -38,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
 
@@ -50,7 +50,13 @@ class PryRescue
       unless Capybara.respond_to?(:reset_sessions_after_rescue!)
         class << Capybara
           alias_method :reset_sessions_after_rescue!, :reset_sessions!
-          def reset_sessions!; end
+          def reset_sessions!
+            return if Capybara.raise_server_errors
+
+            session_pool.reverse_each do |_mode, session|
+              session.server.reset_error!
+            end
+          end
         end
 
         after_filters << Capybara.method(:reset_sessions_after_rescue!)
@@ -67,4 +73,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/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/pry-rescue.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |s|
   s.name          = 'pry-rescue'
-  s.version       = '1.4.5'
+  s.version       = '1.6.0'
   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'
@@ -11,7 +11,7 @@ Gem::Specification.new do |s|
   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?
@@ -20,23 +20,4 @@ Gem::Specification.new do |s|
   s.add_development_dependency 'rspec'
   s.add_development_dependency 'redcarpet'
   s.add_development_dependency 'capybara'
-
-  # SPECIAL DEVELOPMENT GEM FOR OLD RUBY
-  # DONT USE THIS TRICK FOR RUNTIME GEM
-  if Gem::Version.create(RUBY_VERSION) < Gem::Version.create("2.2.2")
-    s.add_development_dependency 'yard', '< 0.9.6'
-    s.add_development_dependency 'rack', ['~> 1.6', '< 1.7']
-  else
-    s.add_development_dependency 'yard'
-  end
-  if Gem::Version.create(RUBY_VERSION) < Gem::Version.create("2.1")
-    # capybara > nokogiri
-    s.add_development_dependency 'nokogiri', ['~> 1.6', '< 1.7.0']
-  end
-  if Gem::Version.create(RUBY_VERSION) < Gem::Version.create("2.0")
-    # capybara > addressable > public_suffix
-    s.add_development_dependency 'public_suffix', ['~> 1.4', '< 1.5']
-    # capybara > mime-types
-    s.add_development_dependency 'mime-types', ['~> 2.6', '< 2.99']
-  end
 end

data/spec/commands_spec.rb

@@ -1,4 +1,5 @@
 require './spec/spec_helper'
+require './spec/fixtures/template_error'
 
 describe "pry-rescue commands" do
   describe "try-again" do
@@ -56,6 +57,30 @@ describe "pry-rescue commands" do
   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

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,54 +2,62 @@ 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
@@ -66,56 +74,56 @@ describe "PryRescue.load" do
     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,5 +1,3 @@
 require 'rspec'
 
-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.5
+  version: 1.6.0
 platform: ruby
 authors:
 - Conrad Irwin
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-02-16 00:00:00.000000000 Z
+date: 2024-01-10 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
@@ -110,20 +110,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'
 description: Allows you to wrap code in Pry::rescue{ } to open a pry session at any
   unhandled exceptions
 email:
@@ -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,9 +167,11 @@ 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:
@@ -201,8 +192,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.6.8
+rubygems_version: 3.0.3.1
 signing_key: 
 specification_version: 4
 summary: Open a pry session on any unhandled exceptions