redis/node-redis
1
0
Fork 0
mirror of https://github.com/redis/node-redis.git synced 2025-12-15 23:55:38 +03:00
Code Activity

fix(ts): xtrim threshold accepts string (#2828)

Browse Source 
* fix(ts): xtrim threshold accepts string

* test: check MINID with text id
This commit is contained in:
Manuel Spigolon
2025-08-15 12:36:04 +02:00
committed by GitHub
parent 3d7668e543
commit 7d905c0027
2 changed files with 8 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
packages/client/lib/commands/XTRIM.spec.ts
View File

@@ -38,6 +38,13 @@ describe('XTRIM', () => {
['XTRIM', 'key', 'MAXLEN', '=', '1', 'LIMIT', '1'] ['XTRIM', 'key', 'MAXLEN', '=', '1', 'LIMIT', '1']
); );
}); });
it('with MINID', () => {
assert.deepEqual(
XTRIM.transformArguments('key', 'MINID', '0-0'),
['XTRIM', 'key', 'MINID', '0-0']
);
});
}); });
testUtils.testAll('xTrim', async client => { testUtils.testAll('xTrim', async client => {

2
packages/client/lib/commands/XTRIM.ts
View File

@@ -12,7 +12,7 @@ export default {
transformArguments( transformArguments(
key: RedisArgument, key: RedisArgument,
strategy: 'MAXLEN' | 'MINID', strategy: 'MAXLEN' | 'MINID',
threshold: number, threshold: number | string,
options?: XTrimOptions options?: XTrimOptions
) { ) {
const args = ['XTRIM', key, strategy]; const args = ['XTRIM', key, strategy];