Fix In-Product when config is empty (#8679)

should not stop the product with empty config

Author: jpbempel

dd-java-agent/agent-debugger/src/main/java/com/datadog/debugger/agent/DefaultProductConfigUpdater.java

@@ -70,10 +70,12 @@ private static void startOrStopFeature(
       LOGGER.debug("Feature {} is explicitly disabled", booleanKey);
       return;
     }
-    if (currentStatus != null && currentStatus) {
-      start.run();
-    } else {
-      stop.run();
+    if (currentStatus != null) {
+      if (currentStatus) {
+        start.run();
+      } else {
+        stop.run();
+      }
     }
   }
 }

dd-java-agent/agent-debugger/src/test/java/com/datadog/debugger/agent/DefaultProductConfigUpdaterTest.java

@@ -24,6 +24,11 @@ public void enableDisable() {
     assertTrue(productConfigUpdater.isExceptionReplayEnabled());
     assertTrue(productConfigUpdater.isCodeOriginEnabled());
     assertTrue(productConfigUpdater.isDistributedDebuggerEnabled());
+    productConfigUpdater.updateConfig(null, null, null, null);
+    assertTrue(productConfigUpdater.isDynamicInstrumentationEnabled());
+    assertTrue(productConfigUpdater.isExceptionReplayEnabled());
+    assertTrue(productConfigUpdater.isCodeOriginEnabled());
+    assertTrue(productConfigUpdater.isDistributedDebuggerEnabled());
     productConfigUpdater.updateConfig(false, false, false, false);
     assertFalse(productConfigUpdater.isDynamicInstrumentationEnabled());
     assertFalse(productConfigUpdater.isExceptionReplayEnabled());