Merge pull request #206 from CABLE-LSM/199-add-getter-for-exe-path

Add getter method for built executable

Author: SeanBryan51

benchcab/fluxsite.py

@@ -264,13 +264,7 @@ def fetch_files(self, verbose=False):
             self.root_dir / internal.NAMELIST_DIR, task_dir, dirs_exist_ok=True
         )
 
-        exe_src = (
-            self.root_dir
-            / internal.SRC_DIR
-            / self.repo.name
-            / "offline"
-            / internal.CABLE_EXE
-        )
+        exe_src = self.repo.get_exe_path()
         exe_dest = task_dir / internal.CABLE_EXE
 
         if verbose:

benchcab/model.py

@@ -76,6 +76,16 @@ def svn_info_show_item(self, item: str) -> str:
         )
         return proc.stdout.strip()
 
+    def get_exe_path(self) -> Path:
+        """Return the path to the built executable."""
+        return (
+            self.root_dir
+            / internal.SRC_DIR
+            / self.name
+            / "offline"
+            / internal.CABLE_EXE
+        )
+
     def custom_build(self, modules: list[str], verbose=False):
         """Build CABLE using a custom build script."""
         build_script_path = (

tests/test_model.py

@@ -46,6 +46,17 @@ def test_undefined_repo_id(self, repo):
             _ = repo.repo_id
 
 
+class TestGetExePath:
+    """Tests for `CableRepository.get_exe_path()`."""
+
+    def test_serial_exe_path(self, repo, mock_cwd):
+        """Success case: get path to serial executable."""
+        assert (
+            repo.get_exe_path()
+            == mock_cwd / internal.SRC_DIR / repo.name / "offline" / internal.CABLE_EXE
+        )
+
+
 class TestCheckout:
     """Tests for `Model.checkout()`."""