Set active_job_report_after_job_retries to true by default

Refs #2598
Refs #2597

Author: solnic

sentry-rails/lib/sentry/rails/configuration.rb

@@ -176,7 +176,7 @@ def initialize
         @enable_db_query_source = true
         @db_query_source_threshold_ms = 100
         @active_support_logger_subscription_items = Sentry::Rails::ACTIVE_SUPPORT_LOGGER_SUBSCRIPTION_ITEMS_DEFAULT.dup
-        @active_job_report_after_job_retries = false
+        @active_job_report_after_job_retries = true
       end
     end
   end

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

@@ -85,7 +85,9 @@ class FailedJobWithRetryOn < FailedJob
 
 RSpec.describe "ActiveJob integration", type: :job do
   before do
-    make_basic_app
+    make_basic_app do |config|
+      config.rails.active_job_report_after_job_retries = false
+    end
   end
 
   let(:event) do
@@ -218,6 +220,7 @@ def post.to_global_id
     before do
       make_basic_app do |config|
         config.traces_sample_rate = 1.0
+        config.rails.active_job_report_after_job_retries = false
       end
     end
 
@@ -261,7 +264,9 @@ def post.to_global_id
 
   context "when DeserializationError happens in user's jobs" do
     before do
-      make_basic_app
+      make_basic_app do |config|
+        config.rails.active_job_report_after_job_retries = false
+      end
     end
 
     class DeserializationErrorJob < ActiveJob::Base
@@ -441,7 +446,16 @@ def perform(event, hint)
       end
     end
 
+
     context "when active_job_report_after_job_retries is false" do
+      before do
+        Sentry.configuration.rails.active_job_report_after_job_retries = false
+      end
+
+      after do
+        Sentry.configuration.rails.active_job_report_after_job_retries = true
+      end
+
       it "reports 3 exceptions" do
         allow(Sentry::Rails::ActiveJobExtensions::SentryReporter)
           .to receive(:capture_exception).and_call_original

sentry-rails/spec/sentry/rails/configuration_spec.rb

@@ -62,7 +62,7 @@ class MySubscriber; end
 
   describe "#active_job_report_after_job_retries" do
     it "has correct default value" do
-      expect(subject.active_job_report_after_job_retries).to eq(false)
+      expect(subject.active_job_report_after_job_retries).to be(true)
     end
   end
 end