fixing extra line bug with complex initializer

closes #103
belav · May 15, 2021 · 3bac5e0 · 3bac5e0
1 parent 514d9b8
commit 3bac5e0
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 6 deletions.
diff --git a/Src/CSharpier.Tests/TestFiles/InitializerExpression/InitializerExpressions.cst b/Src/CSharpier.Tests/TestFiles/InitializerExpression/InitializerExpressions.cst
@@ -19,6 +19,10 @@ class ClassName
     {
         // should break braces correctly
         { 1, "one" },
-        { 2, "two" }
+        { 2, "two" },
+        {
+            3,
+            "ifThisIsReallyLongItShouldntInsertAnExtraLineAboveItttttttttttttttt"
+        }
     };
 }
diff --git a/Src/CSharpier/SyntaxPrinter/SyntaxNodePrinters/InitializerExpression.cs b/Src/CSharpier/SyntaxPrinter/SyntaxNodePrinters/InitializerExpression.cs
@@ -11,11 +11,9 @@ public static class InitializerExpression
         public static Doc Print(InitializerExpressionSyntax node)
         {
             var result = Doc.Concat(
-                node.Kind() == SyntaxKind.ArrayInitializerExpression
-                    ? string.Empty
-                    : node.Kind() == SyntaxKind.ComplexElementInitializerExpression
-                            ? Doc.SoftLine
-                            : Doc.Line,
+                node.Kind() is (SyntaxKind.ArrayInitializerExpression or SyntaxKind.ComplexElementInitializerExpression) 
+                    ? Doc.Null
+                    : Doc.Line,
                 Token.Print(node.OpenBraceToken),
                 Doc.Indent(
                     Doc.Line,