From 7346e55750f9929397432acfa3add02632bb52b4 Mon Sep 17 00:00:00 2001
From: Rich Trott <rtrott@gmail.com>
Date: Thu, 8 Dec 2016 22:46:39 -0800
Subject: [PATCH] test: refactor http pipelined socket test

In test-http-incoming-pipelined-socket-destory:

* setTimeout() with no duration -> setImmediate()
* eliminate unneeded exit listener
* use common.mustCall()
* var -> const/let

PR-URL: https://github.com/nodejs/node/pull/10189
Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
---
 ...-http-incoming-pipelined-socket-destroy.js | 29 ++++++++-----------
 1 file changed, 12 insertions(+), 17 deletions(-)

diff --git a/test/parallel/test-http-incoming-pipelined-socket-destroy.js b/test/parallel/test-http-incoming-pipelined-socket-destroy.js
index 9789e9976c..1465f189dd 100644
--- a/test/parallel/test-http-incoming-pipelined-socket-destroy.js
+++ b/test/parallel/test-http-incoming-pipelined-socket-destroy.js
@@ -1,16 +1,17 @@
 'use strict';
-require('../common');
+const common = require('../common');
 
-var http = require('http');
-var net = require('net');
+const http = require('http');
+const net = require('net');
 
+const seeds = [ 3, 1, 2, 3, 4, 1, 2, 3, 4, 1, 2, 3, 4 ];
 
 // Set up some timing issues where sockets can be destroyed
 // via either the req or res.
-var server = http.createServer(function(req, res) {
+const server = http.createServer(common.mustCall(function(req, res) {
   switch (req.url) {
     case '/1':
-      return setTimeout(function() {
+      return setImmediate(function() {
         req.socket.destroy();
         server.emit('requestDone');
       });
@@ -24,7 +25,7 @@ var server = http.createServer(function(req, res) {
     // in one case, actually send a response in 2 chunks
     case '/3':
       res.write('hello ');
-      return setTimeout(function() {
+      return setImmediate(function() {
         res.end('world!');
         server.emit('requestDone');
       });
@@ -33,7 +34,7 @@ var server = http.createServer(function(req, res) {
       res.destroy();
       server.emit('requestDone');
   }
-});
+}, seeds.length));
 
 
 // Make a bunch of requests pipelined on the same socket
@@ -47,10 +48,9 @@ function generator(seeds) {
 }
 
 
-server.listen(0, function() {
-  var seeds = [ 3, 1, 2, 3, 4, 1, 2, 3, 4, 1, 2, 3, 4 ];
-  var client = net.connect({ port: this.address().port });
-  var done = 0;
+server.listen(0, common.mustCall(function() {
+  const client = net.connect({ port: this.address().port });
+  let done = 0;
   server.on('requestDone', function() {
     if (++done === seeds.length) {
       server.close();
@@ -60,9 +60,4 @@ server.listen(0, function() {
   // immediately write the pipelined requests.
   // Some of these will not have a socket to destroy!
   client.write(generator(seeds));
-});
-
-process.on('exit', function(c) {
-  if (!c)
-    console.log('ok');
-});
+}));