fix: isFinished never set on waiter (#35)

Author: zone117x

src/helpers/__tests__/helpers.test.ts

@@ -1,5 +1,5 @@
 import * as events from 'node:events';
-import { timeout } from '../time';
+import { timeout, waiter } from '../time';
 
 describe('Helper tests', () => {
   test('timeout function should not cause memory leak by accumulating abort listeners on abort', async () => {
@@ -49,4 +49,11 @@ describe('Helper tests', () => {
     // Final check to confirm listeners are cleaned up
     expect(countListeners()).toBe(0);
   });
+
+  test('waiter is finished', async () => {
+    const myWaiter = waiter();
+    myWaiter.finish();
+    await myWaiter;
+    expect(myWaiter.isFinished).toBe(true);
+  });
 });

src/helpers/time.ts

@@ -121,7 +121,7 @@ export function waiter<T = void>(): Waiter<T> {
   });
   const completer = {
     finish: (result: T) => {
-      completer.isFinished = true;
+      void Object.assign(promise, { isFinished: true });
       resolveFn(result);
     },
     isFinished: false,