From 0fdb33c3aa9f4c75d9e15e8a33d9c00116d9052f Mon Sep 17 00:00:00 2001 From: Matt Mastracci Date: Thu, 7 Mar 2024 10:04:39 -0700 Subject: fix(cli): limit test parallelism on Windows to avoid pipe error (#22776) One last attempt to fix the parallelism issue on Windows. --- cli/tools/test/mod.rs | 12 +++++++++++- 1 file changed, 11 insertions(+), 1 deletion(-) (limited to 'cli/tools/test/mod.rs') diff --git a/cli/tools/test/mod.rs b/cli/tools/test/mod.rs index 13cf9f774..4f500df3d 100644 --- a/cli/tools/test/mod.rs +++ b/cli/tools/test/mod.rs @@ -1191,8 +1191,18 @@ async fn test_specifiers( }) }); + // TODO(mmastrac): Temporarily limit concurrency in windows testing to avoid named pipe issue: + // *** Unexpected server pipe failure '"\\\\.\\pipe\\deno_pipe_e30f45c9df61b1e4.1198.222\\0"': 3 + // This is likely because we're hitting some sort of invisible resource limit + // This limit is both in cli/lsp/testing/execution.rs and cli/tools/test/mod.rs + let concurrent = if cfg!(windows) { + std::cmp::min(concurrent_jobs.get(), 4) + } else { + concurrent_jobs.get() + }; + let join_stream = stream::iter(join_handles) - .buffer_unordered(concurrent_jobs.get()) + .buffer_unordered(concurrent) .collect::, tokio::task::JoinError>>>(); let handler = spawn(async move { report_tests(receiver, reporter).await.0 }); -- cgit v1.2.3