Suppress the FileNotFound error when attempting to set the file mode (#498)

* Suppress the FileNotFound error when attempting to set the file mode

* Fix test

Author: faridyagubbayli

kwave/executor.py

@@ -6,11 +6,12 @@
 import h5py
 
 import kwave
+from kwave.options.simulation_execution_options import SimulationExecutionOptions
 from kwave.utils.dotdictionary import dotdict
 
 
 class Executor:
-    def __init__(self, execution_options, simulation_options):
+    def __init__(self, execution_options: SimulationExecutionOptions, simulation_options):
         self.execution_options = execution_options
         self.simulation_options = simulation_options
 
@@ -21,15 +22,15 @@ def __init__(self, execution_options, simulation_options):
         self._make_binary_executable()
 
     def _make_binary_executable(self):
-        try:
-            self.execution_options.binary_path.chmod(self.execution_options.binary_path.stat().st_mode | stat.S_IEXEC)
-        except FileNotFoundError as e:
+        binary_path = self.execution_options.binary_path
+        if not binary_path.exists():
             if kwave.PLATFORM == "darwin" and self.execution_options.is_gpu_simulation:
                 raise ValueError(
-                    "GPU simulations are currently not supported on MacOS. Try running the simulation on CPU by setting is_gpu_simulation=False."
-                ) from e
-            else:
-                raise e
+                    "GPU simulations are currently not supported on MacOS. "
+                    "Try running the simulation on CPU by setting is_gpu_simulation=False."
+                )
+            raise FileNotFoundError(f"Binary not found at {binary_path}")
+        binary_path.chmod(binary_path.stat().st_mode | stat.S_IEXEC)
 
     def run_simulation(self, input_filename: str, output_filename: str, options: str):
         command = [str(self.execution_options.binary_path), "-i", input_filename, "-o", output_filename, options]

tests/test_executor.py

@@ -193,6 +193,7 @@ def test_executor_gpu_cuda_failure_darwin(self):
         # Configure the mock path object
         mock_binary_path = MagicMock(spec=Path)
         mock_binary_path.chmod.side_effect = FileNotFoundError
+        mock_binary_path.exists.return_value = False
 
         # Mock the execution options to use the mocked path
         mock_execution_options = MagicMock()