From 24e0507f5aafcd3f114f1a3181f980e4d3e48015 Mon Sep 17 00:00:00 2001
From: Rich Trott <rtrott@gmail.com>
Date: Mon, 19 Jun 2017 11:56:52 -0700
Subject: [PATCH] test: refactor test-http-set-timeout-server

* Use `common.mustCall()` to track callback invocations
* Remove console.log() statements unrelated to the test
* Add blank line to conform with test-writing guide
---
 test/parallel/test-http-set-timeout-server.js | 14 ++++++--------
 1 file changed, 6 insertions(+), 8 deletions(-)

diff --git a/test/parallel/test-http-set-timeout-server.js b/test/parallel/test-http-set-timeout-server.js
index 1354dc657261eb..b1b9e98ac0baf7 100644
--- a/test/parallel/test-http-set-timeout-server.js
+++ b/test/parallel/test-http-set-timeout-server.js
@@ -21,6 +21,7 @@
 
 'use strict';
 const common = require('../common');
+
 const assert = require('assert');
 const http = require('http');
 const net = require('net');
@@ -30,16 +31,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');
   }
 }
 
@@ -48,7 +46,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();
@@ -71,7 +69,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
   }));
@@ -89,7 +87,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());
   }));
 });
 
@@ -107,7 +105,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());
   }));
 });