Updated the collate.mpi value assertion

ACCESS-NRI · Aug 13, 2024 · ec2b080 · ec2b080
1 parent 8d2e2ac
commit ec2b080
Showing 1 changed file with 5 additions and 2 deletions.
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"
             )