Edit some tests to check the stability of the order (#3943)

Kotlin · Dec 2, 2024 · 9577dc7 · 9577dc7
1 parent ae68eee
commit 9577dc7
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 6 deletions.
diff --git a/...src/test/kotlin/org/jetbrains/dokka/analysis/test/jvm/mixed/SampleMixedJvmAnalysisTest.kt b/...src/test/kotlin/org/jetbrains/dokka/analysis/test/jvm/mixed/SampleMixedJvmAnalysisTest.kt
@@ -72,8 +72,7 @@ class SampleMixedJvmAnalysisTest {
         val secondClasslike = pckg.classlikes[1]
         assertEquals("MyJavaFileInJava", secondClasslike.name)
 
-        // TODO #3250 address unstable order
-        val functions = pckg.functions.sortedBy { it.name }
+        val functions = pckg.functions
         val firstFunction = functions[0]
         assertEquals("bar", firstFunction.name)
 

diff --git a/dokka-subprojects/plugin-base/src/test/kotlin/markdown/LinkTest.kt b/dokka-subprojects/plugin-base/src/test/kotlin/markdown/LinkTest.kt
@@ -776,8 +776,7 @@ class LinkTest : BaseAbstractTest() {
         }
     }
     @Test
-    @OnlyDescriptors("due to #3250 a result DRI is unstable")
-    fun `K1 - link should be stable for overloads in different files`() {
+    fun `link should be stable for overloads in different files`() {
         testInline(
             """
             |/src/main/kotlin/Testing.kt

diff --git a/...n-javadoc/src/test/kotlin/org/jetbrains/dokka/javadoc/JavadocAllClassesTemplateMapTest.kt b/...n-javadoc/src/test/kotlin/org/jetbrains/dokka/javadoc/JavadocAllClassesTemplateMapTest.kt
@@ -8,13 +8,11 @@ import org.jetbrains.dokka.javadoc.pages.AllClassesPage
 import org.jetbrains.dokka.javadoc.pages.LinkJavadocListEntry
 import org.jetbrains.dokka.links.DRI
 import org.jetbrains.dokka.pages.ContentKind
-import org.junit.jupiter.api.Tag
 import kotlin.test.Test
 import kotlin.test.assertEquals
 
 internal class JavadocAllClassesTemplateMapTest : AbstractJavadocTemplateMapTest() {
     @Test
-    @Tag("onlyDescriptors") // https://github.com/Kotlin/dokka/issues/3250
     fun `two classes from different packages`() {
         dualTestTemplateMapInline(
             """