Fix trailing SBT args and add tests

sourcegraph · Sep 25, 2023 · e520f37 · e520f37
1 parent 2044f6e
commit e520f37
Show file tree

Hide file tree

Showing 2 changed files with 31 additions and 4 deletions.
diff --git a/scip-java/src/main/scala/com/sourcegraph/scip_java/buildtools/SbtBuildTool.scala b/scip-java/src/main/scala/com/sourcegraph/scip_java/buildtools/SbtBuildTool.scala
@@ -43,8 +43,14 @@ class SbtBuildTool(index: IndexCommand) extends BuildTool("sbt", index) {
 
   private def unconditionallyGenerateScip(): Int =
     Using.resource(sourcegraphSbtPluginFile()) { _ =>
+      val buildCommand =
+        if (index.buildCommand.isEmpty)
+          List("sourcegraphScip")
+        else
+          index.buildCommand
+
       val sourcegraphScip = index
-        .process(List("sbt", "sourcegraphEnable", "sourcegraphScip"))
+        .process(List("sbt", "sourcegraphEnable") ++ buildCommand)
       val inputDump = index
         .workingDirectory
         .resolve("target")

diff --git a/tests/buildTools/src/test/scala/tests/SbtBuildToolSuite.scala b/tests/buildTools/src/test/scala/tests/SbtBuildToolSuite.scala
@@ -1,6 +1,6 @@
 package tests
 
-abstract class SbtBuildToolSuite(sbtVersion: String)
+abstract class SbtBuildToolSuite(val sbtVersion: String)
     extends BaseBuildToolSuite {
   List("2.11.9", "2.12.18", "2.13.11", "3.3.0").foreach { scalaVersion =>
     checkBuild(
@@ -9,7 +9,7 @@ abstract class SbtBuildToolSuite(sbtVersion: String)
           |scalaVersion := "$scalaVersion"
           |libraryDependencies += "junit" % "junit" % "4.13.2"
           |/project/build.properties
-          |sbt.version=1.5.2
+          |sbt.version=$sbtVersion
           |/src/main/java/example/ExampleJava.java
           |package example;
           |import org.junit.Assert;
@@ -29,8 +29,29 @@ abstract class SbtBuildToolSuite(sbtVersion: String)
       targetRoot = Some("target")
     )
   }
+}
 
+class Sbt_1_BuildToolSuite extends SbtBuildToolSuite("1.5.2") {
+  checkBuild(
+    s"custom-sbt-command=$sbtVersion",
+    s"""|/build.sbt
+        |lazy val bla = project.in(file("bla"))
+        |lazy val blaJS = project.in(file("bla-js")).enablePlugins(ScalaJSPlugin)
+        |/project/plugins.sbt
+        |addSbtPlugin("org.scala-js" % "sbt-scalajs" % "1.14.0")
+        |/project/build.properties
+        |sbt.version=$sbtVersion
+        |/bla/src/main/scala/example/ExampleScala.scala
+        |package example
+        |class ExampleScala()
+        |/bla-js/src/main/scala/example/ExampleScala.scala
+        |package example
+        |class ExampleScala!!!() // this file is intentionally broken
+        |""".stripMargin,
+    expectedSemanticdbFiles = 1,
+    extraArguments = List("--", "bla/compile"),
+    targetRoot = Some("bla/target")
+  )
 }
 
-class Sbt_1_BuildToolSuite extends SbtBuildToolSuite("1.5.2")
 class Sbt_013_BuildToolSuite extends SbtBuildToolSuite("0.13.17")