fix: close memory test failure paths

shakkernerd · shakkernerd · commit 1cd6f81a46ae · 2026-06-26T14:05:07.000+01:00
diff --git a/extensions/memory-core/src/cli.test.ts b/extensions/memory-core/src/cli.test.ts
@@ -2342,7 +2342,7 @@ describe("memory cli", () => {
         lastRecalledAt: "<now>",
         queryHashes: ["<hash>"],
         recallDays: ["<today>"],
-        conceptTags: ["backup", "backups", "glacier"],
+        conceptTags: ["backup", "backups", "glacier", "s3"],
       });
       expect(close).toHaveBeenCalled();
     });
diff --git a/extensions/memory-core/src/memory/manager.sync-errors-do-not-crash.test.ts b/extensions/memory-core/src/memory/manager.sync-errors-do-not-crash.test.ts
@@ -17,18 +17,21 @@ describe("memory manager sync failures", () => {
       unhandled.push(reason);
     };
     process.on("unhandledRejection", handler);
-    const syncSpy = vi
-      .fn()
-      .mockRejectedValueOnce(new Error("openai embeddings failed: 400 bad request"));
-    setTimeout(() => {
-      runDetachedMemorySync(syncSpy, "watch");
-    }, 1);
+    try {
+      const syncSpy = vi
+        .fn()
+        .mockRejectedValueOnce(new Error("openai embeddings failed: 400 bad request"));
+      setTimeout(() => {
+        runDetachedMemorySync(syncSpy, "watch");
+      }, 1);
 
-    await vi.runOnlyPendingTimersAsync();
-    vi.useRealTimers();
-    await syncSpy.mock.results[0]?.value?.catch(() => undefined);
+      await vi.runOnlyPendingTimersAsync();
+      vi.useRealTimers();
+      await syncSpy.mock.results[0]?.value?.catch(() => undefined);
 
-    process.off("unhandledRejection", handler);
-    expect(unhandled).toHaveLength(0);
+      expect(unhandled).toHaveLength(0);
+    } finally {
+      process.off("unhandledRejection", handler);
+    }
   });
 });
