From 3bda368acb57cb37f035579301f135fa3f1b764b Mon Sep 17 00:00:00 2001
From: dovi <dovi.rivkin@gmail.com>
Date: Wed, 3 May 2023 16:48:52 -0400
Subject: [PATCH] fix HGETALL spec

---
 packages/client/lib/commands/HGETALL.spec.ts | 62 +++++++++-----------
 1 file changed, 28 insertions(+), 34 deletions(-)

diff --git a/packages/client/lib/commands/HGETALL.spec.ts b/packages/client/lib/commands/HGETALL.spec.ts
index fcd1a30457..9a9a30feaf 100644
--- a/packages/client/lib/commands/HGETALL.spec.ts
+++ b/packages/client/lib/commands/HGETALL.spec.ts
@@ -1,41 +1,35 @@
 import { strict as assert } from 'assert';
 import testUtils, { GLOBAL } from '../test-utils';
-import { transformReply } from './HGETALL';
 
 describe('HGETALL', () => {
-    describe('transformReply', () => {
-        it('empty', () => {
-            assert.deepEqual(
-                transformReply([]),
-                Object.create(null)
-            );
-        });
 
-        it('with values', () => {
-            assert.deepEqual(
-                transformReply(['key1', 'value1', 'key2', 'value2']),
-                Object.create(null, {
-                    key1: {
-                        value: 'value1',
-                        configurable: true,
-                        enumerable: true,
-                        writable: true
-                    },
-                    key2: {
-                        value: 'value2',
-                        configurable: true,
-                        enumerable: true,
-                        writable: true
-                    }
-                })
-            );
-        });
-    });
+  testUtils.testAll('hGetAll empty', async client => {
+    assert.deepEqual(
+      await client.hGetAll('key'),
+      Object.create(null)
+    );
+  }, {
+    client: GLOBAL.SERVERS.OPEN,
+    cluster: GLOBAL.CLUSTERS.OPEN
+  });
 
-    testUtils.testWithClient('client.hGetAll', async client => {
-        assert.deepEqual(
-            await client.hGetAll('key'),
-            Object.create(null)
-        );
-    }, GLOBAL.SERVERS.OPEN);
+  testUtils.testAll('hGetAll with value', async client => {
+    const [, reply] = await Promise.all([
+      client.hSet('key', 'field', 'value'),
+      client.hGetAll('key')
+    ]);
+    assert.deepEqual(
+      reply,
+      Object.create(null, {
+        field: {
+          value: 'value',
+          enumerable: true,
+          writable: true
+        }
+      })
+    );
+  }, {
+    client: GLOBAL.SERVERS.OPEN,
+    cluster: GLOBAL.CLUSTERS.OPEN
+  });
 });