Fixing inconsistent indentation with parenthisized expression (#1570)

closes #1562

Author: belav

Shell/ReviewBranch.psm1

@@ -53,9 +53,9 @@ function CSH-ReviewBranch {
     $postBranchOutput = (git status 2>&1) | Out-String
     $firstRun = -not $postBranchOutput.Contains("On branch $postBranch")
 
-    $skipValidation = ""
+    $skipValidationParam = ""
     if ($skipValidation -eq $true) {
-        $skipValidation = "--skip-validation"
+        $skipValidationParam = "--skip-validation"
     }
 
     if ($firstRun) {
@@ -76,11 +76,11 @@ function CSH-ReviewBranch {
         & git reset --hard
         & git checkout -b $preBranch
 
-        dotnet $csharpierDllPath . $fastParam --no-cache
+        dotnet $csharpierDllPath format . $skipValidationParam --no-cache
         # there is some weirdness with a couple files with #if where
         # they need to be formatted twice to get them stable
         # it isn't worth fixing in csharpier, because it only really affects this
-        dotnet $csharpierDllPath . $fastParam
+        dotnet $csharpierDllPath format . $skipValidationParam
 
         & git add -A
         & git commit -m "Before $branch"
@@ -99,7 +99,7 @@ function CSH-ReviewBranch {
         & git checkout $postBranch
     }
 
-    dotnet $csharpierDllPath . $fastParam --no-cache
+    dotnet $csharpierDllPath format . $skipValidationParam --no-cache
 
     & git add -A
     & git commit -m "After $branch"

Src/CSharpier.Tests/FormattingTests/TestFiles/cs/ParenthesizedExpressions.test

@@ -12,5 +12,11 @@ class ClassName
                 || currentChar == '|'
                 || currentChar == '&'
             );
+        var b2 = (
+            System.Environment.SpecialFolder.AdminTools
+                is System.Environment.SpecialFolder.AdminTools
+                    or System.Environment.SpecialFolder.AdminTools
+                    or System.Environment.SpecialFolder.AdminTools
+        );
     }
 }

Src/CSharpier/SyntaxPrinter/SyntaxNodePrinters/IsPatternExpression.cs

@@ -4,7 +4,11 @@ internal static class IsPatternExpression
 {
     public static Doc Print(IsPatternExpressionSyntax node, PrintingContext context)
     {
-        if (node.Parent is not (IfStatementSyntax or ParenthesizedExpressionSyntax))
+        if (
+            node.Parent
+            is not (IfStatementSyntax or ParenthesizedExpressionSyntax)
+                or ParenthesizedExpressionSyntax { Parent: EqualsValueClauseSyntax }
+        )
         {
             return Doc.Group(
                 Node.Print(node.Expression, context),