From bec904efba67d8552a112a996d2294224f0b404f Mon Sep 17 00:00:00 2001 From: Brennan Benson <79079362+brennanb2025@users.noreply.github.com> Date: Mon, 3 Aug 2026 22:04:04 -0700 Subject: [PATCH] test(renderer): make detected index perf gate deterministic --- .../lib/worktree-runtime-owner-index.detected-perf.test.ts | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/src/renderer/src/lib/worktree-runtime-owner-index.detected-perf.test.ts b/src/renderer/src/lib/worktree-runtime-owner-index.detected-perf.test.ts index be7389f9d31..cba36ebf6de 100644 --- a/src/renderer/src/lib/worktree-runtime-owner-index.detected-perf.test.ts +++ b/src/renderer/src/lib/worktree-runtime-owner-index.detected-perf.test.ts @@ -129,7 +129,7 @@ describe('detected worktree index performance', () => { expect(walkedReads).toBeGreaterThan(100_000) }) - it('keeps the lookup path an order of magnitude cheaper than the catalog walk', () => { + it('records lookup-path timing without making wall clock a CI gate', () => { const walkPool = freshCatalogPool() const walkMs = measureSweeps((sweep) => { const catalog = walkPool[sweep]! @@ -194,9 +194,6 @@ describe('detected worktree index performance', () => { `${JSON.stringify(report, null, 2)}\n` ) - // Only the warm ratio is asserted: a republished catalog pays one rebuild that costs about - // what a single sweep's walk did, so the fresh leg is parity and is recorded, not gated. - // The measured warm margin is ~100x, so this bound cannot flake under CI load. - expect(walkWarmMs / warmMs).toBeGreaterThan(10) + // Why: shared-runner timing is noisy; the read-count test above is the deterministic CI gate. }) })