listComprehension in method overrides the expression if called twice

Author: angrykoala

.changeset/bright-meals-arrive.md

@@ -0,0 +1,5 @@
+---
+"@neo4j/cypher-builder": major
+---
+
+ListComprehension `.in` method no longer throws if called twice. It will instead override the expression

src/expressions/list/ListComprehension.test.ts

@@ -91,13 +91,24 @@ describe("List comprehension", () => {
         `);
     });
 
-    test("Fails to set a expression twice", () => {
+    test("Overrides if a expression is set twice", () => {
         const variable = new Cypher.Variable();
         const exprVariable = new Cypher.Param([1, 2, 5]);
+        const exprVariable2 = new Cypher.Param([1, 3]);
 
-        expect(() => {
-            new Cypher.ListComprehension(variable).in(exprVariable).in(exprVariable);
-        }).toThrow("Cannot set 2 lists in list comprehension IN");
+        const listComprehension = new Cypher.ListComprehension(variable).in(exprVariable).in(exprVariable2);
+
+        const queryResult = new TestClause(listComprehension).build();
+
+        expect(queryResult.cypher).toMatchInlineSnapshot(`"[var0 IN $param0]"`);
+        expect(queryResult.params).toMatchInlineSnapshot(`
+            {
+              "param0": [
+                1,
+                3,
+              ],
+            }
+        `);
     });
 
     test("Fails to build if no expression is set", () => {

src/expressions/list/ListComprehension.ts

@@ -43,8 +43,8 @@ export class ListComprehension extends CypherASTNode {
         this.variable = variable;
     }
 
+    /** Sets the list expression to be used for the comprehension. If called twice, the expression will be overriden */
     public in(listExpr: Expr): this {
-        if (this.listExpr) throw new Error("Cannot set 2 lists in list comprehension IN");
         this.listExpr = listExpr;
         return this;
     }