playwright 1.55.0-alpha-2025-08-06 → 1.55.0-alpha-2025-08-07

package/lib/index.js

@@ -103,16 +103,12 @@ const playwrightFixtures = {
         }
       });
       await use(browser2);
-      await browser2._wrapApiCall(async () => {
-        await browser2.close({ reason: "Test ended." });
-      }, { internal: true });
+      await browser2.close({ reason: "Test ended." });
       return;
     }
     const browser = await playwright[browserName].launch();
     await use(browser);
-    await browser._wrapApiCall(async () => {
-      await browser.close({ reason: "Test ended." });
-    }, { internal: true });
+    await browser.close({ reason: "Test ended." });
   }, { scope: "worker", timeout: 0 }],
   acceptDownloads: [({ contextOptions }, use) => use(contextOptions.acceptDownloads ?? true), { option: true }],
   bypassCSP: [({ contextOptions }, use) => use(contextOptions.bypassCSP ?? false), { option: true }],
@@ -310,6 +306,7 @@ const playwrightFixtures = {
     const videoMode = normalizeVideoMode(video);
     const captureVideo = shouldCaptureVideo(videoMode, testInfo) && !_reuseContext;
     const contexts = /* @__PURE__ */ new Map();
+    let counter = 0;
     await use(async (options) => {
       const hook = testInfoImpl._currentHookType();
       if (hook === "beforeAll" || hook === "afterAll") {
@@ -326,10 +323,6 @@ const playwrightFixtures = {
         }
       } : {};
       const context = await browser.newContext({ ...videoOptions, ...options });
-      const contextData = { pagesWithVideo: [] };
-      contexts.set(context, contextData);
-      if (captureVideo)
-        context.on("page", (page) => contextData.pagesWithVideo.push(page));
       if (process.env.PW_CLOCK === "frozen") {
         await context._wrapApiCall(async () => {
           await context.clock.install({ time: 0 });
@@ -340,30 +333,36 @@ const playwrightFixtures = {
           await context.clock.install({ time: 0 });
         }, { internal: true });
       }
-      return context;
-    });
-    let counter = 0;
-    const closeReason = testInfo.status === "timedOut" ? "Test timeout of " + testInfo.timeout + "ms exceeded." : "Test ended.";
-    await Promise.all([...contexts.keys()].map(async (context) => {
-      await context._wrapApiCall(async () => {
+      let closed = false;
+      const close = async () => {
+        if (closed)
+          return;
+        closed = true;
+        const closeReason = testInfo.status === "timedOut" ? "Test timeout of " + testInfo.timeout + "ms exceeded." : "Test ended.";
         await context.close({ reason: closeReason });
-      }, { internal: true });
-      const testFailed = testInfo.status !== testInfo.expectedStatus;
-      const preserveVideo = captureVideo && (videoMode === "on" || testFailed && videoMode === "retain-on-failure" || videoMode === "on-first-retry" && testInfo.retry === 1);
-      if (preserveVideo) {
-        const { pagesWithVideo: pagesForVideo } = contexts.get(context);
-        const videos = pagesForVideo.map((p) => p.video()).filter(Boolean);
-        await Promise.all(videos.map(async (v) => {
-          try {
-            const savedPath = testInfo.outputPath(`video${counter ? "-" + counter : ""}.webm`);
-            ++counter;
-            await v.saveAs(savedPath);
-            testInfo.attachments.push({ name: "video", path: savedPath, contentType: "video/webm" });
-          } catch (e) {
-          }
-        }));
-      }
-    }));
+        const testFailed = testInfo.status !== testInfo.expectedStatus;
+        const preserveVideo = captureVideo && (videoMode === "on" || testFailed && videoMode === "retain-on-failure" || videoMode === "on-first-retry" && testInfo.retry === 1);
+        if (preserveVideo) {
+          const { pagesWithVideo: pagesForVideo } = contexts.get(context);
+          const videos = pagesForVideo.map((p) => p.video()).filter((video2) => !!video2);
+          await Promise.all(videos.map(async (v) => {
+            try {
+              const savedPath = testInfo.outputPath(`video${counter ? "-" + counter : ""}.webm`);
+              ++counter;
+              await v.saveAs(savedPath);
+              testInfo.attachments.push({ name: "video", path: savedPath, contentType: "video/webm" });
+            } catch (e) {
+            }
+          }));
+        }
+      };
+      const contextData = { close, pagesWithVideo: [] };
+      if (captureVideo)
+        context.on("page", (page) => contextData.pagesWithVideo.push(page));
+      contexts.set(context, contextData);
+      return { context, close };
+    });
+    await Promise.all([...contexts.values()].map((data) => data.close()));
   }, { scope: "test", title: "context", box: true }],
   _optionContextReuseMode: ["none", { scope: "worker", option: true }],
   _optionConnectOptions: [void 0, { scope: "worker", option: true }],
@@ -374,17 +373,19 @@ const playwrightFixtures = {
     const reuse = mode === "when-possible" && normalizeVideoMode(video) === "off";
     await use(reuse);
   }, { scope: "worker", title: "context", box: true }],
-  context: async ({ playwright, browser, _reuseContext, _contextFactory }, use, testInfo) => {
-    attachConnectedHeaderIfNeeded(testInfo, browser);
+  context: async ({ browser, _reuseContext, _contextFactory }, use, testInfo) => {
+    const browserImpl = browser;
+    attachConnectedHeaderIfNeeded(testInfo, browserImpl);
     if (!_reuseContext) {
-      await use(await _contextFactory());
+      const { context: context2, close } = await _contextFactory();
+      await use(context2);
+      await close();
       return;
     }
-    const defaultContextOptions = playwright.chromium._defaultContextOptions;
-    const context = await browser._newContextForReuse(defaultContextOptions);
+    const context = await browserImpl._wrapApiCall(() => browserImpl._newContextForReuse(), { internal: true });
     await use(context);
     const closeReason = testInfo.status === "timedOut" ? "Test timeout of " + testInfo.timeout + "ms exceeded." : "Test ended.";
-    await browser._disconnectFromReusedContext(closeReason);
+    await browserImpl._wrapApiCall(() => browserImpl._disconnectFromReusedContext(closeReason), { internal: true });
   },
   page: async ({ context, _reuseContext }, use) => {
     if (!_reuseContext) {
@@ -548,7 +549,9 @@ class ArtifactsRecorder {
     const screenshotOptions = typeof screenshot === "string" ? void 0 : screenshot;
     this._startedCollectingArtifacts = Symbol("startedCollectingArtifacts");
     this._screenshotRecorder = new SnapshotRecorder(this, normalizeScreenshotMode(screenshot), "screenshot", "image/png", ".png", async (page, path2) => {
-      await page.screenshot({ ...screenshotOptions, timeout: 5e3, path: path2, caret: "initial" });
+      await page._wrapApiCall(async () => {
+        await page.screenshot({ ...screenshotOptions, timeout: 5e3, path: path2, caret: "initial" });
+      }, { internal: true });
     });
   }
   async willStartTest(testInfo) {
@@ -560,10 +563,10 @@ class ArtifactsRecorder {
     await Promise.all(existingApiRequests.map((c) => this.didCreateRequestContext(c)));
   }
   async didCreateBrowserContext(context) {
-    await this._startTraceChunkOnContextCreation(context.tracing);
+    await this._startTraceChunkOnContextCreation(context, context.tracing);
   }
   async willCloseBrowserContext(context) {
-    await this._stopTracing(context.tracing);
+    await this._stopTracing(context, context.tracing);
     await this._screenshotRecorder.captureTemporary(context);
     await this._takePageSnapshot(context);
   }
@@ -578,17 +581,17 @@ class ArtifactsRecorder {
     if (!page)
       return;
     try {
-      this._pageSnapshot = await page._snapshotForAI({ timeout: 5e3 });
+      await page._wrapApiCall(async () => {
+        this._pageSnapshot = await page._snapshotForAI({ timeout: 5e3 });
+      }, { internal: true });
     } catch {
     }
   }
   async didCreateRequestContext(context) {
-    const tracing2 = context._tracing;
-    await this._startTraceChunkOnContextCreation(tracing2);
+    await this._startTraceChunkOnContextCreation(context, context._tracing);
   }
   async willCloseRequestContext(context) {
-    const tracing2 = context._tracing;
-    await this._stopTracing(tracing2);
+    await this._stopTracing(context, context._tracing);
   }
   async didFinishTestFunction() {
     await this._screenshotRecorder.maybeCapture();
@@ -598,10 +601,9 @@ class ArtifactsRecorder {
     const leftoverContexts = this._playwright._allContexts();
     const leftoverApiRequests = Array.from(this._playwright.request._contexts);
     await Promise.all(leftoverContexts.map(async (context2) => {
-      await this._stopTracing(context2.tracing);
+      await this._stopTracing(context2, context2.tracing);
     }).concat(leftoverApiRequests.map(async (context2) => {
-      const tracing2 = context2._tracing;
-      await this._stopTracing(tracing2);
+      await this._stopTracing(context2, context2._tracing);
     })));
     await this._screenshotRecorder.persistTemporary();
     const context = leftoverContexts[0];
@@ -624,30 +626,34 @@ class ArtifactsRecorder {
       }, void 0);
     }
   }
-  async _startTraceChunkOnContextCreation(tracing2) {
-    const options = this._testInfo._tracing.traceOptions();
-    if (options) {
-      const title = this._testInfo._tracing.traceTitle();
-      const name = this._testInfo._tracing.generateNextTraceRecordingName();
-      if (!tracing2[kTracingStarted]) {
-        await tracing2.start({ ...options, title, name });
-        tracing2[kTracingStarted] = true;
+  async _startTraceChunkOnContextCreation(channelOwner, tracing2) {
+    await channelOwner._wrapApiCall(async () => {
+      const options = this._testInfo._tracing.traceOptions();
+      if (options) {
+        const title = this._testInfo._tracing.traceTitle();
+        const name = this._testInfo._tracing.generateNextTraceRecordingName();
+        if (!tracing2[kTracingStarted]) {
+          await tracing2.start({ ...options, title, name });
+          tracing2[kTracingStarted] = true;
+        } else {
+          await tracing2.startChunk({ title, name });
+        }
       } else {
-        await tracing2.startChunk({ title, name });
-      }
-    } else {
-      if (tracing2[kTracingStarted]) {
-        tracing2[kTracingStarted] = false;
-        await tracing2.stop();
+        if (tracing2[kTracingStarted]) {
+          tracing2[kTracingStarted] = false;
+          await tracing2.stop();
+        }
       }
-    }
+    }, { internal: true });
   }
-  async _stopTracing(tracing2) {
-    if (tracing2[this._startedCollectingArtifacts])
-      return;
-    tracing2[this._startedCollectingArtifacts] = true;
-    if (this._testInfo._tracing.traceOptions() && tracing2[kTracingStarted])
-      await tracing2.stopChunk({ path: this._testInfo._tracing.maybeGenerateNextTraceRecordingPath() });
+  async _stopTracing(channelOwner, tracing2) {
+    await channelOwner._wrapApiCall(async () => {
+      if (tracing2[this._startedCollectingArtifacts])
+        return;
+      tracing2[this._startedCollectingArtifacts] = true;
+      if (this._testInfo._tracing.traceOptions() && tracing2[kTracingStarted])
+        await tracing2.stopChunk({ path: this._testInfo._tracing.maybeGenerateNextTraceRecordingPath() });
+    }, { internal: true });
   }
 }
 function renderTitle(type, method, params, title) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "playwright",
-  "version": "1.55.0-alpha-2025-08-06",
+  "version": "1.55.0-alpha-2025-08-07",
   "description": "A high-level API to automate web browsers",
   "repository": {
     "type": "git",
@@ -56,7 +56,7 @@
   },
   "license": "Apache-2.0",
   "dependencies": {
-    "playwright-core": "1.55.0-alpha-2025-08-06"
+    "playwright-core": "1.55.0-alpha-2025-08-07"
   },
   "optionalDependencies": {
     "fsevents": "2.3.2"