Fix spawn error message logic

agrawal-d · Aug 13, 2020 · 65a1560 · 65a1560
1 parent 0e41f0f
commit 65a1560
Showing 1 changed file with 18 additions and 16 deletions.
diff --git a/src/runs/executions.ts b/src/runs/executions.ts
@@ -36,25 +36,27 @@ export const runTestCase = (
     }, getTimeOutPref());
 
     // Start the binary or the interpreter.
-    try {
-        switch (language.name) {
-            case 'python': {
-                process = spawn(
-                    language.compiler, // 'python3' or 'python' TBD
-                    [binPath, ...language.args],
-                    spawnOpts,
-                );
-                break;
-            }
-            default: {
-                process = spawn(binPath, spawnOpts);
-            }
+    switch (language.name) {
+        case 'python': {
+            process = spawn(
+                language.compiler, // 'python3' or 'python' TBD
+                [binPath, ...language.args],
+                spawnOpts,
+            );
+            break;
+        }
+        default: {
+            process = spawn(binPath, spawnOpts);
         }
-    } catch (err) {
-        vscode.window.showErrorMessage('Could not spawn testcase process.\n');
-        throw err;
     }
 
+    process.on('error', (err) => {
+        console.error(err);
+        vscode.window.showErrorMessage(
+            `Could not launch testcase process. Is '${language.compiler}' in your PATH?`,
+        );
+    });
+
     const begin = Date.now();
     const ret: Promise<Run> = new Promise((resolve) => {
         runningBinaries.push(process);