Fix normalizing IR strings in LSP tests (#978)

timsueberkrueb · web-flow · commit 52cc53c80622 · 2025-05-07T12:05:50.000+02:00
Replacing all occurrences of whitespace by a single space is not a valid
approach since expected and actual output can differ in whitespace (e.g.
newlines).
diff --git a/effekt/jvm/src/test/scala/effekt/LSPTests.scala b/effekt/jvm/src/test/scala/effekt/LSPTests.scala
@@ -52,7 +52,7 @@ class LSPTests extends FunSuite {
    */
   def normalizeIRString(ir: String): String = {
     ir.replaceAll("_\\d+", "_whatever")
-      .replaceAll("\\s+", " ")
+      .replaceAll("\\s+", "")
   }
   
   def assertIREquals(ir: String, expected: String): Unit = {

Original file line number	Diff line number	Diff line change
`@@ -52,7 +52,7 @@ class LSPTests extends FunSuite {`
`52`	`52`	`*/`
`53`	`53`	`def normalizeIRString(ir: String): String = {`
`54`	`54`	`ir.replaceAll("_\\d+", "_whatever")`
`55`		`- .replaceAll("\\s+", " ")`
	`55`	`+ .replaceAll("\\s+", "")`
`56`	`56`	`}`
`57`	`57`
`58`	`58`	`def assertIREquals(ir: String, expected: String): Unit = {`