feat: Allow configuring last-run output file location#40806
Conversation
Allow configuration via environment variable like other reporters Fixes microsoft#40805
|
@microsoft-github-policy-service agree |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
| ['--headed', { description: `Run tests in headed browsers (default: headless)` }], | ||
| ['--ignore-snapshots', { description: `Ignore screenshot and snapshot expectations` }], | ||
| ['--last-failed', { description: `Only re-run the failures` }], | ||
| ['--last-run-file <file>', { description: `Path of the last-run JSON file used by --last-failed (also configurable via PLAYWRIGHT_LAST_RUN_OUTPUT_FILE).` }], |
There was a problem hiding this comment.
--last-run-file -> --last-failed-file
| ['--headed', { description: `Run tests in headed browsers (default: headless)` }], | ||
| ['--ignore-snapshots', { description: `Ignore screenshot and snapshot expectations` }], | ||
| ['--last-failed', { description: `Only re-run the failures` }], | ||
| ['--last-run-file <file>', { description: `Path of the last-run JSON file used by --last-failed (also configurable via PLAYWRIGHT_LAST_RUN_OUTPUT_FILE).` }], |
There was a problem hiding this comment.
It sounds that this option is required for --last-failed, let's rephrase the description to say that this is only needed to change the default last-failed file location.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Test results for "MCP"3 failed 7096 passed, 1104 skipped Merge workflow run. |
Test results for "tests 1"2 flaky41922 passed, 850 skipped Merge workflow run. |
2 participants
Allow configuration via environment variable like other reporters.
Left undocumented for now as lastRun is an internal reporter, so it didn't fit well adding it to the existing reporters doc.
Fixes #40805