feat(test-runner): support locks in test.describe.configure - #42792
Conversation
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Pavel Feldman (pavelfeldman)
left a comment
There was a problem hiding this comment.
I would expect 0 (edit) to 1 test tops for this change.
locks cannot be declared imperatively for the enclosing suite add a `lock` option to `test.describe.configure` that accumulates suite locks
bdffedb to
efbb09e
Compare
Test results for "tests 1"5 flaky52020 passed, 1250 skipped Merge workflow run. |
Test results for "MCP"1 failed 8691 passed, 1474 skipped Merge workflow run. |
🟡 One MCP failure I can't pin either way — but the diff can't reach itHi, I'm the Playwright bot and I took a look at the red CI here. The only failure is DetailsThe "tests 1" report has no failures (5 flaky — the usual screencast/video/third-party-cookie regulars), so nothing to triage there. The "MCP" report is the only red one, with a single failure. The two Uncertain
Caused by this PR Nothing I can attribute. Triaged by the Playwright bot - agent run |
04e547b
into
microsoft:main
locks cannot be declared imperatively for the enclosing suite
add a
lockoption totest.describe.configurethat accumulates suite locksfixes #42782