feat(test-runner): support locks in test.describe.configure

[Devin Rousso (dcrousso)]

locks cannot be declared imperatively for the enclosing suite

add a lock option to test.describe.configure that accumulates suite locks

fixes #42782

[github-actions]

Test results for "tests 1"

8 flaky

⚠️ [chromium-library] › library/tracing.spec.ts:533 › should produce screencast frames fit `@chromium-ubuntu-22.04-arm-node20`
⚠️ [chromium-library] › library/video.spec.ts:521 › screencast › should capture static page in persistent context Radoslav Kirilov (@smoke) `@chromium-ubuntu-22.04-arm-node20`
⚠️ [chromium-library] › library/video.spec.ts:356 › screencast › should work for popups `@realtime-time-library-chromium-linux`
⚠️ [chromium-library] › library/chromium/chromium.spec.ts:373 › should produce network events, routing, and annotations for Service Worker `@chromium-ubuntu-22.04-node24`
⚠️ [chromium-library] › library/video.spec.ts:725 › screencast › should capture full viewport on hidpi `@chromium-ubuntu-22.04-node24`
⚠️ [chromium-library] › library/video.spec.ts:762 › screencast › should work with video+trace `@chromium-ubuntu-22.04-node22`
⚠️ [firefox-library] › library/browsercontext-cookies-third-party.spec.ts:257 › third party 'Partitioned;' cookies `@firefox-ubuntu-22.04-node20`
⚠️ [firefox-library] › library/browsercontext-cookies-third-party.spec.ts:470 › top level 'Partitioned;' cookie and same origin iframe `@firefox-ubuntu-22.04-node20`

52059 passed, 1250 skipped

---

Merge workflow run.

[github-actions]

Test results for "MCP"

2 failed
❌ [chrome] › mcp/cli-webmcp.spec.ts:188 › webmcp-call disambiguates same-name tools in identical same-origin frames @mcp-macos-latest-chrome
❌ [chromium] › mcp/cli-webmcp.spec.ts:135 › webmcp-list stitches tools across frames @mcp-macos-latest-chromium

8690 passed, 1474 skipped

---

Merge workflow run.

[github-actions]

🟢 Both failures are pre-existing WebMCP flakes — the PR is clear

Hi, I'm the Playwright bot and I took a look at the red CI here.

The two failing MCP tests both fail on unrelated PRs and on main with byte-identical errors, including on 07f1a615 — this PR's own base commit. This PR only touches test.describe.configure locks in the test runner, which cli-webmcp.spec.ts doesn't exercise.

Details

The "tests 1" report has no failures (8 flaky, all screencast/video/cookie regulars), so there's nothing to triage there. Only the "MCP" report is red, with two failures — both in mcp/cli-webmcp.spec.ts on the macOS bots.

Pre-existing flake / infra

• [chromium] › mcp/cli-webmcp.spec.ts:135 › webmcp-list stitches tools across frames — same test, same error (Expected substring: "Found 2 WebMCP tool(s)" / Received: "Found 1 WebMCP tool(s)") on PR chore(coverage): istanbul instrumentation for the trace viewer and html reporter #42785 twice: run 35411614391 (d29eeae5) and run 35406976347 (fabe18a1). It passed in 16 of 17 other runs in the results DB.

• [chrome] › mcp/cli-webmcp.spec.ts:188 › webmcp-call disambiguates same-name tools in identical same-origin frames — same test, same error (Expected substring: "- frame: http://localhost:8907/widget.html (frame 1)") on main at 07f1a615, run 35414323273, which is exactly the base commit of this PR. Also on PR fix(skills): narrow npm and npx grants #42789 (3f0fa945), PR test(firefox): cover COOP navigation with history state #42788 (1f166c07), and PR chore(coverage): istanbul instrumentation for the trace viewer and html reporter #42785 (fabe18a1).

Both signatures are the same underlying race: the second frame's tool registration hasn't landed by the time webmcp-list/webmcp-call reads it, so only one tool is seen. Neither failed on retry-zero only by chance — the whole cli-webmcp.spec.ts file has been flipping verdict across unrelated SHAs for the past day.

Caused by this PR

Nothing. The diff is packages/playwright/src/common/testType.ts (the lock option on test.describe.configure), its types/docs, and tests/playwright-test/test-locks.spec.ts. The playwright-test project is green, and the MCP suite doesn't go near suite-level locks.

_{Triaged by the Playwright bot - agent run}