From 79016c5abf50033dcf110c57b4c45aa6b449dd8d Mon Sep 17 00:00:00 2001 From: Rich Trott Date: Thu, 25 Jan 2018 10:42:33 -0800 Subject: [PATCH] test: fix 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 PR-URL: https://github.com/nodejs/node/pull/18382 Reviewed-By: Joyee Cheung Reviewed-By: James M Snell Reviewed-By: Colin Ihrig --- 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 f482b87d2308c3..609ac3ddc02afe 100644 --- a/test/parallel/test-tls-server-verify.js +++ b/test/parallel/test-tls-server-verify.js @@ -306,7 +306,7 @@ function runTest(port, testIndex) { } else { server.close(); successfulTests++; - runTest(port, nextTest++); + runTest(0, nextTest++); } } @@ -325,7 +325,7 @@ function runTest(port, testIndex) { if (clientsCompleted === tcase.clients.length) { server.close(); successfulTests++; - runTest(port, nextTest++); + runTest(0, nextTest++); } }); } @@ -337,7 +337,6 @@ function runTest(port, testIndex) { let nextTest = 0; runTest(0, nextTest++); -runTest(0, nextTest++); process.on('exit', function() {