Skip to content

test: increase test coverage of BufferList#10171

Closed
joyeecheung wants to merge 1 commit intonodejs:masterfrom
joyeecheung:test-buffer-list
Closed

test: increase test coverage of BufferList#10171
joyeecheung wants to merge 1 commit intonodejs:masterfrom
joyeecheung:test-buffer-list

Conversation

@joyeecheung
Copy link
Copy Markdown
Member

Checklist
  • make -j8 test (UNIX), or vcbuild test nosign (Windows) passes
  • commit message follows commit guidelines
Affected core subsystem(s)

test

Description of change

Add tests for edges cases of BufferList

  • test operations when the length is 0
  • test operations when the list only has one element

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

Labels

stream Issues and PRs related to the stream subsystem. test Issues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants