Skip to content

test: improve http res write and end dont take array#20199

Closed
ilsotov wants to merge 1 commit into
nodejs:masterfrom
ilsotov:test-res-write-and-end-dont-take-arr
Closed

test: improve http res write and end dont take array#20199
ilsotov wants to merge 1 commit into
nodejs:masterfrom
ilsotov:test-res-write-and-end-dont-take-arr

Conversation

@ilsotov

@ilsotov ilsotov commented Apr 21, 2018

Copy link
Copy Markdown
Contributor

Checking if the error thrown in res.write and res.end is ERR_INVALID_ARG_TYPE if arg is array.

Checklist

  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines

@nodejs-github-bot nodejs-github-bot added the test Issues and PRs related to the tests. label Apr 21, 2018
@lpinca

lpinca commented Apr 21, 2018

Copy link
Copy Markdown
Member

@lpinca lpinca added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Apr 21, 2018
trivikr pushed a commit that referenced this pull request Apr 23, 2018
PR-URL: #20199
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
@trivikr

trivikr commented Apr 23, 2018

Copy link
Copy Markdown
Member

Landed in 12b90b4

Congratulations @ilsotov on your first commit to Node.js core! 🎉🎉🎉

@trivikr trivikr closed this Apr 23, 2018
@ilsotov ilsotov deleted the test-res-write-and-end-dont-take-arr branch April 23, 2018 15:34
jasnell pushed a commit that referenced this pull request Apr 23, 2018
PR-URL: #20199
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author ready PRs that have at least one approval, no pending requests for changes, and a CI started. test Issues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants