From ca4585f134caf94fb9317010f61de8b4e0b437ee Mon Sep 17 00:00:00 2001 From: Clint Branham Date: Sat, 21 Mar 2026 09:31:40 -0500 Subject: [PATCH] fix(test): add missing Url and Key params to New-PveSdnDns no-session test New-PveSdnDns requires mandatory Url and Key parameters. The no-session test was failing with "missing mandatory parameters" instead of reaching the session check. Co-Authored-By: Claude Opus 4.6 (1M context) --- .../Network/SdnIpamDnsControllerCmdlets.Tests.ps1 | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/PSProxmoxVE.Tests/Network/SdnIpamDnsControllerCmdlets.Tests.ps1 b/tests/PSProxmoxVE.Tests/Network/SdnIpamDnsControllerCmdlets.Tests.ps1 index 9d580ca..6f2fa8a 100644 --- a/tests/PSProxmoxVE.Tests/Network/SdnIpamDnsControllerCmdlets.Tests.ps1 +++ b/tests/PSProxmoxVE.Tests/Network/SdnIpamDnsControllerCmdlets.Tests.ps1 @@ -275,7 +275,7 @@ Describe 'New-PveSdnDns' { Context 'Without active session' { It 'Should throw when no session is active' { Skip-IfMissing 'New-PveSdnDns' - { New-PveSdnDns -Dns 'testdns' -Type 'powerdns' -ErrorAction Stop } | + { New-PveSdnDns -Dns 'testdns' -Type 'powerdns' -Url 'http://localhost:8081' -Key 'testkey' -ErrorAction Stop } | Should -Throw '*No active Proxmox VE session*' } }