Skip to content

Commit

Permalink
test: refactor the code in test-http-keep-alive
Browse files Browse the repository at this point in the history
* use common.mustCall to control the functions execution automatically
* use let and const instead of var
* use assert.strictEqual instead assert.equal
  • Loading branch information
edsadr committed Dec 21, 2016
1 parent a308a2f commit 53a872a
Showing 1 changed file with 25 additions and 25 deletions.
50 changes: 25 additions & 25 deletions test/parallel/test-http-keep-alive.js
Original file line number Diff line number Diff line change
@@ -1,51 +1,51 @@
'use strict';
require('../common');
var assert = require('assert');
var http = require('http');
const common = require('../common');
const assert = require('assert');
const http = require('http');

var body = 'hello world\n';
const server = http.createServer(common.mustCall((req, res) => {
const body = 'hello world\n';

var server = http.createServer(function(req, res) {
res.writeHead(200, {'Content-Length': body.length});
res.write(body);
res.end();
});
}, 3));

var agent = new http.Agent({maxSockets: 1});
var headers = {'connection': 'keep-alive'};
var name;
const agent = new http.Agent({maxSockets: 1});
const headers = {'connection': 'keep-alive'};
let name;

server.listen(0, function() {
server.listen(0, common.mustCall(function() {
name = agent.getName({ port: this.address().port });
http.get({
path: '/', headers: headers, port: this.address().port, agent: agent
}, function(response) {
assert.equal(agent.sockets[name].length, 1);
assert.equal(agent.requests[name].length, 2);
}, common.mustCall((response) => {
assert.strictEqual(agent.sockets[name].length, 1);
assert.strictEqual(agent.requests[name].length, 2);
response.resume();
});
}));

http.get({
path: '/', headers: headers, port: this.address().port, agent: agent
}, function(response) {
assert.equal(agent.sockets[name].length, 1);
assert.equal(agent.requests[name].length, 1);
}, common.mustCall((response) => {
assert.strictEqual(agent.sockets[name].length, 1);
assert.strictEqual(agent.requests[name].length, 1);
response.resume();
});
}));

http.get({
path: '/', headers: headers, port: this.address().port, agent: agent
}, function(response) {
response.on('end', function() {
assert.equal(agent.sockets[name].length, 1);
}, common.mustCall((response) => {
response.on('end', common.mustCall(() => {
assert.strictEqual(agent.sockets[name].length, 1);
assert(!agent.requests.hasOwnProperty(name));
server.close();
});
}));
response.resume();
});
});
}));
}));

process.on('exit', function() {
process.on('exit', () => {
assert(!agent.sockets.hasOwnProperty(name));
assert(!agent.requests.hasOwnProperty(name));
});

0 comments on commit 53a872a

Please sign in to comment.