From e3a4fd1434cba46845622b65f05ef11a0ced0347 Mon Sep 17 00:00:00 2001 From: Rich Trott Date: Thu, 25 Jan 2018 10:42:33 -0800 Subject: [PATCH] test: fix test/parallel/test-tls-server-verify.js on Windows CI The test runs two test cases at a time. This is not relevant to what the test is actually testing. Not sure why doing it that way causes a deadlock on some Windows servers, but running one at a time fixes it. Fixes: https://github.com/nodejs/node/issues/18269 --- test/parallel/test-tls-server-verify.js | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/test/parallel/test-tls-server-verify.js b/test/parallel/test-tls-server-verify.js index 3e5576fda7597f..eeea8b030da719 100644 --- a/test/parallel/test-tls-server-verify.js +++ b/test/parallel/test-tls-server-verify.js @@ -327,7 +327,7 @@ function runTest(port, testIndex) { } else { server.close(); successfulTests++; - runTest(port, nextTest++); + runTest(0, nextTest++); } } @@ -345,7 +345,7 @@ function runTest(port, testIndex) { if (clientsCompleted === tcase.clients.length) { server.close(); successfulTests++; - runTest(port, nextTest++); + runTest(0, nextTest++); } }); } @@ -356,7 +356,6 @@ function runTest(port, testIndex) { let nextTest = 0; runTest(0, nextTest++); -runTest(0, nextTest++); process.on('exit', function() {