test: update http test to use common.mustCall

collin5 · gibfahn · commit 32adb708517c · 2017-12-20T01:14:12.000Z
PR-URL: #17528 Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>
diff --git a/test/parallel/test-http-timeout-overflow.js b/test/parallel/test-http-timeout-overflow.js
@@ -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);
-});
