test: update http test to use common.mustCall

PR-URL: https://github.com/nodejs/node/pull/17528
Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>
This commit is contained in:
Collins Abitekaniza 2017-12-07 21:48:59 +03:00 committed by Anatoli Papirovski
parent c1689dc32c
commit d2626ef6b7
No known key found for this signature in database
GPG Key ID: 614E2E1ABEB4B2C0

View File

@ -20,19 +20,13 @@
// USE OR OTHER DEALINGS IN THE SOFTWARE.
'use strict';
require('../common');
const assert = require('assert');
const common = require('../common');
const http = require('http');
let serverRequests = 0;
let clientRequests = 0;
const server = http.createServer(function(req, res) {
serverRequests++;
const server = http.createServer(common.mustCall(function(req, res) {
res.writeHead(200, { 'Content-Type': 'text/plain' });
res.end('OK');
});
}));
server.listen(0, function() {
function callback() {}
@ -44,10 +38,9 @@ server.listen(0, function() {
}, function(res) {
req.clearTimeout(callback);
res.on('end', function() {
clientRequests++;
res.on('end', common.mustCall(function() {
server.close();
});
}));
res.resume();
});
@ -56,8 +49,3 @@ server.listen(0, function() {
req.setTimeout(0xffffffff, callback);
req.end();
});
process.once('exit', function() {
assert.strictEqual(clientRequests, 1);
assert.strictEqual(serverRequests, 1);
});