Fixing repeated new line being added within switch expressions (#1272)

belav · May 31, 2024 · e911ae4 · e911ae4
1 parent a4dd17d
commit e911ae4
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 1 deletion.
diff --git a/Src/CSharpier.Tests/FormattingTests/TestFiles/cs/SwitchExpressions.test b/Src/CSharpier.Tests/FormattingTests/TestFiles/cs/SwitchExpressions.test
@@ -102,5 +102,13 @@ class ClassName
  // comment
  Some.Two => 2,
  };
+
+ return someValue switch
+ {
+ (true, true) => someOtherValue,
+ (false, false)
+ or someCondition
+ => someValue_____________________________________________________
+ };
  }
 }
diff --git a/Src/CSharpier/SyntaxPrinter/SyntaxNodePrinters/RecursivePattern.cs b/Src/CSharpier/SyntaxPrinter/SyntaxNodePrinters/RecursivePattern.cs
@@ -27,7 +27,13 @@ FormattingContext context
  if (node.PositionalPatternClause != null)
  {
  result.Add(
- node.Parent is SwitchExpressionArmSyntax or CasePatternSwitchLabelSyntax
+ node.Parent
+ is SwitchExpressionArmSyntax
+ or CasePatternSwitchLabelSyntax
+ or BinaryPatternSyntax
+ {
+ Parent: SwitchExpressionArmSyntax or CasePatternSwitchLabelSyntax
+ }
  ? Doc.Null
  : Doc.SoftLine,
  Doc.Group(