Skip to content

Commit 51999d0

Browse files
TrottMylesBorins
authored andcommitted
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: #18269 PR-URL: #18382 Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
1 parent 3a648b7 commit 51999d0

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

test/parallel/test-tls-server-verify.js

+2-3
Original file line numberDiff line numberDiff line change
@@ -327,7 +327,7 @@ function runTest(port, testIndex) {
327327
} else {
328328
server.close();
329329
successfulTests++;
330-
runTest(port, nextTest++);
330+
runTest(0, nextTest++);
331331
}
332332
}
333333

@@ -345,7 +345,7 @@ function runTest(port, testIndex) {
345345
if (clientsCompleted === tcase.clients.length) {
346346
server.close();
347347
successfulTests++;
348-
runTest(port, nextTest++);
348+
runTest(0, nextTest++);
349349
}
350350
});
351351
}
@@ -356,7 +356,6 @@ function runTest(port, testIndex) {
356356

357357
let nextTest = 0;
358358
runTest(0, nextTest++);
359-
runTest(0, nextTest++);
360359

361360

362361
process.on('exit', function() {

0 commit comments

Comments
 (0)