Skip to content

chore: extract large snapshots in the trace as resources#39765

Open
dgozman wants to merge 1 commit intomicrosoft:mainfrom
dgozman:chore-trace-snapshots-as-resources
Open

chore: extract large snapshots in the trace as resources#39765
dgozman wants to merge 1 commit intomicrosoft:mainfrom
dgozman:chore-trace-snapshots-as-resources

Conversation

@dgozman
Copy link
Contributor

@dgozman dgozman commented Mar 19, 2026

No description provided.

Copy link
Member

@pavelfeldman pavelfeldman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This won't be triggered for your new snapshot, you probably want to bump the version with empty modernizer step.

        if (event.version > latestVersion)
          throw new TraceVersionError('The trace was created by a newer version of Playwright and is not supported by this version of the viewer. Please use latest Playwright to open the trace.');

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@dgozman dgozman force-pushed the chore-trace-snapshots-as-resources branch from b58aee8 to 24d442e Compare March 19, 2026 20:30
@dgozman dgozman force-pushed the chore-trace-snapshots-as-resources branch from 24d442e to f7eb9ca Compare March 19, 2026 20:45
@github-actions

This comment has been minimized.

@github-actions
Copy link
Contributor

Test results for "MCP"

5507 passed, 343 skipped


Merge workflow run.

@github-actions
Copy link
Contributor

Test results for "tests 1"

6 flaky ⚠️ [chromium-library] › library/video.spec.ts:358 › screencast › should capture navigation `@chromium-ubuntu-22.04-arm-node20`
⚠️ [chromium-library] › library/trace-viewer.spec.ts:1223 › should display language-specific locators `@chromium-ubuntu-22.04-node24`
⚠️ [firefox-library] › library/inspector/cli-codegen-3.spec.ts:224 › cli codegen › should generate frame locators (4) `@firefox-ubuntu-22.04-node20`
⚠️ [webkit-library] › library/browsertype-connect.spec.ts:758 › launchServer › should upload a folder `@webkit-ubuntu-22.04-node20`
⚠️ [webkit-library] › library/screencast.spec.ts:85 › start reuses existing screencast when video recording is running `@webkit-ubuntu-22.04-node20`
⚠️ [playwright-test] › ui-mode-test-output.spec.ts:118 › should collapse repeated console messages for test `@macos-latest-node20`

38815 passed, 845 skipped


Merge workflow run.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants