fixed tests of expressions-converter for java 11

Author: Jolanrensen

plugins/expressions-converter/build.gradle.kts

@@ -38,6 +38,11 @@ dependencies {
 
 tasks.test {
     useJUnitPlatform()
+    // gets the path to JDK 11 either from gradle.properties or from the system property, defaulting to java.home
+    environment(
+        "JDK_11_0",
+        project.properties["JDK_11_0"] ?: System.getProperty("JDK_11_0", System.getProperty("java.home")),
+    )
     doFirst {
         setLibraryProperty("org.jetbrains.kotlin.test.kotlin-stdlib", "kotlin-stdlib")
         setLibraryProperty("org.jetbrains.kotlin.test.kotlin-reflect", "kotlin-reflect")

plugins/expressions-converter/tests/org/jetbrains/kotlinx/dataframe/AbstractExplainerBlackBoxCodegenTest.kt

@@ -23,16 +23,18 @@ import org.jetbrains.kotlin.test.model.DependencyKind
 import org.jetbrains.kotlin.test.model.FrontendKinds
 import org.jetbrains.kotlin.test.model.TestModule
 import org.jetbrains.kotlin.test.services.RuntimeClasspathProvider
+import org.jetbrains.kotlin.test.services.TemporaryDirectoryManager
 import org.jetbrains.kotlin.test.services.TestServices
 import org.jetbrains.kotlin.test.services.configuration.CommonEnvironmentConfigurator
 import org.jetbrains.kotlin.test.services.configuration.JvmEnvironmentConfigurator
+import org.jetbrains.kotlinx.dataframe.services.TemporaryDirectoryManagerImplFixed
 import java.io.File
 
 open class AbstractExplainerBlackBoxCodegenTest : BaseTestRunner() {
 
     override fun TestConfigurationBuilder.configuration() {
         globalDefaults {
-            frontend = FrontendKinds.ClassicFrontend
+            frontend = FrontendKinds.ClassicAndFIR
             targetPlatform = JvmPlatforms.jvm11
             dependencyKind = DependencyKind.Binary
             targetBackend = TargetBackend.JVM_IR
@@ -65,6 +67,7 @@ open class AbstractExplainerBlackBoxCodegenTest : BaseTestRunner() {
         useConfigurators(::JvmEnvironmentConfigurator, ::CommonEnvironmentConfigurator, ::PluginAnnotationsProvider)
         useCustomRuntimeClasspathProviders(::MyClasspathProvider)
         useAfterAnalysisCheckers(::BlackBoxCodegenSuppressor)
+        useAdditionalService<TemporaryDirectoryManager>(::TemporaryDirectoryManagerImplFixed)
     }
 
     class MyClasspathProvider(testServices: TestServices) : RuntimeClasspathProvider(testServices) {

plugins/expressions-converter/tests/org/jetbrains/kotlinx/dataframe/services/TemporaryDirectoryManagerImplFixed.kt

@@ -0,0 +1,58 @@
+package org.jetbrains.kotlinx.dataframe.services
+
+import org.jetbrains.kotlin.test.services.TemporaryDirectoryManager
+import org.jetbrains.kotlin.test.services.TestServices
+import org.jetbrains.kotlin.test.services.testInfo
+import org.jetbrains.kotlin.test.util.KtTestUtil
+import java.io.File
+import java.nio.file.Paths
+import java.util.Locale
+import kotlin.io.path.ExperimentalPathApi
+import kotlin.io.path.deleteRecursively
+
+// Copied from org.jetbrains.kotlin.test.services.impl.TemporaryDirectoryManagerImpl
+// because it uses NioFiles#deleteRecursively and throws method not found as a result.
+class TemporaryDirectoryManagerImplFixed(testServices: TestServices) : TemporaryDirectoryManager(testServices) {
+    private val cache = mutableMapOf<String, File>()
+    private val rootTempDir = lazy {
+        val testInfo = testServices.testInfo
+        val className = testInfo.className
+        val methodName = testInfo.methodName
+        if (!onWindows && className.length + methodName.length < 255) {
+            return@lazy KtTestUtil.tmpDirForTest(className, methodName)
+        }
+
+        // This code will simplify directory name for windows. This is needed because there can occur errors due to long name
+        val lastDot = className.lastIndexOf('.')
+        val simplifiedClassName = className.substring(lastDot + 1).getOnlyUpperCaseSymbols()
+        val simplifiedMethodName = methodName.getOnlyUpperCaseSymbols()
+        KtTestUtil.tmpDirForTest(simplifiedClassName, simplifiedMethodName)
+    }
+
+    override val rootDir: File
+        get() = rootTempDir.value
+
+    override fun getOrCreateTempDirectory(name: String): File =
+        cache.getOrPut(name) {
+            KtTestUtil.tmpDir(rootDir, name)
+        }
+
+    @OptIn(ExperimentalPathApi::class)
+    override fun cleanupTemporaryDirectories() {
+        cache.clear()
+
+        if (rootTempDir.isInitialized()) {
+            Paths.get(rootDir.path).deleteRecursively()
+        }
+    }
+
+    companion object {
+        private val onWindows: Boolean =
+            System.getProperty("os.name").lowercase(Locale.getDefault()).contains("windows")
+
+        private fun String.getOnlyUpperCaseSymbols(): String =
+            this.filter { it.isUpperCase() || it == '$' }
+                .toList()
+                .joinToString(separator = "")
+    }
+}