Skip to content

Fix: wait for the killed OCR process in the stuck read test - #184

Merged
SandObserver merged 1 commit into
mainfrom
fix/ocr-stuck-read-test-race
Oct 17, 2026
Merged

SandObserver merged 1 commit into
mainfrom
fix/ocr-stuck-read-test-race

Conversation

@SandObserver

Copy link
Copy Markdown
Owner

Summary

  • Wait up to 5 seconds for the killed OCR process to leave the active children list before the stuck read test checks it.

Testing

All green. No new tests; the stuck read test no longer races the process cleanup.

Checklist

  • Format, lint, type checks and tests pass.
  • DESIGN_VERSION bumped and tests/golden written again if rendered images change.
  • CHANGELOG.md updated under Unreleased for user-visible changes.

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying posteryard with  Cloudflare Pages  Cloudflare Pages

Latest commit: 2ab030d
Status: ✅  Deploy successful!
Preview URL: https://5feba48b.posteryard.pages.dev
Branch Preview URL: https://fix-ocr-stuck-read-test-race.posteryard.pages.dev

View logs

@SandObserver
SandObserver merged commit faa968e into main Oct 17, 2026
6 checks passed
@SandObserver
SandObserver deleted the fix/ocr-stuck-read-test-race branch October 17, 2026 18:12
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