IDEA-191500 Inappropriate 'Enum.valueOf()' Smart Type suggestion

java/java-impl/src/com/intellij/psi/filters/types/AssignableFromFilter.java

@@ -112,14 +112,30 @@ public class AssignableFromFilter implements ElementFilter{
                                                                                       expectedType,
                                                                                       false,
                                                                                       PsiUtil.getLanguageLevel(place));
-      if (substitutionForParameter != PsiType.NULL && !(substitutionForParameter instanceof PsiIntersectionType) &&
-        PsiUtil.resolveClassInClassTypeOnly(substitutionForParameter) != parameter) {
+      if (substitutionForParameter != PsiType.NULL &&
+          !isImpossibleIntersection(substitutionForParameter) &&
+          !extendsImpossibleIntersection(PsiUtil.resolveClassInClassTypeOnly(substitutionForParameter)) &&
+          PsiUtil.resolveClassInClassTypeOnly(substitutionForParameter) != parameter) {
         return true;
       }
     }
     return false;
   }
 
+  private static boolean extendsImpossibleIntersection(@Nullable PsiClass psiClass) {
+    if (psiClass instanceof PsiTypeParameter) {
+      PsiClassType[] supers = psiClass.getExtendsListTypes();
+      if (supers.length > 1) {
+        return isImpossibleIntersection(PsiIntersectionType.createIntersection(supers));
+      }
+    }
+    return false;
+  }
+
+  private static boolean isImpossibleIntersection(PsiType intersection) {
+    return intersection instanceof PsiIntersectionType && ((PsiIntersectionType)intersection).getConflictingConjunctsMessage() != null;
+  }
+
   public String toString(){
     return "assignable-from(" + (myType != null ? myType : myClassName) + ")";
   }

java/java-tests/testData/codeInsight/completion/smartType/NoUnrelatedMethodSuggestion.java

@@ -0,0 +1,8 @@
+class Foo {
+  {
+    Foo m = <caret>x;
+  }
+
+
+  static <TX extends Enum<TX>> TX valueOf(Class<TX> enumType, String name);
+}

java/java-tests/testSrc/com/intellij/java/codeInsight/completion/SmartType18CompletionTest.java

@@ -188,6 +188,12 @@ public void testConvertToObjectStream() {
     checkResultByFile("/" + getTestName(false) + "-out.java");
   }
 
+  public void testNoUnrelatedMethodSuggestion() {
+    configureByTestName();
+    myFixture.complete(CompletionType.SMART, 1);
+    assertOrderedEquals(myFixture.getLookupElementStrings(), "this");
+  }
+
   public void testInferFromReturnTypeWhenCompleteInsideArgList() {
     configureByTestName();
     myFixture.complete(CompletionType.SMART, 1);

java/java-tests/testSrc/com/intellij/java/codeInsight/completion/SmartTypeCompletionTest.java

@@ -1282,4 +1282,8 @@ public class SmartTypeCompletionTest extends LightFixtureCompletionTestCase {
   public void testNewMapTypeArguments() { doTest(); }
   public void testNewMapObjectTypeArguments() { doTest(); }
 
+  public void testNoUnrelatedMethodSuggestion() {
+    configureByTestName();
+    assertOrderedEquals(myFixture.getLookupElementStrings(), "this");
+  }
 }