[Build] Don't pass empty test data roots property to tests

Author: demiurg906

compiler/tests-common-new/testFixtures/org/jetbrains/kotlin/test/services/SystemPropertyTestDataRootConfigurator.kt

@@ -8,7 +8,7 @@ package org.jetbrains.kotlin.test.services
 import org.jetbrains.kotlin.codegen.forTestCompile.TestCompilePaths.KOTLIN_TESTDATA_ROOTS
 
 class SystemPropertyTestDataRootConfigurator(testServices: TestServices) : MetaTestConfigurator(testServices) {
-    private val testDataRoots = System.getProperty(KOTLIN_TESTDATA_ROOTS)?.let { property ->
+    private val testDataRoots: Map<String, String>? = System.getProperty(KOTLIN_TESTDATA_ROOTS)?.let { property ->
         buildMap {
             val roots = property.split(";")
             for (root in roots) {

repo/gradle-build-conventions/project-tests-convention/src/main/kotlin/TestCompilerRuntimeArgumentProvider.kt

@@ -145,7 +145,9 @@ abstract class TestCompilerRuntimeArgumentProvider : CommandLineArgumentProvider
             thirdPartyJava8Annotations.orNull?.let { "-D$KOTLIN_THIRDPARTY_JAVA8_ANNOTATIONS_PATH=${it.asFile.absolutePath}" },
             thirdPartyJava9Annotations.orNull?.let { "-D$KOTLIN_THIRDPARTY_JAVA9_ANNOTATIONS_PATH=${it.asFile.absolutePath}" },
             thirdPartyJsr305.orNull?.let { "-D$KOTLIN_THIRDPARTY_JSR305_PATH=${it.asFile.absolutePath}" },
-            testDataMap.get().map { it.key + "=" + it.value }.joinToString(prefix = "-D$KOTLIN_TESTDATA_ROOTS=", separator = ";"),
+            testDataMap.get().takeIf { it.isNotEmpty() }
+                ?.map { it.key + "=" + it.value }
+                ?.joinToString(prefix = "-D$KOTLIN_TESTDATA_ROOTS=", separator = ";"),
         )
     }
-}
+}