Remove second parameter of listComprehension

Author: angrykoala

.changeset/clean-items-greet.md

@@ -0,0 +1,5 @@
+---
+"@neo4j/cypher-builder": major
+---
+
+Remove second parameter of `ListComprehension` in favor of `.in`

src/expressions/list/ListComprehension.test.ts

@@ -25,7 +25,7 @@ describe("List comprehension", () => {
         const variable = new Cypher.Variable();
         const exprVariable = new Cypher.Param([1, 2, 5]);
 
-        const listComprehension = new Cypher.ListComprehension(variable, exprVariable);
+        const listComprehension = new Cypher.ListComprehension(variable).in(exprVariable);
 
         const queryResult = new TestClause(listComprehension).build();
 
@@ -47,7 +47,7 @@ describe("List comprehension", () => {
         const exprVariable = new Cypher.Param([1, 2, 5]);
         const andExpr = Cypher.eq(variable, new Cypher.Param(5));
 
-        const listComprehension = new Cypher.ListComprehension(variable, exprVariable).where(andExpr);
+        const listComprehension = new Cypher.ListComprehension(variable).in(exprVariable).where(andExpr);
 
         const queryResult = new TestClause(listComprehension).build();
 
@@ -96,7 +96,7 @@ describe("List comprehension", () => {
         const exprVariable = new Cypher.Param([1, 2, 5]);
 
         expect(() => {
-            new Cypher.ListComprehension(variable, exprVariable).in(exprVariable);
+            new Cypher.ListComprehension(variable).in(exprVariable).in(exprVariable);
         }).toThrow("Cannot set 2 lists in list comprehension IN");
     });
 

src/expressions/list/ListComprehension.ts

@@ -38,10 +38,9 @@ export class ListComprehension extends CypherASTNode {
     private listExpr: Expr | undefined;
     private mapExpr: Expr | undefined; //  Expression for list mapping
 
-    constructor(variable: Variable, listExpr?: Expr) {
+    constructor(variable: Variable) {
         super();
         this.variable = variable;
-        this.listExpr = listExpr;
     }
 
     public in(listExpr: Expr): this {