test: update test-http-content-length to use countdown

Bamieh · MylesBorins · commit c626f8a71799 · 2018-01-08T02:26:52.000-05:00
PR-URL: #17201 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Evan Lucas <evanlucas@me.com> Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com> Reviewed-By: Jon Moss <me@jonathanmoss.me>
diff --git a/test/parallel/test-http-content-length.js b/test/parallel/test-http-content-length.js
@@ -2,6 +2,7 @@
 require('../common');
 const assert = require('assert');
 const http = require('http');
+const Countdown = require('../common/countdown');
 
 const expectedHeadersMultipleWrites = {
   'connection': 'close',
@@ -18,8 +19,8 @@ const expectedHeadersEndNoData = {
   'content-length': '0',
 };
 
-let receivedRequests = 0;
-const totalRequests = 3;
+
+const countdown = new Countdown(3, () => server.close());
 
 const server = http.createServer(function(req, res) {
   res.removeHeader('Date');
@@ -42,8 +43,7 @@ const server = http.createServer(function(req, res) {
       throw new Error('Unreachable');
   }
 
-  receivedRequests++;
-  if (totalRequests === receivedRequests) server.close();
+  countdown.dec();
 });
 
 server.listen(0, function() {
