Skip to content

Commit fac919e

Browse files
Refactor rule skipping messages in PSScriptAnalyzer tests to use Write-Host for better visibility
1 parent 97f7c4f commit fac919e

File tree

1 file changed

+9
-13
lines changed

1 file changed

+9
-13
lines changed

scripts/tests/PSScriptAnalyzer/PSScriptAnalyzer.Tests.ps1

Lines changed: 9 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -22,27 +22,23 @@ BeforeDiscovery {
2222
$Severeties = $ruleObjects | Select-Object -ExpandProperty Severity -Unique
2323
foreach ($ruleObject in $ruleObjects) {
2424
$skip = if ($ruleObject.RuleName -in $settings.ExcludeRules) {
25-
Write-Verbose "Skipping rule [$($ruleObject.RuleName)] - Because it is in the exclude list" -Verbose
26-
Write-Verbose " Exclude: $($settings.ExcludeRules)" -Verbose
27-
Write-Verbose " Rule is excluded: $($ruleObject.RuleName -in $($settings.ExcludeRules))" -Verbose
25+
Write-Host " - $($ruleObject.RuleName)" -ForegroundColor DarkGray
26+
Write-Host " Skipping rule - Exclude list" -ForegroundColor DarkGray
2827
$true
2928
} elseif ($settings.IncludeRules -and $ruleObject.RuleName -notin $settings.IncludeRules) {
30-
Write-Verbose "Skipping rule [$($ruleObject.RuleName)] - Because it is not in the include list" -Verbose
31-
Write-Verbose " Include: $($settings.IncludeRules)" -Verbose
32-
Write-Verbose " Rule is not included: $($ruleObject.RuleName -notin $($settings.IncludeRules))" -Verbose
29+
Write-Host " - $($ruleObject.RuleName)" -ForegroundColor DarkGray
30+
Write-Host ' Skipping rule - Include list' -ForegroundColor DarkGray
3331
$true
3432
} elseif ($settings.Severity -and $ruleObject.Severity -notin $settings.Severity) {
35-
Write-Verbose "Skipping rule [$($ruleObject.RuleName)] - Because it is not in the severity list" -Verbose
36-
Write-Verbose " Severity: $($settings.Severity)" -Verbose
37-
Write-Verbose " Severity not included: $($ruleObject.Severity -notin $($settings.Severity))" -Verbose
33+
Write-Host " - $($ruleObject.RuleName)" -ForegroundColor DarkGray
34+
Write-Host ' Skipping rule - Severity list' -ForegroundColor DarkGray
3835
$true
3936
} elseif ($settings.Rules -and $settings.Rules.ContainsKey($ruleObject.RuleName) -and -not $settings.Rules[$ruleObject.RuleName].Enable) {
40-
Write-Verbose "Skipping rule [$($ruleObject.RuleName)] - Because it is disabled" -Verbose
41-
Write-Verbose " Rules: $($settings.Rules)" -Verbose
42-
Write-Verbose " Contains: $($settings.Rules.ContainsKey($ruleObject.RuleName))" -Verbose
43-
Write-Verbose " Enable: $($settings.Rules[$ruleObject.RuleName].Enable)" -Verbose
37+
Write-Host " - $($ruleObject.RuleName)" -ForegroundColor DarkGray
38+
Write-Host ' Skipping rule - Disabled' -ForegroundColor DarkGray
4439
$true
4540
} else {
41+
Write-Host " - $($ruleObject.RuleName)" -ForegroundColor Green
4642
$false
4743
}
4844

0 commit comments

Comments
 (0)