Merge pull request #47 from ACCESS-NRI/access-esm1p5-mppnccombine-mpi…

…-fix `ACCESS-ESM1.5` `mppnccombine` MPI Test Fix
ACCESS-NRI · Aug 13, 2024 · 0649090 · 0649090
2 parents 8d2e2ac + 2d538a5
commit 0649090
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/pyproject.toml b/pyproject.toml
@@ -1,6 +1,6 @@
 [project]
 name = "model_config_tests"
-version = "0.0.6"
+version = "0.0.7"
 authors = [
   { name = "ACCESS-NRI" },
 ]

diff --git a/src/model_config_tests/qa/test_access_esm1p5_config.py b/src/model_config_tests/qa/test_access_esm1p5_config.py
@@ -185,6 +185,9 @@ def test_mppnccombine_fast_collate_exe(self, config):
                 "collate.mpi", "config.yaml"
             )
 
-            assert config["collate"]["mpi"], error_field_incorrect(
-                "collate.mpi", "config.yaml", "true"
+            # Loading the yaml into a dict also converts
+            # `collate.mpi:true`/`collate.mpi:false` to `True`/`False` so we
+            # can assert if it is a `bool`.
+            assert isinstance(config["collate"]["mpi"], bool), error_field_incorrect(
+                "collate.mpi", "config.yaml", "true or false"
             )