Generate setters, but tests are failing

1 files changed, 12 insertions, 7 deletions

diff --git a/javaparser-core/src/main/java/com/github/javaparser/ast/expr/BinaryExpr.java b/javaparser-core/src/main/java/com/github/javaparser/ast/expr/BinaryExpr.java
index 3b92261b6..7bc493de0 100644
--- a/javaparser-core/src/main/java/com/github/javaparser/ast/expr/BinaryExpr.java
+++ b/javaparser-core/src/main/java/com/github/javaparser/ast/expr/BinaryExpr.java
@@ -26,6 +26,8 @@ import com.github.javaparser.ast.observer.ObservableProperty;
 import com.github.javaparser.ast.visitor.GenericVisitor;
 import com.github.javaparser.ast.visitor.VoidVisitor;
 
+import static com.github.javaparser.utils.Utils.assertNotNull;
+
 /**
  * An expression with an expression on the left, an expression on the right, and an operator in the middle.
  * It supports the operators that are found the the BinaryExpr.Operator enum.
@@ -95,23 +97,26 @@ public final class BinaryExpr extends Expression {
         return right;
     }
 
-    public BinaryExpr setLeft(Expression left) {
+    public BinaryExpr setLeft(final Expression left) {
+        assertNotNull(left);
         notifyPropertyChange(ObservableProperty.LEFT, this.left, left);
         this.left = left;
-        setAsParentNodeOf(this.left);
+        setAsParentNodeOf(left);
         return this;
     }
 
-    public BinaryExpr setOperator(Operator op) {
-        notifyPropertyChange(ObservableProperty.OPERATOR, this.operator, op);
-        this.operator = op;
+    public BinaryExpr setOperator(final Operator operator) {
+        assertNotNull(operator);
+        notifyPropertyChange(ObservableProperty.OPERATOR, this.operator, operator);
+        this.operator = operator;
         return this;
     }
 
-    public BinaryExpr setRight(Expression right) {
+    public BinaryExpr setRight(final Expression right) {
+        assertNotNull(right);
         notifyPropertyChange(ObservableProperty.RIGHT, this.right, right);
         this.right = right;
-        setAsParentNodeOf(this.right);
+        setAsParentNodeOf(right);
         return this;
     }
 }