From 48ba23af1c9d91a08bca7bf4eea89f5633e472f9 Mon Sep 17 00:00:00 2001 From: Gaetan Craig-Riou Date: Wed, 28 Feb 2024 15:08:57 +1100 Subject: [PATCH] Notify BugSnag when something unexpected happens --- app/controllers/spree/admin/payments_controller.rb | 1 + spec/requests/spree/admin/payments_spec.rb | 1 + 2 files changed, 2 insertions(+) diff --git a/app/controllers/spree/admin/payments_controller.rb b/app/controllers/spree/admin/payments_controller.rb index 4bf6ebcdcc..041dacf8d6 100644 --- a/app/controllers/spree/admin/payments_controller.rb +++ b/app/controllers/spree/admin/payments_controller.rb @@ -60,6 +60,7 @@ module Spree end rescue StandardError => e logger.error e.message + Bugsnag.notify(e) flash[:error] = e.message ensure redirect_to request.referer diff --git a/spec/requests/spree/admin/payments_spec.rb b/spec/requests/spree/admin/payments_spec.rb index 14829a5cde..c8ff325554 100644 --- a/spec/requests/spree/admin/payments_spec.rb +++ b/spec/requests/spree/admin/payments_spec.rb @@ -111,6 +111,7 @@ describe Spree::Admin::PaymentsController, type: :request do # The redirect_do also calls Rails.logger.error expect(Rails.logger).to receive(:error).with("Unexpected !").ordered expect(Rails.logger).to receive(:error).with(/Redirected/).ordered + expect(Bugsnag).to receive(:notify).with(StandardError) put( "/admin/orders/#{order.number}/payments/#{order.payments.first.id}/fire?e=void",