Support multiple parametrs on CONFIG SET (#2042)

* support multiple parametrs on CONFIG SET

* clean code

Co-authored-by: leibale <[email protected]>

Author: Avital-Fine

packages/client/lib/commands/CONFIG_SET.spec.ts

@@ -2,10 +2,23 @@ import { strict as assert } from 'assert';
 import { transformArguments } from './CONFIG_SET';
 
 describe('CONFIG SET', () => {
-    it('transformArguments', () => {
-        assert.deepEqual(
-            transformArguments('parameter', 'value'),
-            ['CONFIG', 'SET', 'parameter', 'value']
-        );
+    describe('transformArguments', () => {
+        it('set one parameter (old version)', () => {
+            assert.deepEqual(
+                transformArguments('parameter', 'value'),
+                ['CONFIG', 'SET', 'parameter', 'value']
+            );
+        });
+
+        it('set muiltiple parameters', () => {
+            assert.deepEqual(
+                transformArguments({
+                    1: 'a',
+                    2: 'b',
+                    3: 'c'
+                }),
+                ['CONFIG', 'SET', '1', 'a', '2', 'b', '3', 'c']
+            );
+        });
     });
 });

packages/client/lib/commands/CONFIG_SET.ts

@@ -1,5 +1,23 @@
-export function transformArguments(parameter: string, value: string): Array<string> {
-    return ['CONFIG', 'SET', parameter, value];
+import { RedisCommandArgument, RedisCommandArguments } from '.';
+
+type SingleParameter = [parameter: RedisCommandArgument, value: RedisCommandArgument];
+
+type MultipleParameters = [config: Record<string, RedisCommandArgument>];
+
+export function transformArguments(
+    ...[parameterOrConfig, value]: SingleParameter | MultipleParameters
+): RedisCommandArguments {
+    const args: RedisCommandArguments = ['CONFIG', 'SET'];
+
+    if (typeof parameterOrConfig === 'string') {
+        args.push(parameterOrConfig, value!);
+    } else {
+        for (const [key, value] of Object.entries(parameterOrConfig)) {
+            args.push(key, value);
+        }
+    }
+
+    return args;
 }
 
 export declare function transformReply(): string;