Add coursier tests

Author: BarkingBad

.github/workflows/ci.yaml

@@ -111,7 +111,7 @@ jobs:
 
       - name: Cmd Tests
         run: |
-          ./project/scripts/sbt ";scala3-bootstrapped/compile ;scala3-bootstrapped/test;sjsSandbox/run;sjsSandbox/test;sjsJUnitTests/test;sjsCompilerTests/test ;sbt-test/scripted scala2-compat/* ;configureIDE ;stdlib-bootstrapped/test:run ;stdlib-bootstrapped-tasty-tests/test"
+          ./project/scripts/sbt ";scala3-bootstrapped/compile ; scala3-compiler-bootstrapped/publishLocal; scala3-bootstrapped/test;sjsSandbox/run;sjsSandbox/test;sjsJUnitTests/test;sjsCompilerTests/test ;sbt-test/scripted scala2-compat/* ;configureIDE ;stdlib-bootstrapped/test:run ;stdlib-bootstrapped-tasty-tests/test"
           ./project/scripts/bootstrapCmdTests
 
       - name: MiMa

.gitignore

@@ -96,3 +96,6 @@ community-build/dotty-community-build-deps
 
 # Bloop
 .bsp
+
+# Coursier
+cs

compiler/src/MainGenericRunner.scala

@@ -73,15 +73,13 @@ object MainGenericRunner {
       )
     case arg :: tail =>
       val line = Try(Source.fromFile(arg).getLines.toList).toOption.flatMap(_.headOption)
-      val newSettings = if arg.endsWith(".scala") || arg.endsWith(".sc") || (line.nonEmpty && shebangscala.matches(line.get))
-        then {
-          settings
-            .withExecuteMode(ExecuteMode.Script)
-            .withTargetScript(arg)
-            .withScriptArgs(tail*)
-        } else
-          settings.withResidualArgs(arg)
-      process(tail, newSettings.withResidualArgs(arg))
+      if arg.endsWith(".scala") || arg.endsWith(".sc") || (line.nonEmpty && shebangscala.matches(line.get)) then
+        settings
+          .withExecuteMode(ExecuteMode.Script)
+          .withTargetScript(arg)
+          .withScriptArgs(tail*)
+      else
+        process(tail, settings.withResidualArgs(arg))
 
   def main(args: Array[String]): Unit =
     val settings = process(args.toList, Settings())

compiler/test/dotty/tools/coursier/CoursierScalaTests.scala

@@ -0,0 +1,77 @@
+package dotty
+package tools
+package scripting
+
+import java.io.File
+import java.nio.file.{Path, Paths, Files}
+import scala.sys.process._
+
+import org.junit.Test
+import org.junit.BeforeClass
+
+import vulpix.TestConfiguration
+
+import dotty.tools.absPath
+import scala.collection.mutable.ListBuffer
+
+class CoursierScalaTests:
+
+  // classpath tests are managed by scripting.ClasspathTests.scala
+  def testFiles = scripts("/scripting").filter { ! _.getName.startsWith("classpath") }
+
+  // Cannot run tests in parallel, more info here: https://stackoverflow.com/questions/6345660/java-executing-bash-script-error-26-text-file-busy
+  @Test def allTests =
+    def scriptArgs() =
+      val scriptPath = scripts("/scripting").find(_.getName == "showArgs.sc").get.absPath
+      val testScriptArgs = Seq("a", "b", "c", "-repl", "-run", "-script", "-debug")
+
+      val args = scriptPath +: testScriptArgs
+      val output = CoursierScalaTests.csCmd(args*)
+      val expectedOutput = List(
+        "arg  0:[a]",
+        "arg  1:[b]",
+        "arg  2:[c]",
+        "arg  3:[-repl]",
+        "arg  4:[-run]",
+        "arg  5:[-script]",
+        "arg  6:[-debug]",
+      )
+      assert(output == expectedOutput)
+    scriptArgs()
+
+    def version() =
+      val output = CoursierScalaTests.csCmd("-version")
+      assert(output.mkString("\n").contains(sys.env("DOTTY_BOOTSTRAPPED_VERSION")))
+    version()
+
+    def emptyArgsEqualsRepl() =
+      val output = CoursierScalaTests.csCmd()
+      assert(output.mkString("\n").contains("Unable to create a system terminal")) // Scala attempted to create REPL so we can assume it is working
+    emptyArgsEqualsRepl()
+
+    def repl() =
+      val output = CoursierScalaTests.csCmd("-repl")
+      assert(output.mkString("\n").contains("Unable to create a system terminal")) // Scala attempted to create REPL so we can assume it is working
+    repl()
+
+
+object CoursierScalaTests:
+
+  def execCmd(command: String, options: String*): List[String] =
+    val cmd = (command :: options.toList).toSeq.mkString(" ")
+    val out = new ListBuffer[String]
+    cmd.!(ProcessLogger(out += _, out += _))
+    out.toList
+
+
+  def csCmd(options: String*): List[String] =
+    val newOptions = options match
+      case Nil => options
+      case _ => "--" +: options
+    execCmd("./cs", (s"""launch "org.scala-lang:scala3-compiler_3:${sys.env("DOTTY_BOOTSTRAPPED_VERSION")}" --main-class "dotty.tools.MainGenericRunner" --property "scala.usejavacp=true"""" +: newOptions)*)
+
+  /** Get coursier script */
+  @BeforeClass def setup(): Unit =
+    val ver = execCmd("uname").head.replace('L', 'l').replace('D', 'd')
+    execCmd("curl", s"-fLo cs https://git.io/coursier-cli-$ver") #&& execCmd("chmod", "+x cs")
+

compiler/test/dotty/tools/utils.scala

@@ -15,6 +15,12 @@ def scripts(path: String): Array[File] = {
   dir.listFiles
 }
 
+extension (f: File) def absPath =
+  f.getAbsolutePath.replace('\\', '/')
+
+extension (str: String) def dropExtension =
+  str.reverse.dropWhile(_ != '.').drop(1).reverse
+
 private def withFile[T](file: File)(action: Source => T): T =
   resource(Source.fromFile(file, UTF_8.name))(action)
 

project/Build.scala

@@ -190,6 +190,7 @@ object Build {
     // Avoid various sbt craziness involving classloaders and parallelism
     run / fork := true,
     Test / fork := true,
+    Test / envVars := Map("DOTTY_BOOTSTRAPPED_VERSION" -> dottyVersion),
     Test / parallelExecution := false,
 
     outputStrategy := Some(StdoutOutput),