browser-ava 2.3.38 → 2.3.40

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "browser-ava",
-  "version": "2.3.38",
+  "version": "2.3.40",
   "publishConfig": {
     "access": "public",
     "provenance": true
@@ -41,6 +41,7 @@
     "commander": "^14.0.1",
     "es-module-lexer": "^1.7.0",
     "globby": "^15.0.0",
+    "json-cyclic": "^1.0.2",
     "koa": "^3.0.1",
     "koa-static": "^5.0.0",
     "playwright": "^1.56.0",

package/src/browser/runtime.mjs

@@ -1,3 +1,4 @@
+import { decycle } from "./node_modules/json-cyclic/dist/index.esm.js";
 import { testModules, test } from "./ava.mjs";
 import {
   calculateSummary,
@@ -18,11 +19,9 @@ Error.prototype.toJSON = primitiveRoString;
 
 function allErrorProperties(key, value) {
   if (value instanceof Error) {
-      const error = {};
-      Object.
-        getOwnPropertyNames(value).
-        forEach((key) => error[key] = value[key]);
-      return error;
+    const error = {};
+    Object.getOwnPropertyNames(value).forEach(key => (error[key] = value[key]));
+    return error;
   }
   return value;
 }
@@ -35,7 +34,7 @@ for (const action of ["log", "info", "error"]) {
 
   console[action] = (...data) => {
     if (ws) {
-      ws.send(JSON.stringify({ action, data }, allErrorProperties));
+      ws.send(JSON.stringify(decycle({ action, data }), allErrorProperties));
     }
     former(...data);
   };
@@ -96,7 +95,9 @@ async function displayTests() {
             .map(a => (a.title || "") + " " + (a.message || ""))
             .join(" ")
         : ""
-    }</span>${t.message ? t.message : ""}${t.stack ? "<br>" + t.stack : ""}</li>`;
+    }</span>${t.message ? t.message : ""}${
+      t.stack ? "<br>" + t.stack : ""
+    }</li>`;
   }
 
   function renderModule(tm) {
@@ -200,7 +201,12 @@ async function runTest(parent, tm, testInstance) {
       testInstance.message = e;
       testInstance.stack = e.stack;
     } finally {
-      ws.send(JSON.stringify({ action: "update", data: testInstance }, allErrorProperties));
+      ws.send(
+        JSON.stringify(
+          decycle({ action: "update", data: testInstance }),
+          allErrorProperties
+        )
+      );
     }
   }
 }
@@ -240,14 +246,21 @@ async function runTestModule(tm) {
 }
 
 async function runTestModules() {
-  try {
-    await Promise.all(testModules.map(tm => runTestModule(tm)));
-  }
-  catch(error) {
-    console.error(error);
+  const results = await Promise.allSettled(
+    testModules.map(tm => runTestModule(tm))
+  );
+
+  let i = 0;
+  for (const result of results) {
+    if (result.status !== "fulfilled") {
+      console.error(testModules[i].url, result.reason);
+      i++;
+    }
   }
 
-  ws.send(JSON.stringify({ action: "result", data: testModules }, allErrorProperties));
+  ws.send(
+    JSON.stringify(decycle({ action: "result", data: testModules }), allErrorProperties)
+  );
 
   displayTests();
 }

package/src/browser-ava-cli.mjs

@@ -12,6 +12,7 @@ import { calculateSummary, summaryMessages } from "./browser/util.mjs";
 import { resolveImport, utf8EncodingOptions } from "./resolver.mjs";
 import { globby } from "globby";
 import chalk from "chalk";
+import { encycle } from "json-cyclic";
 import pkg from "../package.json" with { type: "json" };
 
 const knownBrowsers = {
@@ -106,7 +107,7 @@ program
 
     wss.on("connection", ws => {
       ws.on("message", async content => {
-        const { action, data } = JSON.parse(content);
+        const { action, data } = encycle(JSON.parse(content));
         switch (action) {
           case "info":
             console.info(...data);
@@ -226,6 +227,7 @@ async function createServer(tests, options) {
   app.use(Cors({ origin: "*" }));
 
   app.use(Static(new URL("./browser", import.meta.url).pathname));
+  app.use(Static(new URL("..", import.meta.url).pathname));
 
   app.on("error", console.error);