Fix PSUseConsistentIndentation.PipelineIndentation.None to not remove code when the previous line ended with a pipe (#1746)

bergmeister · web-flow · commit 48909392fb68 · 2022-05-24T15:08:56.000-07:00
* Fix issue 1580

* add test case
diff --git a/Rules/UseConsistentIndentation.cs b/Rules/UseConsistentIndentation.cs
@@ -260,6 +260,7 @@ caused by tokens that require a closing RParen (which are LParen, AtParen and Do
 
                             if (pipelineIndentationStyle == PipelineIndentationStyle.None && PreviousLineEndedWithPipe(tokens, tokenIndex, token))
                             {
+                                onNewLine = false;
                                 continue;
                             }
 
diff --git a/Tests/Rules/UseConsistentIndentation.tests.ps1 b/Tests/Rules/UseConsistentIndentation.tests.ps1
@@ -390,6 +390,11 @@ bar
             @{ IdempotentScriptDefinition = @'
 foo |
     bar
+'@
+            },
+            @{ IdempotentScriptDefinition = @'
+foo |
+    bar -Parameter1
 '@
             }
             ) {

Original file line number	Diff line number	Diff line change
`@@ -260,6 +260,7 @@ caused by tokens that require a closing RParen (which are LParen, AtParen and Do`
`260`	`260`
`261`	`261`	`if (pipelineIndentationStyle == PipelineIndentationStyle.None && PreviousLineEndedWithPipe(tokens, tokenIndex, token))`
`262`	`262`	`{`
	`263`	`+ onNewLine = false;`
`263`	`264`	`continue;`
`264`	`265`	`}`
`265`	`266`