Fix MGET test

Signed-off-by: Andrew Carbonetto <[email protected]>
valkey-io · Nov 15, 2024 · 22ed2ae · 22ed2ae
1 parent 87ae30a
commit 22ed2ae
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 8 deletions.
diff --git a/node/tests/ServerModules.test.ts b/node/tests/ServerModules.test.ts
@@ -2309,8 +2309,6 @@ describe("Server Module Tests", () => {
                         clusterTransaction,
                     );
                 const result = await client.exec(clusterTransaction);
-                console.log("result =====");
-                console.log(result);
 
                 validateTransactionResponse(result, expectedRes);
                 client.close();
@@ -2327,8 +2325,6 @@ describe("Server Module Tests", () => {
                 const expectedRes =
                     await transactionMultiJson(clusterTransaction);
                 const result = await client.exec(clusterTransaction);
-                console.log("result 2 =====");
-                console.log(result);
 
                 validateTransactionResponse(result, expectedRes);
                 client.close();

diff --git a/node/tests/TestUtilities.ts b/node/tests/TestUtilities.ts
@@ -1728,8 +1728,6 @@ export async function transactionMultiJson(
 ): Promise<[string, GlideReturnType][]> {
     const responseData: [string, GlideReturnType][] = [];
     const key = "{key}:1" + uuidv4();
-    const key2 = "{key}:2" + uuidv4();
-    const key3 = "{key}:3" + uuidv4();
     const jsonValue = { a: [1, 2], b: [3, 4], c: "c", d: true };
 
     // JSON.SET to create a key for testing commands.
@@ -1772,8 +1770,18 @@ export async function transactionMultiJson(
     GlideMultiJson.type(baseTransaction, key, { path: "$.a" });
     responseData.push(['type(key, "$.a")', ["array"]]);
 
-    // JSON.MGET -> TODO
-    GlideMultiJson.mget(baseTransaction, [key, key2, key3]);
+    // JSON.MGET
+    const key2 = "{key}:2" + uuidv4();
+    const key3 = "{key}:3" + uuidv4();
+    const jsonValue2 = { b: [3, 4], c: "c", d: true };
+    GlideMultiJson.set(baseTransaction, key2, "$", JSON.stringify(jsonValue2));
+    responseData.push(['set(key2, "$")', "OK"]);
+
+    GlideMultiJson.mget(baseTransaction, [key, key2, key3], "$.a");
+    responseData.push([
+        'json.mget([key, key2, key3], "$.a")',
+        ["[[110,120]]", "[]", null],
+    ]);
 
     // JSON.TOGGLE
     GlideMultiJson.toggle(baseTransaction, key, { path: "$.d" });