effect-distributed-lock 0.0.3 → 0.0.5

package/redis-semaphore/test/src/RedisMutex.test.ts

@@ -0,0 +1,334 @@
+import { expect } from 'chai'
+import { Redis } from 'ioredis'
+import sinon from 'sinon'
+import LostLockError from '../../src/errors/LostLockError'
+import Mutex from '../../src/RedisMutex'
+import { TimeoutOptions } from '../../src/types'
+import { delay } from '../../src/utils/index'
+import { client1 as client, clientMock1 as clientMock } from '../redisClient'
+import { downRedisServer, upRedisServer } from '../shell'
+import {
+  catchUnhandledRejection,
+  throwUnhandledRejection,
+  unhandledRejectionSpy
+} from '../unhandledRejection'
+
+const timeoutOptions: TimeoutOptions = {
+  lockTimeout: 300,
+  acquireTimeout: 100,
+  refreshInterval: 80,
+  retryInterval: 10
+}
+
+describe('Mutex', () => {
+  it('should fail on invalid arguments', () => {
+    expect(() => new Mutex(null as unknown as Redis, 'key')).to.throw(
+      '"client" is required'
+    )
+    expect(() => new Mutex(client, '')).to.throw('"key" is required')
+    expect(() => new Mutex(client, 1 as unknown as string)).to.throw(
+      '"key" must be a string'
+    )
+    expect(() => new Mutex(client, 'key', { identifier: '' })).to.throw(
+      'identifier must be not empty random string'
+    )
+    expect(
+      () => new Mutex(client, 'key', { acquiredExternally: true })
+    ).to.throw('acquiredExternally=true meanless without custom identifier')
+    expect(
+      () =>
+        new Mutex(client, 'key', {
+          externallyAcquiredIdentifier: '123',
+          identifier: '123'
+        })
+    ).to.throw(
+      'Invalid usage. Use custom identifier and acquiredExternally: true'
+    )
+    expect(
+      () =>
+        new Mutex(client, 'key', {
+          externallyAcquiredIdentifier: '123',
+          acquiredExternally: true,
+          identifier: '123'
+        })
+    ).to.throw(
+      'Invalid usage. Use custom identifier and acquiredExternally: true'
+    )
+  })
+  it('should set default options', () => {
+    expect(new Mutex(client, 'key', {})).to.be.ok
+    expect(new Mutex(client, 'key')).to.be.ok
+  })
+  it('should set random UUID as identifier', () => {
+    expect(new Mutex(client, 'key').identifier).to.match(
+      /^[0-9a-fA-F]{8}\b-[0-9a-fA-F]{4}\b-[0-9a-fA-F]{4}\b-[0-9a-fA-F]{4}\b-[0-9a-fA-F]{12}$/
+    )
+  })
+  it('should add identifier suffix', () => {
+    expect(
+      new Mutex(client, 'key', { identifierSuffix: 'abc' }).identifier
+    ).to.match(
+      /^[0-9a-fA-F]{8}\b-[0-9a-fA-F]{4}\b-[0-9a-fA-F]{4}\b-[0-9a-fA-F]{4}\b-[0-9a-fA-F]{12}-abc$/
+    )
+  })
+  it('should use custom identifier if provided', () => {
+    expect(
+      new Mutex(client, 'key', { identifier: 'abc' }).identifier
+    ).to.be.eql('abc')
+  })
+  it('should acquire and release lock', async () => {
+    const mutex = new Mutex(client, 'key')
+    expect(mutex.isAcquired).to.be.false
+    await mutex.acquire()
+    expect(mutex.isAcquired).to.be.true
+    expect(await client.get('mutex:key')).to.be.eql(mutex.identifier)
+    await mutex.release()
+    expect(mutex.isAcquired).to.be.false
+    expect(await client.get('mutex:key')).to.be.eql(null)
+  })
+  it('should reject after timeout', async () => {
+    const mutex1 = new Mutex(client, 'key', timeoutOptions)
+    const mutex2 = new Mutex(client, 'key', timeoutOptions)
+    await mutex1.acquire()
+    await expect(mutex2.acquire()).to.be.rejectedWith(
+      'Acquire mutex mutex:key timeout'
+    )
+    await mutex1.release()
+    expect(await client.get('mutex:key')).to.be.eql(null)
+  })
+  it('should return false for tryAcquire after timeout', async () => {
+    const mutex1 = new Mutex(client, 'key', timeoutOptions)
+    const mutex2 = new Mutex(client, 'key', timeoutOptions)
+    await mutex1.acquire()
+    const result = await mutex2.tryAcquire()
+    expect(result).to.be.false
+    await mutex1.release()
+    expect(await client.get('mutex:key')).to.be.eql(null)
+  })
+  it('should return true for successful tryAcquire', async () => {
+    const mutex = new Mutex(client, 'key', timeoutOptions)
+    const result = await mutex.tryAcquire()
+    expect(result).to.be.true
+    await mutex.release()
+    expect(await client.get('mutex:key')).to.be.eql(null)
+  })
+  it('should refresh lock every refreshInterval ms until release', async () => {
+    const mutex = new Mutex(client, 'key', timeoutOptions)
+    await mutex.acquire()
+    await delay(400)
+    expect(await client.get('mutex:key')).to.be.eql(mutex.identifier)
+    await mutex.release()
+    expect(await client.get('mutex:key')).to.be.eql(null)
+  })
+  it('should stop refreshing lock every refreshInterval ms if stopped', async () => {
+    const mutex = new Mutex(client, 'key', timeoutOptions)
+    await mutex.acquire()
+    mutex.stopRefresh()
+    await delay(400)
+    expect(await client.get('mutex:key')).to.be.eql(null)
+  })
+  it('should not call _refresh if already refreshing', async () => {
+    const mutex = new Mutex(client, 'key', timeoutOptions)
+    let callCount = 0
+    Object.assign(mutex, {
+      _refresh: () =>
+        delay(100).then(() => {
+          callCount++
+          return true
+        })
+    })
+    await mutex.acquire()
+    await delay(400)
+    expect(callCount).to.be.eql(2) // not floor(400/80) = 9
+  })
+
+  it('should support externally acquired mutex (deprecated interface)', async () => {
+    const externalMutex = new Mutex(client, 'key', {
+      ...timeoutOptions,
+      refreshInterval: 0
+    })
+    const localMutex = new Mutex(client, 'key', {
+      ...timeoutOptions,
+      externallyAcquiredIdentifier: externalMutex.identifier
+    })
+    await externalMutex.acquire()
+    await localMutex.acquire()
+    await delay(400)
+    expect(await client.get('mutex:key')).to.be.eql(localMutex.identifier)
+    await localMutex.release()
+    expect(await client.get('mutex:key')).to.be.eql(null)
+  })
+  it('should support externally acquired mutex', async () => {
+    const externalMutex = new Mutex(client, 'key', {
+      ...timeoutOptions,
+      refreshInterval: 0
+    })
+    const localMutex = new Mutex(client, 'key', {
+      ...timeoutOptions,
+      identifier: externalMutex.identifier,
+      acquiredExternally: true
+    })
+    await externalMutex.acquire()
+    await localMutex.acquire()
+    await delay(400)
+    expect(await client.get('mutex:key')).to.be.eql(localMutex.identifier)
+    await localMutex.release()
+    expect(await client.get('mutex:key')).to.be.eql(null)
+  })
+  describe('lost lock case', () => {
+    beforeEach(() => {
+      catchUnhandledRejection()
+    })
+    afterEach(() => {
+      throwUnhandledRejection()
+    })
+    it('should throw unhandled error if lock was lost between refreshes (another instance acquired)', async () => {
+      const mutex = new Mutex(client, 'key', timeoutOptions)
+      await mutex.acquire()
+      expect(mutex.isAcquired).to.be.true
+      await client.set('mutex:key', '222') // another instance
+      await delay(200)
+      expect(mutex.isAcquired).to.be.false
+      expect(unhandledRejectionSpy).to.be.called
+      expect(unhandledRejectionSpy.firstCall.firstArg instanceof LostLockError)
+        .to.be.true
+    })
+    it('should throw unhandled error if lock was lost between refreshes (lock expired)', async () => {
+      const mutex = new Mutex(client, 'key', timeoutOptions)
+      await mutex.acquire()
+      expect(mutex.isAcquired).to.be.true
+      await client.del('mutex:key') // expired
+      await delay(200)
+      expect(mutex.isAcquired).to.be.false
+      expect(unhandledRejectionSpy).to.be.called
+      expect(unhandledRejectionSpy.firstCall.firstArg instanceof LostLockError)
+        .to.be.true
+    })
+    it('should call onLockLost callback if provided (another instance acquired)', async () => {
+      const onLockLostCallback = sinon.spy(function (this: Mutex) {
+        expect(this.isAcquired).to.be.false
+      })
+      const mutex = new Mutex(client, 'key', {
+        ...timeoutOptions,
+        onLockLost: onLockLostCallback
+      })
+      await mutex.acquire()
+      expect(mutex.isAcquired).to.be.true
+      await client.set('mutex:key', '222') // another instance
+      await delay(200)
+      expect(mutex.isAcquired).to.be.false
+      expect(unhandledRejectionSpy).to.not.called
+      expect(onLockLostCallback).to.be.called
+      expect(onLockLostCallback.firstCall.firstArg instanceof LostLockError).to
+        .be.true
+    })
+    it('should call onLockLost callback if provided (lock expired)', async () => {
+      const onLockLostCallback = sinon.spy(function (this: Mutex) {
+        expect(this.isAcquired).to.be.false
+      })
+      const mutex = new Mutex(client, 'key', {
+        ...timeoutOptions,
+        onLockLost: onLockLostCallback
+      })
+      await mutex.acquire()
+      expect(mutex.isAcquired).to.be.true
+      await client.del('mutex:key') // expired
+      await delay(200)
+      expect(mutex.isAcquired).to.be.false
+      expect(unhandledRejectionSpy).to.not.called
+      expect(onLockLostCallback).to.be.called
+      expect(onLockLostCallback.firstCall.firstArg instanceof LostLockError).to
+        .be.true
+    })
+  })
+  it('should be reusable', async function () {
+    this.timeout(10000)
+    const mutex = new Mutex(client, 'key', timeoutOptions)
+
+    /* Lifecycle 1 */
+    await mutex.acquire()
+    await delay(300)
+    expect(await client.get('mutex:key')).to.be.eql(mutex.identifier)
+    await mutex.release()
+    expect(await client.get('mutex:key')).to.be.eql(null)
+    await delay(300)
+    expect(await client.get('mutex:key')).to.be.eql(null)
+
+    await delay(300)
+
+    /* Lifecycle 2 */
+    await mutex.acquire()
+    await delay(300)
+    expect(await client.get('mutex:key')).to.be.eql(mutex.identifier)
+    await mutex.release()
+    expect(await client.get('mutex:key')).to.be.eql(null)
+    await delay(300)
+    expect(await client.get('mutex:key')).to.be.eql(null)
+
+    await delay(300)
+
+    /* Lifecycle 3 */
+    await mutex.acquire()
+    await delay(300)
+    expect(await client.get('mutex:key')).to.be.eql(mutex.identifier)
+    await mutex.release()
+    expect(await client.get('mutex:key')).to.be.eql(null)
+    await delay(300)
+    expect(await client.get('mutex:key')).to.be.eql(null)
+  })
+  describe('[Node shutdown]', () => {
+    beforeEach(() => {
+      catchUnhandledRejection()
+    })
+    afterEach(async () => {
+      throwUnhandledRejection()
+      await upRedisServer(1)
+    })
+    it('should lost lock when node become alive', async function () {
+      this.timeout(60000)
+      const onLockLostCallback = sinon.spy(function (this: Mutex) {
+        expect(this.isAcquired).to.be.false
+      })
+      const mutex1 = new Mutex(client, 'key', {
+        ...timeoutOptions,
+        onLockLost: onLockLostCallback
+      })
+      await mutex1.acquire()
+      await downRedisServer(1)
+
+      await delay(1000)
+      // lock expired now
+
+      await upRedisServer(1)
+      // mutex was expired, key was deleted in redis
+      // give refresh mechanism time to detect lock lost
+      // (includes client reconnection time)
+      await delay(1000)
+
+      expect(await client.get('mutex:key')).to.be.eql(null)
+      expect(onLockLostCallback).to.be.called
+      expect(onLockLostCallback.firstCall.firstArg instanceof LostLockError).to
+        .be.true
+
+      // lock was not reacquired by mutex1, so mutex2 can acquire the lock
+
+      const mutex2 = new Mutex(client, 'key', timeoutOptions)
+      await mutex2.acquire()
+      expect(await client.get('mutex:key')).to.be.eql(mutex2.identifier)
+
+      await Promise.all([mutex1.release(), mutex2.release()])
+    })
+  })
+  describe('ioredis-mock support', async () => {
+    it('should acquire and release lock', async () => {
+      const mutex = new Mutex(clientMock, 'key')
+      expect(mutex.isAcquired).to.be.false
+      await mutex.acquire()
+      expect(mutex.isAcquired).to.be.true
+      expect(await clientMock.get('mutex:key')).to.be.eql(mutex.identifier)
+      await mutex.release()
+      expect(mutex.isAcquired).to.be.false
+      expect(await clientMock.get('mutex:key')).to.be.eql(null)
+    })
+  })
+})

package/redis-semaphore/test/src/RedisSemaphore.test.ts

@@ -0,0 +1,367 @@
+import { expect } from 'chai'
+import { Redis } from 'ioredis'
+import sinon from 'sinon'
+import LostLockError from '../../src/errors/LostLockError'
+import Semaphore from '../../src/RedisSemaphore'
+import { TimeoutOptions } from '../../src/types'
+import { delay } from '../../src/utils/index'
+import { client1 as client, clientMock1 as clientMock } from '../redisClient'
+import { downRedisServer, upRedisServer } from '../shell'
+import {
+  catchUnhandledRejection,
+  throwUnhandledRejection,
+  unhandledRejectionSpy
+} from '../unhandledRejection'
+
+const timeoutOptions: TimeoutOptions = {
+  lockTimeout: 300,
+  acquireTimeout: 100,
+  refreshInterval: 80,
+  retryInterval: 10
+}
+
+describe('Semaphore', () => {
+  it('should fail on invalid arguments', () => {
+    expect(() => new Semaphore(null as unknown as Redis, 'key', 5)).to.throw(
+      '"client" is required'
+    )
+    expect(() => new Semaphore(client, '', 5)).to.throw('"key" is required')
+    expect(() => new Semaphore(client, 1 as unknown as string, 5)).to.throw(
+      '"key" must be a string'
+    )
+    expect(() => new Semaphore(client, 'key', 0)).to.throw(
+      '"limit" is required'
+    )
+    expect(
+      () => new Semaphore(client, 'key', '10' as unknown as number)
+    ).to.throw('"limit" must be a number')
+  })
+  it('should acquire and release semaphore', async () => {
+    const semaphore1 = new Semaphore(client, 'key', 2)
+    const semaphore2 = new Semaphore(client, 'key', 2)
+    expect(semaphore1.isAcquired).to.be.false
+    expect(semaphore2.isAcquired).to.be.false
+
+    await semaphore1.acquire()
+    expect(semaphore1.isAcquired).to.be.true
+    await semaphore2.acquire()
+    expect(semaphore2.isAcquired).to.be.true
+    expect(await client.zrange('semaphore:key', 0, -1)).to.have.members([
+      semaphore1.identifier,
+      semaphore2.identifier
+    ])
+
+    await semaphore1.release()
+    expect(semaphore1.isAcquired).to.be.false
+    expect(await client.zrange('semaphore:key', 0, -1)).to.be.eql([
+      semaphore2.identifier
+    ])
+    await semaphore2.release()
+    expect(semaphore2.isAcquired).to.be.false
+    expect(await client.zcard('semaphore:key')).to.be.eql(0)
+  })
+  it('should reject after timeout', async () => {
+    const semaphore1 = new Semaphore(client, 'key', 1, timeoutOptions)
+    const semaphore2 = new Semaphore(client, 'key', 1, timeoutOptions)
+    await semaphore1.acquire()
+    await expect(semaphore2.acquire()).to.be.rejectedWith(
+      'Acquire semaphore semaphore:key timeout'
+    )
+    await semaphore1.release()
+    expect(await client.get('semaphore:key')).to.be.eql(null)
+  })
+  it('should refresh lock every refreshInterval ms until release', async () => {
+    const semaphore1 = new Semaphore(client, 'key', 2, timeoutOptions)
+    const semaphore2 = new Semaphore(client, 'key', 2, timeoutOptions)
+    await semaphore1.acquire()
+    await semaphore2.acquire()
+    await delay(400)
+    expect(await client.zrange('semaphore:key', 0, -1)).to.have.members([
+      semaphore1.identifier,
+      semaphore2.identifier
+    ])
+    await semaphore1.release()
+    expect(await client.zrange('semaphore:key', 0, -1)).to.be.eql([
+      semaphore2.identifier
+    ])
+    await semaphore2.release()
+    expect(await client.zcard('semaphore:key')).to.be.eql(0)
+  })
+  it('should stop refreshing lock if stopped', async () => {
+    const semaphore1 = new Semaphore(client, 'key', 2, timeoutOptions)
+    const semaphore2 = new Semaphore(client, 'key', 2, timeoutOptions)
+    await semaphore1.acquire()
+    await semaphore2.acquire()
+    await semaphore1.stopRefresh()
+    await delay(400)
+    expect(await client.zrange('semaphore:key', 0, -1)).to.be.eql([
+      semaphore2.identifier
+    ])
+    await semaphore2.stopRefresh()
+    await delay(400)
+    expect(await client.zcard('semaphore:key')).to.be.eql(0)
+  })
+  it('should acquire maximum LIMIT semaphores', async () => {
+    const s = () =>
+      new Semaphore(client, 'key', 3, {
+        acquireTimeout: 1000,
+        lockTimeout: 50,
+        retryInterval: 10,
+        refreshInterval: 0 // disable refresh
+      })
+    const pr1 = Promise.all([s().acquire(), s().acquire(), s().acquire()])
+    await delay(5)
+    const pr2 = Promise.all([s().acquire(), s().acquire(), s().acquire()])
+    await pr1
+    const ids1 = await client.zrange('semaphore:key', 0, -1)
+    expect(ids1.length).to.be.eql(3)
+    await pr2
+    const ids2 = await client.zrange('semaphore:key', 0, -1)
+    expect(ids2.length).to.be.eql(3)
+    expect(ids2)
+      .to.not.include(ids1[0])
+      .and.not.include(ids1[1])
+      .and.not.include(ids1[2])
+  })
+  it('should support externally acquired semaphore (deprecated interface)', async () => {
+    const externalSemaphore = new Semaphore(client, 'key', 3, {
+      ...timeoutOptions,
+      refreshInterval: 0
+    })
+    const localSemaphore = new Semaphore(client, 'key', 3, {
+      ...timeoutOptions,
+      externallyAcquiredIdentifier: externalSemaphore.identifier
+    })
+    await externalSemaphore.acquire()
+    await localSemaphore.acquire()
+    await delay(400)
+    expect(await client.zrange('semaphore:key', 0, -1)).to.have.members([
+      localSemaphore.identifier
+    ])
+    await localSemaphore.release()
+    expect(await client.zcard('semaphore:key')).to.be.eql(0)
+  })
+  it('should support externally acquired semaphore', async () => {
+    const externalSemaphore = new Semaphore(client, 'key', 3, {
+      ...timeoutOptions,
+      refreshInterval: 0
+    })
+    const localSemaphore = new Semaphore(client, 'key', 3, {
+      ...timeoutOptions,
+      identifier: externalSemaphore.identifier,
+      acquiredExternally: true
+    })
+    await externalSemaphore.acquire()
+    await localSemaphore.acquire()
+    await delay(400)
+    expect(await client.zrange('semaphore:key', 0, -1)).to.have.members([
+      localSemaphore.identifier
+    ])
+    await localSemaphore.release()
+    expect(await client.zcard('semaphore:key')).to.be.eql(0)
+  })
+  describe('lost lock case', () => {
+    beforeEach(() => {
+      catchUnhandledRejection()
+    })
+    afterEach(() => {
+      throwUnhandledRejection()
+    })
+    it('should throw unhandled error if lock is lost between refreshes', async () => {
+      const semaphore = new Semaphore(client, 'key', 3, timeoutOptions)
+      await semaphore.acquire()
+      await client.del('semaphore:key')
+      await client.zadd(
+        'semaphore:key',
+        Date.now(),
+        'aaa',
+        Date.now(),
+        'bbb',
+        Date.now(),
+        'ccc'
+      )
+      await delay(200)
+      expect(unhandledRejectionSpy).to.be.called
+      expect(unhandledRejectionSpy.firstCall.firstArg instanceof LostLockError)
+        .to.be.true
+    })
+    it('should call onLockLost callback if provided', async () => {
+      const onLockLostCallback = sinon.spy(function (this: Semaphore) {
+        expect(this.isAcquired).to.be.false
+      })
+      const semaphore = new Semaphore(client, 'key', 3, {
+        ...timeoutOptions,
+        onLockLost: onLockLostCallback
+      })
+      await semaphore.acquire()
+      expect(semaphore.isAcquired).to.be.true
+      await client.del('semaphore:key')
+      await client.zadd(
+        'semaphore:key',
+        Date.now(),
+        'aaa',
+        Date.now(),
+        'bbb',
+        Date.now(),
+        'ccc'
+      )
+      await delay(200)
+      expect(semaphore.isAcquired).to.be.false
+      expect(unhandledRejectionSpy).to.not.called
+      expect(onLockLostCallback).to.be.called
+      expect(onLockLostCallback.firstCall.firstArg instanceof LostLockError).to
+        .be.true
+    })
+  })
+  describe('reusable', () => {
+    it('autorefresh enabled', async () => {
+      const semaphore1 = new Semaphore(client, 'key', 2, timeoutOptions)
+      const semaphore2 = new Semaphore(client, 'key', 2, timeoutOptions)
+
+      await semaphore1.acquire()
+      await semaphore2.acquire()
+      await delay(300)
+      await semaphore1.release()
+      await semaphore2.release()
+
+      await delay(300)
+
+      await semaphore1.acquire()
+      await semaphore2.acquire()
+      await delay(300)
+      await semaphore1.release()
+      await semaphore2.release()
+
+      await delay(300)
+
+      await semaphore1.acquire()
+      await semaphore2.acquire()
+      await delay(300)
+      await semaphore1.release()
+      await semaphore2.release()
+    })
+
+    it('autorefresh disabled', async () => {
+      const noRefreshOptions = {
+        ...timeoutOptions,
+        refreshInterval: 0,
+        acquireTimeout: 10
+      }
+      const semaphore1 = new Semaphore(client, 'key', 2, noRefreshOptions)
+      const semaphore2 = new Semaphore(client, 'key', 2, noRefreshOptions)
+      const semaphore3 = new Semaphore(client, 'key', 2, noRefreshOptions)
+
+      await semaphore1.acquire()
+      await semaphore2.acquire()
+      await delay(300)
+      await semaphore1.release()
+      await semaphore2.release()
+
+      await delay(300)
+
+      // [0/2]
+      await semaphore1.acquire()
+      // [1/2]
+      await delay(80)
+      await semaphore2.acquire()
+      // [2/2]
+      await expect(semaphore3.acquire()).to.be.rejectedWith(
+        'Acquire semaphore semaphore:key timeout'
+      ) // rejectes after 10ms
+
+      // since semaphore1.acquire() elapsed 80ms (delay) + 10ms (semaphore3 timeout)
+      // semaphore1 will expire after 300 - 90 = 210ms
+      await delay(210)
+
+      // [1/2]
+      await semaphore3.acquire()
+    })
+  })
+  describe('[Node shutdown]', () => {
+    beforeEach(() => {
+      catchUnhandledRejection()
+    })
+    afterEach(async () => {
+      throwUnhandledRejection()
+      await upRedisServer(1)
+    })
+    it('should lost lock when node become alive', async function () {
+      this.timeout(60000)
+      const onLockLostCallbacks = [1, 2, 3].map(() =>
+        sinon.spy(function (this: Semaphore) {
+          expect(this.isAcquired).to.be.false
+        })
+      )
+      const semaphores1 = [1, 2, 3].map(
+        (n, i) =>
+          new Semaphore(client, 'key', 3, {
+            ...timeoutOptions,
+            onLockLost: onLockLostCallbacks[i]
+          })
+      )
+
+      await Promise.all(semaphores1.map(s => s.acquire()))
+
+      await downRedisServer(1)
+      console.log('SHUT DOWN')
+
+      await delay(1000)
+
+      await upRedisServer(1)
+      console.log('ONLINE')
+
+      // semaphore was expired, key was deleted in redis
+      // give refresh mechanism time to detect lock lost
+      // (includes reconnection time)
+      await delay(1000)
+
+      const data = await client.zrange('semaphore:key', 0, -1, 'WITHSCORES')
+      expect(data).to.be.eql([])
+
+      // console.log(data)
+      // expect(data).to.include(semaphore11.identifier)
+      // expect(data).to.include(semaphore12.identifier)
+      // expect(data).to.include(semaphore13.identifier)
+
+      // lock was not reacquired by semaphore1[1-3], so semaphore2 can acquire the lock
+
+      const semaphore2 = new Semaphore(client, 'key', 3, timeoutOptions)
+      await semaphore2.acquire()
+
+      expect(await client.zrange('semaphore:key', 0, -1)).to.have.members([
+        semaphore2.identifier
+      ])
+
+      await Promise.all([
+        ...semaphores1.map(s => s.release()),
+        semaphore2.release()
+      ])
+    })
+  })
+  describe('ioredis-mock support', async () => {
+    it('should acquire and release semaphore', async () => {
+      const semaphore1 = new Semaphore(clientMock, 'key', 2)
+      const semaphore2 = new Semaphore(clientMock, 'key', 2)
+      expect(semaphore1.isAcquired).to.be.false
+      expect(semaphore2.isAcquired).to.be.false
+
+      await semaphore1.acquire()
+      expect(semaphore1.isAcquired).to.be.true
+      await semaphore2.acquire()
+      expect(semaphore2.isAcquired).to.be.true
+      expect(await clientMock.zrange('semaphore:key', 0, -1)).to.have.members([
+        semaphore1.identifier,
+        semaphore2.identifier
+      ])
+
+      await semaphore1.release()
+      expect(semaphore1.isAcquired).to.be.false
+      expect(await clientMock.zrange('semaphore:key', 0, -1)).to.be.eql([
+        semaphore2.identifier
+      ])
+      await semaphore2.release()
+      expect(semaphore2.isAcquired).to.be.false
+      expect(await clientMock.zcard('semaphore:key')).to.be.eql(0)
+    })
+  })
+})