Skip to content

test(middleware): cover accept negotiation input validation - #1839

Merged
Jagadeeshftw merged 1 commit into
Fluxora-Org:mainfrom
Ova-Klik:test/accept-negotiation-input-validation
Oct 21, 2026
Merged

Jagadeeshftw merged 1 commit into
Fluxora-Org:mainfrom
Ova-Klik:test/accept-negotiation-input-validation

Conversation

@Ova-Klik

Copy link
Copy Markdown
Contributor

Summary

Closes #1779

Add test coverage for malformed input handling in src/middleware/acceptNegotiation.ts.

Changes

  • Cover malformed, empty, and oversized inputs.
  • Assert typed errors and useful rejection reasons.
  • Verify rejected inputs do not mutate state.
  • Confirm valid input retains its existing behavior.
  • Keep changes focused on tests unless a test demonstrates a genuine defect.

Validation

Run pnpm test and confirm the test suite passes.

@drips-wave

drips-wave Bot commented Oct 19, 2026

Copy link
Copy Markdown

@Ova-Klik Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@Jagadeeshftw

Copy link
Copy Markdown
Contributor

Looks good to me, @Ova-Klik — nice work. Bringing this in.

@Jagadeeshftw
Jagadeeshftw merged commit 713f365 into Fluxora-Org:main Oct 21, 2026
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.

Harden the input validation in acceptNegotiation

2 participants