Skip to content

Commit

Permalink
[CI] Use 'list' Playwright reporter
Browse files Browse the repository at this point in the history
When running the tests locally, the current 'html' reporter is fine.
But I've been confused with it in CI logs where there was a failure,
presumably a silent retry and finally a report of the tests passing.

I think in CI we would have more clarity with an alternate reporter.
Let's try 'list'. If it does not work well we can fall-back on the
simple 'dot'.

Signed-off-by: Marc Dumais <[email protected]>
  • Loading branch information
marcdumais-work committed Feb 1, 2024
1 parent e9b07c5 commit 806c79c
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion playwright.config.ts
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ export default defineConfig({
/* Opt out of parallel tests on CI. */
workers: process.env.CI ? 1 : undefined,
/* Reporter to use. See https://playwright.dev/docs/test-reporters */
reporter: 'html',
reporter: process.env.CI ? 'list' : 'html',
/* Shared settings for all the projects below. See https://playwright.dev/docs/api/class-testoptions. */
use: {
/* Collect trace when retrying the failed test. See https://playwright.dev/docs/trace-viewer */
Expand Down

0 comments on commit 806c79c

Please sign in to comment.