fix: handle deleteMany with empty keys list (#51)

* Handle deleteMany with empty keys list

* Create khaki-days-clean.md

---------

Co-authored-by: Julien Ripouteau <[email protected]>

Author: simoneNEMO

.changeset/khaki-days-clean.md

@@ -0,0 +1,5 @@
+---
+"bentocache": patch
+---
+
+Handle deleteMany with empty keys list

packages/bentocache/src/drivers/database/database.ts

@@ -156,6 +156,7 @@ export class DatabaseDriver extends BaseDriver implements CacheDriver<true> {
    * Delete multiple keys from the cache
    */
   async deleteMany(keys: string[]) {
+    if (keys.length === 0) return true
     await this.#initialized
 
     keys = keys.map((key) => this.getItemKey(key))

packages/bentocache/src/drivers/dynamodb.ts

@@ -267,6 +267,7 @@ export class DynamoDbDriver extends BaseDriver implements CacheDriver {
    * Delete multiple keys from the cache
    */
   async deleteMany(keys: string[]) {
+    if (keys.length === 0) return true
     await Promise.all(keys.map((key) => this.delete(key)))
     return true
   }

packages/bentocache/src/drivers/file/file.ts

@@ -225,6 +225,7 @@ export class FileDriver extends BaseDriver implements CacheDriver {
    * Delete multiple keys from the cache
    */
   async deleteMany(keys: string[]) {
+    if (keys.length === 0) return true
     await Promise.all(keys.map((key) => this.delete(key)))
     return true
   }

packages/bentocache/src/drivers/memory.ts

@@ -131,6 +131,7 @@ export class MemoryDriver extends BaseDriver implements L1CacheDriver {
    * Delete multiple keys from the cache
    */
   deleteMany(keys: string[]) {
+    if (keys.length === 0) return true
     for (const key of keys) this.delete(key)
     return true
   }

packages/bentocache/src/drivers/redis.ts

@@ -138,6 +138,7 @@ export class RedisDriver extends BaseDriver implements L2CacheDriver {
    * Delete multiple keys from the cache
    */
   async deleteMany(keys: string[]) {
+    if (keys.length === 0) return true
     await this.#connection.del(keys.map((key) => this.getItemKey(key)))
     return true
   }

packages/bentocache/tests/helpers/driver_test_suite.ts

@@ -103,6 +103,11 @@ export function registerCacheDriverTestSuite(options: {
     assert.deepEqual(await cache.get('key2'), undefined)
   })
 
+  test('deleteMany() return true when no keys are provided', async ({ assert }) => {
+    const result = await cache.deleteMany([])
+    assert.isFalse(result)
+  })
+
   test('delete() returns true when key is removed', async ({ assert }) => {
     await cache.set('key1', 'value1')
     const result = await cache.delete('key1')