Fix handling of missing command by dotenv run

Changes for users:

- (BREAKING) `dotenv run` exits with status code 1 instead of 2 if the command
  provided as argument can't be found.
- `dotenv run` prints a friendly error message instead of a stack trace
  if the command provided as argument can't be found.

Notes:

The existing test case was wrongly testing for that situation: it was
actually observing a "missing env file" error, not a "command not found
error". I thus added an appropriate test case for full coverage.

Author: bbc2

src/dotenv/cli.py

@@ -210,9 +210,18 @@ def run_command(command: List[str], env: Dict[str, str]) -> None:
     if sys.platform == "win32":
         # execvpe on Windows returns control immediately
         # rather than once the command has finished.
-        p = Popen(command, universal_newlines=True, bufsize=0, shell=False, env=cmd_env)
+        try:
+            p = Popen(command, universal_newlines=True, bufsize=0, shell=False, env=cmd_env)
+        except FileNotFoundError:
+            print(f"Command not found: {command[0]}", file=sys.stderr)
+            sys.exit(1)
+
         _, _ = p.communicate()
 
         sys.exit(p.returncode)
     else:
-        os.execvpe(command[0], args=command, env=cmd_env)
+        try:
+            os.execvpe(command[0], args=command, env=cmd_env)
+        except FileNotFoundError:
+            print(f"Command not found: {command[0]}", file=sys.stderr)
+            sys.exit(1)

tests/test_cli.py

@@ -246,7 +246,18 @@ def test_run_without_cmd(cli):
     assert "Invalid value for '-f'" in result.output
 
 
-def test_run_with_invalid_cmd(cli):
+def test_run_with_invalid_cmd(cli, dotenv_path):
+    result = cli.invoke(dotenv_cli, ["--file", dotenv_path, "run", "i_do_not_exist"])
+
+    assert result.exit_code == 1
+    assert "Command not found: i_do_not_exist" in result.output
+
+
+def test_run_with_env_missing_and_invalid_cmd(cli):
+    """
+    Check that an .env file missing takes precedence over a command not found error.
+    """
+
     result = cli.invoke(dotenv_cli, ["run", "i_do_not_exist"])
 
     assert result.exit_code == 2