Enable tests to be run individually

Author: belcar-s

src/commands/tests.sidebar.ts

@@ -6,6 +6,7 @@ import {
   wrapCommand,
 } from "../nova_utils.ts";
 import TestsDataProvider, {
+  TestFile,
   UnexpectedLogError,
 } from "../tests/TestsDataProvider.ts";
 
@@ -152,8 +153,11 @@ export function registerRun(testsDataProvider: TestsDataProvider) {
     wrapCommand(run),
   );
 
-  function run() {
-    testsDataProvider.runTests();
+  async function run() {
+    // This command is only available when `TestFile`s, exclusively, are selected.
+    const selected = testsDataProvider.treeView.selection as TestFile[];
+    await testsDataProvider.runTests(selected.map((file) => file.path));
+    testsDataProvider.treeView.reload();
   }
 }
 

src/tests/TestsDataProvider.ts

@@ -171,6 +171,7 @@ export default class TestsDataProvider implements TreeDataProvider<Element> {
       throw new Error("This function requires a workspace path.");
     }
 
+    const ranAll = !tests;
     const paths = tests ?? this.files.map((file) => file.path);
     const args = ["test", "-A"];
 
@@ -246,7 +247,6 @@ export default class TestsDataProvider implements TreeDataProvider<Element> {
 
     const onExit = new Promise<TestFile[]>((resolve, reject) => {
       denoProcess.onDidExit(() => {
-        // TODO: explore the dangers regarding tests that take long to execute
         if (loggingError) {
           reject(loggingError);
         } else {
@@ -261,25 +261,28 @@ export default class TestsDataProvider implements TreeDataProvider<Element> {
             }
           }
 
-          const paths = output.map((file) => file.path);
-          const missingFiles = this.files.filter(
-            (file) => !paths.includes(file.path),
-          );
-          for (
-            const file of missingFiles
-          ) {
-            file.children = [new Header("Failed to run")];
-          }
+          if (ranAll) {
+            const paths = output.map((file) => file.path);
+            const missingFiles = this.files.filter(
+              (file) => !paths.includes(file.path),
+            );
+            for (
+              const file of missingFiles
+            ) {
+              file.children = [new Header("Failed to run")];
+            }
 
-          if (missingFiles.length) {
-            const configurationErrorNotificationRequest =
-              new NotificationRequest(
-                "co.gwil.deno.notifications.unexpectedEmptiness",
-              );
-            configurationErrorNotificationRequest.title = "Check the console.";
-            configurationErrorNotificationRequest.body =
-              "Deno may be failing to run some tests. Check the extension console for logging.";
-            nova.notifications.add(configurationErrorNotificationRequest);
+            if (missingFiles.length) {
+              const configurationErrorNotificationRequest =
+                new NotificationRequest(
+                  "co.gwil.deno.notifications.unexpectedEmptiness",
+                );
+              configurationErrorNotificationRequest.title =
+                "Check the console.";
+              configurationErrorNotificationRequest.body =
+                "Deno may be failing to run some tests. Check the extension console for logging.";
+              nova.notifications.add(configurationErrorNotificationRequest);
+            }
           }
 
           resolve(output);