Fixing weird edge that involved initializers + invocations on them.

closes #802
belav · Feb 4, 2023 · b96da47 · b96da47
1 parent c32bbfd
commit b96da47
Show file tree

Hide file tree

Showing 2 changed files with 47 additions and 1 deletion.
diff --git a/Src/CSharpier.Tests/FormattingTests/TestFiles/InitializerExpressions.cst b/Src/CSharpier.Tests/FormattingTests/TestFiles/InitializerExpressions.cst
@@ -162,6 +162,48 @@ class ClassName
                 ? one
                 : two
         };
+
+        var someObject = new SomeObject
+        {
+            SomeArray = new SomeOtherObject[]
+            {
+                new SomeOtherObject { SomeProperty = 1 },
+                new SomeOtherObject()
+            }
+                .CallMethod()
+                .CallMethod()
+        };
+
+        var someObject = new SomeObject
+        {
+            SomeArray = new SomeOtherObject
+            {
+                new SomeOtherObject { SomeProperty = 1 },
+                new SomeOtherObject()
+            }
+                .CallMethod()
+                .CallMethod()
+        };
+
+        var someObject = new SomeObject
+        {
+            SomeArray = new SomeOtherObject[]
+            {
+                new SomeOtherObject { SomeProperty = 1 },
+                new SomeOtherObject()
+            }
+        };
+
+        var someObject = new SomeObject
+        {
+            SomeArray = new SomeOtherObject[]
+            {
+                new SomeOtherObject_________________________________(),
+                new SomeOtherObject_________________________________()
+            }
+                .CallMethod()
+                .CallMethod()
+        };
     }
 
     private SomeObject someObject =

diff --git a/Src/CSharpier/SyntaxPrinter/SyntaxNodePrinters/InvocationExpression.cs b/Src/CSharpier/SyntaxPrinter/SyntaxNodePrinters/InvocationExpression.cs
@@ -68,7 +68,11 @@ or PostfixUnaryExpressionSyntax
             PrintIndentedGroup(node, groups.Skip(shouldMergeFirstTwoGroups ? 2 : 1).ToList())
         );
 
-        return oneLine.Skip(1).Any(DocUtilities.ContainsBreak)
+        return
+            oneLine.Skip(1).Any(DocUtilities.ContainsBreak)
+            || groups[0].Any(
+                o => o.Node is ArrayCreationExpressionSyntax or ObjectCreationExpressionSyntax
+            )
             ? expanded
             : Doc.ConditionalGroup(Doc.Concat(oneLine), expanded);
     }