Skip to content

Increase timeout for system tests from 10 to 15 minutes#19703

Merged
seanbudd merged 1 commit into
betafrom
bumpTimeOut
Feb 26, 2026
Merged

Increase timeout for system tests from 10 to 15 minutes#19703
seanbudd merged 1 commit into
betafrom
bumpTimeOut

Conversation

@seanbudd

@seanbudd seanbudd commented Feb 26, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

Link to issue number:

None

Summary of the issue:

As of today, system tests are now regularly running longer than 10min.

Todays actions release may be related

Description of user facing changes:

None

Description of developer facing changes:

fix up time out for system tests

Description of development approach:

Code Review Checklist:

  • Documentation:
    • Change log entry
    • User Documentation
    • Developer / Technical Documentation
    • Context sensitive help for GUI changes
  • Testing:
    • Unit tests
    • System (end to end) tests
    • Manual testing
  • UX of all users considered:
    • Speech
    • Braille
    • Low Vision
    • Different web browsers
    • Localization in other languages / culture than English
  • API is compatible with existing add-ons.
  • Security precautions taken.

@seanbudd seanbudd requested a review from a team as a code owner February 26, 2026 07:25
Copilot AI reviewed Feb 26, 2026
View reviewed changes

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.

Pull request overview

Adjusts CI to accommodate longer-running NVDA system tests by increasing the GitHub Actions job timeout, reducing spurious failures when tests exceed the previous 10-minute limit.

Changes:

  • Increase the systemTests job timeout from 10 to 15 minutes in the CI workflow.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@seanbudd seanbudd merged commit 261639c into beta Feb 26, 2026
70 of 75 checks passed
@seanbudd seanbudd deleted the bumpTimeOut branch February 26, 2026 08:27
@github-actions github-actions Bot added this to the 2026.2 milestone Feb 26, 2026
seanbudd added a commit that referenced this pull request Feb 27, 2026
@seanbudd
Revert "Increase timeout for system tests from 10 to 15 minutes (#19703…
679763c 
…)"

This reverts commit 261639c.
@seanbudd seanbudd mentioned this pull request Feb 27, 2026
Merged
SaschaCowley pushed a commit that referenced this pull request Feb 27, 2026
@seanbudd
Revert "Increase timeout for system tests from 10 to 15 minutes" (#19715
963f29b 
)

### Reverts PR
Reverts #19703

### Issues fixed
None

### Issues reopened
None

### Reason for revert

PR was unnecessary.
See #19714 for more information

### Can this PR be reimplemented? If so, what is required for the next
attempt
@seanbudd seanbudd modified the milestones: 2026.2, 2026.1 Feb 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@SaschaCowley SaschaCowley SaschaCowley approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

2026.1

Development

Successfully merging this pull request may close these issues.

3 participants

@seanbudd @SaschaCowley