Skip to content

fix(browser): clear screenshot stability timeout - #11659

Merged
sheremet-va merged 1 commit into
mainfrom
fix/screenshot-timeout-leak
Oct 20, 2026
Merged

sheremet-va merged 1 commit into
mainfrom
fix/screenshot-timeout-leak

Conversation

@sheremet-va

Copy link
Copy Markdown
Member

toMatchScreenshot left its stability timer running after the screenshot won the race, so the process stayed alive until timeout elapsed. The timer is now cleared once the race settles.

Fixes #11656

@netlify

netlify Bot commented Oct 20, 2026

Copy link
Copy Markdown

✅ Deploy Preview for vitest-dev ready!

Name Link
🔨 Latest commit ac100c7
🔍 Latest deploy log https://app.netlify.com/projects/vitest-dev/deploys/6ad727b4f2193c15dae10d47
😎 Deploy Preview https://deploy-preview-11659--vitest-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@sheremet-va
sheremet-va merged commit fbe37c0 into main Oct 20, 2026
31 checks passed
@sheremet-va
sheremet-va deleted the fix/screenshot-timeout-leak branch October 20, 2026 12:54
@github-actions github-actions Bot locked and limited conversation to collaborators Nov 4, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

toMatchScreenshot leaves its stability timeout running, delaying exit by up to 10s

1 participant