diff options
author | Rich Trott <rtrott@gmail.com> | 2018-01-25 10:42:33 -0800 |
---|---|---|
committer | Rich Trott <rtrott@gmail.com> | 2018-01-26 10:57:00 -0800 |
commit | 1ecd2be2e65d8d33ed6f110422b6795cdf066f01 (patch) | |
tree | c9e151d590c0df5ff20d6f0a45df82bb12aaed8c | |
parent | 082f9525d92532867bc7172acebdf9b89a715dab (diff) | |
download | android-node-v8-1ecd2be2e65d8d33ed6f110422b6795cdf066f01.tar.gz android-node-v8-1ecd2be2e65d8d33ed6f110422b6795cdf066f01.tar.bz2 android-node-v8-1ecd2be2e65d8d33ed6f110422b6795cdf066f01.zip |
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 <joyeec9h3@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
-rw-r--r-- | test/parallel/test-tls-server-verify.js | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/test/parallel/test-tls-server-verify.js b/test/parallel/test-tls-server-verify.js index 3e5576fda7..eeea8b030d 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() { |