Skip to content

chore(test-runner): replace TestInfoError.matcherResult with errorContext#40603

Merged
pavelfeldman merged 1 commit intomicrosoft:mainfrom
pavelfeldman:chore-testinfoerror-error-context
May 4, 2026
Merged

chore(test-runner): replace TestInfoError.matcherResult with errorContext#40603
pavelfeldman merged 1 commit intomicrosoft:mainfrom
pavelfeldman:chore-testinfoerror-error-context

Conversation

@pavelfeldman
Copy link
Copy Markdown
Member

Summary

  • Drop TestInfoError.matcherResult from the serialized error.
  • Add TestInfoError.errorContext: string carrying the aria snapshot of the receiver at matcher-failure time.
  • Runtime ExpectError.matcherResult (custom matcher API) is unchanged.

…text

`TestInfoError.matcherResult` is removed. The aria snapshot of the
receiver at the time of an `expect(...)` matcher failure now lives
on the new top-level `TestInfoError.errorContext: string` property;
the rest of the matcher fields are dropped from the serialized form.

The runtime `ExpectError.matcherResult` (used by custom matchers) is
unchanged.
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 4, 2026

Test results for "MCP"

1 failed
❌ [chrome] › mcp/autowait.spec.ts:19 › racy navigation destroys context @mcp-windows-latest-chrome

6875 passed, 1015 skipped


Merge workflow run.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 4, 2026

Test results for "tests 1"

2 flaky ⚠️ [chromium-library] › library/video.spec.ts:647 › screencast › should capture full viewport `@chromium-ubuntu-22.04-node20`
⚠️ [playwright-test] › ui-mode-trace.spec.ts:812 › should update state on subsequent run `@ubuntu-latest-node20`

41642 passed, 784 skipped


Merge workflow run.

@pavelfeldman pavelfeldman merged commit 2b2e505 into microsoft:main May 4, 2026
44 of 45 checks passed
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