Skip to content

test: clean up transport recovery timers - #26

Merged
danyel117 merged 1 commit into
mainfrom
fix/transport-test-cleanup
Aug 20, 2026
Merged

test: clean up transport recovery timers#26
danyel117 merged 1 commit into
mainfrom
fix/transport-test-cleanup

Conversation

@danyel117

Copy link
Copy Markdown
Contributor

Summary

  • dispose each transport-error test plugin instance so scheduled recovery timers cannot outlive afterEach
  • stop scheduling retries in the classification loop by using the one-failure ceiling
  • widen polling deadlines to tolerate loaded CI runners while preserving bounded assertions

Context

The publish workflow for #25 repeatedly completed the new recovery tests successfully, then failed when leaked timers wrote after their test directory had been removed.

Validation

  • 20 focused stress runs passed
  • bun run lint
  • bun run typecheck
  • bun test (190 pass)
  • bun run build
  • bun run pack:dry-run
  • Claude Opus review: no merge blocker

@danyel117
danyel117 merged commit 9121596 into main Aug 20, 2026
4 checks passed
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