Skip to content

Commit

Permalink
udpated playwright config to increase retries and limit browsers
Browse files Browse the repository at this point in the history
  • Loading branch information
aasimsyed committed Apr 23, 2024
1 parent fa35f4f commit 67b5dfa
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 12 deletions.
1 change: 0 additions & 1 deletion .github/workflows/pr_ci_frontend.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,6 @@ jobs:
working-directory: ./frontend
run: |
yarn playwright install --with-deps
yarn playwright install msedge
- name: Run Playwright tests
working-directory: ./frontend
Expand Down
20 changes: 9 additions & 11 deletions frontend/playwright.config.ts
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ export default defineConfig({
/* Fail the build on CI if you accidentally left test.only in the source code. */
forbidOnly: !!process.env.CI,
/* Retry on CI only */
retries: process.env.CI ? 2 : 0,
retries: process.env.CI ? 4 : 0,
/* Opt out of parallel tests on CI. */
workers: process.env.CI ? 1 : undefined,
/* Reporter to use. See https://playwright.dev/docs/test-reporters */
Expand All @@ -46,20 +46,18 @@ export default defineConfig({
name: "chromium",
use: { ...devices["Desktop Chrome"] },
},

{
name: "firefox",
use: { ...devices["Desktop Firefox"] },
},

{
name: "webkit",
use: { ...devices["Desktop Safari"] },
},
}, /*
{
name: "Microsoft Edge",
use: { ...devices["Desktop Edge"], channel: "msedge" },
},
{
name: "firefox",
use: { ...devices["Desktop Firefox"] },
}, */

/* Test against mobile viewports. */
{
Expand All @@ -69,17 +67,16 @@ export default defineConfig({
{
name: "Mobile Safari",
use: { ...devices["iPhone 12"], isMobile: true },
},
}, /*
{
name: "Mobile Samsung",
use: { ...devices["Galaxy S20"], isMobile: true },
},
{
name: "Mobile iPad",
use: { ...devices["iPad (gen 8)"], isMobile: true },
},
}, */

/* Test against branded browsers. */
/* Test against branded browsers. */
// {
// name: 'Microsoft Edge',
Expand All @@ -90,6 +87,7 @@ export default defineConfig({
// use: { ...devices['Desktop Chrome'], channel: 'chrome' },
// },
],

/* Run your local dev server before starting the tests */
// webServer: {
// command: 'npm run start',
Expand Down

0 comments on commit 67b5dfa

Please sign in to comment.