Skip to content

Commit 4dfb159

Browse files
author
Kapil Borle
committed
Fix tests of PlaceOpenBrace rule
1 parent ea38998 commit 4dfb159

File tree

1 file changed

+15
-14
lines changed

1 file changed

+15
-14
lines changed

Tests/Rules/PlaceOpenBrace.tests.ps1

Lines changed: 15 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,14 @@
11
Import-Module PSScriptAnalyzer
2-
$ruleName = "PSPlaceOpenBrace"
2+
$settings = @{
3+
IncludeRules = @("PSPlaceOpenBrace")
4+
Rules = @{
5+
PSPlaceOpenBrace = @{
6+
Enable = $true
7+
OnSameLine = $true
8+
}
9+
}
10+
}
11+
312

413
Describe "PlaceOpenBrace on same line" {
514
Context "When an open brace must be on the same line" {
@@ -10,7 +19,7 @@ function foo ($param1)
1019
1120
}
1221
'@
13-
$violations = Invoke-ScriptAnalyzer -ScriptDefinition $def -IncludeRule $ruleName
22+
$violations = Invoke-ScriptAnalyzer -ScriptDefinition $def -Settings $settings
1423
}
1524

1625
It "Should find a violation" {
@@ -29,18 +38,10 @@ function foo ($param1) {
2938
3039
}
3140
'@
32-
$params = @{
33-
ScriptDefinition = $def
34-
IncludeRule = $ruleName
35-
Settings = @{
36-
rules = @{
37-
PSPlaceOpenBrace = @{
38-
OnSameLine = $false
39-
}
40-
}
41-
}
42-
}
43-
$violations = Invoke-ScriptAnalyzer @params
41+
42+
$settingsNewLine = $settings.Clone()
43+
$settingsNewLine["Rules"]["PSPlaceOpenBrace"]["OnSameLine"] = $false
44+
$violations = Invoke-ScriptAnalyzer -ScriptDefinition $def -Settings $settingsNewLine
4445
}
4546

4647
It "Should find a violation" {

0 commit comments

Comments
 (0)