fix: debugThreadPool & shouldCheckCRDAndValidateLocalModel should work (#1015)

Fixes #1014

Author: metacosm

operator-framework-core/src/main/java/io/javaoperatorsdk/operator/api/config/Utils.java

@@ -64,11 +64,28 @@ public static boolean isValidateCustomResourcesEnvVarSet() {
   }
 
   public static boolean shouldCheckCRDAndValidateLocalModel() {
-    return Boolean.getBoolean(System.getProperty(CHECK_CRD_ENV_KEY, "true"));
+    return getBooleanFromSystemPropsOrDefault(CHECK_CRD_ENV_KEY, true);
   }
 
   public static boolean debugThreadPool() {
-    return Boolean.getBoolean(System.getProperty(DEBUG_THREAD_POOL_ENV_KEY, "false"));
+    return getBooleanFromSystemPropsOrDefault(DEBUG_THREAD_POOL_ENV_KEY, false);
+  }
+
+  static boolean getBooleanFromSystemPropsOrDefault(String propertyName, boolean defaultValue) {
+    var property = System.getProperty(propertyName);
+    if (property == null) {
+      return defaultValue;
+    } else {
+      property = property.trim().toLowerCase();
+      switch (property) {
+        case "true":
+          return true;
+        case "false":
+          return false;
+        default:
+          return defaultValue;
+      }
+    }
   }
 
   public static Class<?> getFirstTypeArgumentFromExtendedClass(Class<?> clazz) {

operator-framework-core/src/test/java/io/javaoperatorsdk/operator/api/config/UtilsTest.java

@@ -12,6 +12,62 @@
 
 class UtilsTest {
 
+  @Test
+  void shouldCheckCRDAndValidateLocalModelByDefault() {
+    assertTrue(Utils.shouldCheckCRDAndValidateLocalModel());
+  }
+
+  @Test
+  void shouldNotDebugThreadPoolByDefault() {
+    assertFalse(Utils.debugThreadPool());
+  }
+
+  @Test
+  void askingForNonexistentPropertyShouldReturnDefault() {
+    final var key = "foo";
+    assertNull(System.getProperty(key));
+    assertFalse(Utils.getBooleanFromSystemPropsOrDefault(key, false));
+    assertTrue(Utils.getBooleanFromSystemPropsOrDefault(key, true));
+  }
+
+  @Test
+  void askingForExistingPropertyShouldReturnItIfBoolean() {
+    final var key = "foo";
+    try {
+      System.setProperty(key, "true");
+      assertNotNull(System.getProperty(key));
+      assertTrue(Utils.getBooleanFromSystemPropsOrDefault(key, false));
+      assertTrue(Utils.getBooleanFromSystemPropsOrDefault(key, true));
+
+      System.setProperty(key, "TruE");
+      assertTrue(Utils.getBooleanFromSystemPropsOrDefault(key, false));
+      assertTrue(Utils.getBooleanFromSystemPropsOrDefault(key, true));
+
+      System.setProperty(key, " \tTRUE  ");
+      assertTrue(Utils.getBooleanFromSystemPropsOrDefault(key, false));
+      assertTrue(Utils.getBooleanFromSystemPropsOrDefault(key, true));
+
+      System.setProperty(key, " \nFalSe \t ");
+      assertFalse(Utils.getBooleanFromSystemPropsOrDefault(key, false));
+      assertFalse(Utils.getBooleanFromSystemPropsOrDefault(key, true));
+    } finally {
+      System.clearProperty(key);
+    }
+  }
+
+  @Test
+  void askingForExistingNonBooleanPropertyShouldReturnDefaultValue() {
+    final var key = "foo";
+    try {
+      System.setProperty(key, "bar");
+      assertNotNull(System.getProperty(key));
+      assertFalse(Utils.getBooleanFromSystemPropsOrDefault(key, false));
+      assertTrue(Utils.getBooleanFromSystemPropsOrDefault(key, true));
+    } finally {
+      System.clearProperty(key);
+    }
+  }
+
   @Test
   void getsFirstTypeArgumentFromExtendedClass() {
     Class<?> res =