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

PR-URL: #17201 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Evan Lucas <[email protected]> Reviewed-By: Gireesh Punathil <[email protected]> Reviewed-By: Jon Moss <[email protected]>
nodejs · Jan 23, 2018 · ec1e260 · ec1e260
1 parent d56b589
commit ec1e260
Showing 1 changed file with 4 additions and 4 deletions.
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() {