fest: return subprocess command completed process object

src/whispr/utils/process.py

@@ -5,7 +5,7 @@
 from whispr.logging import logger
 
 
-def execute_command(command: tuple, no_env: bool, secrets: dict):
+def execute_command(command: tuple, no_env: bool, secrets: dict) -> subprocess.CompletedProcess[bytes]:
     """Executes a Unix/Windows command.
     Arg: `no_env` decides whether secrets are passed vai environment or K:V pairs in command arguments.
     """
@@ -23,6 +23,7 @@ def execute_command(command: tuple, no_env: bool, secrets: dict):
             os.environ.update(secrets)
 
         sp = subprocess.run(usr_command, env=os.environ, shell=False, check=True)
+        return sp
     except subprocess.CalledProcessError as e:
         logger.error(
             f"Encountered a problem while running command: '{command[0]}'. Aborting."

tests/test_process_utils.py

@@ -34,14 +34,24 @@ def test_execute_command_with_env(
         self, mock_env_update, mock_subprocess_run, mock_logger
     ):
         """Test execute_command with `no_env=False`, passing secrets via environment variables."""
-        execute_command(self.command, no_env=False, secrets=self.secrets)
+        completed_process = subprocess.CompletedProcess(
+            args=["echo", "Hello"], returncode=0, stdout=b"Hello\n", stderr=b""
+        )
+        mock_subprocess_run.return_value = completed_process
+
+        result = execute_command(self.command, no_env=False, secrets=self.secrets)
 
         mock_env_update.assert_called_once_with(self.secrets)
         expected_command = ["echo", "Hello"]
         mock_subprocess_run.assert_called_once_with(
             expected_command, env=os.environ, shell=False, check=True
         )
 
+        self.assertIsInstance(result, subprocess.CompletedProcess)
+        self.assertEqual(
+            type(result.stdout), bytes
+        )  # Additional sanity check on stdout type
+
     @patch("whispr.utils.process.logger", new_callable=lambda: MagicMock())
     @patch("subprocess.run", side_effect=subprocess.CalledProcessError(1, "test"))
     def test_execute_command_called_process_error(