Add more tests

Author: Fabrice-TIERCELIN

samples/src/test/java/org/autorefactor/jdt/internal/ui/fix/samples_in/BreakRatherThanPassiveIterationsSample.java

@@ -26,6 +26,7 @@
 package org.autorefactor.jdt.internal.ui.fix.samples_in;
 
 public class BreakRatherThanPassiveIterationsSample {
+    private int[] innerArray = new int[10];
 
     private int crazyInteger = 0;
 
@@ -55,6 +56,19 @@ public String addBreakInForeachLoop(int[] array) {
         return isFound ? "The result has been found" : "The result has not been found";
     }
 
+    public String addBreakWithField() {
+        boolean isFound = false;
+
+        for (int i : this.innerArray) {
+            if (i == 42) {
+                // Keep this comment
+                isFound = true;
+            }
+        }
+
+        return isFound ? "The result has been found" : "The result has not been found";
+    }
+
     public String addBreakWithoutBlock(int[] array) {
         boolean isFound = false;
 

samples/src/test/java/org/autorefactor/jdt/internal/ui/fix/samples_in/XORRatherThanDuplicateConditionsSample.java

@@ -68,6 +68,12 @@ public void replaceDuplicateConditionsWithExpressions(int i1, int i2, int i3, in
         boolean newBoolean4 = (i1 == i2) && (i3 < i4) || (i1 != i2) && (i4 <= i3);
     }
 
+    public void replaceDuplicateConditionsWithFields() {
+        // Keep this comment
+        boolean newBoolean1 = (staticField > 0) && (staticField < 100) || (staticField <= 0) && (staticField >= 100);
+        boolean newBoolean2 = (staticField > 0) && (staticField < 100) || (staticField >= 100) && !(staticField > 0);
+    }
+
     public void doNotReplaceDuplicateConditionsWithMethods(List<String> myList) {
         boolean newBoolean1 = myList.remove("lorem") && !myList.remove("ipsum") || !myList.remove("lorem")
                 && myList.remove("ipsum");

samples/src/test/java/org/autorefactor/jdt/internal/ui/fix/samples_out/BreakRatherThanPassiveIterationsSample.java

@@ -26,6 +26,7 @@
 package org.autorefactor.jdt.internal.ui.fix.samples_out;
 
 public class BreakRatherThanPassiveIterationsSample {
+    private int[] innerArray = new int[10];
 
     private int crazyInteger = 0;
 
@@ -57,6 +58,20 @@ public String addBreakInForeachLoop(int[] array) {
         return isFound ? "The result has been found" : "The result has not been found";
     }
 
+    public String addBreakWithField() {
+        boolean isFound = false;
+
+        for (int i : this.innerArray) {
+            if (i == 42) {
+                // Keep this comment
+                isFound = true;
+                break;
+            }
+        }
+
+        return isFound ? "The result has been found" : "The result has not been found";
+    }
+
     public String addBreakWithoutBlock(int[] array) {
         boolean isFound = false;
 

samples/src/test/java/org/autorefactor/jdt/internal/ui/fix/samples_out/XORRatherThanDuplicateConditionsSample.java

@@ -68,6 +68,12 @@ public void replaceDuplicateConditionsWithExpressions(int i1, int i2, int i3, in
         boolean newBoolean4 = (i1 == i2) == (i3 < i4);
     }
 
+    public void replaceDuplicateConditionsWithFields() {
+        // Keep this comment
+        boolean newBoolean1 = (staticField > 0) == (staticField < 100);
+        boolean newBoolean2 = (staticField > 0) == (staticField < 100);
+    }
+
     public void doNotReplaceDuplicateConditionsWithMethods(List<String> myList) {
         boolean newBoolean1 = myList.remove("lorem") && !myList.remove("ipsum") || !myList.remove("lorem")
                 && myList.remove("ipsum");