Skip to content

chore: Update timeout on playwright tests#1210

Merged
straker merged 1 commit into
developfrom
straker-patch-1
Aug 22, 2025
Merged

chore: Update timeout on playwright tests#1210
straker merged 1 commit into
developfrom
straker-patch-1

Conversation

@straker
Copy link
Copy Markdown
Contributor

@straker straker commented Aug 20, 2025

The latest 2 runs of playwright 18 node test timed out after 15mins. A 3rd run took 9 mins. Since a completed run is so close to the timeout, decided to increase the timeout to avoid failures.

No QA required

The latest runs of playwright 18 node test [timed out after 15mins](https://github.com/dequelabs/axe-core-npm/actions/runs/17102920148/job/48527777041?pr=1196) before it completed. A 3rd run [took 14 mins](https://github.com/dequelabs/axe-core-npm/actions/runs/17102920148/job/48528901298?pr=1196). Since a completed run is so close to the timeout, decided to increase the timeout to avoid failures.
@straker straker requested a review from a team as a code owner August 20, 2025 21:07
Copy link
Copy Markdown
Member

@Garbee Garbee left a comment

Choose a reason for hiding this comment

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

I wonder what caused such a jump to occur in the timing needs... Approving, but we may want to put a task on the backlog as a low priority thing to just try and determine where the time is going. If it is a natural increase from new features, fine. But if it was a jump out of nowhere it indicates we might have a problem.

@straker straker merged commit 7c07558 into develop Aug 22, 2025
36 of 38 checks passed
@straker straker deleted the straker-patch-1 branch August 22, 2025 15:00
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.

2 participants