SymDB components spec: cover symbol_database wiring and shutdown

p-ddsign · p-ddsign · commit b259d2192b82 · 2026-05-18T12:16:22.000-04:00
#5717 wired SymbolDatabase::Component into Core::Configuration::Components (build at @symbol_database, shutdown! during teardown) but did not extend components_spec.rb to cover the new wiring. Add an @symbol_database describe block with two contexts: - symbol_database disabled → components.symbol_database is nil - symbol_database + remote config enabled → components.symbol_database is a Datadog::SymbolDatabase::Component (skipped on JRuby and Ruby < 2.6 per the feature's support matrix) Also extend the #shutdown! test to assert that components.symbol_database receives shutdown! when present, matching the pattern already used for dynamic_instrumentation, appsec, etc.
diff --git a/spec/datadog/core/configuration/components_spec.rb b/spec/datadog/core/configuration/components_spec.rb
@@ -173,6 +173,33 @@
         end
       end
     end
+
+    describe '@symbol_database' do
+      context 'when symbol_database is disabled' do
+        before { settings.symbol_database.enabled = false }
+
+        it 'does not build a symbol_database component' do
+          expect(components.symbol_database).to be nil
+        end
+      end
+
+      context 'when symbol_database is enabled with remote config' do
+        before(:all) do
+          skip 'Symbol database requires MRI Ruby 2.6+' if PlatformHelpers.jruby? || RUBY_VERSION < '2.6'
+        end
+
+        before do
+          settings.symbol_database.enabled = true
+          settings.remote.enabled = true
+        end
+
+        after { components.symbol_database&.shutdown! }
+
+        it 'builds a SymbolDatabase::Component' do
+          expect(components.symbol_database).to be_a(Datadog::SymbolDatabase::Component)
+        end
+      end
+    end
   end
 
   describe '::build_health_metrics' do
@@ -631,6 +658,7 @@ def self.on_load(*)
         expect(components.remote).to receive(:shutdown!) unless components.remote.nil?
         expect(components.profiler).to receive(:shutdown!) unless components.profiler.nil?
         expect(components.dynamic_instrumentation).to receive(:shutdown!) unless components.dynamic_instrumentation.nil?
+        expect(components.symbol_database).to receive(:shutdown!) unless components.symbol_database.nil?
         expect(components.appsec).to receive(:shutdown!) unless components.appsec.nil?
         expect(components.runtime_metrics).to receive(:stop)
           .with(true, close_metrics: false)
