Refactored "runAndGetOutputWithoutErrors" into "waitForCleanOutout"

Author: kraenhansen

packages/e2e-tests/test/e2e-snapshot.spec.ts

@@ -2,7 +2,6 @@ import {
   skipIfApiStrict,
   startSharedTestServer,
 } from '../../../testing/integration-testing-hooks';
-import { TestShell } from './test-shell';
 import { expect } from 'chai';
 
 const setDifference = <T>(a: T[], b: T[]) => a.filter((e) => !b.includes(e));
@@ -57,7 +56,7 @@ describe('e2e snapshot support', function () {
       ] = (
         await Promise.all(
           argLists.map((args) =>
-            TestShell.runAndGetOutputWithoutErrors({ args })
+            this.startTestShell({ args }).waitForCleanOutput()
           )
         )
       ).map((output) =>

packages/e2e-tests/test/test-shell.ts

@@ -50,7 +50,7 @@ export class TestShell {
   /**
    * Starts a test shell.
    *
-   * Beware that the caller is responsible for calling {@link TestShell.kill} (and potentially {@link TestShell.waitForExit}).
+   * Beware that the caller is responsible for calling {@link kill} (and potentially {@link waitForExit}).
    *
    * Consider calling the `startTestShell` function on a {@link Mocha.Context} instead, as that manages the lifetime the shell
    * and ensures it gets killed eventually.
@@ -108,15 +108,6 @@ export class TestShell {
     return shell;
   }
 
-  static async runAndGetOutputWithoutErrors(
-    options: TestShellOptions
-  ): Promise<string> {
-    const shell = this.start(options);
-    await shell.waitForExit();
-    shell.assertNoErrors();
-    return shell.output;
-  }
-
   debugInformation() {
     return {
       pid: this.process.pid,
@@ -232,6 +223,15 @@ export class TestShell {
     return this._onClose;
   }
 
+  /**
+   * Waits for the shell to exit, asserts no errors and returns the output.
+   */
+  async waitForCleanOutput(): Promise<string> {
+    await this.waitForExit();
+    this.assertNoErrors();
+    return this.output;
+  }
+
   async waitForPromptOrExit(): Promise<TestShellStartupResult> {
     return Promise.race([
       this.waitForPrompt().then(() => ({ state: 'prompt' } as const)),