test: fast fail on uncaught exception to produce better error message

It would be better to fail the test on uncaught exception compared to
timeout.

Author: kateinoigakukun

packages/npm-packages/ruby-wasm-wasi/test-e2e/integrations/browser-script.spec.ts

@@ -1,13 +1,14 @@
 import { test, expect, Page } from "@playwright/test";
 
-import { setupDebugLog, setupProxy, waitForRubyVM } from "../support";
+import { setupDebugLog, setupProxy, setupUncaughtExceptionRejection, expectUncaughtException } from "../support";
 
 if (!process.env.RUBY_NPM_PACKAGE_ROOT) {
   test.skip("skip", () => {});
 } else {
-  test.beforeEach(async ({ context }) => {
+  test.beforeEach(async ({ context, page }) => {
     setupDebugLog(context);
     setupProxy(context);
+    setupUncaughtExceptionRejection(page);
   });
 
   const resolveBinding = async (page: Page, name: string) => {
@@ -35,6 +36,8 @@ if (!process.env.RUBY_NPM_PACKAGE_ROOT) {
     });
 
     test("JS::Object#await throws error on default attr", async ({ page }) => {
+      expectUncaughtException(page);
+
       await page.setContent(`
       <script src="https://cdn.jsdelivr.net/npm/@ruby/head-wasm-wasi@latest/dist/browser.script.iife.js"></script>
       <script type="text/ruby">
@@ -60,4 +63,6 @@ if (!process.env.RUBY_NPM_PACKAGE_ROOT) {
       expect(await resolve()).toBe("ok");
     });
   });
+  test.describe('data-eval="async" crash', () => {
+  });
 }

packages/npm-packages/ruby-wasm-wasi/test-e2e/support.ts

@@ -28,3 +28,17 @@ export const setupProxy = (context: BrowserContext) => {
     });
   });
 };
+
+export const { setupUncaughtExceptionRejection, expectUncaughtException } = (() => {
+  const rejectUncaughtException = (err: Error) => {
+    expect(err).toEqual(undefined);
+  }
+  return {
+    setupUncaughtExceptionRejection: (page: Page) => {
+      page.on("pageerror", rejectUncaughtException);
+    },
+    expectUncaughtException: (page: Page) => {
+      page.off("pageerror", rejectUncaughtException);
+    },
+  }
+})()