[active_job] patch retry_job to report all errors during retries

Author: solnic

sentry-rails/lib/sentry/rails/active_job.rb

@@ -17,6 +17,13 @@ def already_supported_by_sentry_integration?
         Sentry.configuration.rails.skippable_job_adapters.include?(self.class.queue_adapter.class.to_s)
       end
 
+      def retry_job(error:, **opts)
+        unless Sentry.configuration.rails.active_job_report_after_job_retries
+          SentryReporter.capture_exception(self, error)
+        end
+        super
+      end
+
       class SentryReporter
         OP_NAME = "queue.active_job"
         SPAN_ORIGIN = "auto.queue.active_job"

sentry-rails/spec/sentry/rails/activejob_spec.rb

@@ -446,9 +446,11 @@ def perform(event, hint)
         allow(Sentry::Rails::ActiveJobExtensions::SentryReporter)
           .to receive(:capture_exception).and_call_original
 
-        assert_performed_jobs 3 do
-          FailedJobWithRetryOn.perform_later rescue nil
-        end
+        FailedJobWithRetryOn.perform_later rescue nil
+
+        perform_enqueued_jobs
+        perform_enqueued_jobs
+        perform_enqueued_jobs rescue nil
 
         expect(Sentry::Rails::ActiveJobExtensions::SentryReporter)
           .to have_received(:capture_exception)
@@ -469,9 +471,11 @@ def perform(event, hint)
         allow(Sentry::Rails::ActiveJobExtensions::SentryReporter)
           .to receive(:capture_exception).and_call_original
 
-        assert_performed_jobs 3 do
-          FailedJobWithRetryOn.perform_later rescue nil
-        end
+        FailedJobWithRetryOn.perform_later rescue nil
+
+        perform_enqueued_jobs
+        perform_enqueued_jobs
+        perform_enqueued_jobs rescue nil
 
         expect(Sentry::Rails::ActiveJobExtensions::SentryReporter)
           .to have_received(:capture_exception)