From cdce65dfd5e090b9610cdb273ca7c51c470f957b Mon Sep 17 00:00:00 2001 From: Mohamed Boudra Date: Sun, 16 Aug 2026 07:48:03 +0000 Subject: [PATCH] test(app): stabilize archive descriptor assertions The test rebuilt expected provider rows with fresh timestamps, so its result depended on both fixture calls landing in the same millisecond. Assert the preserved descriptor identities instead, matching the behavior under test without clock dependence. --- packages/app/src/subagents/archive-finished.test.ts | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/packages/app/src/subagents/archive-finished.test.ts b/packages/app/src/subagents/archive-finished.test.ts index 6611b93f8a..10df655d95 100644 --- a/packages/app/src/subagents/archive-finished.test.ts +++ b/packages/app/src/subagents/archive-finished.test.ts @@ -400,9 +400,11 @@ describe("createArchiveFinishedSubagents", () => { }); it("dismisses finished provider rows locally without removing their descriptors", async () => { + const finishedDescriptor = provider("finished"); + const runningDescriptor = provider("running", "running"); const descriptors = new Map([ - ["finished", provider("finished")], - ["running", provider("running", "running")], + ["finished", finishedDescriptor], + ["running", runningDescriptor], ]); const dismissed: string[][] = []; const archive = createArchiveFinishedSubagents([...descriptors.values()], { @@ -420,7 +422,7 @@ describe("createArchiveFinishedSubagents", () => { }); expect(dismissed).toEqual([["finished"]]); - expect(descriptors.get("finished")).toEqual(provider("finished")); - expect(descriptors.get("running")).toEqual(provider("running", "running")); + expect(descriptors.get("finished")).toBe(finishedDescriptor); + expect(descriptors.get("running")).toBe(runningDescriptor); }); });