add single static import: ensure accessible class (from reference) is chosen (IDEA-144708)

java/java-impl/src/com/intellij/codeInsight/intention/impl/AddSingleMemberStaticImportAction.java

@@ -169,6 +169,9 @@ public class AddSingleMemberStaticImportAction extends BaseElementAtCaretIntenti
         if (qResolved instanceof PsiVariable) {
           aClass = PsiUtil.resolveClassInClassTypeOnly(((PsiVariable)qResolved).getType());
         }
+        else if (qResolved instanceof PsiClass) {
+          aClass = (PsiClass)qResolved;
+        }
       }
     }
     return aClass;

java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/addSingleStaticImport/InaccessibleSuper.java

@@ -0,0 +1,6 @@
+import foo.*;
+class Class2 {
+  public static void main(String[] args) {
+    Bar.f<caret>oo();
+  }
+}

java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/addSingleStaticImport/InaccessibleSuper_after.java

@@ -0,0 +1,9 @@
+import foo.*;
+
+import static foo.Bar.foo;
+
+class Class2 {
+  public static void main(String[] args) {
+    f<caret>oo();
+  }
+}

java/java-tests/testSrc/com/intellij/java/codeInsight/intention/AddSingleStaticImportActionTest.java

@@ -29,6 +29,12 @@ public class AddSingleStaticImportActionTest extends JavaCodeInsightFixtureTestC
     doTest("Add static import for 'impl.FooImpl.foo'");
   }
 
+  public void testInaccessibleSuper() {
+    myFixture.addClass("package foo; class Foo {public static void foo(){}}");
+    myFixture.addClass("package foo; public class Bar extends Foo {}");
+    doTest("Add static import for 'foo.Bar.foo'");
+  }
+
   public void testInsideParameterizedReference() {
     myFixture.addClass("package foo; " +
                        "public class Class1 {" +