Skip to content

feat(list-reporter): add printFailuresInline option#40875

Open
pavelfeldman wants to merge 1 commit into
microsoft:mainfrom
pavelfeldman:list-print-failures-inline
Open

feat(list-reporter): add printFailuresInline option#40875
pavelfeldman wants to merge 1 commit into
microsoft:mainfrom
pavelfeldman:list-print-failures-inline

Conversation

@pavelfeldman
Copy link
Copy Markdown
Member

Summary

  • New printFailuresInline list-reporter option (also PLAYWRIGHT_LIST_PRINT_FAILURES_INLINE) that prints failure details right after each failed test instead of batching them at the end.
  • End-of-run epilogue suppresses the failure repeat when the option is on.

Print failure details immediately after each failed test instead of
batching them at the end of the run. Useful for long suites where you
want to read failures as they happen.

Opt in via the reporter config or PLAYWRIGHT_LIST_PRINT_FAILURES_INLINE.
@github-actions
Copy link
Copy Markdown
Contributor

Test results for "MCP"

1 failed
❌ [msedge] › mcp/annotate.spec.ts:297 › should annotate via direct browser_annotate MCP call @mcp-windows-latest-msedge

7098 passed, 1104 skipped


Merge workflow run.

@github-actions
Copy link
Copy Markdown
Contributor

Test results for "tests 1"

3 flaky ⚠️ [chromium-page] › page/page-history.spec.ts:95 › goBack/goForward should work with bfcache-able pages `@ubuntu-22.04-chromium-tip-of-tree`
⚠️ [chromium-library] › library/video.spec.ts:719 › screencast › should work with video+trace `@chromium-ubuntu-22.04-arm-node20`
⚠️ [chromium-library] › library/browsertype-connect.spec.ts:1000 › run-server › socks proxy › should proxy local.playwright requests `@chromium-ubuntu-22.04-node24`

41963 passed, 850 skipped


Merge workflow run.

const wasPaused = this._paused.delete(result);
if (!wasPaused)
this._updateTestLine(test, result);
if (!wasPaused && this._printFailuresInline && !this.willRetry(test) && (test.outcome() === 'flaky' || test.outcome() === 'unexpected' || result.status === 'interrupted'))
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

why print it for flaky?

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