Skip to content

Commit 3f00576

Browse files
Refactor rule skipping messages in PSScriptAnalyzer tests to use ANSI coloring for improved visibility
1 parent a7f544e commit 3f00576

File tree

1 file changed

+14
-10
lines changed

1 file changed

+14
-10
lines changed

scripts/tests/PSScriptAnalyzer/PSScriptAnalyzer.Tests.ps1

Lines changed: 14 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -26,27 +26,31 @@ BeforeDiscovery {
2626
$ruleObjects = Get-ScriptAnalyzerRule -Verbose:$false | Sort-Object -Property Severity, CommonName
2727
$Severeties = $ruleObjects | Select-Object -ExpandProperty Severity -Unique
2828

29-
Write-Verbose ($PSStyle.OutputRendering.ToString()) -Verbose
29+
$PSStyle.OutputRendering = 'Ansi'
30+
$darkGrey = $PSStyle.Foreground.FromRgb(85, 85, 85)
31+
$green = $PSStyle.Foreground.Green
32+
$reset = $PSStyle.Reset
3033

3134
foreach ($ruleObject in $ruleObjects) {
3235
if ($ruleObject.RuleName -in $settings.ExcludeRules) {
33-
Write-Host " - $($ruleObject.RuleName)" -ForegroundColor DarkGray
34-
Write-Host ' Skipping rule - Exclude list' -ForegroundColor DarkGray
36+
Write-Host "$darkGrey - $($ruleObject.RuleName)$reset"
37+
Write-Host "$darkGrey Skipping rule - Exclude list$reset"
3538
$skip = $true
3639
} elseif ($settings.IncludeRules -and $ruleObject.RuleName -notin $settings.IncludeRules) {
37-
Write-Host " - $($ruleObject.RuleName)" -ForegroundColor DarkGray
38-
Write-Host ' Skipping rule - Include list' -ForegroundColor DarkGray
40+
Write-Host "$darkGrey - $($ruleObject.RuleName)$reset"
41+
Write-Host "$darkGrey Skipping rule - Include list$reset"
3942
$skip = $true
4043
} elseif ($settings.Severity -and $ruleObject.Severity -notin $settings.Severity) {
41-
Write-Host " - $($ruleObject.RuleName)" -ForegroundColor DarkGray
42-
Write-Host ' Skipping rule - Severity list' -ForegroundColor DarkGray
44+
Write-Host "$darkGrey - $($ruleObject.RuleName)$reset"
45+
Write-Host "$darkGrey Skipping rule - Severity list$reset"
4346
$skip = $true
4447
} elseif ($settings.Rules -and $settings.Rules.ContainsKey($ruleObject.RuleName) -and -not $settings.Rules[$ruleObject.RuleName].Enable) {
45-
Write-Host " - $($ruleObject.RuleName)" -ForegroundColor DarkGray
46-
Write-Host ' Skipping rule - Disabled' -ForegroundColor DarkGray
48+
Write-Host "$darkGrey - $($ruleObject.RuleName)$reset"
49+
Write-Host "$darkGrey Skipping rule - Disabled$reset"
4750
$skip = $true
4851
} else {
49-
Write-Host " - $($ruleObject.RuleName)" -ForegroundColor Green
52+
Write-Host "$green + $($ruleObject.RuleName)$reset"
53+
Write-Host "$green Including rule$reset"
5054
$skip = $false
5155
}
5256

0 commit comments

Comments
 (0)