Skip to content

Commit cc96056

Browse files
Optimize CI: Chromium-only runs, lower timeout, fewer retries, remove test.slow() (#30)
* Initial plan * Optimize CI: Chromium-only, retries 1, timeout 90s, remove test.slow() Co-authored-by: bg-playground <259109604+bg-playground@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: bg-playground <259109604+bg-playground@users.noreply.github.com>
1 parent 66f5778 commit cc96056

2 files changed

Lines changed: 26 additions & 27 deletions

File tree

automated-testing/playwright.config.js

Lines changed: 26 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ const { devices } = require('@playwright/test');
22

33
module.exports = {
44
testDir: 'tests',
5-
timeout: 120000,
5+
timeout: 90000,
66
expect: {
77
timeout: 5000,
88
},
@@ -12,7 +12,7 @@ module.exports = {
1212
forbidOnly: !!process.env.CI,
1313

1414
/* Retry on CI only */
15-
retries: process.env.CI ? 2 : 0,
15+
retries: process.env.CI ? 1 : 0,
1616

1717
/* Opt out of parallel tests on CI. */
1818
workers: process.env.CI ? 1 : undefined,
@@ -45,27 +45,28 @@ module.exports = {
4545
navigationTimeout: 60000,
4646
},
4747

48-
projects: [
49-
{
50-
name: 'chromium',
51-
use: { ...devices['Desktop Chrome'] },
52-
},
53-
{
54-
name: 'firefox',
55-
use: { ...devices['Desktop Firefox'] },
56-
},
57-
{
58-
name: 'webkit',
59-
use: { ...devices['Desktop Safari'] },
60-
},
61-
{
62-
name: 'Mobile Chrome',
63-
use: { ...devices['Pixel 5'] },
64-
},
65-
// Only run Mobile Safari locally, not in CI
66-
...(!process.env.CI ? [{
67-
name: 'Mobile Safari',
68-
use: { ...devices['iPhone 12'] },
69-
}] : []),
70-
],
48+
projects: process.env.CI
49+
? [{ name: 'chromium', use: { ...devices['Desktop Chrome'] } }]
50+
: [
51+
{
52+
name: 'chromium',
53+
use: { ...devices['Desktop Chrome'] },
54+
},
55+
{
56+
name: 'firefox',
57+
use: { ...devices['Desktop Firefox'] },
58+
},
59+
{
60+
name: 'webkit',
61+
use: { ...devices['Desktop Safari'] },
62+
},
63+
{
64+
name: 'Mobile Chrome',
65+
use: { ...devices['Pixel 5'] },
66+
},
67+
{
68+
name: 'Mobile Safari',
69+
use: { ...devices['iPhone 12'] },
70+
},
71+
],
7172
};

automated-testing/tests/ui/pim.spec.js

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,6 @@ import { DashboardPage } from '../../pages/DashboardPage';
44
import { PIMPage, AddEmployeePage } from '../../pages/PIMPage';
55

66
test.describe('PIM Module Tests', () => {
7-
test.slow(); // Triple timeout — tests depend on external demo site
8-
97
let loginPage;
108
let dashboardPage;
119
let pimPage;

0 commit comments

Comments
 (0)