Skip to content

Commit

Permalink
test: refactor test-http-set-timeout-server
Browse files Browse the repository at this point in the history
* Use `common.mustCall()` to track callback invocations
* Remove console.log() statements unrelated to the test
* Add blank line to conform with test-writing guide

PR-URL: #13802
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Refael Ackermann <[email protected]>
  • Loading branch information
Trott authored and MylesBorins committed Aug 16, 2017
1 parent 389f294 commit 842b84c
Showing 1 changed file with 6 additions and 8 deletions.
14 changes: 6 additions & 8 deletions test/parallel/test-http-set-timeout-server.js
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
'use strict';
const common = require('../common');

const assert = require('assert');
const http = require('http');
const net = require('net');
Expand All @@ -9,16 +10,13 @@ const tests = [];
function test(fn) {
if (!tests.length)
process.nextTick(run);
tests.push(fn);
tests.push(common.mustCall(fn));
}

function run() {
const fn = tests.shift();
if (fn) {
console.log('# %s', fn.name);
fn(run);
} else {
console.log('ok');
}
}

Expand All @@ -27,7 +25,7 @@ test(function serverTimeout(cb) {
// just do nothing, we should get a timeout event.
});
server.listen(common.mustCall(function() {
http.get({ port: server.address().port }).on('error', common.noop);
http.get({ port: server.address().port }).on('error', common.mustCall());
}));
const s = server.setTimeout(50, common.mustCall(function(socket) {
socket.destroy();
Expand All @@ -50,7 +48,7 @@ test(function serverRequestTimeout(cb) {
server.listen(common.mustCall(function() {
const port = server.address().port;
const req = http.request({ port: port, method: 'POST' });
req.on('error', common.noop);
req.on('error', common.mustCall());
req.write('Hello');
// req is in progress
}));
Expand All @@ -68,7 +66,7 @@ test(function serverResponseTimeout(cb) {
});
server.listen(common.mustCall(function() {
const port = server.address().port;
http.get({ port: port }).on('error', common.noop);
http.get({ port: port }).on('error', common.mustCall());
}));
});

Expand All @@ -86,7 +84,7 @@ test(function serverRequestNotTimeoutAfterEnd(cb) {
});
server.listen(common.mustCall(function() {
const port = server.address().port;
http.get({ port: port }).on('error', common.noop);
http.get({ port: port }).on('error', common.mustCall());
}));
});

Expand Down

0 comments on commit 842b84c

Please sign in to comment.