Skip to content

[Refactor] Roll out stream helpers to Unbound spec - #1090

Merged
edelauna merged 1 commit into
mainfrom
refactor/unbound-stream-test-utils-06q4nypl2c73c
Aug 1, 2026
Merged

[Refactor] Roll out stream helpers to Unbound spec#1090
edelauna merged 1 commit into
mainfrom
refactor/unbound-stream-test-utils-06q4nypl2c73c

Conversation

@zoomote

@zoomote zoomote Bot commented Aug 1, 2026

Copy link
Copy Markdown
Contributor

Created by Roomote. Follow up by mentioning @roomote, in the web UI, or in Discord.

What changed

  • Refactored src/api/providers/__tests__/unbound.spec.ts to replace inline async-iterator mock streams with asyncStreamFrom and pure stream collection/drain loops with collectStream.
  • Reduced the unbound.spec.ts ESLint suppressions count after removing the obsolete chunks: any[] declarations.

Why this change was made

This continues the approved test-utils rollout with another provider spec, using the same narrow rule: extract mechanical stream setup and collection, while leaving assertions and request-shape checks explicit.

Impact

No user-facing behavior change. The Unbound spec follows the shared stream-helper pattern, and the targeted spec, src lint/typecheck, and full Zoo-Code test suite passed.

Related PRs

@zoomote

zoomote Bot commented Aug 1, 2026

Copy link
Copy Markdown
Contributor Author

No code issues found. See task

Reviewed 52450f5

@codecov

codecov Bot commented Aug 1, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@edelauna
edelauna marked this pull request as ready for review August 1, 2026 14:41
@edelauna
edelauna enabled auto-merge August 1, 2026 14:42
@edelauna
edelauna added this pull request to the merge queue Aug 1, 2026
Merged via the queue into main with commit 725fafd Aug 1, 2026
24 checks passed
@edelauna
edelauna deleted the refactor/unbound-stream-test-utils-06q4nypl2c73c branch August 1, 2026 14:54
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