fix bloom info command update

didn't update tests, and got types wrong

Author: sjpotter

packages/bloom/lib/commands/bloom/INFO.spec.ts

@@ -17,10 +17,10 @@ describe('BF.INFO', () => {
     ]);
 
     assert.equal(typeof reply, 'object');
-    assert.equal(reply.capacity, 100);
-    assert.equal(typeof reply.size, 'number');
-    assert.equal(typeof reply.numberOfFilters, 'number');
-    assert.equal(typeof reply.numberOfInsertedItems, 'number');
-    assert.equal(typeof reply.expansionRate, 'number');
+    assert.equal(reply['Capacity'], 100);
+    assert.equal(typeof reply['Size'], 'number');
+    assert.equal(typeof reply['Number of filters'], 'number');
+    assert.equal(typeof reply['Number of items inserted'], 'number');
+    assert.equal(typeof reply['Expansion rate'], 'number');
   }, GLOBAL.SERVERS.OPEN);
 });

packages/bloom/lib/commands/cuckoo/INFO.spec.ts

@@ -17,13 +17,13 @@ describe('CF.INFO', () => {
     ]);
 
     assert.equal(typeof reply, 'object');
-    assert.equal(typeof reply.size, 'number');
-    assert.equal(typeof reply.numberOfBuckets, 'number');
-    assert.equal(typeof reply.numberOfFilters, 'number');
-    assert.equal(typeof reply.numberOfInsertedItems, 'number');
-    assert.equal(typeof reply.numberOfDeletedItems, 'number');
-    assert.equal(typeof reply.bucketSize, 'number');
-    assert.equal(typeof reply.expansionRate, 'number');
-    assert.equal(typeof reply.maxIteration, 'number');
+    assert.equal(typeof reply['Size'], 'number');
+    assert.equal(typeof reply['Number of buckets'], 'number');
+    assert.equal(typeof reply['Number of filters'], 'number');
+    assert.equal(typeof reply['Number of items inserted'], 'number');
+    assert.equal(typeof reply['Number of items deleted'], 'number');
+    assert.equal(typeof reply['Bucket size'], 'number');
+    assert.equal(typeof reply['Expansion rate'], 'number');
+    assert.equal(typeof reply['Max iterations'], 'number');
   }, GLOBAL.SERVERS.OPEN);
 });

packages/bloom/lib/commands/cuckoo/INFO.ts

@@ -12,27 +12,16 @@ export type CfInfoReplyMap = TuplesToMapReply<[
   [SimpleStringReply<'Max iterations'>, NumberReply]
 ]>;
 
-export interface CfInfoReply {
-  size: NumberReply;
-  numberOfBuckets: NumberReply;
-  numberOfFilters: NumberReply;
-  numberOfInsertedItems: NumberReply;
-  numberOfDeletedItems: NumberReply;
-  bucketSize: NumberReply;
-  expansionRate: NumberReply;
-  maxIteration: NumberReply;
-}
-
 export default {
   FIRST_KEY_INDEX: 1,
   IS_READ_ONLY: true,
   transformArguments(key: RedisArgument) {
     return ['CF.INFO', key];
   },
   transformReply: {
-    2: (reply: UnwrapReply<Resp2Reply<CfInfoReplyMap>>, _, typeMapping?: TypeMapping): CfInfoReply => {
-      return transformInfoV2Reply<CfInfoReply>(reply, typeMapping);
+    2: (reply: UnwrapReply<Resp2Reply<CfInfoReplyMap>>, _, typeMapping?: TypeMapping): CfInfoReplyMap => {
+      return transformInfoV2Reply<CfInfoReplyMap>(reply, typeMapping);
     },
-    3: undefined as unknown as () => CfInfoReply
+    3: undefined as unknown as () => CfInfoReplyMap
   }
 } as const satisfies Command;

packages/bloom/lib/commands/t-digest/INFO.spec.ts

@@ -16,12 +16,15 @@ describe('TDIGEST.INFO', () => {
       client.tDigest.info('key')
     ]);
 
-    assert(typeof reply.capacity, 'number');
-    assert(typeof reply.mergedNodes, 'number');
-    assert(typeof reply.unmergedNodes, 'number');
-    assert(typeof reply.mergedWeight, 'number');
-    assert(typeof reply.unmergedWeight, 'number');
-    assert(typeof reply.totalCompression, 'number');
-    assert(typeof reply.totalCompression, 'number');
+    assert(typeof reply, 'object');
+    assert(typeof reply['Compression'], 'number');
+    assert(typeof reply['Capacity'], 'number');
+    assert(typeof reply['Merged nodes'], 'number');
+    assert(typeof reply['Unmerged nodes'], 'number');
+    assert(typeof reply['Merged weight'], 'number');
+    assert(typeof reply['Unmerged weight'], 'number');
+    assert(typeof reply['Observations'], 'number');
+    assert(typeof reply['Total compressions'], 'number');
+    assert(typeof reply['Memory usage'], 'number');
   }, GLOBAL.SERVERS.OPEN);
 });

packages/bloom/lib/commands/t-digest/INFO.ts

@@ -13,28 +13,16 @@ export type TdInfoReplyMap = TuplesToMapReply<[
   [SimpleStringReply<'Memory usage'>, NumberReply]
 ]>;
 
-export interface TdInfoReply {
-  compression: NumberReply;
-  capacity: NumberReply;
-  mergedNodes: NumberReply;
-  unmergedNodes: NumberReply;
-  mergedWeight: NumberReply;
-  unmergedWeight: NumberReply;
-  observations: NumberReply,
-  totalCompression: NumberReply;
-  memoryUsage: NumberReply;
-}
-
 export default {
   FIRST_KEY_INDEX: 1,
   IS_READ_ONLY: true,
   transformArguments(key: RedisArgument) {
     return ['TDIGEST.INFO', key];
   },
   transformReply: {
-    2: (reply: UnwrapReply<Resp2Reply<TdInfoReplyMap>>, _, typeMapping?: TypeMapping): TdInfoReply => {
-      return transformInfoV2Reply<TdInfoReply>(reply, typeMapping);
+    2: (reply: UnwrapReply<Resp2Reply<TdInfoReplyMap>>, _, typeMapping?: TypeMapping): TdInfoReplyMap => {
+      return transformInfoV2Reply<TdInfoReplyMap>(reply, typeMapping);
     },
-    3: undefined as unknown as () => TdInfoReply
+    3: undefined as unknown as () => TdInfoReplyMap
   }
 } as const satisfies Command;