Skip to content

test: cover response-size limiter context propagation - #7879

Open
CharlieTLe with Copilot wants to merge 1 commit into
masterfrom
copilot/inspect-existing-tests-add-focused-tests
Open

CharlieTLe with Copilot wants to merge 1 commit into
masterfrom
copilot/inspect-existing-tests-add-focused-tests

Conversation

Copilot AI commented Oct 7, 2026

Copy link
Copy Markdown
Contributor

What this PR does:

Adds focused tests for previously uncovered response-size limiter context behavior.

  • Shared budget: Child contexts enforce one aggregate limit, accepting exactly 4096 bytes and rejecting further additions.
  • Isolation: Separate queries retain independent budgets.
  • Fallback: Contexts without a limiter allow unrestricted response bytes.

Limiter package tests pass; focused tests pass 20 repetitions with race detection. No production changes.

Which issue(s) this PR fixes:

Checklist

  • Tests updated
  • Documentation added
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]
  • docs/configuration/v1-guarantees.md updated if this PR introduces experimental flags

Signed-off-by: GitHub <noreply@github.com>

Co-authored-by: CharlieTLe <3375195+CharlieTLe@users.noreply.github.com>

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants