gsadmin/PSProxmoxVE
1
0
Fork 0
mirror of https://github.com/GoodOlClint/PSProxmoxVE.git synced 2026-07-26 07:58:14 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: remove ValidateSet assertion from Set-PveHaRule Type test

Browse Source 
The ValidateSet attribute was intentionally removed from the Type
parameter during merge review to allow future rule types without
code changes. Updated test to only assert Type is mandatory.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
Clint Branham
2026-03-26 08:18:47 -05:00
parent d37569533d
commit 12ca01cd09
1 changed files with 1 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
tests/PSProxmoxVE.Tests/HA/HaCmdlets.Tests.ps1
+1 -3
View File
@@ -736,14 +736,12 @@ Describe 'Set-PveHaRule' -Tag 'Unit' {
$isMandatory = $p.ParameterSets.Values | Where-Object { $_.IsMandatory }
$isMandatory | Should -Not -BeNullOrEmpty
}
It 'Should have mandatory Type parameter with ValidateSet' {
It 'Should have mandatory Type parameter' {
Skip-IfMissing 'Set-PveHaRule'
$p = $script:Cmd.Parameters['Type']
$p | Should -Not -BeNullOrEmpty
$isMandatory = $p.ParameterSets.Values | Where-Object { $_.IsMandatory }
$isMandatory | Should -Not -BeNullOrEmpty
$vsAttr = $p.Attributes | Where-Object { $_ -is [System.Management.Automation.ValidateSetAttribute] }
$vsAttr | Should -Not -BeNullOrEmpty
}
It 'Should have optional State parameter with ValidateSet' {
Skip-IfMissing 'Set-PveHaRule'