update operand javadocs, tidy test imports and helper placement

Author: andyfelder16

client-java/controller/src/main/java/org/evomaster/client/java/controller/neo4j/conditions/Operand.java

@@ -1,9 +1,12 @@
 package org.evomaster.client.java.controller.neo4j.conditions;
 
 /**
- * One side of a comparison in a WHERE clause: a {@link PropertyOperand} resolved from the
- * matched element, a {@link LiteralOperand} constant, or a {@link RawOperand} kept unchanged.
- * Modelling each side explicitly lets {@code n.age > m.age} be told apart from {@code n.age > "m.age"}.
+ * One operand of a condition: a side of a WHERE comparison or an inline property value. Concrete
+ * kinds are {@link PropertyOperand} (a {@code variable.key} resolved from the matched element),
+ * {@link LiteralOperand} (a constant), {@link ArithmeticOperand} (an arithmetic expression),
+ * {@link ListOperand} (an {@code IN} list), and {@link RawOperand} (kept verbatim when not
+ * decomposed). Modelling each explicitly lets {@code n.age > m.age} be told apart from
+ * {@code n.age > "m.age"}.
  */
 public interface Operand {
 }

client-java/controller/src/main/java/org/evomaster/client/java/controller/neo4j/conditions/RawOperand.java

@@ -3,8 +3,8 @@
 import java.util.Objects;
 
 /**
- * An operand kept unchanged because the model does not decompose it: arithmetic such as
- * 25 + 5, a function call, or a list. Carries the original text so nothing is dropped.
+ * An operand kept unchanged because the model does not decompose it: a function call, a parameter,
+ * or string concatenation. Carries the original text so nothing is dropped.
  */
 public final class RawOperand implements Operand {
 

client-java/controller/src/test/java/org/evomaster/client/java/controller/neo4j/parser/Cypher25AntlrParserTest.java

@@ -4,8 +4,8 @@
 import org.evomaster.client.java.controller.neo4j.operations.MatchOperation;
 import org.evomaster.client.java.controller.neo4j.operations.PatternEdge;
 import org.evomaster.client.java.controller.neo4j.operations.QuantifiedPathPattern;
-import org.junit.jupiter.api.Test;
 import org.evomaster.client.java.controller.neo4j.parser.cypher25.Cypher25AntlrParser;
+import org.junit.jupiter.api.Test;
 
 import static org.junit.jupiter.api.Assertions.*;
 
@@ -36,6 +36,13 @@ private <T extends CypherCondition> long countOf(MatchOperation op, Class<T> typ
         return op.getConditions().stream().filter(type::isInstance).count();
     }
 
+    private ComparisonCondition comparison(MatchOperation op) {
+        return (ComparisonCondition) op.getConditions().stream()
+                .filter(ComparisonCondition.class::isInstance)
+                .findFirst()
+                .orElseThrow(() -> new AssertionError("no ComparisonCondition found"));
+    }
+
     @Test
     void testAllNodes() {
         MatchOperation op = parse("MATCH (n) RETURN n");
@@ -799,10 +806,4 @@ void testSyntaxErrorFails() {
         assertFails("MATCH (n:Person RETURN n"); // missing closing paren
     }
 
-    private ComparisonCondition comparison(MatchOperation op) {
-        return (ComparisonCondition) op.getConditions().stream()
-                .filter(ComparisonCondition.class::isInstance)
-                .findFirst()
-                .orElseThrow(() -> new AssertionError("no ComparisonCondition found"));
-    }
 }