test: refactored http test to use countdown

mithunsasidharan · gibfahn · commit 4ebd75bce9ca · 2017-12-19T22:25:40.000Z
PR-URL: #17241 Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com> Reviewed-By: Anatoli Papirovski <apapirovski@mac.com> Reviewed-By: Anna Henningsen <anna@addaleax.net>
diff --git a/test/parallel/test-http-end-throw-socket-handling.js b/test/parallel/test-http-end-throw-socket-handling.js
@@ -21,6 +21,7 @@
 
 'use strict';
 const common = require('../common');
+const Countdown = require('../common/countdown');
 
 // Make sure that throwing in 'end' handler doesn't lock
 // up the socket forever.
@@ -29,10 +30,10 @@ const common = require('../common');
 // the same, we should not be so brittle and easily broken.
 
 const http = require('http');
+const countdown = new Countdown(10, () => server.close());
 
-let n = 0;
 const server = http.createServer((req, res) => {
-  if (++n === 10) server.close();
+  countdown.dec();
   res.end('ok');
 });
 
