Skip to content

Commit

Permalink
test: fix test/parallel/test-tls-server-verify.js on Windows CI
Browse files Browse the repository at this point in the history
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: #18269
  • Loading branch information
Trott committed Jan 26, 2018
1 parent bea5f26 commit e3a4fd1
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions test/parallel/test-tls-server-verify.js
Expand Up @@ -327,7 +327,7 @@ function runTest(port, testIndex) {
} else {
server.close();
successfulTests++;
runTest(port, nextTest++);
runTest(0, nextTest++);
}
}

Expand All @@ -345,7 +345,7 @@ function runTest(port, testIndex) {
if (clientsCompleted === tcase.clients.length) {
server.close();
successfulTests++;
runTest(port, nextTest++);
runTest(0, nextTest++);
}
});
}
Expand All @@ -356,7 +356,6 @@ function runTest(port, testIndex) {

let nextTest = 0;
runTest(0, nextTest++);
runTest(0, nextTest++);


process.on('exit', function() {
Expand Down

0 comments on commit e3a4fd1

Please sign in to comment.