redis/node-redis
1
0
Fork 0
mirror of https://github.com/redis/node-redis.git synced 2025-08-06 02:15:48 +03:00
Code Activity

Flatten array arguments in legacyMode multi commands (#2064)

Browse Source
This commit is contained in:
Joe Price
2022-04-06 08:43:32 -07:00
committed by GitHub
parent d8e9da0f8e
commit 329885b4ae
2 changed files with 15 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
packages/client/lib/client/index.spec.ts
View File

@@ -294,6 +294,20 @@ describe('Client', () => {
}
}
});
testUtils.testWithClient('client.multi.{command}.exec should flatten array arguments', async client => {
assert.deepEqual(
await client.multi()
.sAdd('a', ['b', 'c'])
.v4.exec(),
[2])
}, {
...GLOBAL.SERVERS.OPEN,
clientOptions: {
legacyMode: true
}
});
});
describe('events', () => {

2
packages/client/lib/client/multi-command.ts
View File

@@ -87,7 +87,7 @@ export default class RedisClientMultiCommand {
#defineLegacyCommand(name: string): void {
this.v4[name] = (this as any)[name].bind(this.v4);
(this as any)[name] =
(...args: Array<unknown>): void => (this as any).addCommand(name, args);
(...args: Array<unknown>): void => (this as any).addCommand(name, ...args);
}
commandsExecutor(command: RedisCommand, args: Array<unknown>): this {