Fix two intermittently failing library tests (closes #90)

Library.close() now returns a promise that resolves once the work it
started has finished: an in-flight refresh with its cache write, the ML
data fetch, and running precache sweeps. The interval test could see a
refresh's new state, close, and remove the directory while the write was
still running. Every library test and CLI command now awaits close(), and
tests hold each of the three writes open to prove close() waits for it.

The precache test waited for its stub source to be called, but the cache
records a file only after checking it on disk, so status() could lag. It
now waits for both fills to report "done".

Model: opus-5-5

test/library/mldata.test.ts

@@ -478,19 +478,24 @@ describe("Library ML-data fetch on refresh", () => {
             cacheDirectory,
             refreshIntervalSeconds: 3600,
         });
-        await started;
+        try {
+            await started;
 
-        let closed = false;
+            let closed = false;
-        const closing = lib.close().then(() => {
+            const closing = lib.close().then(() => {
-            closed = true;
+                closed = true;
-        });
+            });
-        await new Promise((r) => setTimeout(r, 20));
+            await new Promise((r) => setTimeout(r, 20));
-        expect(closed).toBe(false);
+            expect(closed).toBe(false);
 
-        release();
+            release();
-        await closing;
+            await closing;
-        expect(existsSync(join(cacheDirectory, "mldata", "1001.json"))).toBe(
+            expect(
-            true,
+                existsSync(join(cacheDirectory, "mldata", "1001.json")),
-        );
+            ).toBe(true);
+        } finally {
+            release();
+            await lib.close();
+        }
     });
 });

test/library/precache.test.ts

@@ -471,17 +471,22 @@ describe("Precache through Library.open", () => {
             refreshIntervalSeconds: 3600,
             precacheOriginals: false,
         });
-        const destination = await started;
+        try {
+            const destination = await started;
 
-        let closed = false;
+            let closed = false;
-        const closing = lib.close().then(() => {
+            const closing = lib.close().then(() => {
-            closed = true;
+                closed = true;
-        });
+            });
-        await new Promise((r) => setTimeout(r, 20));
+            await new Promise((r) => setTimeout(r, 20));
-        expect(closed).toBe(false);
+            expect(closed).toBe(false);
 
-        release();
+            release();
-        await closing;
+            await closing;
-        expect(existsSync(destination)).toBe(true);
+            expect(existsSync(destination)).toBe(true);
+        } finally {
+            release();
+            await lib.close();
+        }
     });
 });