1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

test: fix test-tls-server-verify.js on Windows CI

Browse Source 
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>
This commit is contained in:
Rich Trott 2018-01-25 10:42:33 -08:00
parent 082f9525d9
commit 1ecd2be2e6
1 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
test/parallel/test-tls-server-verify.js
View File

@ -327,7 +327,7 @@ function runTest(port, testIndex) {
} else { } else {
server.close(); server.close();
successfulTests++; successfulTests++;
runTest(port, nextTest++); runTest(0, nextTest++);
} }
} }
@ -345,7 +345,7 @@ function runTest(port, testIndex) {
if (clientsCompleted === tcase.clients.length) { if (clientsCompleted === tcase.clients.length) {
server.close(); server.close();
successfulTests++; successfulTests++;
runTest(port, nextTest++); runTest(0, nextTest++);
} }
}); });
} }
@ -356,7 +356,6 @@ function runTest(port, testIndex) {
let nextTest = 0; let nextTest = 0;
runTest(0, nextTest++); runTest(0, nextTest++);
runTest(0, nextTest++);
process.on('exit', function() { process.on('exit', function() {