fix(memory): update mutex owner when releasing then acquiring again

Close #12

Author: Julien-R44

packages/verrou/src/drivers/memory.ts

@@ -84,10 +84,11 @@ export class MemoryStore implements LockStore {
   async delete(key: string, owner: string) {
     const mutex = this.#locks.get(key)
 
-    if (!mutex || !mutex.releaser) throw new E_LOCK_NOT_OWNED()
+    if (!mutex || !mutex.releaser) return
     if (mutex.owner !== owner) throw new E_LOCK_NOT_OWNED()
 
     mutex.releaser()
+    this.#locks.delete(key)
   }
 
   /**

packages/verrou/src/test_suite.ts

@@ -149,6 +149,28 @@ export function registerStoreTestSuite(options: {
     assert.deepEqual(result, '42')
   })
 
+  test('run is exclusive with two factories', async ({ assert }) => {
+    const provider = new LockFactory(options.createStore())
+    const lock1 = provider.createLock('foo')
+    const lock2 = provider.createLock('foo')
+
+    let flag = false
+    lock1.run(async () => {
+      await sleep(500)
+      flag = true
+    })
+
+    assert.isFalse(flag)
+
+    const [, result] = await lock2.run(async () => {
+      assert.isTrue(flag)
+      return '42'
+    })
+
+    assert.isTrue(flag)
+    assert.deepEqual(result, '42')
+  })
+
   test('exceptions during run do not leave mutex in locked state', async ({ assert }) => {
     const provider = new LockFactory(options.createStore())
     const lock = provider.createLock('foo')