test: use countdown timer

PR-URL: #17326 Refs: #17169 Reviewed-By: Jon Moss <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]>
nodejs · Jan 30, 2018 · fc89cea · fc89cea
1 parent 0250e1b
commit fc89cea
Showing 1 changed file with 4 additions and 10 deletions.
diff --git a/test/parallel/test-http-response-status-message.js b/test/parallel/test-http-response-status-message.js
@@ -24,8 +24,7 @@ require('../common');
 const assert = require('assert');
 const http = require('http');
 const net = require('net');
-
-let testsComplete = 0;
+const Countdown = require('../common/countdown');
 
 const testCases = [
   { path: '/200', statusMessage: 'OK',
@@ -59,6 +58,8 @@ const server = net.createServer(function(connection) {
   });
 });
 
+const countdown = new Countdown(testCases.length, () => server.close());
+
 function runTest(testCaseIndex) {
   const testCase = testCases[testCaseIndex];
 
@@ -71,12 +72,9 @@ function runTest(testCaseIndex) {
     assert.strictEqual(testCase.statusMessage, response.statusMessage);
 
     response.on('end', function() {
-      testsComplete++;
-
+      countdown.dec();
       if (testCaseIndex + 1 < testCases.length) {
         runTest(testCaseIndex + 1);
-      } else {
-        server.close();
       }
     });
 
@@ -85,7 +83,3 @@ function runTest(testCaseIndex) {
 }
 
 server.listen(0, function() { runTest(0); });
-
-process.on('exit', function() {
-  assert.strictEqual(testCases.length, testsComplete);
-});