Skip to content

fix(tests): reduce default test timeout when Percy is disabled#3440

Open
rohitpaulk wants to merge 1 commit intomainfrom
reduce-test-timeout-percy-disabled
Open

fix(tests): reduce default test timeout when Percy is disabled#3440
rohitpaulk wants to merge 1 commit intomainfrom
reduce-test-timeout-percy-disabled

Conversation

@rohitpaulk
Copy link
Member

@rohitpaulk rohitpaulk commented Dec 29, 2025

Decrease QUnit test timeout from 5000ms to 3000ms for cases
when Percy is not enabled. This speeds up test runs by avoiding
unnecessary wait time while maintaining sufficient timeout for
test stability.


Note

Reduces default QUnit timeout to speed up local/CI tests without Percy.

  • In tests/test-helper.js, updates QUnit.config.testTimeout to config.x.percyIsEnabled ? 20000 : 3000 (from 5000).

Written by Cursor Bugbot for commit d696781. This will update automatically on new commits. Configure here.

Decrease QUnit test timeout from 5000ms to 3000ms for cases 
when Percy is not enabled. This speeds up test runs by avoiding
unnecessary wait time while maintaining sufficient timeout for
test stability.
@github-actions
Copy link

Test Results

  1 files  ±0    1 suites  ±0   7m 0s ⏱️ ±0s
675 tests ±0  610 ✅  - 10  55 💤 ±0   0 ❌ ± 0  10 🔥 +10 
690 runs  ±0  615 ✅  - 20  55 💤 ±0  10 ❌ +10  10 🔥 +10 

For more details on these errors, see this check.

Results for commit d696781. ± Comparison against base commit 572a127.

@codecov
Copy link

codecov bot commented Dec 29, 2025

❌ 10 Tests Failed:

Tests completed Failed Passed Skipped
620 10 610 55
View the top 3 failed test(s) by shortest run time
Chrome 142.0::Acceptance | course-page | try-other-language: can try other language from repository setup page (regression)
Stack Traces | 1.26s run time
global failure: Error: Assertion occurred after test finished.
> Test: can try other language
> Message: Promise rejected during "can try other language": Assertion occurred after test finished.
> Test: can try other language
> Message: API requests match expected sequence after second poll

Chrome 142.0::Acceptance | course-page | try-other-language: can try other language
Stack Traces | 3.06s run time
Test took longer than 3000ms; test timed out.
Chrome 142.0::Acceptance | course-page | course-stage-comments: comment has correct user label
Stack Traces | 3.31s run time
Test took longer than 3000ms; test timed out.
Chrome 142.0::Acceptance | concepts-test: clicking on the upcoming concept cards works properly
Stack Traces | 3.35s run time
Test took longer than 3000ms; test timed out.
Chrome 142.0::Acceptance | concepts-test: anonymous users can view concepts not linked to a concept group
Stack Traces | 3.56s run time
Test took longer than 3000ms; test timed out.
Chrome 142.0::Acceptance | concepts-test: anonymous users can view concepts linked to a concept group
Stack Traces | 3.56s run time
Test took longer than 3000ms; test timed out.
Chrome 142.0::Acceptance | concepts-test: users can interact with concepts, and the expected elements are focused
Stack Traces | 3.57s run time
Test took longer than 3000ms; test timed out.
Chrome 142.0::Acceptance | concepts-test: can view concepts
Stack Traces | 3.7s run time
Test took longer than 3000ms; test timed out.
Chrome 142.0::Acceptance | course-page | code-examples | vote: can downvote code examples
Stack Traces | 3.81s run time
Test took longer than 3000ms; test timed out.
Chrome 142.0::Acceptance | course-page | switch-repository: can switch repository
Stack Traces | 4.16s run time
Test took longer than 3000ms; test timed out.

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

@codecov
Copy link

codecov bot commented Dec 30, 2025

Bundle Report

Changes will increase total bundle size by 1.64kB (0.0%) ⬆️. This is within the configured threshold ✅

Detailed changes
Bundle name Size Change
client-array-push 38.79MB 1.64kB (0.0%) ⬆️

Affected Assets, Files, and Routes:

view changes for bundle: client-array-push

Assets Changed:

Asset Name Size Change Total Size Change (%)
assets/chunk.*.js 82 bytes 40.76kB 0.2%
assets/chunk.*.js 1.56kB 3.32MB 0.05%

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant