#378 Get a nice error message when running diff-quality with a tool w…

…hich is not installed (#380) * #378 Add shell=True to check for program existence * Update command_runner.py Black is upset, looks like the change is simple enough * #378 Explicitly check for FileNotFound instead --------- Co-authored-by: Matt Bachmann <[email protected]>
Bachmann1234 · Dec 16, 2023 · d6ec6af · d6ec6af
1 parent c37cdbe
commit d6ec6af
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/diff_cover/command_runner.py b/diff_cover/command_runner.py
@@ -51,8 +51,13 @@ def run_command_for_code(command):
     """
     Returns command's exit code.
     """
-    process = subprocess.Popen(command, stdout=subprocess.PIPE, stderr=subprocess.PIPE)
-    process.communicate()
+    try:
+        process = subprocess.Popen(
+            command, stdout=subprocess.PIPE, stderr=subprocess.PIPE
+        )
+        process.communicate()
+    except FileNotFoundError:
+        return 1
     return process.returncode