Node: fix pubsub IT by making sure cluster mode tests run only with c…

…luster mode enabled flag (#2317) * Signed-off-by: Yury-Fridlyand <[email protected]> * Signed-off-by: Yury-Fridlyand <[email protected]>
valkey-io · Sep 19, 2024 · 1c3aa0d · 1c3aa0d
1 parent e37308d
commit 1c3aa0d
Showing 1 changed file with 8 additions and 12 deletions.
diff --git a/node/tests/PubSub.test.ts b/node/tests/PubSub.test.ts
@@ -4018,7 +4018,7 @@ describe("PubSub", () => {
      *
      * @param clusterMode - Indicates if the test should be run in cluster mode.
      */
-    it.each([true, false])(
+    it.each([true])(
         "test pubsub numsub and shardnumsub separation_%p",
         async (clusterMode) => {
             //const clusterMode = false;
@@ -4070,17 +4070,13 @@ describe("PubSub", () => {
                 });
 
                 // Test pubsubShardnumsub
-                if (clusterMode) {
-                    const shardSubscribers = await (
-                        client2 as GlideClusterClient
-                    ).pubsubShardNumSub([regularChannel, shardChannel]);
-                    expect(
-                        convertGlideRecordToRecord(shardSubscribers),
-                    ).toEqual({
-                        [regularChannel]: 0,
-                        [shardChannel]: 2,
-                    });
-                }
+                const shardSubscribers = await (
+                    client2 as GlideClusterClient
+                ).pubsubShardNumSub([regularChannel, shardChannel]);
+                expect(convertGlideRecordToRecord(shardSubscribers)).toEqual({
+                    [regularChannel]: 0,
+                    [shardChannel]: 2,
+                });
             } finally {
                 if (client1) {
                     await clientCleanup(client1, pubSub!);