test(nextjs): Move playwright deps installation into CI pipeline

Author: Luca Forstner

.github/workflows/build.yml

@@ -490,6 +490,7 @@ jobs:
           NODE_VERSION: ${{ matrix.node }}
         run: |
           cd packages/nextjs
+          yarn playwright install-deps
           yarn test:integration
 
   # Ember tests are separate from the rest because they are the slowest part of the test suite, and making them a

packages/nextjs/package.json

@@ -68,10 +68,9 @@
     "test:build": "yarn ts-node test/buildProcess/runTest.ts",
     "test:unit": "jest",
     "test:integration": "./test/run-integration-tests.sh && yarn test:types",
-    "test:integration:ci": "run-s test:integration:clean test:integration:client:ci test:integration:server",
     "test:integration:prepare": "(cd test/integration && yarn build && yarn start)",
     "test:integration:clean": "(cd test/integration && rimraf .cache node_modules build)",
-    "test:integration:client": "yarn playwright install-deps && yarn playwright test test/integration/test/client/",
+    "test:integration:client": "yarn playwright test test/integration/test/client/",
     "test:integration:client:ci": "yarn test:integration:client --browser='all' --reporter='line'",
     "test:integration:server": "export NODE_OPTIONS='--stack-trace-limit=25' && jest --config=test/integration/jest.config.js test/integration/test/server/",
     "test:types": "cd test/types && yarn test",

packages/nextjs/playwright.config.ts

@@ -1,7 +1,7 @@
 import type { PlaywrightTestConfig } from '@playwright/test';
 
 const config: PlaywrightTestConfig = {
-  retries: 2,
+  retries: 0, // We do not accept flakes.
   timeout: 12000,
   use: {
     baseURL: 'http://localhost:3000',