From 6096f97dd73f44f324f632b3be59ccb0b9d46443 Mon Sep 17 00:00:00 2001
From: Jarred Sumner <jarred@jarredsumner.com>
Date: Sun, 2 Aug 2026 06:33:27 -0700
Subject: [PATCH] test: remove expected-durations.test.ts

It asserted no js/*/test/parallel/ entry in test/expected-durations.json
exceeds 500 ms, assuming the generator clamps every concurrent-phase span.
The generator only clamps inter-dispatch gaps; explicit `(N.Ns)` timed
headers are recorded as-is, so a genuinely slow parallel test (e.g.
test-fs-read-stream-pos.js at 74 s on musl) fails the test after any
regeneration. The remaining assertions only re-validated the shape of a
file that is produced by scripts/update-test-durations.mjs.

No-Verification-Needed: deletes a test file only
---
 test/internal/expected-durations.test.ts | 65 ------------------------
 1 file changed, 65 deletions(-)
 delete mode 100644 test/internal/expected-durations.test.ts

diff --git a/test/internal/expected-durations.test.ts b/test/internal/expected-durations.test.ts
deleted file mode 100644
index 5563af80b0c2..000000000000
--- a/test/internal/expected-durations.test.ts
+++ /dev/null
@@ -1,65 +0,0 @@
-import { describe, expect, test } from "bun:test";
-import { readFileSync } from "node:fs";
-import { join } from "node:path";
-
-// Guards the checked-in table that drives LPT shard bin-packing in
-// scripts/runner.node.mjs. Regenerate via scripts/update-test-durations.mjs.
-const table = JSON.parse(readFileSync(join(import.meta.dir, "..", "expected-durations.json"), "utf8"));
-const entries = Object.entries(table).filter(([k]) => k !== "_meta") as [string, Record<string, number>][];
-
-describe("test/expected-durations.json", () => {
-  test("every lane the runner selects is declared and populated", () => {
-    expect(table._meta).toBeObject();
-    expect(table._meta.lanes).toBeObject();
-    // runner.node.mjs lane selection: asan / musl / windows / default.
-    for (const lane of ["default", "asan", "musl", "windows"]) {
-      expect(table._meta.lanes[lane]).toBeString();
-      const populated = entries.filter(([, e]) => typeof e[lane] === "number").length;
-      expect(populated).toBeGreaterThan(1000);
-    }
-  });
-
-  test("keys are relative test paths, not runner retry/error labels", () => {
-    // Same predicate parseLog() uses to reject `[N/M] <path> - code 1` /
-    // `[N/M] <path> [attempt #2]` headers; anything it lets through must be a
-    // forward-slash relative path ending at a test file extension.
-    const isTestPath = (k: string) => !k.startsWith("/") && !k.includes("\\") && /\.(?:[cm]?[jt]sx?|json)$/.test(k);
-    const bad = entries.map(([k]) => k).filter(k => !isTestPath(k));
-    expect(bad).toEqual([]);
-  });
-
-  test("covers the parallel-safe phase and clamps its spans", () => {
-    // js/{node,bun}/test/parallel/ run N-wide and log without a `--- ` group
-    // prefix; a parser that only matches `--- [N/M]` drops ~3k entries here.
-    const parallelSafe = entries.filter(
-      ([k]) => k.startsWith("js/node/test/parallel/") || k.startsWith("js/bun/test/parallel/"),
-    );
-    expect(parallelSafe.length).toBeGreaterThan(1000);
-    // Concurrent-phase spans are inter-dispatch gaps; without the clamp the
-    // last-dispatched file on each shard absorbs the tail drain / a sibling's
-    // retry backoff and reads as multiple seconds for a ~50 ms test.
-    const unclamped = parallelSafe.filter(([, e]) => Object.values(e).some(ms => ms > 500)).map(([k]) => k);
-    expect(unclamped).toEqual([]);
-  });
-
-  test("every entry is {lane: non-negative ms}", () => {
-    const lanes = new Set(Object.keys(table._meta.lanes));
-    const bad: string[] = [];
-    for (const [key, entry] of entries) {
-      if (entry === null || typeof entry !== "object") {
-        bad.push(`${key}: not an object`);
-        continue;
-      }
-      const entryLanes = Object.keys(entry);
-      if (entryLanes.length === 0) bad.push(`${key}: no lanes`);
-      for (const lane of entryLanes) {
-        if (!lanes.has(lane)) bad.push(`${key}.${lane}: undeclared lane`);
-        const ms = entry[lane];
-        if (typeof ms !== "number" || !(ms >= 0)) bad.push(`${key}.${lane}: ${ms}`);
-      }
-    }
-    expect(bad).toEqual([]);
-    // Loose lower bound: the runner currently shards ~5k files.
-    expect(entries.length).toBeGreaterThan(3000);
-  });
-});