diff options
author | Runite618 <matthew_mo@hotmail.co.uk> | 2017-08-13 20:23:20 +0100 |
---|---|---|
committer | Tobias Nießen <tniessen@tnie.de> | 2017-08-16 12:06:47 +0200 |
commit | d4374ad6514b5825021597eb3fccdce241b66f70 (patch) | |
tree | 2c0adee7dd1e07f313328479009ff48913a2aa4f /test/async-hooks/test-httpparser.response.js | |
parent | 3f5d944ace8b0a6ac2f4c76a5873a9bb6ec27b2f (diff) | |
download | android-node-v8-d4374ad6514b5825021597eb3fccdce241b66f70.tar.gz android-node-v8-d4374ad6514b5825021597eb3fccdce241b66f70.tar.bz2 android-node-v8-d4374ad6514b5825021597eb3fccdce241b66f70.zip |
test: refactor async-hooks/test-httparser tests
CRLF variable was defined but only used on line 22 so the variable
was deleted and placed inside line 22 as a string literal. This
was in file test-httpparser.request.js
On line 46 there's a function declared that takes 3 arguments but
none of them are ever used so removed. This is in file
test-httpparser.response.js
PR-URL: https://github.com/nodejs/node/pull/14818
Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
Reviewed-By: Alexey Orlenko <eaglexrlnk@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Diffstat (limited to 'test/async-hooks/test-httpparser.response.js')
-rw-r--r-- | test/async-hooks/test-httpparser.response.js | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/test/async-hooks/test-httpparser.response.js b/test/async-hooks/test-httpparser.response.js index 5723ac6fbd..d56e3e1fec 100644 --- a/test/async-hooks/test-httpparser.response.js +++ b/test/async-hooks/test-httpparser.response.js @@ -43,7 +43,7 @@ function onheadersComplete() { 'when onheadersComplete called'); } -function onbody(buf, start, len) { +function onbody() { checkInvocations(httpparser, { init: 1, before: 2, after: 1 }, 'when onbody called'); tick(1, common.mustCall(tick1)); |