Skip to content

Conversation

@cte
Copy link
Collaborator

@cte cte commented Jun 21, 2025

Description

Increased to 2m per suite, and 20m overall.


Important

Increase e2e test suite timeout to 2 minutes per suite and 20 minutes overall, and centralize timeout configuration using setDefaultSuiteTimeout().

  • Timeout Adjustments:
    • Increase suite-level timeout to 2 minutes and overall timeout to 20 minutes in index.ts.
    • Introduce setDefaultSuiteTimeout() in test-utils.ts to set default suite timeout to 120,000 ms.
  • Test Suite Modifications:
    • Apply setDefaultSuiteTimeout() to test suites in extension.test.ts, modes.test.ts, and task.test.ts.
    • Remove individual test timeouts in apply-diff.test.ts, execute-command.test.ts, and 8 other test files.
  • Miscellaneous:
    • Add test-utils.ts to centralize timeout configuration.

This description was created by Ellipsis for 3e8483a. You can customize this summary. It will automatically update as commits are pushed.

@cte cte requested review from jr and mrubens as code owners June 21, 2025 07:36
@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Jun 21, 2025
@cte cte merged commit bccf11f into main Jun 21, 2025
10 checks passed
@cte cte deleted the cte/increase-e2e-test-timeouts branch June 21, 2025 07:57
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Jun 21, 2025
@github-project-automation github-project-automation bot moved this from Triage to Done in Roo Code Roadmap Jun 21, 2025
cte added a commit that referenced this pull request Jun 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:M This PR changes 30-99 lines, ignoring generated files.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants