Add tests to check new line after close brace violation

Author: Kapil Borle

Tests/Rules/PlaceCloseBrace.tests.ps1

@@ -3,6 +3,7 @@ $ruleConfiguration = @{
     Enable = $true
     NoEmptyLineBefore = $true
     IgnoreOneLineBlock = $true
+    NewLineAfter = $true
 }
 
 $settings = @{
@@ -19,6 +20,9 @@ Describe "PlaceCloseBrace" {
 function foo {
     Write-Output "close brace not on a new line"}
 '@
+            $ruleConfiguration.'NoEmptyLineBefore' = $false
+            $ruleConfiguration.'IgnoreOneLineBlock' = $false
+            $ruleConfiguration.'NewLineAfter' = $false
             $violations = Invoke-ScriptAnalyzer -ScriptDefinition $def -Settings $settings
         }
 
@@ -39,6 +43,9 @@ function foo {
 
 }
 '@
+            $ruleConfiguration.'NoEmptyLineBefore' = $true
+            $ruleConfiguration.'IgnoreOneLineBlock' = $false
+            $ruleConfiguration.'NewLineAfter' = $false
             $violations = Invoke-ScriptAnalyzer -ScriptDefinition $def -Settings $settings
         }
 
@@ -56,6 +63,9 @@ function foo {
             $def = @'
 $hashtable = @{a = 1; b = 2}
 '@
+            $ruleConfiguration.'NoEmptyLineBefore' = $false
+            $ruleConfiguration.'IgnoreOneLineBlock' = $true
+            $ruleConfiguration.'NewLineAfter' = $false
             $violations = Invoke-ScriptAnalyzer -ScriptDefinition $def -Settings $settings
         }
 
@@ -71,6 +81,9 @@ $hashtable = @{
     a = 1
     b = 2}
 '@
+            $ruleConfiguration.'NoEmptyLineBefore' = $false
+            $ruleConfiguration.'IgnoreOneLineBlock' = $true
+            $ruleConfiguration.'NewLineAfter' = $false
             $violations = Invoke-ScriptAnalyzer -ScriptDefinition $def -Settings $settings
         }
 
@@ -101,4 +114,24 @@ $x = if ($true) { "blah" } else { "blah blah" }
             $violations.Count | Should Be 0
         }
     }
+
+    Context "When a close brace should be follow a new line" {
+        BeforeAll {
+            $def = @'
+if (Test-Path "blah") {
+    "blah"
+} else {
+    "blah blah"
+}
+'@
+            $ruleConfiguration.'NoEmptyLineBefore' = $false
+            $ruleConfiguration.'IgnoreOneLineBlock' = $false
+            $ruleConfiguration.'NewLineAfter' = $true
+            $violations = Invoke-ScriptAnalyzer -ScriptDefinition $def -Settings $settings
+        }
+
+        It "Should find two violations" {
+            $violations.Count | Should Be 2
+        }
+    }
 }