feat(list reporter): add printWorkerIndex option to prefix output with worker index - #42886
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.
Yury Semikhatsky (yury-s)
left a comment
Yury Semikhatsky (yury-s)
left a comment
There was a problem hiding this comment.
Should it be printPid as discussed ?
588b6d1 to
48fca50
Compare
Test results for "tests 1"8 flaky52140 passed, 1241 skipped Merge workflow run. |
Test results for "MCP"2 failed 8703 passed, 1474 skipped Merge workflow run. |
🟢 CI is clear — both failures are known flakesHi, I'm the Playwright bot and I took a look at the failing CI. The two MCP failures, DetailsOverall: the latest reports are 2 failed in Pre-existing flake / infra
Triaged by the Playwright bot - agent run |
| console.log('line1\\nline2'); | ||
| }); | ||
| `, | ||
| }, { workers: 1 }, { PW_TEST_DEBUG_REPORTERS: '1', PLAYWRIGHT_FORCE_TTY: '80' }); |
There was a problem hiding this comment.
run with 2 workers?
| if (!this._printWorkerIndex) | ||
| return ''; | ||
| const workerIndex = result.workerIndex >= 0 ? String(result.workerIndex) : ' '; | ||
| return this.screen.colors.dim(`[${workerIndex}]`) + ' '; |
There was a problem hiding this comment.
maybe add some padding?
91eeb0c
into
microsoft:main
2 participants
Summary
printWorkerIndex(envPLAYWRIGHT_LIST_PRINT_WORKER_INDEX) prefixes every test line, stdio line and inline failure with the index of the worker that produced it.