test: use Countdown in http test

PR-URL: #17506 Reviewed-By: Anatoli Papirovski <[email protected]> Reviewed-By: Jon Moss <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
nodejs · Dec 12, 2017 · f5cbc68 · f5cbc68
1 parent 0329caa
commit f5cbc68
Showing 1 changed file with 3 additions and 4 deletions.
diff --git a/test/parallel/test-http-incoming-pipelined-socket-destroy.js b/test/parallel/test-http-incoming-pipelined-socket-destroy.js
@@ -21,11 +21,13 @@
 
 'use strict';
 const common = require('../common');
+const Countdown = require('../common/countdown');
 
 const http = require('http');
 const net = require('net');
 
 const seeds = [ 3, 1, 2, 3, 4, 1, 2, 3, 4, 1, 2, 3, 4 ];
+const countdown = new Countdown(seeds.length, () => server.close());
 
 // Set up some timing issues where sockets can be destroyed
 // via either the req or res.
@@ -72,11 +74,8 @@ function generator(seeds) {
 
 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();
-    }
+    countdown.dec();
   });
 
   // immediately write the pipelined requests.