fix: Add back Pipeline#length

Closes #1584
redis · May 21, 2022 · f7c43f8 · f7c43f8
1 parent 91ed2d8
commit f7c43f8
Show file tree

Hide file tree

Showing 4 changed files with 14 additions and 4 deletions.
diff --git a/bin/template.ts b/bin/template.ts
@@ -11,8 +11,9 @@ export interface ResultTypes<Result, Context> {
   pipeline: ChainableCommander;
 }
 
-export interface ChainableCommander
-  extends RedisCommander<{ type: "pipeline" }> {}
+export interface ChainableCommander extends RedisCommander<{ type: "pipeline" }> {
+  length: number;
+}
 
 export type ClientContext = { type: keyof ResultTypes<unknown, unknown> };
 export type Result<T, Context extends ClientContext> =

diff --git a/lib/utils/RedisCommander.ts b/lib/utils/RedisCommander.ts
@@ -11,8 +11,9 @@ export interface ResultTypes<Result, Context> {
   pipeline: ChainableCommander;
 }
 
-export interface ChainableCommander
-  extends RedisCommander<{ type: "pipeline" }> {}
+export interface ChainableCommander extends RedisCommander<{ type: "pipeline" }> {
+  length: number;
+}
 
 export type ClientContext = { type: keyof ResultTypes<unknown, unknown> };
 export type Result<T, Context extends ClientContext> =

diff --git a/test/functional/pipeline.ts b/test/functional/pipeline.ts
@@ -400,6 +400,7 @@ describe("pipeline", () => {
 
       const pipeline1 = redis
         .pipeline()
+        // @ts-expect-error
         .multi()
         .set("foo", "bar")
         .get("foo")

diff --git a/test/typing/pipeline.test-d.ts b/test/typing/pipeline.test-d.ts
@@ -24,3 +24,10 @@ expectType<RETURN_TYPE>(
     ])
     .exec()
 );
+
+expectType<number>(
+  redis.pipeline([
+    ["set", Buffer.from("foo"), "bar"],
+    ["incrby", "foo", 42],
+  ]).length
+);