From 71ceb91c9d65b970794e9245d15ea77c513168bc Mon Sep 17 00:00:00 2001 From: Wesley Wigham Date: Mon, 20 Nov 2017 15:47:46 -0800 Subject: [PATCH] Parallel runner shouldnt print nan and freeze on unestimatable tests when above batch count (#20176) --- src/harness/parallel/host.ts | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/harness/parallel/host.ts b/src/harness/parallel/host.ts index e278d267e1f..eca719628c7 100644 --- a/src/harness/parallel/host.ts +++ b/src/harness/parallel/host.ts @@ -208,8 +208,8 @@ namespace Harness.Parallel.Host { workers.push(child); } - // It's only really worth doing an initial batching if there are a ton of files to go through - if (totalFiles > 1000) { + // It's only really worth doing an initial batching if there are a ton of files to go through (and they have estimates) + if (totalFiles > 1000 && batchSize > 0) { console.log("Batching initial test lists..."); const batches: { runner: TestRunnerKind | "unittest", file: string, size: number }[][] = new Array(batchCount); const doneBatching = new Array(batchCount);