Clean up syntax in tests

Author: som-snytt

presentation-compiler/test/dotty/tools/pc/tests/completion/CompletionExtensionSuite.scala

@@ -86,7 +86,7 @@ class CompletionExtensionSuite extends BaseCompletionSuite:
 
   @Test def `filter-by-type-old` =
     check(
-      """|package example
+     """|package example
         |
         |object enrichments:
         |  implicit class A(num: Int):
@@ -99,7 +99,6 @@ class CompletionExtensionSuite extends BaseCompletionSuite:
      """|identity: String (implicit)
         |""".stripMargin, // identity2 won't be available
       filter = _.contains("(implicit)")
-
     )
 
   @Test def `filter-by-type-subtype` =

presentation-compiler/test/dotty/tools/pc/tests/completion/CompletionKeywordSuite.scala

@@ -57,7 +57,7 @@ class CompletionKeywordSuite extends BaseCompletionSuite:
         |  **/
         |}
         |""".stripMargin,
-     "",
+      "",
       includeCommitCharacter = true
     )
 

presentation-compiler/test/dotty/tools/pc/tests/completion/CompletionSuite.scala

@@ -1360,7 +1360,7 @@ class CompletionSuite extends BaseCompletionSuite:
           | val t: TT@@
           |}
           |""".stripMargin,
-    "TTT[A <: Int] = List[A]"
+      "TTT[A <: Int] = List[A]"
     )
 
   @Test def `type-lambda` =

presentation-compiler/test/dotty/tools/pc/tests/hover/HoverDocSuite.scala

@@ -19,20 +19,20 @@ class HoverDocSuite extends BaseHoverSuite:
 
   @Test def `doc` =
     check(
-      """object a {
-        |  <<java.util.Collections.empty@@List[Int]>>
-        |}
-        |""".stripMargin,
-     """|**Expression type**:
-        |```scala
-        |java.util.List[Int]
-        |```
-        |**Symbol signature**:
-        |```scala
-        |final def emptyList[T](): java.util.List[T]
-        |```
-        |Found documentation for java/util/Collections#emptyList().
-        |""".stripMargin,
+      """|object a {
+         |  <<java.util.Collections.empty@@List[Int]>>
+         |}
+         |""".stripMargin,
+      """|**Expression type**:
+         |```scala
+         |java.util.List[Int]
+         |```
+         |**Symbol signature**:
+         |```scala
+         |final def emptyList[T](): java.util.List[T]
+         |```
+         |Found documentation for java/util/Collections#emptyList().
+         |""".stripMargin,
     )
 
   @Test def `doc-parent` =
@@ -47,7 +47,6 @@ class HoverDocSuite extends BaseHoverSuite:
          |```
          |Found documentation for scala/collection/LinearSeqOps#headOption().
          |""".stripMargin,
-
     )
 
   @Test def `java-method` =
@@ -61,7 +60,7 @@ class HoverDocSuite extends BaseHoverSuite:
          |```
          |Found documentation for java/lang/String#substring().
          |""".stripMargin
-  )
+    )
 
   @Test def `object` =
     check(

scaladoc/test/dotty/tools/scaladoc/site/TemplateFileTests.scala

@@ -208,26 +208,26 @@ class TemplateFileTests:
     testTemplate(
       """# Hello {{ msg }}!""",
       ext = "md"
-    ) { t =>
+    ): t =>
       assertEquals(
         """<section id="hello-there">
-        |<h1 class="h500"><a href="#hello-there" class="anchor"></a>Hello there!</h1>
-        |</section>""".stripMargin,
-      t.resolveInner(RenderingContext(Map("msg" -> "there"))).code.trim())
-    }
+          |<h1 class="h500"><a href="#hello-there" class="anchor"></a>Hello there!</h1>
+          |</section>""".stripMargin,
+        t.resolveInner(RenderingContext(Map("msg" -> "there"))).code.trim()
+      )
 
   @Test
   def mixedTemplates() : Unit =
     testTemplate(
       """# Hello {{ msg }}!""",
       ext = "md"
-    ) { t =>
+    ): t =>
       assertEquals(
         """<section id="hello-there2">
-        |<h1 class="h500"><a href="#hello-there2" class="anchor"></a>Hello there2!</h1>
-        |</section>""".stripMargin,
-      t.resolveInner(RenderingContext(Map("msg" -> "there2"))).code.trim())
-    }
+          |<h1 class="h500"><a href="#hello-there2" class="anchor"></a>Hello there2!</h1>
+          |</section>""".stripMargin,
+        t.resolveInner(RenderingContext(Map("msg" -> "there2"))).code.trim()
+      )
 
   @Test
   def htmlOnly(): Unit =

tests/neg/i22527.scala

@@ -23,3 +23,8 @@ def g: Unit =
       class X extends AnyRef, Serializable // error
       27 // error
   )
+
+def onlyIf(x: Int): Unit =
+  callme(
+    if (x > 0)
+      true, "fail") // error syntax is broken after old-style conditional

tests/pos/i22527.scala

@@ -41,7 +41,7 @@ def orElse(x: Int): Unit =
 @nowarn("msg=Unit")
 def onlyIf(x: Int): Unit =
   callme(
-    if (x > 0)
+    if (x > 0) then // then syntax required
       true, "fail") // warn value discard
 
 def h(xs: List[Int]) =

tests/warn/i22527.scala

@@ -36,7 +36,7 @@ def orElse(x: Int): Unit =
 
 def onlyIf(x: Int): Unit =
   callme(
-    if (x > 0)
+    if x > 0 then
       true, "fail") // warn value discard
 
 def h(xs: List[Int]) =