Skip to content

CI: fail a hung TestCafe test after 3 minutes instead of waiting for the job timeout - #35046

Open
EugeniyKiyashko wants to merge 1 commit into
DevExpress:mainfrom
EugeniyKiyashko:chore/testcafe-test-execution-timeout
Open

CI: fail a hung TestCafe test after 3 minutes instead of waiting for the job timeout#35046
EugeniyKiyashko wants to merge 1 commit into
DevExpress:mainfrom
EugeniyKiyashko:chore/testcafe-test-execution-timeout

Conversation

@EugeniyKiyashko

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI lite review requested due to automatic review settings September 3, 2026 19:41
@EugeniyKiyashko EugeniyKiyashko changed the title Fail a hung TestCafe test after 3 minutes instead of waiting for the job timeout CI: Fail a hung TestCafe test after 3 minutes instead of waiting for the job timeout Sep 3, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

The change is small, localized to the TestCafe runner configuration, and aligns directly with the stated goal of failing hung tests sooner.

Pull request overview

This PR adds a per-test execution timeout to the TestCafe E2E runner so that individual hung tests fail quickly (after 3 minutes) rather than blocking the entire CI job until the overall job timeout.

Changes:

  • Introduced a TEST_EXECUTION_TIMEOUT constant set to 3 minutes (in ms).
  • Passed testExecutionTimeout into TestCafe runOptions to enforce the per-test timeout.
File summaries
File Description
e2e/testcafe-devextreme/runner.ts Adds a 3-minute per-test execution timeout to fail hung TestCafe tests earlier in CI.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@EugeniyKiyashko EugeniyKiyashko self-assigned this Sep 3, 2026
@EugeniyKiyashko EugeniyKiyashko changed the title CI: Fail a hung TestCafe test after 3 minutes instead of waiting for the job timeout CI: fail a hung TestCafe test after 3 minutes instead of waiting for the job timeout Sep 3, 2026
@alexslavr
alexslavr added this pull request to the merge queue Sep 4, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Sep 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants