Skip @JvmDefault in UnimplementedKotlinInterfaceMemberAnnotator

#KT-23967 Fixed

idea/idea-analysis/src/org/jetbrains/kotlin/idea/java/UnimplementedKotlinInterfaceMemberAnnotator.kt

@@ -29,6 +29,7 @@ import com.intellij.psi.*
 import org.jetbrains.kotlin.asJava.classes.KtLightClassForSourceDeclaration
 import org.jetbrains.kotlin.asJava.elements.KtLightMethod
 import org.jetbrains.kotlin.idea.KotlinLanguage
+import org.jetbrains.kotlin.resolve.annotations.JVM_DEFAULT_FQ_NAME
 import org.jetbrains.kotlin.utils.ifEmpty
 
 class UnimplementedKotlinInterfaceMemberAnnotator : Annotator {
@@ -51,10 +52,14 @@ class UnimplementedKotlinInterfaceMemberAnnotator : Annotator {
         }.ifEmpty { return null }
 
         val kotlinSuperClass = generateSequence(psiClass) { it.superClass }.firstOrNull { it is KtLightClassForSourceDeclaration }
-                               ?: return signaturesFromKotlinInterfaces.first().method as? KtLightMethod
 
-        val signaturesVisibleThroughKotlinSuperClass = kotlinSuperClass.visibleSignatures
-        return signaturesFromKotlinInterfaces.firstOrNull { it !in signaturesVisibleThroughKotlinSuperClass }?.method as? KtLightMethod
+        val signaturesVisibleThroughKotlinSuperClass = kotlinSuperClass?.visibleSignatures ?: emptyList()
+        return signaturesFromKotlinInterfaces.firstOrNull {
+            it !in signaturesVisibleThroughKotlinSuperClass &&
+                    it.method.annotations.none { annotation ->
+                        annotation.qualifiedName == JVM_DEFAULT_FQ_NAME.asString()
+                    }
+        }?.method as? KtLightMethod
     }
 
     private fun report(method: KtLightMethod, holder: AnnotationHolder, psiClass: PsiClass) {

idea/testData/kotlinAndJavaChecker/javaAgainstKotlin/ExtendClassWithJvmDefaultImplementation.java

@@ -0,0 +1,11 @@
+package test;
+
+public class ExtendClassWithJvmDefaultImplementation {
+    public static class ExtendClass extends KotlinClass {
+
+    }
+
+    public static class ImplementInterface implements KotlinInterface {
+
+    }
+}

idea/testData/kotlinAndJavaChecker/javaAgainstKotlin/ExtendClassWithJvmDefaultImplementation.kt

@@ -0,0 +1,17 @@
+package test
+
+interface KotlinInterface {
+    @JvmDefault
+    fun bar() {
+
+    }
+
+//    @JvmDefault
+//    val foo: String
+//        get() = "123"
+}
+
+
+abstract class KotlinClass : KotlinInterface {
+
+}

idea/testData/kotlinAndJavaChecker/javaAgainstKotlin/ExtendClassWithJvmDefaultImplementation.txt

@@ -0,0 +1,2 @@
+// LANGUAGE_LEVEL 1.8
+// WITH_RUNTIME

idea/tests/org/jetbrains/kotlin/checkers/JavaAgainstKotlinBinariesCheckerTestGenerated.java

@@ -74,6 +74,11 @@ public class JavaAgainstKotlinBinariesCheckerTestGenerated extends AbstractJavaA
         runTest("idea/testData/kotlinAndJavaChecker/javaAgainstKotlin/ExtendClassWithDefaultImplementation_1_8.kt");
     }
 
+    @TestMetadata("ExtendClassWithJvmDefaultImplementation.kt")
+    public void testExtendClassWithJvmDefaultImplementation() throws Exception {
+        runTest("idea/testData/kotlinAndJavaChecker/javaAgainstKotlin/ExtendClassWithJvmDefaultImplementation.kt");
+    }
+
     @TestMetadata("ExtendingMutableInterfaces.kt")
     public void testExtendingMutableInterfaces() throws Exception {
         runTest("idea/testData/kotlinAndJavaChecker/javaAgainstKotlin/ExtendingMutableInterfaces.kt");

idea/tests/org/jetbrains/kotlin/checkers/JavaAgainstKotlinSourceCheckerTestGenerated.java

@@ -76,6 +76,11 @@ public class JavaAgainstKotlinSourceCheckerTestGenerated extends AbstractJavaAga
             runTest("idea/testData/kotlinAndJavaChecker/javaAgainstKotlin/ExtendClassWithDefaultImplementation_1_8.kt");
         }
 
+        @TestMetadata("ExtendClassWithJvmDefaultImplementation.kt")
+        public void testExtendClassWithJvmDefaultImplementation() throws Exception {
+            runTest("idea/testData/kotlinAndJavaChecker/javaAgainstKotlin/ExtendClassWithJvmDefaultImplementation.kt");
+        }
+
         @TestMetadata("ExtendingMutableInterfaces.kt")
         public void testExtendingMutableInterfaces() throws Exception {
             runTest("idea/testData/kotlinAndJavaChecker/javaAgainstKotlin/ExtendingMutableInterfaces.kt");