Deleted old compat tests, made api dumps unversioned

Author: Mr3zee

krpc/krpc-test/src/jvmTest/kotlin/kotlinx/rpc/krpc/test/api/ApiTestContext.kt

@@ -1,11 +1,10 @@
 /*
- * Copyright 2023-2024 JetBrains s.r.o and contributors. Use of this source code is governed by the Apache 2.0 license.
+ * Copyright 2023-2025 JetBrains s.r.o and contributors. Use of this source code is governed by the Apache 2.0 license.
  */
 
 package kotlinx.rpc.krpc.test.api
 
-import kotlinx.rpc.krpc.test.api.ApiVersioningTest.Companion.CURRENT_CLASS_DUMPS_DIR
-import kotlinx.rpc.krpc.test.api.ApiVersioningTest.Companion.LATEST_CLASS_DUMPS_DIR
+import kotlinx.rpc.krpc.test.api.ApiVersioningTest.Companion.CLASS_DUMPS_DIR
 import kotlinx.rpc.krpc.test.api.util.StringGoldContent
 import kotlinx.rpc.krpc.test.api.util.checkGold
 import kotlin.reflect.KClass
@@ -27,8 +26,8 @@ class ApiTestContext {
         sampled.add(clazz)
 
         val log = checkGold(
-            latestDir = LATEST_CLASS_DUMPS_DIR,
-            currentDir = CURRENT_CLASS_DUMPS_DIR,
+            latestDir = CLASS_DUMPS_DIR,
+            currentDir = CLASS_DUMPS_DIR,
             filename = clazz.simpleName!!,
             content = StringGoldContent(currentContent),
             parseGoldFile = { StringGoldContent(it) },

krpc/krpc-test/src/jvmTest/kotlin/kotlinx/rpc/krpc/test/api/ApiVersioningTest.kt

@@ -1,25 +1,17 @@
 /*
- * Copyright 2023-2024 JetBrains s.r.o and contributors. Use of this source code is governed by the Apache 2.0 license.
+ * Copyright 2023-2025 JetBrains s.r.o and contributors. Use of this source code is governed by the Apache 2.0 license.
  */
 
 package kotlinx.rpc.krpc.test.api
 
-import kotlinx.coroutines.flow.toList
 import kotlinx.rpc.krpc.internal.CancellationType
 import kotlinx.rpc.krpc.internal.KrpcMessage
 import kotlinx.rpc.krpc.internal.KrpcPlugin
 import kotlinx.rpc.krpc.internal.KrpcPluginKey
 import kotlinx.rpc.krpc.test.api.util.GoldUtils.NewLine
-import kotlinx.rpc.krpc.test.plainFlow
-import org.jetbrains.krpc.test.api.util.SamplingData
 import org.junit.Test
 import java.nio.file.Path
 import kotlin.io.path.Path
-import kotlin.io.path.isDirectory
-import kotlin.io.path.listDirectoryEntries
-import kotlin.io.path.name
-import kotlin.test.Ignore
-import kotlin.test.assertEquals
 import kotlin.test.fail
 
 class ApiVersioningTest {
@@ -45,82 +37,10 @@ class ApiVersioningTest {
         testEnum<CancellationType>()
     }
 
-    @Test
-    fun testEchoSampling() = wireSamplingTest("echo") {
-        sample {
-            val response = echo("Hello", SamplingData("data"))
-            assertEquals(SamplingData("data"), response)
-        }
-    }
-
-    @Test
-    @Ignore("Flow sampling tests are too unstable. Ignored until better fix")
-    fun testClientStreamSampling() = wireSamplingTest("clientStream") {
-        sample {
-            val response = clientStream(plainFlow { it }).joinToString()
-            val expected = List(5) { it }.joinToString()
-
-            assertEquals(expected, response)
-        }
-    }
-
-    @Test
-    @Ignore("Flow sampling tests are too unstable. Ignored until better fix")
-    fun testServerStreamSampling() = wireSamplingTest("serverStream") {
-        sample {
-            val response = serverFlow().toList().joinToString()
-            val expected = List(5) { SamplingData("data") }.joinToString()
-
-            assertEquals(expected, response)
-        }
-    }
-
-    @Test
-    fun testCallExceptionSampling() = wireSamplingTest("callException") {
-        // ignore protobuf here, as it's hard to properly sample stacktrace
-        // in Json we can just cut it out
-        sample(SamplingFormat.Json) {
-            try {
-                callException()
-                fail("Expected exception to be thrown")
-            } catch (e: IllegalStateException) {
-                assertEquals("Server exception", e.message)
-            }
-        }
-    }
-
     companion object {
-        val LIBRARY_VERSION_DIR = System.getenv("LIBRARY_VERSION")?.versionToDirName()
-            ?: error("Expected LIBRARY_VERSION env variable")
+        val CLASS_DUMPS_DIR: Path = Path("src/jvmTest/resources/class_dumps/")
 
-        val CURRENT_CLASS_DUMPS_DIR: Path = Path("src/jvmTest/resources/class_dumps/")
-            .resolve(LIBRARY_VERSION_DIR)
-
-        val LATEST_CLASS_DUMPS_DIR: Path = Path("src/jvmTest/resources/class_dumps/")
-            .latestVersionOrCurrent()
-
-        val WIRE_DUMPS_DIR: Path = Path("src/jvmTest/resources/wire_dumps/")
         val INDEXED_ENUM_DUMPS_DIR: Path = Path("src/jvmTest/resources/indexed_enum_dumps/")
-
-        private fun String.versionToDirName(): String {
-            return replace('.', '_').replace('-', '_').substringBefore("-")
-        }
-
-        fun Path.latestVersionOrCurrent(): Path {
-            return listDirectoryEntries()
-                .filter { it.isDirectory() }
-                .sortedWith { a, b ->
-                    val aBeta = a.name.contains("beta")
-                    val bBeta = b.name.contains("beta")
-                    when {
-                        aBeta && bBeta -> a.compareTo(b)
-                        aBeta -> -1
-                        bBeta -> 1
-                        else -> a.name.substringBefore("-").compareTo(b.name.substringBefore("-"))
-                    }
-                }.lastOrNull()
-                ?: resolve(LIBRARY_VERSION_DIR)
-        }
     }
 }