Add requested changes from code review

CABLE-LSM · Sep 20, 2023 · 24b7fe4 · 24b7fe4
1 parent ef75799
commit 24b7fe4
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/benchcab/benchcab.py b/benchcab/benchcab.py
@@ -201,10 +201,8 @@ def build(self):
                     modules=self.config["modules"], verbose=self.args.verbose
                 )
             else:
-                print(
-                    f"Compiling CABLE {'with MPI' if internal.MPI else 'serially'} for "
-                    f"realisation {repo.name}..."
-                )
+                build_mode = "with MPI" if internal.MPI else "serially"
+                print(f"Compiling CABLE {build_mode} for realisation {repo.name}...")
                 repo.pre_build(verbose=self.args.verbose)
                 repo.run_build(
                     modules=self.config["modules"], verbose=self.args.verbose

diff --git a/tests/test_repository.py b/tests/test_repository.py
@@ -228,6 +228,7 @@ def test_post_build():
     (tmp_dir / internal.CABLE_EXE).touch()
     repo = get_mock_repo()
     repo.post_build()
+    assert not (offline_dir / ".tmp" / internal.CABLE_EXE).exists()
     assert (offline_dir / internal.CABLE_EXE).exists()
 
     # Success case: test non-verbose standard output