Allow cross compilation tests with Scala 2 compiler

Author: nicolasstucki

compiler/test/dotc/run-scala-js.blacklist

@@ -0,0 +1,2 @@
+# Cross version tests (`..._c2.13.12.scala`) are not supported for Scala 2 with Scala.js.
+i18884

compiler/test/dotty/tools/TestSources.scala

@@ -27,12 +27,14 @@ object TestSources {
 
   def runFromTastyBlacklistFile: String = "compiler/test/dotc/run-from-tasty.blacklist"
   def runTestPicklingBlacklistFile: String = "compiler/test/dotc/run-test-pickling.blacklist"
+  def runScalaJSBlacklistFile: String = "compiler/test/dotc/run-scala-js.blacklist"
   def runTestRecheckExcludesFile: String = "compiler/test/dotc/run-test-recheck.excludes"
   def runLazyValsAllowlistFile: String = "compiler/test/dotc/run-lazy-vals-tests.allowlist"
 
 
   def runFromTastyBlacklisted: List[String] = loadList(runFromTastyBlacklistFile)
   def runTestPicklingBlacklisted: List[String] = loadList(runTestPicklingBlacklistFile)
+  def runScalaJSBlacklistFileBlacklisted: List[String] = loadList(runScalaJSBlacklistFile)
   def runTestRecheckExcluded: List[String] = loadList(runTestRecheckExcludesFile)
   def runLazyValsAllowlist: List[String] = loadList(runLazyValsAllowlistFile)
 

compiler/test/dotty/tools/vulpix/ParallelTesting.scala

@@ -610,17 +610,38 @@ trait ParallelTesting extends RunnerOrchestration { self =>
           .run()
           .mkString(JFile.pathSeparator)
 
-      val stdlibClasspath = artifactClasspath("org.scala-lang", "scala3-library_3")
-      val scalacClasspath = artifactClasspath("org.scala-lang", "scala3-compiler_3")
-
       val pageWidth = TestConfiguration.pageWidth - 20
-      val flags1 = flags.copy(defaultClassPath = stdlibClasspath)
-        .withClasspath(targetDir.getPath)
-        .and("-d", targetDir.getPath)
-        .and("-pagewidth", pageWidth.toString)
 
-      val scalacCommand = Array("java", "-cp", scalacClasspath, "dotty.tools.dotc.Main")
-      val command = scalacCommand ++ flags1.all ++ files.map(_.getAbsolutePath)
+      val fileArgs = files.map(_.getAbsolutePath)
+
+      def scala2Command(): Array[String] = {
+        assert(!flags.options.contains("-scalajs"),
+          "Compilation tests with Scala.js on Scala 2 are not supported.\nThis test can be blacklisted in compiler/test/dotc/*-scala-js.blacklist")
+        val stdlibClasspath = artifactClasspath("org.scala-lang", "scala-library")
+        val scalacClasspath = artifactClasspath("org.scala-lang", "scala-compiler")
+        val flagsArgs = flags
+          .copy(options = Array.empty, defaultClassPath = stdlibClasspath)
+          .withClasspath(targetDir.getPath)
+          .and("-d", targetDir.getPath)
+          .all
+        val scalacCommand = Array("java", "-cp", scalacClasspath, "scala.tools.nsc.Main")
+        scalacCommand ++ flagsArgs ++ fileArgs
+      }
+
+      def scala3Command(): Array[String] = {
+        val stdlibClasspath = artifactClasspath("org.scala-lang", "scala3-library_3")
+        val scalacClasspath = artifactClasspath("org.scala-lang", "scala3-compiler_3")
+        val flagsArgs = flags
+          .copy(defaultClassPath = stdlibClasspath)
+          .withClasspath(targetDir.getPath)
+          .and("-d", targetDir.getPath)
+          .and("-pagewidth", pageWidth.toString)
+          .all
+        val scalacCommand = Array("java", "-cp", scalacClasspath, "dotty.tools.dotc.Main")
+        scalacCommand ++ flagsArgs ++ fileArgs
+      }
+
+      val command = if compiler.startsWith("2") then scala2Command() else scala3Command()
       val process = Runtime.getRuntime.exec(command)
 
       val reporter = mkReporter

sjs-compiler-tests/test/scala/dotty/tools/dotc/ScalaJSCompilationTests.scala

@@ -59,7 +59,7 @@ class ScalaJSCompilationTests extends ParallelTesting {
   @Test def runScalaJS: Unit = {
     implicit val testGroup: TestGroup = TestGroup("runScalaJS")
     aggregateTests(
-      compileFilesInDir("tests/run", scalaJSOptions),
+      compileFilesInDir("tests/run", scalaJSOptions, FileFilter.exclude(TestSources.runScalaJSBlacklistFileBlacklisted)),
     ).checkRuns()
   }
 }