Remove "runAndGetOutputWithoutErrors"

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));
@@ -56,9 +55,12 @@ describe('e2e snapshot support', function () {
         atReplEvalTime,
       ] = (
         await Promise.all(
-          argLists.map((args) =>
-            TestShell.runAndGetOutputWithoutErrors({ args })
-          )
+          argLists.map(async (args) => {
+            const shell = this.startTestShell({ args });
+            await shell.waitForExit();
+            shell.assertNoErrors();
+            return shell.output;
+          })
         )
       ).map((output) =>
         (JSON.parse(output) as string[])

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,