[psi] move PsiKDocCoverageTest to api module

^KT-78613

Author: dimonchik0036

.github/CODEOWNERS

@@ -159,7 +159,6 @@
 /compiler/testData/parseCodeFragment/ @JetBrains/kotlin-frontend
 /compiler/testData/psi/ @JetBrains/kotlin-frontend
 /compiler/testData/psiUtil/ @JetBrains/kotlin-frontend
-/compiler/testData/psiKdocCoverage/ @JetBrains/kotlin-analysis-api
 /compiler/testData/recursiveProcessor/ @JetBrains/kotlin-frontend
 /compiler/testData/reflection/ @JetBrains/kotlin-libraries [email protected]
 /compiler/testData/repl/ @JetBrains/kotlin-frontend
@@ -178,8 +177,6 @@
 /compiler/testData/type-checker-test.kt @JetBrains/kotlin-frontend
 /compiler/testData/type-substitutor.kt @JetBrains/kotlin-frontend
 /compiler/tests/ @JetBrains/kotlin-frontend @JetBrains/kotlin-jvm @JetBrains/kotlin-native @JetBrains/kotlin-wasm @JetBrains/kotlin-common-backend @JetBrains/kotlin-js
-/compiler/tests/org/jetbrains/kotlin/psi/PsiKDocCoverageTest.kt @JetBrains/kotlin-analysis-api
-/compiler/tests/org/jetbrains/kotlin/util/KDocCoverageTest.kt @JetBrains/kotlin-analysis-api
 /compiler/tests-against-klib/ @JetBrains/kotlin-jvm
 /compiler/tests-common/ @JetBrains/kotlin-frontend @JetBrains/kotlin-jvm @JetBrains/kotlin-native @JetBrains/kotlin-wasm @JetBrains/kotlin-common-backend @JetBrains/kotlin-js
 /compiler/tests-common/tests/org/jetbrains/kotlin/asJava/ @JetBrains/kotlin-analysis-api

.idea/runConfigurations/Check_PSI_KDoc_Coverage.xml

@@ -193,7 +193,6 @@
 /compiler/testData/parseCodeFragment/ "Kotlin Frontend"
 /compiler/testData/psi/ "Kotlin Frontend"
 /compiler/testData/psiUtil/ "Kotlin Frontend"
-/compiler/testData/psiKdocCoverage/ "Kotlin Analysis API"
 /compiler/testData/recursiveProcessor/ "Kotlin Frontend"
 /compiler/testData/reflection/ "Kotlin Libraries" [email protected]
 /compiler/testData/repl/ "Kotlin Frontend"
@@ -212,8 +211,6 @@
 /compiler/testData/type-checker-test.kt "Kotlin Frontend"
 /compiler/testData/type-substitutor.kt "Kotlin Frontend"
 /compiler/tests/ "Kotlin Frontend" "Kotlin JVM" "Kotlin Native" "Kotlin Wasm" "Kotlin Common Backend" "Kotlin JS"
-/compiler/tests/org/jetbrains/kotlin/psi/PsiKDocCoverageTest.kt "Kotlin Analysis API"
-/compiler/tests/org/jetbrains/kotlin/util/KDocCoverageTest.kt "Kotlin Analysis API"
 /compiler/tests-against-klib/ "Kotlin JVM"
 /compiler/tests-common/ "Kotlin Frontend" "Kotlin JVM" "Kotlin Native" "Kotlin Wasm" "Kotlin Common Backend" "Kotlin JS"
 /compiler/tests-common/tests/org/jetbrains/kotlin/asJava/ "Kotlin Analysis API"

.space/CODEOWNERS

@@ -28,6 +28,7 @@ dependencies {
     testApi(platform(libs.junit.bom))
     testImplementation(libs.junit.jupiter.api)
     testRuntimeOnly(libs.junit.jupiter.engine)
+
     testImplementation(projectTests(":compiler:psi:psi-api"))
     testImplementation(projectTests(":compiler"))
 }

analysis/analysis-api/build.gradle.kts

@@ -5,18 +5,18 @@
 
 package org.jetbrains.kotlin.analysis.api.test
 
-import org.jetbrains.kotlin.util.KDocCoverageTest
+import org.jetbrains.kotlin.AbstractKDocCoverageTest
 import org.junit.jupiter.api.Test
 
-class AnalysisApiKDocCoverageTest : KDocCoverageTest() {
+class AnalysisApiKDocCoverageTest : AbstractKDocCoverageTest() {
     override val sourceDirectories: List<DocumentationLocations> = listOf(
         DocumentationLocations(
-            listOf("/analysis/analysis-api/src/org/jetbrains/kotlin/analysis/api"),
-            "/analysis/analysis-api/api/analysis-api.undocumented"
+            listOf("analysis/analysis-api/src/org/jetbrains/kotlin/analysis/api"),
+            "analysis/analysis-api/api/analysis-api.undocumented",
         ),
         DocumentationLocations(
-            listOf("/analysis/analysis-api-platform-interface/src/org/jetbrains/kotlin/analysis/api/platform"),
-            "/analysis/analysis-api/api/analysis-api-platform-interface.undocumented"
+            listOf("analysis/analysis-api-platform-interface/src/org/jetbrains/kotlin/analysis/api/platform"),
+            "analysis/analysis-api/api/analysis-api-platform-interface.undocumented",
         ),
     )
 

analysis/analysis-api/tests/org/jetbrains/kotlin/analysis/api/test/AnalysisApiKDocCoverageTest.kt

@@ -17,6 +17,8 @@ dependencies {
     testApi(platform(libs.junit.bom))
     testImplementation(libs.junit.jupiter.api)
     testRuntimeOnly(libs.junit.jupiter.engine)
+
+    testImplementation(projectTests(":compiler"))
 }
 
 sourceSets {

compiler/testData/psiKdocCoverage/psi.undocumented renamed to compiler/psi/psi-api/api/psi.undocumented

@@ -3,7 +3,7 @@
  * Use of this source code is governed by the Apache 2.0 license that can be found in the license/LICENSE.txt file.
  */
 
-package org.jetbrains.kotlin.util
+package org.jetbrains.kotlin
 
 import com.intellij.lang.jvm.JvmModifier
 import com.intellij.openapi.vfs.StandardFileSystems
@@ -40,7 +40,8 @@ import java.io.File
  * If the lack of documentation for some declaration is intentional,
  * the developer has to manually add this declaration to the master file.
  */
-abstract class KDocCoverageTest : KtUsefulTestCase() {
+abstract class AbstractKDocCoverageTest : KtUsefulTestCase() {
+    @OptIn(K1Deprecation::class)
     protected fun doTest() {
         val environment = KotlinCoreEnvironment.createForParallelTests(
             testRootDisposable,
@@ -52,7 +53,7 @@ abstract class KDocCoverageTest : KtUsefulTestCase() {
         val homeDir = KtTestUtil.getHomeDirectory()
 
         sourceDirectories.forEach { (sourceCodeDirectoryPaths, outputFilePath) ->
-            val roots = sourceCodeDirectoryPaths.map { File(homeDir + it) }
+            val roots = sourceCodeDirectoryPaths.map { File(homeDir, it) }
 
             val actualText = buildList {
                 for (root in roots) {
@@ -79,7 +80,7 @@ abstract class KDocCoverageTest : KtUsefulTestCase() {
                 }
             }.sorted().joinToString("\n")
 
-            val expectedFile = File(homeDir + outputFilePath)
+            val expectedFile = File(homeDir, outputFilePath)
             KotlinTestUtils.assertEqualsToFile(
                 "Some newer public declarations from `$roots` are undocumented. Please, consider either documenting them or adding them to `$outputFilePath`",
                 expectedFile,

compiler/psi/psi-api/build.gradle.kts

@@ -5,19 +5,21 @@
 
 package org.jetbrains.kotlin.psi
 
+import org.jetbrains.kotlin.AbstractKDocCoverageTest
 import org.jetbrains.kotlin.name.FqName
-import org.jetbrains.kotlin.util.KDocCoverageTest
+import org.junit.jupiter.api.Test
 
-class PsiKDocCoverageTest() : KDocCoverageTest() {
+class PsiKDocCoverageTest : AbstractKDocCoverageTest() {
     override val sourceDirectories: List<DocumentationLocations> = listOf(
         DocumentationLocations(
             listOf(
-                "/compiler/psi/psi-api/src/org/jetbrains/kotlin",
-                "/compiler/psi/psi-impl/src/org/jetbrains/kotlin",
-                "/compiler/psi/psi-utils/src/org/jetbrains/kotlin",
-                "/compiler/psi/psi-frontend-utils/src/org/jetbrains/kotlin",
+                "compiler/psi/psi-api/src/org/jetbrains/kotlin",
+                "compiler/psi/psi-impl/src/org/jetbrains/kotlin",
+                "compiler/psi/psi-utils/src/org/jetbrains/kotlin",
+                "compiler/psi/psi-frontend-utils/src/org/jetbrains/kotlin",
             ),
-            "/compiler/testData/psiKdocCoverage/psi.undocumented")
+            "compiler/psi/psi-api/api/psi.undocumented",
+        )
     )
 
     override val ignoredPackages: List<FqName> = listOf(
@@ -28,7 +30,8 @@ class PsiKDocCoverageTest() : KDocCoverageTest() {
         FqName("org.jetbrains.kotlin.kdoc.parser"),
     )
 
+    @Test
     fun testKDocCoverage() {
         doTest()
     }
-}
+}