Skip to content

test: increase coverage of Buffer.transcode - #10437

Closed
joyeecheung wants to merge 1 commit into
nodejs:masterfrom
joyeecheung:test-empty-transcode
Closed

test: increase coverage of Buffer.transcode#10437
joyeecheung wants to merge 1 commit into
nodejs:masterfrom
joyeecheung:test-empty-transcode

Conversation

@joyeecheung

Copy link
Copy Markdown
Member
Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • commit message follows commit guidelines
Affected core subsystem(s)

test, buffer

Description of change

Adds test for transcoding an empty buffer.

Adds test for transcoding an empty buffer.
@nodejs-github-botnodejs-github-bot added test Issues and PRs related to the tests. dont-land-on-v7.x labels Dec 24, 2016
@mscdexmscdex added the buffer Issues and PRs related to the buffer subsystem. label Dec 24, 2016

@TrottTrott left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM if CI is green.

@Trott

Copy link
Copy Markdown
Member

@evanlucasevanlucas left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM Thanks!

jasnell pushed a commit that referenced this pull request Dec 24, 2016
Adds test for transcoding an empty buffer.
PR-URL: #10437
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
@jasnell

Copy link
Copy Markdown
Member

Landed in 904b66d

@jasnelljasnell closed this Dec 24, 2016
@joyeecheung
joyeecheung deleted the test-empty-transcode branch January 2, 2017 05:28
italoacasas pushed a commit to italoacasas/node that referenced this pull request Jan 18, 2017
Adds test for transcoding an empty buffer.
PR-URL: nodejs#10437
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
@jasnelljasnell mentioned this pull request Apr 4, 2017
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bufferIssues and PRs related to the buffer subsystem.testIssues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants

@joyeecheung@Trott@jasnell@evanlucas@targos@cjihrig@mscdex@nodejs-github-bot