Correctly handle files with a single test

Also, notify the user of empty files.

Author: belcar-s

src/commands/tests.sidebar.ts

@@ -66,6 +66,7 @@ export function registerRunAll(testsDataProvider: TestsDataProvider) {
           "The tests' outcomes are unknown.";
         dissatisfactoryOutcomeNotificationRequest.body =
           "This feature is fragile, and relies on the extension's compatibility with your particular Deno version. If you're using an outdated version of Deno, or an outdated version of the extension, try updating.";
+        dissatisfactoryOutcomeNotificationRequest.actions = ["OK"];
         nova.notifications.add(dissatisfactoryOutcomeNotificationRequest);
       } else {
         // shown as a Nova message regardless; doesn't produce a crash

src/tests/TestsDataProvider.ts

@@ -42,10 +42,10 @@ class Test implements Element {
 export class UnexpectedLogError extends Error {}
 export class TestFile implements Element {
   path: string;
-  children: Test[];
+  children: Element[];
   shouldDisambiguate: boolean;
 
-  constructor(path: string, children: [], shouldDisambiguate: boolean) {
+  constructor(path: string, children: Element[], shouldDisambiguate: boolean) {
     this.path = path;
     this.children = children;
     this.shouldDisambiguate = shouldDisambiguate;
@@ -165,29 +165,35 @@ export default class TestsDataProvider implements TreeDataProvider<Element> {
     const denoProcess = new Process("/usr/bin/env", options);
 
     // This must be very fragile!
-    const INTRODUCTION_REGEXP = /running \d+ tests from ([^]+)/;
+    const INTRODUCTION_REGEXP = /running (\d+) tests? from ([^]+)/;
     const TEST_REGEXP = /([^]+) \.\.\. (FAILED|ok) \(\d+[A-Za-z]+\)$/;
+    const CONTROL_REGEXP =
+      // deno-lint-ignore no-control-regex
+      /[\u001b\u009b][[\]#;?()]*(?:(?:(?:[^\W_]*;?[^\W_]*)\u0007)|(?:(?:[0-9]{1,4}(;[0-9]{0,4})*)?[~0-9=<>cf-nqrtyA-PRZ]))/g;
+
     const output: TestFile[] = [];
+
+    let loggingError: UnexpectedLogError | null = null;
     denoProcess.onStdout((line) => {
       // remove newline
       line = line.slice(0, -1);
+      console.log(line);
       // remove control (?) characters that make output colorful
       line = line.replace(
-        // deno-lint-ignore no-control-regex
-        /[\u001b\u009b][[\]#;?()]*(?:(?:(?:[^\W_]*;?[^\W_]*)\u0007)|(?:(?:[0-9]{1,4}(;[0-9]{0,4})*)?[~0-9=<>cf-nqrtyA-PRZ]))/g,
+        CONTROL_REGEXP,
         "",
       );
       const introduction = line.match(INTRODUCTION_REGEXP);
       const test = line.match(TEST_REGEXP);
 
       if (introduction) {
-        const [, relativePath] = introduction;
+        const [, count, relativePath] = introduction;
         output.push(
           new TestFile(
             nova.path.normalize(
               nova.path.join(nova.workspace.path!, relativePath),
             ),
-            [],
+            count == "0" ? [new Header("No tests")] : [],
             false, // TODO
           ),
         );
@@ -197,27 +203,26 @@ export default class TestsDataProvider implements TreeDataProvider<Element> {
 
         const currentFile = output.at(-1);
         if (!currentFile) {
-          throw new UnexpectedLogError("Unexpected logging. Oops.");
+          if (!loggingError) {
+            loggingError = new UnexpectedLogError(
+              "Unexpected logging. Oops.",
+            );
+          }
+          return;
         }
         currentFile.children.push(new Test(name, passed));
       }
     });
 
-    const onExit = new Promise((resolve) => {
+    const onExit = new Promise((resolve, reject) => {
       denoProcess.onDidExit(() => {
         // TODO: explore the dangers regarding tests that take long to execute
-        console.log(
-          JSON.stringify(
-            output.map((file) => ({
-              path: file.path,
-              tests: file.children.map((test) => test.passed),
-            })),
-            undefined,
-            "\t",
-          ),
-        );
         this.files = output;
-        resolve(output);
+        if (loggingError) {
+          reject(loggingError);
+        } else {
+          resolve(output);
+        }
       });
     });