summaryrefslogtreecommitdiff
path: root/test/parallel/test-http-parser.js
diff options
context:
space:
mode:
authorGibson Fahnestock <gib@uk.ibm.com>2017-01-08 13:19:00 +0000
committerGibson Fahnestock <gib@uk.ibm.com>2017-01-11 11:43:52 +0000
commit7a0e462f9facfff8ccd7b37eb5b65db1c2b2f55f (patch)
treea971102d320e17e6cb3d00c48fe708b2b86c8136 /test/parallel/test-http-parser.js
parent1ef401ce92d6195878b9d041cc969612628f5852 (diff)
downloadandroid-node-v8-7a0e462f9facfff8ccd7b37eb5b65db1c2b2f55f.tar.gz
android-node-v8-7a0e462f9facfff8ccd7b37eb5b65db1c2b2f55f.tar.bz2
android-node-v8-7a0e462f9facfff8ccd7b37eb5b65db1c2b2f55f.zip
test: use eslint to fix var->const/let
Manually fix issues that eslint --fix couldn't do automatically. PR-URL: https://github.com/nodejs/node/pull/10685 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Roman Reiss <me@silverwind.io>
Diffstat (limited to 'test/parallel/test-http-parser.js')
-rw-r--r--test/parallel/test-http-parser.js32
1 files changed, 16 insertions, 16 deletions
diff --git a/test/parallel/test-http-parser.js b/test/parallel/test-http-parser.js
index 8617a093fb..e1215fbd13 100644
--- a/test/parallel/test-http-parser.js
+++ b/test/parallel/test-http-parser.js
@@ -6,14 +6,14 @@ const binding = process.binding('http_parser');
const methods = binding.methods;
const HTTPParser = binding.HTTPParser;
-var CRLF = '\r\n';
-var REQUEST = HTTPParser.REQUEST;
-var RESPONSE = HTTPParser.RESPONSE;
+const CRLF = '\r\n';
+const REQUEST = HTTPParser.REQUEST;
+const RESPONSE = HTTPParser.RESPONSE;
-var kOnHeaders = HTTPParser.kOnHeaders | 0;
-var kOnHeadersComplete = HTTPParser.kOnHeadersComplete | 0;
-var kOnBody = HTTPParser.kOnBody | 0;
-var kOnMessageComplete = HTTPParser.kOnMessageComplete | 0;
+const kOnHeaders = HTTPParser.kOnHeaders | 0;
+const kOnHeadersComplete = HTTPParser.kOnHeadersComplete | 0;
+const kOnBody = HTTPParser.kOnBody | 0;
+const kOnMessageComplete = HTTPParser.kOnMessageComplete | 0;
// The purpose of this test is not to check HTTP compliance but to test the
// binding. Tests for pathological http messages should be submitted
@@ -22,7 +22,7 @@ var kOnMessageComplete = HTTPParser.kOnMessageComplete | 0;
function newParser(type) {
- var parser = new HTTPParser(type);
+ const parser = new HTTPParser(type);
parser.headers = [];
parser.url = '';
@@ -47,7 +47,7 @@ function newParser(type) {
function mustCall(f, times) {
- var actual = 0;
+ let actual = 0;
process.setMaxListeners(256);
process.on('exit', function() {
@@ -63,7 +63,7 @@ function mustCall(f, times) {
function expectBody(expected) {
return mustCall(function(buf, start, len) {
- var body = '' + buf.slice(start, start + len);
+ const body = '' + buf.slice(start, start + len);
assert.equal(body, expected);
});
}
@@ -447,11 +447,11 @@ function expectBody(expected) {
assert.strictEqual(expected_body, '');
}
- for (var i = 1; i < request.length - 1; ++i) {
- var a = request.slice(0, i);
+ for (let i = 1; i < request.length - 1; ++i) {
+ const a = request.slice(0, i);
console.error('request.slice(0, ' + i + ') = ',
JSON.stringify(a.toString()));
- var b = request.slice(i);
+ const b = request.slice(i);
console.error('request.slice(' + i + ') = ',
JSON.stringify(b.toString()));
test(a, b);
@@ -571,11 +571,11 @@ function expectBody(expected) {
// Test parser 'this' safety
// https://github.com/joyent/node/issues/6690
assert.throws(function() {
- var request = Buffer.from(
+ const request = Buffer.from(
'GET /hello HTTP/1.1' + CRLF +
CRLF);
- var parser = newParser(REQUEST);
- var notparser = { execute: parser.execute };
+ const parser = newParser(REQUEST);
+ const notparser = { execute: parser.execute };
notparser.execute(request, 0, request.length);
}, TypeError);