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(test): add -Confirm:$false to Suspend/Restart container Pester tests

Browse Source 
Suspend-PveContainer and Restart-PveContainer gained ConfirmImpact.High
in F062/F063 fixes. The "Should throw when no session is active" Pester
tests invoke these cmdlets without -Confirm:$false, causing the
confirmation prompt to fire before the session check — failing in
non-interactive CI on all platforms.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
Clint Branham
2026-03-23 15:09:58 -05:00
parent 339ffaf0bd
commit d74f8c952a
2 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
tests/PSProxmoxVE.Tests/Containers/ContainerGapCmdlets.Tests.ps1
+1 -1
View File
@@ -70,7 +70,7 @@ Describe 'Suspend-PveContainer' -Tag 'Unit' {
Context 'Without active session' {
It 'Should throw when no session is active' {
Skip-IfMissing 'Suspend-PveContainer'
{ Suspend-PveContainer -Node 'pve' -VmId 100 -ErrorAction Stop } |
{ Suspend-PveContainer -Node 'pve' -VmId 100 -Confirm:$false -ErrorAction Stop } |
Should -Throw '*No active Proxmox VE session*'
}
}
tests/PSProxmoxVE.Tests/Containers/ContainerLifecycle.Tests.ps1
+1 -1
View File
@@ -288,7 +288,7 @@ Describe 'Restart-PveContainer' {
Context 'Without active session' {
It 'Should throw when no session is active' {
Skip-IfMissing 'Restart-PveContainer'
{ Restart-PveContainer -Node 'pve-node1' -VmId 200 -ErrorAction Stop } |
{ Restart-PveContainer -Node 'pve-node1' -VmId 200 -Confirm:$false -ErrorAction Stop } |
Should -Throw '*No active Proxmox VE session*'
}
}