stores-com / petty-cache / 30875846718

100%

main: 100%

Build Type

Pull #70

github

Committed by freshlogic

Commit Message

Add promise support to the semaphore functions

Convert all six semaphore methods to the executor pattern with linear
async internals: mutex lock acquired up front, try/finally guarantees
the unlock (errors ignored, matching the previous behavior), pool reads
and writes go through promisified client calls, and acquireLock's
async.retry is now a plain retry loop. retrieveOrCreate's options.size
function supports both callback and async signatures via executeFunc.
The semaphore bind loop was a no-op on arrow functions and is removed.

Also update the mutex.lock test stubs to honor the method's dual
callback/promise contract, and fix a flaky bulkFetch test: it used
fixed keys, so values left in Redis by a previous run could expire
mid-test between the first fetch and the post-memory-expiry fetch,
triggering the should-not-be-called cache-miss function.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

Pull Request Pull Request #70: Add promise support to fetch

Coverage Stats

330 of 332 branches covered (99.4%)

Branch coverage included in aggregate %.

410 of 412 new or added lines in 1 file covered. (99.51%)

1 existing line in 1 file now uncovered.

1118 of 1121 relevant lines covered (99.73%)

55.47 hits per line