Keep property with method

closes #1010
belav · Dec 16, 2023 · 50f99f6 · 50f99f6
1 parent a7d151d
commit 50f99f6
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 23 deletions.
diff --git a/Src/CSharpier.Tests/FormattingTests/TestFiles/cs/MemberChain_PropertiesConsistent.test b/Src/CSharpier.Tests/FormattingTests/TestFiles/cs/MemberChain_PropertiesConsistent.test
@@ -1,14 +1,15 @@
-var someVariable = someObject
-    .Property
-    .CallMethod(someValue => someValue.SomeProperty == someOtherValue___________________________);
+var someVariable = someObject.Property.CallMethod(someValue =>
+    someValue.SomeProperty == someOtherValue___________________________
+);
 
 var someVariable = someObject
     .Property()
     .CallMethod(someValue => someValue.SomeProperty == someOtherValue___________________________);
 
 var someVariable = someObject
-    .Property
-    .CallMethod(someValue => someValue.SomeProperty == someOtherValue___________________________)
+    .Property.CallMethod(someValue =>
+        someValue.SomeProperty == someOtherValue___________________________
+    )
     .CallMethod();
 
 var someVariable = someObject
@@ -24,8 +25,9 @@ var someVariable = this.Property.CallMethod(someValue =>
 var someVariable = this.Property()
     .CallMethod(someValue => someValue.SomeProperty == someOtherValue___________________________);
 
-var someVariable = this.Property
-    .CallMethod(someValue => someValue.SomeProperty == someOtherValue___________________________)
+var someVariable = this.Property.CallMethod(someValue =>
+    someValue.SomeProperty == someOtherValue___________________________
+)
     .CallMethod();
 
 var someVariable = this.Property()

diff --git a/Src/CSharpier.Tests/FormattingTests/TestFiles/cs/MemberChains.test b/Src/CSharpier.Tests/FormattingTests/TestFiles/cs/MemberChains.test
@@ -52,13 +52,11 @@ class ClassName
             .Where(o => someLongCondition__________________________);
 
         var someValue = someOtherValue!
-            .Thing!
-            .Where(o => someLongCondition__________________________)
+            .Thing!.Where(o => someLongCondition__________________________)
             .Where(o => someLongCondition__________________________);
 
         var someValue = someOtherValue
-            .Thing
-            .Where(o => someLongCondition__________________________)
+            .Thing.Where(o => someLongCondition__________________________)
             .Where(o => someLongCondition__________________________);
 
         var someValue = someOtherValue
@@ -83,8 +81,7 @@ class ClassName
             );
 
         roleNames
-            .Value
-            .Where(o => o.SomeProperty____________________________________)
+            .Value.Where(o => o.SomeProperty____________________________________)
             .Select(o => o.SomethingElse);
 
         return someCondition
@@ -112,11 +109,9 @@ class ClassName
             someParameter____________________________________
         )!;
 
-        var someVariable = someObject
-            .Property
-            .CallMethod(someValue =>
-                someValue.SomeProperty == someOtherValue___________________________________
-            );
+        var someVariable = someObject.Property.CallMethod(someValue =>
+            someValue.SomeProperty == someOtherValue___________________________________
+        );
 
         CallMethod(
                 firstParameter____________________________,
@@ -236,8 +231,7 @@ class ClassName
             .CallMethod__________________();
 
         someThing_______________________
-            .Property
-            .CallMethod__________________()
+            .Property.CallMethod__________________()
             .CallMethod__________________();
 
         someThing_______________________
@@ -246,8 +240,7 @@ class ClassName
             .CallMethod__________________();
 
         someThing_______________________
-            .Property!
-            .CallMethod__________________()
+            .Property!.CallMethod__________________()
             .CallMethod__________________();
     }
 }
diff --git a/Src/CSharpier/SyntaxPrinter/SyntaxNodePrinters/InvocationExpression.cs b/Src/CSharpier/SyntaxPrinter/SyntaxNodePrinters/InvocationExpression.cs
@@ -296,7 +296,6 @@ or ConditionalAccessExpressionSyntax
                 printedNodes[index].Node
                 is (
                     InvocationExpressionSyntax
-                    or MemberAccessExpressionSyntax
                     or ElementAccessExpressionSyntax
                     or MemberBindingExpressionSyntax
                 )