Remove expect_any_instance_of(...)

Wrap `default_rescue_handler` with `method`
ruby-grape · Jul 23, 2024 · 9dce428 · 9dce428
1 parent fb67ea9
commit 9dce428
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 3 deletions.
diff --git a/lib/grape/middleware/error.rb b/lib/grape/middleware/error.rb
@@ -125,7 +125,7 @@ def run_rescue_handler(handler, error, endpoint)
         elsif response.is_a?(Rack::Response)
           response
         else
-          run_rescue_handler(:default_rescue_handler, Grape::Exceptions::InvalidResponse.new, endpoint)
+          run_rescue_handler(method(:default_rescue_handler), Grape::Exceptions::InvalidResponse.new, endpoint)
         end
       end
 

diff --git a/spec/grape/api_spec.rb b/spec/grape/api_spec.rb
@@ -2196,8 +2196,6 @@ def foo
     it 'uses default_rescue_handler to handle invalid response from rescue_from' do
       subject.rescue_from(:all) { 'error' }
       subject.get('/') { raise }
-
-      expect_any_instance_of(Grape::Middleware::Error).to receive(:default_rescue_handler).and_call_original
       get '/'
       expect(last_response).to be_server_error
       expect(last_response.body).to eql 'Invalid response'