From fb12e74972017ded3e18a1838d03c450b4d55d77 Mon Sep 17 00:00:00 2001 From: linonetwo Date: Fri, 1 May 2026 23:26:54 +0800 Subject: [PATCH] fix(ci): remove hardcoded timeout, use GitHub Actions default - Remove timeout-minutes: 40 from E2E test step - Let GitHub Actions use default 360 minute timeout - Actual test duration controlled by calibration-based step timeouts - Calibration dynamically adjusts timeouts based on CI performance --- .github/workflows/test.yml | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index c6504999..fadc2478 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -68,6 +68,8 @@ jobs: - name: Run E2E tests # E2E GUI tests with Electron on Linux require a virtual framebuffer, upgrade screen size from time to time. + # No timeout specified - uses GitHub Actions default (360 minutes) + # Actual test duration is controlled by calibration-based step timeouts in cucumber run: xvfb-run --auto-servernum --server-args="-screen 0 2560x1440x24" pnpm run test:e2e env: CI: true @@ -75,10 +77,6 @@ jobs: # Set Chinese locale for i18n testing LANG: zh_CN.UTF-8 LC_ALL: zh_CN.UTF-8 - # Dynamic timeout based on calibration result. If calibration succeeds, timeout is calculated. - # If calibration fails, fallback multiplier (4.0×) is used: 25s × 4.0 × 65 scenarios / 60 ≈ 108min - # Conservative 40min allows room for both calibrated and fallback scenarios - timeout-minutes: 40 # Upload test artifacts (screenshots, logs) - name: Upload test artifacts