Merge pull request #837 from tgodzik/add-stacktrace

improvement: Add stack trace to failed results

Author: tgodzik

modules/core/src/main/scala/ch/epfl/scala/debugadapter/testing/TestSuiteEventHandler.scala

@@ -35,13 +35,14 @@ object TestSuiteEventHandler {
         case Status.Failure =>
           val failedMsg =
             TestUtils.printThrowable(e.throwable()).getOrElse("")
+          val stackTrace = TestUtils.printStackTrace(e.throwable()).getOrElse("")
           val formatted =
             TestSuiteEventHandler.formatError(
               name,
               failedMsg,
               indentSize = 0
             )
-          SingleTestResult.Failed(name, e.duration, formatted)
+          SingleTestResult.Failed(name, e.duration, formatted, stackTrace, location = null)
         case _ =>
           SingleTestResult.Skipped(name)
       }

modules/core/src/main/scala/ch/epfl/scala/debugadapter/testing/TestSuiteSummary.scala

@@ -10,6 +10,11 @@ final case class TestSuiteSummary(
     tests: java.util.List[SingleTestSummary]
 )
 
+final class TestLocation(
+    val file: String,
+    val line: Int
+)
+
 /**
  * Sealed hierarchy that models 3 possible outcomes of single test case.
  * I wanted to model this a discriminated union type and due to gson serialization,
@@ -40,10 +45,18 @@ object SingleTestResult {
       val kind: String,
       val testName: String,
       val duration: Long,
-      val error: String
+      val error: String,
+      val stackTrace: String,
+      val location: TestLocation
   ) extends SingleTestSummary
   object Failed {
-    def apply(testName: String, duration: Long, error: String): Failed =
-      new Failed("failed", testName, duration, error)
+    def apply(
+        testName: String,
+        duration: Long,
+        error: String,
+        stackTrace: String,
+        location: TestLocation
+    ): Failed =
+      new Failed("failed", testName, duration, error, stackTrace, location)
   }
 }

modules/core/src/main/scala/ch/epfl/scala/debugadapter/testing/TestUtils.scala

@@ -1,6 +1,9 @@
 package ch.epfl.scala.debugadapter.testing
 
-import sbt.testing._
+import sbt.testing.*
+
+import java.io.ByteArrayOutputStream
+import java.io.PrintStream
 
 object TestUtils {
   def printSelector(selector: Selector): Option[String] = selector match {
@@ -17,6 +20,18 @@ object TestUtils {
     else Some(stripTestFrameworkSpecificInformation(opt.get().getMessage))
   }
 
+  def printStackTrace(opt: OptionalThrowable): Option[String] = {
+    if (opt.isEmpty) None
+    else
+      Some {
+        val writer = new StringBuffer
+        val outputStream = new ByteArrayOutputStream()
+        val printStream = new PrintStream(outputStream)
+        opt.get().printStackTrace(printStream)
+        outputStream.toString
+      }
+  }
+
   private val specs2Prefix = "java.lang.Exception: "
   private val utestPrefix = "utest.AssertionError: "
   private val scalaTestPrefix = "org.scalatest.exceptions.TestFailedException: "