@smoothdeploy/playwright 1.60.1 → 1.61.0

package/lib/index.js

@@ -402,16 +402,6 @@ const playwrightFixtures = {
         }
       } : {};
       const context = await browser.newContext({ ...videoOptions, ...options });
-      if (process.env.PW_CLOCK === "frozen") {
-        await context._wrapApiCall(async () => {
-          await context.clock.install({ time: 0 });
-          await context.clock.pauseAt(1e3);
-        }, { internal: true });
-      } else if (process.env.PW_CLOCK === "realtime") {
-        await context._wrapApiCall(async () => {
-          await context.clock.install({ time: 0 });
-        }, { internal: true });
-      }
       let closed = false;
       const close = async () => {
         if (closed)
@@ -419,8 +409,7 @@ const playwrightFixtures = {
         closed = true;
         const closeReason = testInfo.status === "timedOut" ? "Test timeout of " + testInfo.timeout + "ms exceeded." : "Test ended.";
         await context.close({ reason: closeReason });
-        const testFailed = testInfo.status !== testInfo.expectedStatus;
-        const preserveVideo = captureVideo && (videoMode === "on" || testFailed && videoMode === "retain-on-failure" || videoMode === "on-first-retry" && testInfo.retry === 1);
+        const preserveVideo = captureVideo && shouldPreserveVideo(videoMode, testInfo);
         if (preserveVideo) {
           const { pagesWithVideo: pagesForVideo } = contexts.get(context);
           const videos = pagesForVideo.map((p) => p.video()).filter((video2) => !!video2);
@@ -490,7 +479,23 @@ function normalizeVideoMode(video) {
   return videoMode;
 }
 function shouldCaptureVideo(videoMode, testInfo) {
-  return videoMode === "on" || videoMode === "retain-on-failure" || videoMode === "on-first-retry" && testInfo.retry === 1;
+  return videoMode === "on" || videoMode === "retain-on-failure" || videoMode === "retain-on-failure-and-retries" || videoMode === "on-first-retry" && testInfo.retry === 1 || videoMode === "on-all-retries" && testInfo.retry > 0 || videoMode === "retain-on-first-failure" && testInfo.retry === 0;
+}
+function shouldPreserveVideo(videoMode, testInfo) {
+  const testFailed = testInfo.status !== testInfo.expectedStatus;
+  switch (videoMode) {
+    case "on":
+    case "on-first-retry":
+    case "on-all-retries":
+      return true;
+    case "retain-on-failure":
+    case "retain-on-first-failure":
+      return testFailed;
+    case "retain-on-failure-and-retries":
+      return testFailed || testInfo.retry > 0;
+    default:
+      return false;
+  }
 }
 function normalizeScreenshotMode(screenshot) {
   if (!screenshot)

package/lib/loader/loaderProcessEntry.js

@@ -24,7 +24,7 @@ var LoaderMain = class extends import_common.ProcessRunner {
     return { fileSuite: fileSuite._deepSerialize(), testErrors };
   }
   async getCompilationCacheFromLoader() {
-    await import_common.esm.incorporateCompilationCache();
+    await import_common.transform.incorporateCompilationCache();
     return import_common.cc.serializeCompilationCache();
   }
 };

package/lib/matchers/expect.js

@@ -11976,8 +11976,7 @@ var NonConfigProperties = [
   "fullPage",
   "mask",
   "maskColor",
-  "omitBackground",
-  "timeout"
+  "omitBackground"
 ];
 var SnapshotHelper = class {
   constructor(state, testInfo, matcherName, locator, anonymousSnapshotExtension, configOptions, nameOrOptions, optOptions) {
@@ -12596,6 +12595,7 @@ var import_fs2 = __toESM(require("fs"));
 var import_path2 = __toESM(require("path"));
 var { escapeTemplateString, isString: isString3 } = require("playwright-core/lib/coreBundle").iso;
 var { existsAsync } = require("playwright-core/lib/coreBundle").utils;
+var kImpossibleAriaMatch = `- none "Generating new baseline"`;
 async function toMatchAriaSnapshot(receiver, expectedParam, options = {}) {
   const matcherName = "toMatchAriaSnapshot";
   expectTypes(receiver, ["Page", "Locator"], matcherName);
@@ -12620,14 +12620,14 @@ async function toMatchAriaSnapshot(receiver, expectedParam, options = {}) {
     expected = await import_fs2.default.promises.readFile(expectedPath, "utf8").catch(() => "");
     timeout = expectedParam?.timeout ?? this.timeout;
   }
-  const generateMissingBaseline = updateSnapshots === "missing" && !expected;
-  if (generateMissingBaseline) {
-    if (this.isNot) {
-      const message2 = `Matchers using ".not" can't generate new baselines`;
-      return { pass: this.isNot, message: () => message2, name: "toMatchAriaSnapshot" };
-    } else {
-      expected = `- none "Generating new baseline"`;
-    }
+  const isMissingBaseline = updateSnapshots === "missing" && !expected;
+  if (isMissingBaseline && this.isNot) {
+    const message2 = `Matchers using ".not" can't generate new baselines`;
+    return { pass: this.isNot, message: () => message2, name: "toMatchAriaSnapshot" };
+  }
+  const generateBaseline = !this.isNot && (updateSnapshots === "all" || isMissingBaseline);
+  if (generateBaseline) {
+    expected = kImpossibleAriaMatch;
   }
   expected = unshift(expected);
   const globalChildren = expectConfig().toMatchAriaSnapshot?.children;
@@ -12668,12 +12668,12 @@ async function toMatchAriaSnapshot(receiver, expectedParam, options = {}) {
   if (errorMessage)
     return { pass: this.isNot, message, name: "toMatchAriaSnapshot", expected };
   if (!this.isNot) {
-    if (updateSnapshots === "all" || updateSnapshots === "changed" && pass === this.isNot || generateMissingBaseline) {
+    if (generateBaseline || updateSnapshots === "changed" && pass === this.isNot) {
       if (expectedPath) {
         await import_fs2.default.promises.mkdir(import_path2.default.dirname(expectedPath), { recursive: true });
         await import_fs2.default.promises.writeFile(expectedPath, typedReceived.regex, "utf8");
         const relativePath = import_path2.default.relative(process.cwd(), expectedPath);
-        if (updateSnapshots === "missing") {
+        if (isMissingBaseline) {
           const message3 = `A snapshot doesn't exist at ${relativePath}, writing actual.`;
           return { pass: true, message: () => "", name: "toMatchAriaSnapshot", softError: new Error(message3), shouldNotRetryTest: true };
         }
@@ -12684,7 +12684,7 @@ async function toMatchAriaSnapshot(receiver, expectedParam, options = {}) {
         const suggestedRebaseline = `\`
 ${escapeTemplateString(indent2(typedReceived.regex, "{indent}  "))}
 {indent}\``;
-        if (updateSnapshots === "missing") {
+        if (isMissingBaseline) {
           const message2 = "A snapshot is not provided, generating new baseline.";
           return { pass: true, message: () => "", name: "toMatchAriaSnapshot", suggestedRebaseline, softError: new Error(message2), shouldNotRetryTest: true };
         }
@@ -12818,9 +12818,11 @@ function createExpect(info) {
       newInfo.isSoft = configuration.soft;
     return createExpect(newInfo);
   };
-  expectFn.soft = (actual, messageOrOptions) => {
-    return createMatchers(actual, { ...info, isSoft: true }, messageOrOptions);
-  };
+  Object.defineProperty(expectFn, "soft", {
+    configurable: true,
+    enumerable: true,
+    get: () => info.isSoft ? expectFn : createExpect({ ...info, isSoft: true })
+  });
   expectFn.poll = (actual, messageOrOptions) => {
     const poll = isString4(messageOrOptions) ? {} : messageOrOptions || {};
     return createMatchers(actual, { ...info, poll: { timeout: poll.timeout, intervals: poll.intervals } }, messageOrOptions);

package/lib/matchers/expect.js.txt

@@ -1,5 +1,5 @@
 # packages/playwright/lib/matchers/expect.js
-# total: 513.7 KB
+# total: 513.8 KB
 
 ## Inlined (64)
    7.9 KB  node_modules/@babel/code-frame/lib/index.js
@@ -56,14 +56,14 @@
    7.6 KB  node_modules/stack-utils/index.js
    0.4 KB  node_modules/stack-utils/node_modules/escape-string-regexp/index.js
    6.8 KB  node_modules/to-regex-range/index.js
-  10.3 KB  packages/playwright/src/matchers/expect.ts
+  10.4 KB  packages/playwright/src/matchers/expect.ts
   45.2 KB  packages/playwright/src/matchers/expectLibrary.ts
    3.3 KB  packages/playwright/src/matchers/matcherHint.ts
   16.0 KB  packages/playwright/src/matchers/matchers.ts
    1.3 KB  packages/playwright/src/matchers/toBeTruthy.ts
    2.1 KB  packages/playwright/src/matchers/toEqual.ts
    2.4 KB  packages/playwright/src/matchers/toHaveURL.ts
-   5.6 KB  packages/playwright/src/matchers/toMatchAriaSnapshot.ts
+   5.7 KB  packages/playwright/src/matchers/toMatchAriaSnapshot.ts
   15.1 KB  packages/playwright/src/matchers/toMatchSnapshot.ts
    3.0 KB  packages/playwright/src/matchers/toMatchText.ts
 

package/lib/mcp/test/testContext.js

@@ -89,6 +89,7 @@ class TestContext {
   async _enqueue(fn) {
     const next = this._testOpQueue.then(fn);
     this._testOpQueue = next.then(() => {
+    }, () => {
     });
     return await next;
   }
@@ -199,7 +200,8 @@ class TestContext {
         releaseStdio();
       }
     };
-    const abortPromise = signal ? signalToPromise(signal).promise.then(() => "interrupted") : new Promise(() => {
+    const abort = signal ? signalToPromise(signal) : void 0;
+    const abortPromise = abort ? abort.promise.then(() => "interrupted") : new Promise(() => {
     });
     try {
       const reporter = new MCPListReporter({ configDir, screen, includeTestId: true });
@@ -225,6 +227,8 @@ class TestContext {
       testRunnerAndScreen.output.push(String(e));
       await cleanup();
       return { output: testRunnerAndScreen.output.join("\n"), status };
+    } finally {
+      abort?.dispose();
     }
     await cleanup();
     return { output: testRunnerAndScreen.output.join("\n"), status };

package/lib/program.js

@@ -46,8 +46,11 @@ function addTestCommand(program2) {
     return command;
   });
   command.action(async (args, opts) => {
+    const dashDashIndex = process.argv.indexOf("--");
+    const postDashCount = dashDashIndex >= 0 ? process.argv.length - 1 - dashDashIndex : 0;
+    const testFilters = args.slice(0, args.length - postDashCount);
     try {
-      await (0, import_testActions.runTests)(args, opts);
+      await (0, import_testActions.runTests)(testFilters, opts);
     } catch (e) {
       console.error(e);
       gracefullyProcessExitDoNotHang(1);
@@ -144,7 +147,7 @@ function addInitAgentsCommand(program2) {
   const command = program2.command("init-agents");
   command.description("Initialize repository agents");
   const option = command.createOption("--loop <loop>", "Agentic loop provider");
-  option.choices(["claude", "copilot", "opencode", "vscode", "vscode-legacy"]);
+  option.choices(["claude", "codex", "copilot", "opencode", "vscode", "vscode-legacy"]);
   command.addOption(option);
   command.option("-c, --config <file>", `Configuration file to find a project to use for seed test`);
   command.option("--project <project>", "Project to use for seed test");
@@ -157,6 +160,8 @@ function addInitAgentsCommand(program2) {
       await import_generateAgents.VSCodeGenerator.init(loadedConfig, opts.project);
     } else if (opts.loop === "claude") {
       await import_generateAgents.ClaudeGenerator.init(loadedConfig, opts.project, opts.prompts);
+    } else if (opts.loop === "codex") {
+      await import_generateAgents.CodexGenerator.init(loadedConfig, opts.project, opts.prompts);
     } else {
       await import_generateAgents.CopilotGenerator.init(loadedConfig, opts.project, opts.prompts);
       return;
@@ -174,10 +179,11 @@ const testOptions = [
   ["--fully-parallel", { description: `Run all tests in parallel (default: false)` }],
   ["--global-timeout <timeout>", { description: `Maximum time this test suite can run in milliseconds (default: unlimited)` }],
   ["-g, --grep <grep>", { description: `Only run tests matching this regular expression (default: ".*")` }],
-  ["--grep-invert <grep>", { description: `Only run tests that do not match this regular expression` }],
+  ["-G, --grep-invert <grep>", { description: `Only run tests that do not match this regular expression` }],
   ["--headed", { description: `Run tests in headed browsers (default: headless)` }],
   ["--ignore-snapshots", { description: `Ignore screenshot and snapshot expectations` }],
   ["--last-failed", { description: `Only re-run the failures` }],
+  ["--last-failed-file <file>", { description: `Override the default path for the last-run JSON file used with --last-failed (default: <outputDir>/.last-run.json). Same as PLAYWRIGHT_LAST_RUN_OUTPUT_FILE environment variable.` }],
   ["--list", { description: `Collect all the tests and report them, but do not run` }],
   ["--max-failures <N>", { description: `Stop after the first N failures` }],
   ["--no-deps", { description: `Do not run project dependencies` }],