diff options
author | Li Haoyi <haoyi.sg@gmail.com> | 2018-02-04 10:29:29 -0800 |
---|---|---|
committer | Li Haoyi <haoyi.sg@gmail.com> | 2018-02-04 10:29:29 -0800 |
commit | e34b2cd25ced6dfef6dbcbcadcfed978494f2630 (patch) | |
tree | 5bfa5560d874d507d2a27e921d5489f1f39e09c0 /scalalib/src | |
parent | ce0a5c2869da538b79d6adb9e28dd46741953889 (diff) | |
download | mill-e34b2cd25ced6dfef6dbcbcadcfed978494f2630.tar.gz mill-e34b2cd25ced6dfef6dbcbcadcfed978494f2630.tar.bz2 mill-e34b2cd25ced6dfef6dbcbcadcfed978494f2630.zip |
Fix ScalaWorker test runs to make them properly break the build
Diffstat (limited to 'scalalib/src')
-rw-r--r-- | scalalib/src/mill/scalalib/ScalaModule.scala | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/scalalib/src/mill/scalalib/ScalaModule.scala b/scalalib/src/mill/scalalib/ScalaModule.scala index a5637220..23d7345b 100644 --- a/scalalib/src/mill/scalalib/ScalaModule.scala +++ b/scalalib/src/mill/scalalib/ScalaModule.scala @@ -261,9 +261,12 @@ trait ScalaModule extends mill.Module with TaskModule { outer => object TestModule{ def handleResults(doneMsg: String, results: Seq[TestRunner.Result]) = { - if (results.count(Set("Error", "Failure")) == 0) Result.Success((doneMsg, results)) - else { - val grouped = results.map(_.status).groupBy(x => x).mapValues(_.length).filter(_._2 != 0).toList.sorted + + val badStatuses = results.map(_.status).filter(Set("Error", "Failure")) + if (badStatuses.isEmpty) { + Result.Success((doneMsg, results)) + } else { + val grouped = badStatuses.groupBy(x => x).mapValues(_.length).toList.sorted Result.Failure(grouped.map{case (k, v) => k + ": " + v}.mkString(",")) } |