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 bveen confused with it in CI logs where the 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 <marc.dumais@ericsson.com>
  • Loading branch information
marcdumais-work committed Jan 31, 2024
1 parent e9b07c5 commit 81f4a67
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 81f4a67

Please sign in to comment.