Skip to content

Commit 79016c5

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 86fbebe commit 79016c5

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
@@ -306,7 +306,7 @@ function runTest(port, testIndex) {
306306
} else {
307307
server.close();
308308
successfulTests++;
309-
runTest(port, nextTest++);
309+
runTest(0, nextTest++);
310310
}
311311
}
312312

@@ -325,7 +325,7 @@ function runTest(port, testIndex) {
325325
if (clientsCompleted === tcase.clients.length) {
326326
server.close();
327327
successfulTests++;
328-
runTest(port, nextTest++);
328+
runTest(0, nextTest++);
329329
}
330330
});
331331
}
@@ -337,7 +337,6 @@ function runTest(port, testIndex) {
337337

338338
let nextTest = 0;
339339
runTest(0, nextTest++);
340-
runTest(0, nextTest++);
341340

342341

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

0 commit comments

Comments
 (0)