perf(active_job): boot the dummy app once per spec group

The harness was calling make_basic_app in its around-each block,
which creates a fresh Rails::Application subclass and runs every
initializer on each example. With 98 AJ examples that overhead dwarfed
the actual test work — and worse, it left behind state (Sidekiq's
@config_blocks list, accumulated routes, lingering Rails::Application
subclasses) that made each subsequent make_basic_app a little
slower. Under Ruby 3.4 + Rails 8.1.3 the per-example time grew 3× over
the run, pushing the full sentry-rails CI past the 15-min timeout.

Hoist make_basic_app to before(:all) and replicate the per-example
bits of Sentry::Rails::Railtie's after_initialize hook in the around
block — re-init Sentry, re-activate tracing / structured logging,
re-register the AJ event handlers. The one-time extensions
(controller methods, streaming reporter, backtrace cleanup, etc.) were
already installed by the initial make_basic_app and persist for the
group.

Also memoize the SidekiqAdapter instance in the :sidekiq context.
Each SidekiqAdapter.new appended to Sidekiq's internal @config_blocks
list and added an on(:quiet) callback; creating a fresh adapter per
example was unnecessary global churn.

Result: spec/active_job goes from 33s → 0.8s, and the full
sentry-rails spec task (Ruby 3.4 + Rails 8.1.3) goes from 9:22 to
2:31 — well under the CI limit.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: solnic

sentry-rails/spec/active_job/support/harness.rb

@@ -17,8 +17,52 @@
   let(:adapter) { adapter }
   let(:configure_sentry) { proc { } }
 
+  # Boot the dummy Rails app ONCE per example group. Each +make_basic_app+
+  # call creates a new +Rails::Application+ subclass and re-runs every
+  # initializer — including Sidekiq's railtie (which appends two entries
+  # to +Sidekiq.@config_blocks+) and Rails' route-drawing (which also
+  # accumulates). Repeating that for every example caused per-example
+  # time to grow ~3× over the run, which is what pushed the
+  # Ruby 3.4 + Rails 8.1.3 CI matrix past the 15-min timeout.
+  #
+  # We reproduce the relevant per-example pieces of the Sentry/Rails
+  # railtie's +config.after_initialize+ block below (re-init Sentry,
+  # re-activate tracing/structured logging, re-register AJ event
+  # handlers) so each example still gets a fresh Sentry configuration.
+  before(:all) do
+    make_basic_app
+  end
+
   around do |example|
-    make_basic_app(&configure_sentry)
+    Sentry.init do |config|
+      config.release = "beta"
+      config.dsn = "http://12345:67890@sentry.localdomain:3000/sentry/42"
+      config.transport.transport_class = Sentry::DummyTransport
+      config.background_worker_threads = 0
+      config.include_local_variables = true
+      configure_sentry.call(config, ::Rails.application) if configure_sentry
+    end
+
+    # Mirror the bits of Sentry::Rails::Railtie's after_initialize hook
+    # that need to run AFTER Sentry.init each example — the one-time
+    # extensions (controller methods, streaming reporter, backtrace
+    # cleanup callback, etc.) were already wired up by the initial
+    # make_basic_app in before(:all) and persist for the rest of the
+    # group.
+    if Sentry.configuration.tracing_enabled? && Sentry.configuration.instrumenter == :sentry
+      Sentry::Rails::Tracing.register_subscribers(Sentry.configuration.rails.tracing_subscribers)
+      Sentry::Rails::Tracing.subscribe_tracing_events
+      Sentry::Rails::Tracing.patch_active_support_notifications
+    end
+
+    if Sentry.configuration.rails.structured_logging.enabled? && Sentry.configuration.enable_logs
+      Sentry::Rails::StructuredLogging.attach(Sentry.configuration.rails.structured_logging)
+    end
+
+    if defined?(Sentry::Rails::ActiveJobExtensions)
+      Sentry::Rails::ActiveJobExtensions::SentryReporter.register_event_handlers
+    end
+
     setup_sentry_test
 
     boot_adapter(adapter)

sentry-rails/spec/active_job/support/sidekiq_adapter_context.rb

@@ -30,8 +30,15 @@
 end
 
 RSpec.shared_context "sidekiq adapter" do
+  # Instantiated once. Each SidekiqAdapter.new appends blocks to
+  # Sidekiq's internal @config_blocks list (configure_server) and adds
+  # an on(:quiet) callback (configure_client). Creating a fresh
+  # adapter per example accumulated several hundred entries across a
+  # full suite run and showed up as compounding per-example slowdown.
+  SIDEKIQ_ADAPTER_FOR_TEST = ::ActiveJob::QueueAdapters::SidekiqAdapter.new
+
   def queue_adapter_for_test
-    ::ActiveJob::QueueAdapters::SidekiqAdapter.new
+    SIDEKIQ_ADAPTER_FOR_TEST
   end
 
   # Scope fake mode to this example only — the block form of +fake!+