Added test for second press completion for inaccessible

idea/ide-common/src/org/jetbrains/kotlin/idea/codeInsight/ReferenceVariantsHelper.kt

@@ -66,7 +66,6 @@ public class ReferenceVariantsHelper(
         val variants = getReferenceVariantsNoVisibilityFilter(expression, kindFilter, useRuntimeReceiverType, nameFilter)
                 .filter(visibilityFilter)
         return ShadowedDeclarationsFilter(context, moduleDescriptor, project).filter(variants, expression)
-        //TODO: if visibility filter is empty, filter out shadowed can work incorrectly!
     }
 
     private fun getReferenceVariantsNoVisibilityFilter(

idea/idea-completion/testData/basic/multifile/MoreSpecificExtensionIsPrivate/MoreSpecificExtensionIsPrivate.dependency.kt

@@ -0,0 +1,4 @@
+package dependency
+
+fun Any.xxx(): Int = 1
+private fun ppp.C.xxx(): Int = 1

idea/idea-completion/testData/basic/multifile/MoreSpecificExtensionIsPrivate/MoreSpecificExtensionIsPrivate.kt

@@ -0,0 +1,14 @@
+package ppp
+
+import dependency.*
+
+class C
+
+fun foo(c: C) {
+    c.xx<caret>
+}
+
+// we should not include inaccessible extension even on the second completion because the call won't resolve into it anyway
+// INVOCATION_COUNT: 2
+// EXIST: { lookupString: "xxx", itemText: "xxx", tailText: "() for Any in dependency", typeText: "Int" }
+// NOTHING_ELSE

idea/idea-completion/tests/org/jetbrains/kotlin/idea/completion/test/MultiFileJvmBasicCompletionTestGenerated.java

@@ -113,6 +113,12 @@ public class MultiFileJvmBasicCompletionTestGenerated extends AbstractMultiFileJ
         doTest(fileName);
     }
 
+    @TestMetadata("MoreSpecificExtensionIsPrivate")
+    public void testMoreSpecificExtensionIsPrivate() throws Exception {
+        String fileName = JetTestUtils.navigationMetadata("idea/idea-completion/testData/basic/multifile/MoreSpecificExtensionIsPrivate/");
+        doTest(fileName);
+    }
+
     @TestMetadata("NoAutoInsertionOfNotImported")
     public void testNoAutoInsertionOfNotImported() throws Exception {
         String fileName = JetTestUtils.navigationMetadata("idea/idea-completion/testData/basic/multifile/NoAutoInsertionOfNotImported/");