From 8241a015755e3980e87b298df7d7a53456ebc6ab Mon Sep 17 00:00:00 2001 From: Kiko Beats Date: Mon, 8 Apr 2024 21:45:31 +0200 Subject: [PATCH] test: improve coverage --- src/index.js | 1 - src/keys.js | 3 ++- src/plans.js | 3 ++- test/keys.js | 1 - 4 files changed, 4 insertions(+), 4 deletions(-) diff --git a/src/index.js b/src/index.js index 1105592..d69b27c 100644 --- a/src/index.js +++ b/src/index.js @@ -5,7 +5,6 @@ const createKeys = require('./keys') const createPlans = require('./plans') module.exports = ({ serialize = JSONB.stringify, deserialize = JSONB.parse, redis = new Map() } = {}) => { - if (!redis) throw TypeError('The argument `store` is required.') const plans = createPlans({ serialize, deserialize, redis }) const keys = createKeys({ serialize, deserialize, redis, plans }) return { keys, plans } diff --git a/src/keys.js b/src/keys.js index e6de9c7..0a590bd 100644 --- a/src/keys.js +++ b/src/keys.js @@ -67,7 +67,8 @@ module.exports = ({ serialize, deserialize, plans, redis } = {}) => { assert(plan === null, `The key \`${keyId}\` is associated with the plan \`${getKey.plan}\``) } const isDeleted = (await redis.del(getKey(keyId, { verify: true }))) === 1 - return assert(isDeleted, `The key \`${keyId}\` does not exist.`) || isDeleted + assert(isDeleted, `The key \`${keyId}\` does not exist.`) + return isDeleted } /** diff --git a/src/plans.js b/src/plans.js index b329db2..dfb8a7e 100644 --- a/src/plans.js +++ b/src/plans.js @@ -66,7 +66,8 @@ module.exports = ({ serialize, deserialize, redis } = {}) => { */ const del = async planId => { const isDeleted = (await redis.del(getKey(planId, { validate: true }))) === 1 - return assert(isDeleted, `The plan \`${planId}\` does not exist.`) || isDeleted + assert(isDeleted, `The plan \`${planId}\` does not exist.`) + return isDeleted } /** diff --git a/test/keys.js b/test/keys.js index cab98d2..95a8be5 100644 --- a/test/keys.js +++ b/test/keys.js @@ -13,7 +13,6 @@ const { keys, plans } = openkey({ redis: new Redis() }) test.beforeEach(async () => { const keys = await redis.keys(`${KEY_PREFIX}*`) - if (keys.length > 0) await redis.del(keys) })