Fixing extra new line inside case (#1202)

closes #1192 Co-authored-by: Lasath Fernando <[email protected]>
belav · Mar 11, 2024 · 8949282 · 8949282
1 parent 5e12370
commit 8949282
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 2 deletions.
diff --git a/Src/CSharpier.Tests/FormattingTests/TestFiles/cs/SwitchStatements.test b/Src/CSharpier.Tests/FormattingTests/TestFiles/cs/SwitchStatements.test
@@ -112,5 +112,20 @@ public class ClassName
             case (_, SomeValue) when !isEquality:
                 return;
         }
+
+        switch (someValue)
+        {
+            case 0:
+            {
+                // dedented because the only statement is a block
+                break;
+            }
+
+            case 1:
+                {
+                    // indented because there are two statements, a block then a break
+                }
+                break;
+        }
     }
 }
diff --git a/Src/CSharpier/SyntaxPrinter/SyntaxNodePrinters/SwitchSection.cs b/Src/CSharpier/SyntaxPrinter/SyntaxNodePrinters/SwitchSection.cs
@@ -8,15 +8,15 @@ public static Doc Print(SwitchSectionSyntax node, FormattingContext context)
         {
             Doc.Join(Doc.HardLine, node.Labels.Select(o => Node.Print(o, context)))
         };
-        if (node.Statements.Count == 1 && node.Statements[0] is BlockSyntax blockSyntax)
+        if (node.Statements is [BlockSyntax blockSyntax])
         {
             docs.Add(Block.Print(blockSyntax, context));
         }
         else
         {
             docs.Add(
                 Doc.Indent(
-                    Doc.HardLine,
+                    node.Statements.First() is BlockSyntax ? Doc.Null : Doc.HardLine,
                     Doc.Join(
                         Doc.HardLine,
                         node.Statements.Select(o => Node.Print(o, context)).ToArray()