@lvce-editor/test-with-playwright-worker 14.7.0 → 15.0.0

package/dist/workerMain.js

@@ -962,27 +962,39 @@ const runTest = async ({
   traceFocus
 }) => {
   const start = performance.now();
-  const url = getUrlFromTestFile(test, port, traceFocus);
-  await page.goto(url, {
-    waitUntil: 'networkidle'
-  });
-  const testOverlay = page.locator('#TestOverlay');
-  await expect(testOverlay).toBeVisible({
-    timeout
-  });
-  const text = await testOverlay.textContent();
-  const testOverlayState = await testOverlay.getAttribute('data-state');
-  // @ts-ignore
-  const testState = getTestState(testOverlayState);
-  const end = performance.now();
-  return {
+  try {
+    const url = getUrlFromTestFile(test, port, traceFocus);
+    await page.goto(url, {
+      waitUntil: 'networkidle'
+    });
+    const testOverlay = page.locator('#TestOverlay');
+    await expect(testOverlay).toBeVisible({
+      timeout
+    });
+    const text = await testOverlay.textContent();
+    const testOverlayState = await testOverlay.getAttribute('data-state');
     // @ts-ignore
-    ...testState,
-    end,
-    error: text,
-    name: test,
-    start
-  };
+    const testState = getTestState(testOverlayState);
+    const end = performance.now();
+    return {
+      // @ts-ignore
+      ...testState,
+      end,
+      error: text,
+      name: test,
+      start
+    };
+  } catch (error) {
+    const end = performance.now();
+    const message = error instanceof Error ? error.message : `${error}`;
+    return {
+      end,
+      error: message,
+      name: test,
+      start,
+      status: Fail
+    };
+  }
 };
 
 /**
@@ -1055,6 +1067,11 @@ const lockedPorts = {
 // and a new young set for locked ports are created.
 const releaseOldLockedPortsIntervalMs = 1000 * 15;
 
+// Keep `reserve` deliberately process-wide by port number.
+// It is meant to avoid in-process races, not to model every possible
+// IPv4/IPv6 or host-specific bind combination.
+const reservedPorts = new Set();
+
 // Lazily create timeout on first use
 let timeout;
 const getLocalHosts = () => {
@@ -1101,11 +1118,16 @@ const getAvailablePort = async (options, hosts) => {
   }
   return options.port;
 };
+const isLockedPort = port => lockedPorts.old.has(port) || lockedPorts.young.has(port) || reservedPorts.has(port);
 const portCheckSequence = function* (ports) {
   yield 0; // Fall back to 0 if anything else failed
 };
 async function getPorts(options) {
   let exclude = new Set();
+  const {
+    reserve,
+    ...netOptions
+  } = {};
   if (timeout === undefined) {
     timeout = setTimeout(() => {
       timeout = undefined;
@@ -1125,19 +1147,23 @@ async function getPorts(options) {
         continue;
       }
       let availablePort = await getAvailablePort({
-        ...options,
+        ...netOptions,
         port
       }, hosts); // eslint-disable-line no-await-in-loop
-      while (lockedPorts.old.has(availablePort) || lockedPorts.young.has(availablePort)) {
+      while (isLockedPort(availablePort)) {
         if (port !== 0) {
           throw new Locked(port);
         }
         availablePort = await getAvailablePort({
-          ...options,
+          ...netOptions,
           port
         }, hosts); // eslint-disable-line no-await-in-loop
       }
-      lockedPorts.young.add(availablePort);
+      if (reserve) {
+        reservedPorts.add(availablePort);
+      } else {
+        lockedPorts.young.add(availablePort);
+      }
       return availablePort;
     } catch (error) {
       if (!['EADDRINUSE', 'EACCES'].includes(error.code) && !(error instanceof Locked)) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lvce-editor/test-with-playwright-worker",
-  "version": "14.7.0",
+  "version": "15.0.0",
   "description": "Worker package for test-with-playwright",
   "repository": {
     "type": "git",
@@ -12,7 +12,7 @@
   "type": "module",
   "main": "dist/workerMain.js",
   "dependencies": {
-    "@playwright/test": "1.58.2"
+    "@playwright/test": "1.59.0"
   },
   "engines": {
     "node": ">=24"