From ec2b0806a7956fa8a55ee60c66f5c6ccd30c23a4 Mon Sep 17 00:00:00 2001 From: Tommy Gatti Date: Tue, 13 Aug 2024 15:33:15 +1000 Subject: [PATCH] Updated the `collate.mpi` value assertion --- src/model_config_tests/qa/test_access_esm1p5_config.py | 7 +++++-- 1 file changed, 5 insertions(+), 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 index 86e6dec..e2890ba 100644 --- 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" )