Return exit code 1 when build fails for test

VirtusLab · Nov 3, 2022 · 9c12ca1 · 9c12ca1
1 parent 5793497
commit 9c12ca1
Show file tree

Hide file tree

Showing 3 changed files with 24 additions and 0 deletions.
diff --git a/modules/build/src/main/scala/scala/build/Builds.scala b/modules/build/src/main/scala/scala/build/Builds.scala
@@ -14,6 +14,7 @@ final case class Builds(
     }
   def get(scope: Scope): Option[Build] =
     builds.find(_.scope == scope)
+  def isFail: Boolean = !builds.forall(_.success)
 
   def all: Seq[Build] =
     builds ++ crossBuilds.flatten

diff --git a/modules/cli/src/main/scala/scala/cli/commands/Test.scala b/modules/cli/src/main/scala/scala/cli/commands/Test.scala
@@ -153,6 +153,10 @@ object Test extends ScalaCommand[TestOptions] {
           actionableDiagnostics = actionableDiagnostics
         )
           .orExit(logger)
+      if (builds.isFail) {
+        System.err.println("Compilation failed")
+        sys.exit(1)
+      }
       maybeTest(builds, allowExit = true)
     }
   }

diff --git a/modules/integration/src/test/scala/scala/cli/integration/TestTestDefinitions.scala b/modules/integration/src/test/scala/scala/cli/integration/TestTestDefinitions.scala
@@ -314,6 +314,25 @@ abstract class TestTestDefinitions(val scalaVersionOpt: Option[String])
     }
   }
 
+  test("failing test return code when compiling error") {
+    val inputs = TestInputs(
+      os.rel / "MyTests.scala" ->
+        """//> using lib "org.scalameta::munit::0.7.29"
+          |
+          |class SomeTest extends munit.FunSuite {
+          |  test("failig") {
+          |   val s: String = 1
+          |   assert(true == true)
+          | }
+          |}
+          |""".stripMargin
+    )
+    inputs.fromRoot { root =>
+      val res = os.proc(TestUtil.cli, "test", extraOptions, ".").call(cwd = root, check = false)
+      expect(res.exitCode == 1)
+    }
+  }
+
   test("utest") {
     successfulUtestInputs.fromRoot { root =>
       val output = os.proc(TestUtil.cli, "test", extraOptions, ".").call(cwd = root).out.text()