updating tests

Author: mhlidd

internal-api/src/main/java/datadog/trace/api/Config.java

@@ -652,6 +652,8 @@
 import datadog.trace.api.profiling.ProfilingEnablement;
 import datadog.trace.api.rum.RumInjectorConfig;
 import datadog.trace.api.rum.RumInjectorConfig.PrivacyLevel;
+import datadog.trace.api.telemetry.ConfigInversionMetricCollectorImpl;
+import datadog.trace.api.telemetry.ConfigInversionMetricCollectorProvider;
 import datadog.trace.api.telemetry.OtelEnvMetricCollectorImpl;
 import datadog.trace.api.telemetry.OtelEnvMetricCollectorProvider;
 import datadog.trace.bootstrap.config.provider.CapturedEnvironmentConfigSource;
@@ -1245,6 +1247,7 @@ public static String getHostName() {
   static {
     // Bind telemetry collector to config module before initializing ConfigProvider
     OtelEnvMetricCollectorProvider.register(OtelEnvMetricCollectorImpl.getInstance());
+    ConfigInversionMetricCollectorProvider.register(ConfigInversionMetricCollectorImpl.getInstance());
   }
 
   // Read order: System Properties -> Env Variables, [-> properties file], [-> default value]

utils/config-utils/src/main/java/datadog/trace/api/telemetry/ConfigInversionMetricCollectorProvider.java

@@ -7,7 +7,7 @@ private ConfigInversionMetricCollectorProvider() {}
 
   public static ConfigInversionMetricCollector get() {
     if (INSTANCE == null) {
-      throw new IllegalStateException("OtelEnvMetricCollectorService has not been registered.");
+      throw new IllegalStateException("ConfigInversionMetricCollectorService has not been registered.");
     }
     return INSTANCE;
   }

utils/config-utils/src/test/java/datadog/trace/config/inversion/ConfigHelperTest.java

@@ -12,6 +12,8 @@
 import java.util.List;
 import java.util.Map;
 import java.util.Set;
+
+import datadog.trace.api.telemetry.ConfigInversionMetricCollectorProvider;
 import org.junit.jupiter.api.AfterAll;
 import org.junit.jupiter.api.BeforeAll;
 import org.junit.jupiter.api.Test;
@@ -41,6 +43,7 @@ public class ConfigHelperTest {
   @BeforeAll
   static void setUp() {
     // Set up test configurations using SupportedConfigurationSource
+//    ConfigInversionMetricCollectorProvider.register(ConfigInversionMetricCollectorImpl.getInstance());
     Set<String> testSupported =
         new HashSet<>(Arrays.asList(TEST_DD_VAR, TEST_OTEL_VAR, TEST_REGULAR_VAR));
 
@@ -118,18 +121,6 @@ void testMainConfigPrecedence() {
     setEnvVar(ALIAS_DD_VAR, null);
   }
 
-  @Test
-  void testUnsupportedDDEnvironmentVariable() {
-    setEnvVar(UNSUPPORTED_DD_VAR, "");
-
-    assertNull(ConfigHelper.getEnvironmentVariable(UNSUPPORTED_DD_VAR));
-    Map<String, String> result = ConfigHelper.getEnvironmentVariables();
-    assertFalse(result.containsKey(UNSUPPORTED_DD_VAR));
-
-    // Cleanup
-    setEnvVar(UNSUPPORTED_DD_VAR, null);
-  }
-
   @Test
   void testNonDDAliases() {
     setEnvVar(NON_DD_ALIAS_VAR, NON_DD_ALIAS_VAL);
@@ -149,19 +140,6 @@ void testAliasesWithoutPresentAliases() {
     assertFalse(result.containsKey(ALIAS_DD_VAR));
   }
 
-  @Test
-  void testUnsupportedButNotStrict() {
-    ConfigHelper.setConfigInversionStrict(ConfigInversionStrictStyle.WARNING);
-    setEnvVar(UNSUPPORTED_DD_VAR, "loose");
-
-    // Should fall through and return the env var even though it's unsupported
-    assertEquals("loose", ConfigHelper.getEnvironmentVariable(UNSUPPORTED_DD_VAR));
-
-    // Cleanup
-    ConfigHelper.setConfigInversionStrict(ConfigInversionStrictStyle.STRICT);
-    setEnvVar(UNSUPPORTED_DD_VAR, null);
-  }
-
   @Test
   void testAliasWithEmptyList() {
     Map<String, List<String>> aliasMap = new HashMap<>();