fix(test): close leaked embedded worker in bug-23 (CI unhandled error)

bug-23-scheduler-job-name created a new Worker('test-job-name', {embedded}) inside a Promise executor and never closed it, so it polled for the whole suite. After the 2.8.14 commit added 9 test files, the suite timeline shifted enough that a stray poll coincided with another test's global disk-IO injection: pull -> getSharedManager -> migrate threw SQLITE_IOERR, handlePullError emitted 'error' with no listener -> 'Unhandled error between tests' (exit 1). Capture + await worker.close() to remove the leak. Test-only; no shipped-code change, so still 2.8.14.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

Author: egeominotti

test/bug-23-scheduler-job-name.test.ts

@@ -57,8 +57,9 @@ describe('Bug #23: upsertJobScheduler job.name should use jobTemplate.name', ()
     // Wait for the cron to fire
     await Bun.sleep(200);
 
+    let worker: Worker | undefined;
     const receivedName = await new Promise<string>((resolve) => {
-      const worker = new Worker(
+      worker = new Worker(
         'test-job-name',
         async (job) => {
           resolve(job.name);
@@ -67,6 +68,11 @@ describe('Bug #23: upsertJobScheduler job.name should use jobTemplate.name', ()
         { embedded: true }
       );
     });
+    // Close the worker — otherwise it keeps polling forever (leaked across the
+    // whole suite) and a stray poll during another test's disk-IO injection
+    // surfaces as an "Unhandled error between tests" (emit('error') with no
+    // listener). See CI failure on 2.8.14.
+    await worker?.close();
 
     expect(receivedName).toBe('send-newsletter');
   });