Fix violations for one line switch conditionals (#709)

Consider the following expression:
switch ($x) {
{"a" -eq $_} {"a";break}
{"b" -eq $_} {"b";break}
}

If OnSameLine switch is `on`, this would get formatted to:
switch ($x) { {"a" -eq $_} {"a";break} {"b" -eq $_} {"b";break}
}

With this commit, it ignores these one-line conditional expression and so the output will be the same as the input in this case, i.e.,
switch ($x) {
{"a" -eq $_} {"a";break}
{"b" -eq $_} {"b";break}
}

Author: Kapil Borle

Rules/PlaceOpenBrace.cs

@@ -206,7 +206,8 @@ private IEnumerable<DiagnosticRecord> FindViolationsForBraceShouldBeOnSameLine(
             for (int k = 2; k < tokens.Length; k++)
             {
                 if (tokens[k].Kind == TokenKind.LCurly
-                    && tokens[k - 1].Kind == TokenKind.NewLine)
+                    && tokens[k - 1].Kind == TokenKind.NewLine
+                    && !tokensToIgnore.Contains(tokens[k]))
                 {
                     yield return new DiagnosticRecord(
                         GetError(Strings.PlaceOpenBraceErrorShouldBeOnSameLine),

Tests/Rules/PlaceOpenBrace.tests.ps1

@@ -35,6 +35,24 @@ function foo ($param1)
         }
     }
 
+    Context "When an open brace must be on the same line in a switch statement" {
+        BeforeAll {
+            $def = @'
+switch ($x) {
+    {"b"} {"b"; break;}
+    {"a"} {"a"; break;}
+}
+'@
+            $ruleConfiguration.'OnSameLine' = $true
+            $ruleConfiguration.'IgnoreOneLineBlock' = $true
+            $violations = Invoke-ScriptAnalyzer -ScriptDefinition $def -Settings $settings
+        }
+
+        It "Should not find a violation" {
+            $violations.Count | Should Be 0
+        }
+    }
+
     Context "When an open brace must be on a new line" {
         BeforeAll {
             $def = @'