IDEA-185548 Inspection to remove excessive Comparator.comparing call

java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/redundantComparing/afterComparing.java

@@ -0,0 +1,6 @@
+// "Remove 'comparing' call" "true"
+import java.util.*;
+
+class Test {
+  Comparator<String> cmp = Comparator.comparing(String::length).thenComparing(s -> s.charAt(1));
+}

java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/redundantComparing/afterComparingInt.java

@@ -0,0 +1,6 @@
+// "Remove 'comparingInt' call and use 'thenComparingInt'" "true"
+import java.util.*;
+
+class Test {
+  Comparator<String> cmp = Comparator.comparing(String::length).thenComparingInt(s -> s.charAt(1));
+}

java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/redundantComparing/afterCustomInterface.java

@@ -0,0 +1,11 @@
+// "Remove 'comparing' call" "true"
+import java.util.Comparator;
+import java.util.function.Function;
+
+public class MyFile {
+  interface Foo extends Function<String, Integer> {}
+
+  void test(Foo comparator) {
+    Comparator<String> cmp = Comparator.comparing(String::length).thenComparing(comparator);
+  }
+}

java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/redundantComparing/beforeComparing.java

@@ -0,0 +1,6 @@
+// "Remove 'comparing' call" "true"
+import java.util.*;
+
+class Test {
+  Comparator<String> cmp = Comparator.comparing(String::length).thenComparing(Comparator.c<caret>omparing(s -> s.charAt(1)));
+}

java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/redundantComparing/beforeComparingInt.java

@@ -0,0 +1,6 @@
+// "Remove 'comparingInt' call and use 'thenComparingInt'" "true"
+import java.util.*;
+
+class Test {
+  Comparator<String> cmp = Comparator.comparing(String::length).thenComparing(Comparator.c<caret>omparingInt(s -> s.charAt(1)));
+}

java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/redundantComparing/beforeCustomInterface.java

@@ -0,0 +1,11 @@
+// "Remove 'comparing' call" "true"
+import java.util.Comparator;
+import java.util.function.Function;
+
+public class MyFile {
+  interface Foo extends Function<String, Integer> {}
+
+  void test(Foo comparator) {
+    Comparator<String> cmp = Comparator.comparing(String::length).thenComparing(Comparator.comp<caret>aring(comparator));
+  }
+}

java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/redundantComparing/beforeCustomInterfaceConflict.java

@@ -0,0 +1,12 @@
+// "Remove 'comparing' call" "false"
+import java.util.Comparator;
+import java.util.function.Function;
+
+public class MyFile {
+  interface Foo extends Function<String, Integer>, Comparator<String> {}
+
+  void test(Foo comparator) {
+    // replacement will make the call ambiguous, thus not suggested
+    Comparator<String> cmp = Comparator.comparing(String::length).thenComparing(Comparator.comp<caret>aring(comparator));
+  }
+}