Skip to content

Commit 1d5ac10

Browse files
committed
👷 default local e2e to chromium only
Make `yarn test:e2e` run chromium only by default to keep the local feedback loop fast. CI still runs the full browser matrix via the new `test:e2e:ci` script. Other browsers remain available locally via `yarn test:e2e --project=firefox` (etc.).
1 parent b7df5f9 commit 1d5ac10

2 files changed

Lines changed: 3 additions & 2 deletions

File tree

‎.gitlab-ci.yml‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -255,7 +255,7 @@ e2e:
255255
- yarn build:apps
256256
# Browsers are pre-installed in the CI image. If playwright (current or pinned 1.40.1)
257257
# is upgraded without rebuilding the image, this job will crash — rebuild the image to fix it.
258-
- FORCE_COLOR=1 PW_BROWSER=$BROWSER yarn test:e2e --project=$BROWSER
258+
- FORCE_COLOR=1 PW_BROWSER=$BROWSER yarn test:e2e:ci --project=$BROWSER
259259
after_script:
260260
- node ./scripts/test/export-test-result.ts e2e
261261

‎package.json‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,8 @@
3131
"test:unit:watch": "yarn test:unit --no-single-run",
3232
"test:unit:bs": "node --env-file-if-exists=.env ./scripts/test/bs-wrapper.ts karma start test/unit/karma.bs.conf.js",
3333
"test:e2e:init": "yarn build && yarn build:apps && yarn playwright install --with-deps && yarn dlx -p playwright@1.40.1 playwright install chromium firefox webkit",
34-
"test:e2e": "playwright test --config test/e2e/playwright.config.ts",
34+
"test:e2e": "playwright test --config test/e2e/playwright.config.ts --project=chromium",
35+
"test:e2e:ci": "playwright test --config test/e2e/playwright.config.ts",
3536
"test:compat:tsc": "node scripts/check-typescript-compatibility.ts",
3637
"test:compat:ssr": "scripts/cli check_server_side_rendering_compatibility",
3738
"test:performance": "yarn test:e2e:init && playwright test --config test/performance/playwright.config.ts",

0 commit comments

Comments
 (0)