Repository navigation
fix(test): record a separate video for each test and page - #43980
Pavel Feldman (pavelfeldman) wants to merge 1 commit into
Conversation
Test fixtures now start and stop the screencast video for every page during the test, instead of using the recordVideo context option. This covers pages that are created in beforeAll and shared between tests. Fixes: microsoft#43915
Test results for "tests 1"1 failed 2 flaky23608 passed, 705 skipped Merge workflow run. |
🟢 The failure is a pre-existing flakeHi, I'm the Playwright bot. I compared the latest CI report with this PR and the refreshed test history. DetailsThe only failed test has an identical failure on Pre-existing flake
The same test, on the same CI runner, failed in a main-branch run on October 28, at commit The PR changes shared artifact fixtures, but this test enables tracing only, leaves video off, and creates no browser pages. The test and fixture teardown scheduler are unchanged. The earlier identical failure is the evidence for the flake verdict. The report's two other tests passed on retry; neither is a remaining failure. CI failures assessed by the Playwright bot - agent run Triaged by the Playwright bot - agent run |
Test results for "MCP"5194 passed, 616 skipped Merge workflow run. |
Summary
recordVideocontext option. Pages created inbeforeAlland shared between tests get a video per test, and pages closed during the test keep their video.Screencast.stop()saves the video when the page is already closed, as long as the context is open.page.video()is no longer available for fixture pages; videos are attached to the test results instead.Supersedes #43924.
Fixes #43915