test: remove playwright multipliers

test: disable e2e retries
2025-01-03 13:42:07 -05:00 · 2025-01-03 13:16:18 -05:00
2 changed files with 4 additions and 13 deletions
--- a/.github/workflows/main.yml
+++ b/.github/workflows/main.yml
@@ -368,13 +368,7 @@ jobs:
        run: pnpm exec playwright install-deps chromium

      - name: E2E Tests
-        uses: nick-fields/retry@v3
-        with:
-          retry_on: any
-          max_attempts: 5
-          timeout_minutes: 20
-          command: PLAYWRIGHT_JSON_OUTPUT_NAME=results_${{ matrix.suite }}.json pnpm test:e2e:prod:ci ${{ matrix.suite }}
-          on_retry_command: pnpm clean:build && pnpm install --no-frozen-lockfile && pnpm build:all
+        run: pnpm test:e2e:prod:ci ${{ matrix.suite }}
        env:
          PLAYWRIGHT_JSON_OUTPUT_NAME: results_${{ matrix.suite }}.json
          NEXT_TELEMETRY_DISABLED: 1
--- a/test/playwright.config.ts
+++ b/test/playwright.config.ts
@@ -8,12 +8,9 @@ const dirname = path.dirname(filename)

 dotenv.config({ path: path.resolve(dirname, 'test.env') })

-let multiplier = process.env.CI ? 3 : 0.5
-let smallMultiplier = process.env.CI ? 2 : 0.75
-
-export const TEST_TIMEOUT_LONG = 640000 * multiplier // 8*3 minutes - used as timeOut for the beforeAll
-export const TEST_TIMEOUT = 30000 * multiplier
-export const EXPECT_TIMEOUT = 6000 * smallMultiplier
+export const TEST_TIMEOUT_LONG = process.env.CI ? 300_000 : 60_000
+export const TEST_TIMEOUT = process.env.CI ? 30_000 : 15_000
+export const EXPECT_TIMEOUT = 10_000
 export const POLL_TOPASS_TIMEOUT = EXPECT_TIMEOUT * 4 // That way expect.poll() or expect().toPass can retry 4 times. 4x higher than default expect timeout => can retry 4 times if retryable expects are used inside

 export default defineConfig({
Author	SHA1	Message	Date
Elliot DeNolf	156e45999b	test: remove playwright multipliers	2025-01-03 13:42:07 -05:00
Elliot DeNolf	0dd05c6f0d	test: disable e2e retries	2025-01-03 13:16:18 -05:00