Browse Source

test: refector parallel/test-http.js

* favor ’===’ over in ’==’
* favor ’assert.strictEqual’ over ’assert.equal’
* favor ’const’ over ’var’

PR-URL: https://github.com/nodejs/node/pull/8471
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
v4.x
Junshu Okamoto 8 years ago
committed by Myles Borins
parent
commit
8da2dcb70a
  1. 45
      test/parallel/test-http.js

45
test/parallel/test-http.js

@ -1,30 +1,30 @@
'use strict'; 'use strict';
require('../common'); require('../common');
var assert = require('assert'); const assert = require('assert');
var http = require('http'); const http = require('http');
var url = require('url'); const url = require('url');
var responses_sent = 0; var responses_sent = 0;
var responses_recvd = 0; var responses_recvd = 0;
var body0 = ''; var body0 = '';
var body1 = ''; var body1 = '';
var server = http.Server(function(req, res) { const server = http.Server(function(req, res) {
if (responses_sent == 0) { if (responses_sent === 0) {
assert.equal('GET', req.method); assert.strictEqual('GET', req.method);
assert.equal('/hello', url.parse(req.url).pathname); assert.strictEqual('/hello', url.parse(req.url).pathname);
console.dir(req.headers); console.dir(req.headers);
assert.equal(true, 'accept' in req.headers); assert.strictEqual(true, 'accept' in req.headers);
assert.equal('*/*', req.headers['accept']); assert.strictEqual('*/*', req.headers['accept']);
assert.equal(true, 'foo' in req.headers); assert.strictEqual(true, 'foo' in req.headers);
assert.equal('bar', req.headers['foo']); assert.strictEqual('bar', req.headers['foo']);
} }
if (responses_sent == 1) { if (responses_sent === 1) {
assert.equal('POST', req.method); assert.strictEqual('POST', req.method);
assert.equal('/world', url.parse(req.url).pathname); assert.strictEqual('/world', url.parse(req.url).pathname);
this.close(); this.close();
} }
@ -41,14 +41,14 @@ var server = http.Server(function(req, res) {
server.listen(0); server.listen(0);
server.on('listening', function() { server.on('listening', function() {
var agent = new http.Agent({ port: this.address().port, maxSockets: 1 }); const agent = new http.Agent({ port: this.address().port, maxSockets: 1 });
http.get({ http.get({
port: this.address().port, port: this.address().port,
path: '/hello', path: '/hello',
headers: {'Accept': '*/*', 'Foo': 'bar'}, headers: {'Accept': '*/*', 'Foo': 'bar'},
agent: agent agent: agent
}, function(res) { }, function(res) {
assert.equal(200, res.statusCode); assert.strictEqual(200, res.statusCode);
responses_recvd += 1; responses_recvd += 1;
res.setEncoding('utf8'); res.setEncoding('utf8');
res.on('data', function(chunk) { body0 += chunk; }); res.on('data', function(chunk) { body0 += chunk; });
@ -56,13 +56,13 @@ server.on('listening', function() {
}); });
setTimeout(function() { setTimeout(function() {
var req = http.request({ const req = http.request({
port: server.address().port, port: server.address().port,
method: 'POST', method: 'POST',
path: '/world', path: '/world',
agent: agent agent: agent
}, function(res) { }, function(res) {
assert.equal(200, res.statusCode); assert.strictEqual(200, res.statusCode);
responses_recvd += 1; responses_recvd += 1;
res.setEncoding('utf8'); res.setEncoding('utf8');
res.on('data', function(chunk) { body1 += chunk; }); res.on('data', function(chunk) { body1 += chunk; });
@ -74,12 +74,11 @@ server.on('listening', function() {
process.on('exit', function() { process.on('exit', function() {
console.error('responses_recvd: ' + responses_recvd); console.error('responses_recvd: ' + responses_recvd);
assert.equal(2, responses_recvd); assert.strictEqual(2, responses_recvd);
console.error('responses_sent: ' + responses_sent); console.error('responses_sent: ' + responses_sent);
assert.equal(2, responses_sent); assert.strictEqual(2, responses_sent);
assert.equal('The path was /hello', body0); assert.strictEqual('The path was /hello', body0);
assert.equal('The path was /world', body1); assert.strictEqual('The path was /world', body1);
}); });

Loading…
Cancel
Save