test(functional): monitor result compare lowercase more changes - rel…

…ated to issue #1671 Signed-off-by: Boaz Sade <boaz@dragonflydb.io>
redis · Nov 6, 2022 · 44ff5db · 44ff5db
1 parent 0add883
commit 44ff5db
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 10 deletions.
diff --git a/test/functional/monitor.ts b/test/functional/monitor.ts
@@ -15,7 +15,7 @@ describe("monitor", () => {
           return;
         }
         monitor.on("monitor", function (time, args) {
-          expect(args[0]).to.eql("get");
+          expect(args[0]).to.match(/get/i);
           expect(args[1]).to.eql("foo");
           redis.disconnect();
           monitor.disconnect();
@@ -61,7 +61,7 @@ describe("monitor", () => {
         return;
       }
       monitor.on("monitor", (_time, args) => {
-        if (args[0] === "set") {
+        if (args[0] === "set" || args[0] === "SET") {
           redis.disconnect();
           monitor.disconnect();
           done();

diff --git a/test/functional/pipeline.ts b/test/functional/pipeline.ts
@@ -323,7 +323,7 @@ describe("pipeline", () => {
       redis.disconnect();
 
       expectedCommands.forEach((expectedCommand, j) => {
-        expectedCommand.forEach((arg, i) => expect(arg).to.eql(commands[j][i]));
+        expectedCommand.forEach((arg, i) => expect(arg).to.eql(commands[j][i].toLowerCase()));
       });
     });
 
@@ -388,9 +388,7 @@ describe("pipeline", () => {
       redis2.disconnect();
 
       const expected = ["set", "eval", "get"];
-      commands.forEach((c, i) => {
-        expect(c[0]).to.equal(expected[i]);
-      });
+      expect(commands.map((c) => c[0].toLowerCase())).to.have.members(expected);
     });
   });
 

diff --git a/test/functional/scripting.ts b/test/functional/scripting.ts
@@ -204,7 +204,7 @@ describe("scripting", () => {
     });
 
     const expectedComands = ["evalsha", "eval", "get", "evalsha", "get"];
-    expect(commands.map((c) => c[0])).to.eql(expectedComands);
+    expect(commands.map((c) => c[0].toLowerCase())).to.have.members(expectedComands);
   });
 
   it("should load scripts first before execution of pipeline", async () => {
@@ -230,7 +230,7 @@ describe("scripting", () => {
 
     const expectedComands = ["evalsha", "get", "eval", "set", "get"];
 
-    expect(commands.map((c) => c[0])).to.eql(expectedComands);
+    expect(commands.map((c) => c[0].toLowerCase())).to.have.members(expectedComands);
   });
 
   it("does not fallback to EVAL in regular transaction", async () => {
@@ -260,7 +260,7 @@ describe("scripting", () => {
     spy.restore();
     expect(spy.callCount).to.equal(4);
     const expectedComands = ["multi", "evalsha", "evalsha", "exec"];
-    expect(commands.map((c) => c[0])).to.eql(expectedComands);
+    expect(commands.map((c) => c[0].toLowerCase())).to.have.members(expectedComands);
   });
 
   it("does not fallback to EVAL in manual transaction", async () => {
@@ -284,7 +284,7 @@ describe("scripting", () => {
     spy.restore();
     expect(spy.callCount).to.equal(4);
     const expectedComands = ["multi", "evalsha", "evalsha", "exec"];
-    expect(commands.map((c) => c[0])).to.eql(expectedComands);
+    expect(commands.map((c) => c[0].toLowerCase())).to.have.members(expectedComands);
   });
 
   it("should support key prefixing", (done) => {