Better logging in tests

Author: alexarchambault

frontend/src/test/scala/bloop/util/TestUtil.scala

@@ -207,12 +207,15 @@ object TestUtil {
   def getBloopConfigDir(buildName: String): Path = {
     val baseDirURL = ThisClassLoader.getResource(buildName)
     if (baseDirURL == null) {
-      sys.error(s"Project ${buildName} does not exist in test resources")
+      sys.error(s"Project ${buildName} does not exist in test resources (baseDirURL=$baseDirURL)")
     } else {
       val baseDir = java.nio.file.Paths.get(baseDirURL.toURI)
       val bloopConfigDir = baseDir.resolve("bloop-config")
       if (Files.exists(bloopConfigDir)) bloopConfigDir
-      else sys.error(s"Project ${buildName} does not exist in test resources")
+      else
+        sys.error(
+          s"Bloop config dir for project ${buildName} does not exist in test resources (bloopConfigDir=$bloopConfigDir)"
+        )
     }
   }
 
@@ -595,14 +598,18 @@ object TestUtil {
       .notification(endpoints.Build.showMessage) {
         case bsp.ShowMessageParams(bsp.MessageType.Log, _, o, msg) => logger.debug(fmt(msg, o))
         case bsp.ShowMessageParams(bsp.MessageType.Info, _, o, msg) => logger.info(fmt(msg, o))
-        case bsp.ShowMessageParams(bsp.MessageType.Warning, _, o, msg) => logger.warn(fmt(msg, o))
-        case bsp.ShowMessageParams(bsp.MessageType.Error, _, o, msg) => logger.error(fmt(msg, o))
+        case bsp.ShowMessageParams(bsp.MessageType.Warning, _, o, msg) =>
+          System.err.println(fmt(msg, o))
+        case bsp.ShowMessageParams(bsp.MessageType.Error, _, o, msg) =>
+          System.err.println(fmt(msg, o))
       }
       .notification(endpoints.Build.logMessage) {
         case bsp.LogMessageParams(bsp.MessageType.Log, _, o, msg) => logger.debug(fmt(msg, o))
         case bsp.LogMessageParams(bsp.MessageType.Info, _, o, msg) => logger.info(fmt(msg, o))
-        case bsp.LogMessageParams(bsp.MessageType.Warning, _, o, msg) => logger.warn(fmt(msg, o))
-        case bsp.LogMessageParams(bsp.MessageType.Error, _, o, msg) => logger.error(fmt(msg, o))
+        case bsp.LogMessageParams(bsp.MessageType.Warning, _, o, msg) =>
+          System.err.println(fmt(msg, o))
+        case bsp.LogMessageParams(bsp.MessageType.Error, _, o, msg) =>
+          System.err.println(fmt(msg, o))
       }
 
     // Lsp4s fails if we try to repeat a handler for a given notification