ODSC-44021: Fix developer mode logic. Reduce testing time (#251)

Author: KshitizLohia

ads/feature_store/common/spark_session_singleton.py

@@ -11,6 +11,7 @@
 import os
 
 from ads.common.oci_client import OCIClientFactory
+from ads.feature_store.common.utils.utility import get_env_bool
 
 try:
     from delta import configure_spark_with_delta_pip
@@ -33,7 +34,7 @@
 
 
 def developer_enabled():
-    return os.getenv("DEVELOPER_MODE")
+    return get_env_bool("DEVELOPER_MODE", False)
 
 
 class SingletonMeta(type):

ads/feature_store/common/utils/utility.py

@@ -1,6 +1,7 @@
 #!/usr/bin/env python
 # -*- coding: utf-8; -*-
 import copy
+import os
 # Copyright (c) 2023 Oracle and/or its affiliates.
 # Licensed under the Universal Permissive License v 1.0 as shown at https://oss.oracle.com/licenses/upl/
 
@@ -62,6 +63,25 @@ def get_execution_engine_type(
         else ExecutionEngine.SPARK
     )
 
+def get_env_bool(env_var: str, default: bool = False) -> bool:
+    """
+    :param env_var: Environment variable name
+    :param default: Default environment variable value
+    :return: Value of the boolean env variable
+    """
+    env_val = os.getenv(env_var)
+    if env_val is None:
+        env_val = default
+    else:
+        env_val = env_val.lower()
+        if env_val == "true":
+            env_val = True
+        elif env_val == "false":
+            env_val = False
+        else:
+            raise ValueError("For environment variable: {0} only string values T/true or F/false are allowed but: \
+                {1} was provided.".format(env_var, env_val))
+    return env_val
 
 def get_metastore_id(feature_store_id: str):
     """