fix: test logging unhandled rejection from teardown

Author: AriPerkkio

src/fixtures/index.ts

@@ -24,6 +24,14 @@ export const test = base.extend<{
 
     await use(webcontainer);
 
+    addEventListener("unhandledrejection", (event) => {
+      if (event.reason instanceof Error) {
+        if (event.reason.message === "Process aborted") {
+          // TODO: Remove, debugging
+          console.log("Webcontainer teardown error", event.reason);
+        }
+      }
+    });
     await webcontainer.teardown();
   },
 });

src/fixtures/webcontainer.ts

@@ -7,6 +7,7 @@ import {
 export class WebContainer {
   private _instancePromise?: WebContainerApi;
   private _isReady: Promise<void>;
+  private _onExit: (() => Promise<unknown>)[] = [];
 
   constructor() {
     this._isReady = WebContainerApi.boot({}).then((instance) => {
@@ -47,6 +48,8 @@ export class WebContainer {
   }
 
   async teardown() {
+    await Promise.all(this._onExit.map((fn) => fn()));
+
     this._instance.teardown();
     this._instancePromise = undefined;
   }
@@ -64,6 +67,17 @@ export class WebContainer {
       }),
     );
 
+    // make sure any long-living processes are terminated before teardown, e.g. "npm run dev" commands
+    this._onExit.push(() => {
+      // @ts-ignore -- internal
+      if (process._process != null) {
+        console.log("Terminating process of", { command, args });
+        process.kill();
+      }
+
+      return process.exit;
+    });
+
     await process.exit;
 
     return output.trim();