From 4c6df99b6a31ff38d820d6c60daf68f6451e7706 Mon Sep 17 00:00:00 2001 From: Ethan Atkins Date: Thu, 15 Aug 2019 17:06:15 -0700 Subject: [PATCH] Run scripted watch tests with launcher on windows The sbt-coursier plugin now uses a shaded jansi and it seems to cause problems with scripted tests if they aren't run using the launcher. On my vm, all of the watch tests succeeded after this change and they all crashed the jvm before. --- .../src/main/scala/sbt/scriptedtest/ScriptedTests.scala | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/scripted-sbt-redux/src/main/scala/sbt/scriptedtest/ScriptedTests.scala b/scripted-sbt-redux/src/main/scala/sbt/scriptedtest/ScriptedTests.scala index f83f02093..3afdc0c82 100644 --- a/scripted-sbt-redux/src/main/scala/sbt/scriptedtest/ScriptedTests.scala +++ b/scripted-sbt-redux/src/main/scala/sbt/scriptedtest/ScriptedTests.scala @@ -19,7 +19,7 @@ import scala.collection.parallel.ForkJoinTaskSupport import scala.util.control.NonFatal import sbt.internal.scripted._ import sbt.internal.io.Resources -import sbt.internal.util.{ BufferedLogger, ConsoleOut, FullLogger } +import sbt.internal.util.{ BufferedLogger, ConsoleOut, FullLogger, Util } import sbt.io.syntax._ import sbt.io.{ DirectoryFilter, HiddenFileFilter, IO } import sbt.io.FileFilter._ @@ -238,6 +238,8 @@ final class ScriptedTests( case "source-dependencies/linearization" => LauncherBased // sbt/Package$ case "source-dependencies/named" => LauncherBased // sbt/Package$ case "source-dependencies/specialized" => LauncherBased // sbt/Package$ + case gn if gn.startsWith("watch/") && Util.isWindows => + LauncherBased // there is an issue with jansi and coursier case "watch/commands" => LauncherBased // java.lang.ClassNotFoundException: javax.tools.DiagnosticListener when run with java 11 and an old sbt launcher case "watch/managed" => LauncherBased // sbt/Package$