IDEA-178788 Suggestion code replace from list iteration to List.removeAll

java/java-impl/src/com/intellij/codeInspection/bulkOperation/JdkBulkMethodInfoProvider.java

@@ -26,8 +26,7 @@ import java.util.stream.Stream;
  */
 public class JdkBulkMethodInfoProvider implements BulkMethodInfoProvider {
   private static BulkMethodInfo[] INFOS = {
-    new BulkMethodInfo(CommonClassNames.JAVA_UTIL_COLLECTION, "add", "addAll"),
-    new BulkMethodInfo(CommonClassNames.JAVA_UTIL_LIST, "remove", "removeAll")
+    new BulkMethodInfo(CommonClassNames.JAVA_UTIL_COLLECTION, "add", "addAll")
   };
 
   @NotNull

java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/useBulkOperation/afterAddAllWhile.java

@@ -0,0 +1,10 @@
+// "Replace iteration with bulk 'Collection.addAll' call" "true"
+import java.util.*;
+
+public class Collect {
+  static class Person {}
+
+  void collectNames(List<Person> persons, Collection<Person> toAdd){
+      persons.addAll(toAdd);
+  }
+}

java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/useBulkOperation/afterRemoveAll.java

@@ -1,10 +0,0 @@
-// "Replace iteration with bulk 'List.removeAll' call" "true"
-import java.util.*;
-
-public class Collect {
-  static class Person {}
-
-  void collectNames(List<Person> persons, Collection<Person> toRemove){
-      persons.removeAll(toRemove);
-  }
-}

java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/useBulkOperation/afterRemoveAllWhile.java

@@ -1,10 +0,0 @@
-// "Replace iteration with bulk 'List.removeAll' call" "true"
-import java.util.*;
-
-public class Collect {
-  static class Person {}
-
-  void collectNames(List<Person> persons, Collection<Person> toRemove){
-      persons.removeAll(toRemove);
-  }
-}

java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/useBulkOperation/beforeAddAllWhile.java

@@ -0,0 +1,14 @@
+// "Replace iteration with bulk 'Collection.addAll' call" "true"
+import java.util.*;
+
+public class Collect {
+  static class Person {}
+
+  void collectNames(List<Person> persons, Collection<Person> toAdd){
+    Iterator<Person> it = toAdd.iterator();
+    while (it.hasNext()) {
+      Person person = it.next();
+      persons<caret>.add(person);
+    }
+  }
+}

java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/useBulkOperation/beforeRemoveAll.java

@@ -1,11 +0,0 @@
-// "Replace iteration with bulk 'List.removeAll' call" "true"
-import java.util.*;
-
-public class Collect {
-  static class Person {}
-
-  void collectNames(List<Person> persons, Collection<Person> toRemove){
-    for (Iterator<Person> it = toRemove.iterator(); it.hasNext(); )
-      persons<caret>.remove(it.next());
-  }
-}

java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/useBulkOperation/beforeRemoveAllWhile.java

@@ -1,14 +0,0 @@
-// "Replace iteration with bulk 'List.removeAll' call" "true"
-import java.util.*;
-
-public class Collect {
-  static class Person {}
-
-  void collectNames(List<Person> persons, Collection<Person> toRemove){
-    Iterator<Person> it = toRemove.iterator();
-    while (it.hasNext()) {
-      Person person = it.next();
-      persons<caret>.remove(person);
-    }
-  }
-}