Merge branch 'optional_runnability_checks' into develop

LourensVeen · LourensVeen · commit 5b93f6f07ffb · 2026-02-05T11:43:56.000+01:00
diff --git a/ymmsl/v0_2/configuration.py b/ymmsl/v0_2/configuration.py
@@ -181,7 +181,7 @@ def update(self, overlay: 'Configuration') -> None:
         self.checkpoints.update(overlay.checkpoints)
         self.resume.update(overlay.resume)
 
-    def check_consistent(self) -> None:
+    def check_consistent(self, check_runnable: bool = True) -> None:
         """Checks that the configuration is internally consistent.
 
         This checks:
@@ -197,6 +197,10 @@ def check_consistent(self) -> None:
 
         If any of these requirements is false, this function will raise a RuntimeError
         with an explanation of the problem.
+
+        Args:
+            check_runnable: if False, skip the checks for whether component
+                implementations exist and whether resources have been requested.
         """
         errors = list()
 
@@ -207,11 +211,13 @@ def check_consistent(self) -> None:
         errors.extend(self._check_duplicate_implementations())
 
         component_paths = self._component_paths()
-        errors.extend(self._check_implementations_exist(component_paths))
+        if check_runnable:
+            errors.extend(self._check_implementations_exist(component_paths))
         errors.extend(self._check_consistent_ports(component_paths))
         errors.extend(self._check_custom_implementations(component_paths))
         errors.extend(self._check_consistent_settings(component_paths))
-        errors.extend(self._check_resources(component_paths))
+        if check_runnable:
+            errors.extend(self._check_resources(component_paths))
 
         if errors:
             raise RuntimeError(
diff --git a/ymmsl/v0_2/tests/test_configuration.py b/ymmsl/v0_2/tests/test_configuration.py
@@ -372,6 +372,11 @@ def test_check_inconsistent_implementation_ports(
 
     assert len(str(e.value).split('\n')) == 8
 
+    with pytest.raises(RuntimeError) as e:
+        config_inconsistent_impl_ports.check_consistent(False)
+
+    assert len(str(e.value).split('\n')) == 7
+
 
 def test_check_consistent_custom_implementations(
         config_consistent_custom_impls: Configuration) -> None:
@@ -425,3 +430,5 @@ def test_check_inconsistent_resources(
         config_inconsistent_resources.check_consistent()
 
     assert len(str(e.value).split('\n')) == 4
+
+    config_inconsistent_resources.check_consistent(False)
