fix #628

remove -indent and -rewrite options when evaluating expression
scalacenter · May 1, 2024 · 4cc7019 · 4cc7019
1 parent b6bdfe6
commit 4cc7019
Show file tree

Hide file tree

Showing 3 changed files with 26 additions and 3 deletions.
diff --git a/modules/core/src/main/scala/ch/epfl/scala/debugadapter/internal/DebugTools.scala b/modules/core/src/main/scala/ch/epfl/scala/debugadapter/internal/DebugTools.scala
@@ -120,7 +120,7 @@ object DebugTools {
     debuggee.managedEntries.flatMap(loadExpressionCompiler).toMap
   }
 
-  private val optionsToRemove = Set("-Xfatal-warnings", "-Werror")
+  private val optionsToRemove = Set("-Xfatal-warnings", "-Werror", "-indent", "-rewrite")
 
   private def prepareOptions(options: Seq[String], toAdd: Seq[String]) = {
     val withoutRemoved = options.filter(o => !optionsToRemove.contains(o))

diff --git a/modules/tests/src/main/scala/ch/epfl/scala/debugadapter/testfmk/DebugTestSuite.scala b/modules/tests/src/main/scala/ch/epfl/scala/debugadapter/testfmk/DebugTestSuite.scala
@@ -229,8 +229,8 @@ trait DebugTest extends CommonUtils {
       continueIfPaused()
       // This is flaky, terminated can happen before exited
       if (!GithubUtils.isCI()) {
-        client.exited()
-        client.terminated()
+        client.exited(timeout = 4.seconds)
+        client.terminated(timeout = 4.seconds)
       }
     }
 

diff --git a/modules/tests/src/test/scala/ch/epfl/scala/debugadapter/ScalaEvaluationTests.scala b/modules/tests/src/test/scala/ch/epfl/scala/debugadapter/ScalaEvaluationTests.scala
@@ -2692,4 +2692,27 @@ abstract class ScalaEvaluationTests(scalaVersion: ScalaVersion) extends DebugTes
     implicit val debuggee: TestingDebuggee = TestingDebuggee.mainClass(source, "example.Main", scalaVersion)
     check(Breakpoint(6), if (isScala2) Breakpoint(6) else NoStep(), Evaluation.success("msg", "hello"))
   }
+
+  test("i628: rewrite indent options") {
+    assume(scalaVersion.isScala3)
+    val source =
+      """|package example
+         |
+         |object Main:
+         |  def main(args: Array[String]): Unit =
+         |    last(List(1))
+         |
+         |  def last[A](l: List[A]): A = l match
+         |    case lst :: Nil => lst
+         |    case _ :: tail => last(tail)
+         |    case Nil => throw new NoSuchElementException("last of empty list")
+         |""".stripMargin
+    implicit val debuggee: TestingDebuggee =
+      TestingDebuggee.mainClass(source, "example.Main", scalaVersion, Seq("-rewrite", "-indent"))
+    check(
+      Breakpoint(8),
+      Evaluation.success("l.size", 1),
+      Breakpoint(8)
+    )
+  }
 }