Skip to content

test(packet): add unit test suite for packet payload buffer size validation - #5557

Open
gcoinstash-cmd wants to merge 1 commit into
socketio:mainfrom
gcoinstash-cmd:test/packet-payload-length-bounds
Open

gcoinstash-cmd wants to merge 1 commit into
socketio:mainfrom
gcoinstash-cmd:test/packet-payload-length-bounds

Conversation

@gcoinstash-cmd

Copy link
Copy Markdown

Summary

Adds isolated unit tests for packet payload buffer size clamping:

  • Valid payload byte bounds [0, maxPayload]
  • Upper bound rejection
  • Negative / NaN input rejection

@darrachequesne

Copy link
Copy Markdown
Member

Hi! Thanks for the pull request. Could you please explain the usage of the validatePacketPayloadSize() method?

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