Revert "Allow tests to be run in parallel."

This reverts commit 21d81f35

compiler/tests-common/tests/org/jetbrains/kotlin/checkers/AbstractForeignAnnotationsNoAnnotationInClasspathTest.kt

@@ -22,11 +22,7 @@ import org.jetbrains.kotlin.test.KotlinTestUtils
 import java.io.File
 
 abstract class AbstractForeignAnnotationsNoAnnotationInClasspathTest : AbstractForeignAnnotationsTest() {
-    // This should be executed after setUp runs, since setUp changes the root folder
-    // for temporary files.
-    private val compiledJavaPath by lazy {
-        KotlinTestUtils.tmpDir("java-compiled-files")
-    }
+    private val compiledJavaPath = KotlinTestUtils.tmpDir("java-compiled-files")
 
     override fun getExtraClasspath(): List<File> {
         val foreignAnnotations = createJarWithForeignAnnotations()

compiler/tests-common/tests/org/jetbrains/kotlin/checkers/KotlinMultiFileTestWithJava.java

@@ -43,6 +43,13 @@ public abstract class KotlinMultiFileTestWithJava<M, F> extends KtUsefulTestCase
         coroutinesPackage = "";
     }
 
+    @Override
+    protected void tearDown() throws Exception {
+        if (javaFilesDir != null) FileUtil.delete(javaFilesDir);
+        if (kotlinSourceRoot != null) FileUtil.delete(kotlinSourceRoot);
+        super.tearDown();
+    }
+
     public class ModuleAndDependencies {
         final M module;
         final List<String> dependencies;

compiler/tests-common/tests/org/jetbrains/kotlin/test/KotlinTestUtils.java

@@ -413,7 +413,9 @@ public class KotlinTestUtils {
 
     @NotNull
     public static File tmpDirForTest(@NotNull String testClassName, @NotNull String testName) throws IOException {
-        return normalizeFile(FileUtil.createTempDirectory(testClassName, testName, false));
+        File answer = normalizeFile(FileUtil.createTempDirectory(testClassName, testName));
+        deleteOnShutdown(answer);
+        return answer;
     }
 
     @NotNull
@@ -423,7 +425,10 @@ public class KotlinTestUtils {
 
     @NotNull
     public static File tmpDir(String name) throws IOException {
-        return normalizeFile(FileUtil.createTempDirectory(name, "", false));
+        // We should use this form. otherwise directory will be deleted on each test.
+        File answer = normalizeFile(FileUtil.createTempDirectory(new File(System.getProperty("java.io.tmpdir")), name, ""));
+        deleteOnShutdown(answer);
+        return answer;
     }
 
     private static File normalizeFile(File file) throws IOException {
@@ -433,6 +438,18 @@ public class KotlinTestUtils {
         return file.getCanonicalFile();
     }
 
+    private static void deleteOnShutdown(File file) {
+        if (filesToDelete.isEmpty()) {
+            ShutDownTracker.getInstance().registerShutdownTask(() -> {
+                for (File victim : filesToDelete) {
+                    FileUtil.delete(victim);
+                }
+            });
+        }
+
+        filesToDelete.add(file);
+    }
+
     @NotNull
     public static KtFile createFile(@NotNull @NonNls String name, @NotNull String text, @NotNull Project project) {
         String shortName = name.substring(name.lastIndexOf('/') + 1);

compiler/tests-common/tests/org/jetbrains/kotlin/test/testFramework/KtUsefulTestCase.java

@@ -76,7 +76,7 @@ public abstract class KtUsefulTestCase extends TestCase {
     private static final String ourPathToKeep = null;
     private final List<String> myPathsToKeep = new ArrayList<>();
 
-    private File myTempDir;
+    private String myTempDir;
 
     static {
         // Radar #5755208: Command line Java applications need a way to launch without a Dock icon.
@@ -100,8 +100,8 @@ public abstract class KtUsefulTestCase extends TestCase {
         String testName =  FileUtil.sanitizeFileName(getTestName(true));
         if (StringUtil.isEmptyOrSpaces(testName)) testName = "";
         testName = new File(testName).getName(); // in case the test name contains file separators
-        myTempDir = FileUtil.createTempDirectory(TEMP_DIR_MARKER, testName, false);
-        FileUtil.resetCanonicalTempPathCache(myTempDir.getPath());
+        myTempDir = new File(ORIGINAL_TEMP_DIR, TEMP_DIR_MARKER + testName).getPath();
+        FileUtil.resetCanonicalTempPathCache(myTempDir);
         boolean isStressTest = isStressTest();
         ApplicationInfoImpl.setInStressTest(isStressTest);
         // turn off Disposer debugging for performance tests
@@ -119,7 +119,7 @@ public abstract class KtUsefulTestCase extends TestCase {
             Disposer.setDebugMode(oldDisposerDebug);
             FileUtil.resetCanonicalTempPathCache(ORIGINAL_TEMP_DIR);
             if (hasTmpFilesToKeep()) {
-                File[] files = myTempDir.listFiles();
+                File[] files = new File(myTempDir).listFiles();
                 if (files != null) {
                     for (File file : files) {
                         if (!shouldKeepTmpFile(file)) {
@@ -127,8 +127,9 @@ public abstract class KtUsefulTestCase extends TestCase {
                         }
                     }
                 }
-            } else {
-                FileUtil.delete(myTempDir);
+            }
+            else {
+                FileUtil.delete(new File(myTempDir));
             }
         }