Move test for keys starting with _ to the top

Signed-off-by: Ankita Katiyar <ankitakatiyar2401@gmail.com>

kedro/config/omegaconf_config.py

@@ -323,6 +323,10 @@ def _register_globals_resolver(self):
 
     def _get_globals_value(self, variable, default_value):
         """Return the globals values to the resolver"""
+        if variable.startswith("_"):
+            raise InterpolationResolutionError(
+                "Keys starting with '_' are not supported for globals."
+            )
         keys = variable.split(".")
         value = self["globals"]
         for k in keys:
@@ -334,8 +338,6 @@ def _get_globals_value(self, variable, default_value):
                     )
                     return default_value
                 msg = f"Globals key '{variable}' not found and no default value provided. "
-                if variable.startswith("_"):
-                    msg += "Keys starting with '_' are not supported for globals."
                 raise InterpolationResolutionError(msg)
         return value
 

tests/config/test_omegaconf_config.py

@@ -794,6 +794,6 @@ def test_bad_globals_underscore(self, tmp_path):
         conf = OmegaConfigLoader(tmp_path, default_run_env="")
         with pytest.raises(
             InterpolationResolutionError,
-            match=r"Globals key '_ignore' not found and no default value provided. Keys starting with '_' are not supported for globals.",
+            match=r"Keys starting with '_' are not supported for globals.",
         ):
             conf["parameters"]["param2"]