Address delayed release review findings

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Author: MariusStorhaug

src/functions/private/Auth/Test-DomeneshopContextName.ps1

@@ -33,6 +33,10 @@ function Test-DomeneshopContextName {
         [string] $Context
     )
 
+    if ([string]::IsNullOrWhiteSpace($Context)) {
+        throw 'Context name cannot be empty or whitespace.'
+    }
+
     if ($Context -eq '__Domeneshop.Config') {
         throw "Context name [$Context] is reserved for Domeneshop module configuration."
     }

src/functions/public/Dns/Get-DomeneshopDnsRecord.ps1

@@ -22,7 +22,7 @@ function Get-DomeneshopDnsRecord {
         You can't pipe objects to Get-DomeneshopDnsRecord.
 
         .OUTPUTS
-        System.Object[]
+        System.Object
 
         The matching Domeneshop DNS records.
 
@@ -32,7 +32,7 @@ function Get-DomeneshopDnsRecord {
         .LINK
         https://api.domeneshop.no/docs/
     #>
-    [OutputType([object[]])]
+    [OutputType([object])]
     [CmdletBinding(DefaultParameterSetName = 'List')]
     param(
         # The numeric identifier of the domain that owns the records.

src/functions/public/Domains/Get-DomeneshopDomain.ps1

@@ -20,7 +20,7 @@ function Get-DomeneshopDomain {
         You can't pipe objects to Get-DomeneshopDomain.
 
         .OUTPUTS
-        System.Object[]
+        System.Object
 
         The matching Domeneshop domain records.
 
@@ -30,7 +30,7 @@ function Get-DomeneshopDomain {
         .LINK
         https://api.domeneshop.no/docs/
     #>
-    [OutputType([object[]])]
+    [OutputType([object])]
     [CmdletBinding(DefaultParameterSetName = 'List')]
     param(
         # A domain-name filter for list requests.

src/functions/public/Forwards/Add-DomeneshopForward.ps1

@@ -9,7 +9,7 @@ function Add-DomeneshopForward {
         .EXAMPLE
         $forward = @{
             host = 'www'
-            ('URL'.ToLowerInvariant()) = [uri]::new('HTTPS'.ToLowerInvariant() + '://example.net')
+            ('URL'.ToLowerInvariant()) = 'HTTPS'.ToLowerInvariant() + '://example.net'
         }
         Add-DomeneshopForward -DomainID 42 -Forward $forward
 

src/functions/public/Forwards/Get-DomeneshopForward.ps1

@@ -22,7 +22,7 @@ function Get-DomeneshopForward {
         You can't pipe objects to Get-DomeneshopForward.
 
         .OUTPUTS
-        System.Object[]
+        System.Object
 
         The matching Domeneshop HTTP forwards.
 
@@ -32,7 +32,7 @@ function Get-DomeneshopForward {
         .LINK
         https://api.domeneshop.no/docs/
     #>
-    [OutputType([object[]])]
+    [OutputType([object])]
     [CmdletBinding(DefaultParameterSetName = 'List')]
     param(
         # The numeric identifier of the domain that owns the forwards.

src/functions/public/Forwards/Set-DomeneshopForward.ps1

@@ -9,7 +9,7 @@ function Set-DomeneshopForward {
         .EXAMPLE
         $forward = @{
             host = 'www'
-            ('URL'.ToLowerInvariant()) = [uri]::new('HTTPS'.ToLowerInvariant() + '://example.org')
+            ('URL'.ToLowerInvariant()) = 'HTTPS'.ToLowerInvariant() + '://example.org'
         }
         Set-DomeneshopForward -DomainID 42 -ForwardHost 'www' -Forward $forward
 

src/functions/public/Invoices/Get-DomeneshopInvoice.ps1

@@ -22,7 +22,7 @@ function Get-DomeneshopInvoice {
         You can't pipe objects to Get-DomeneshopInvoice.
 
         .OUTPUTS
-        System.Object[]
+        System.Object
 
         The matching Domeneshop invoices.
 
@@ -32,7 +32,7 @@ function Get-DomeneshopInvoice {
         .LINK
         https://api.domeneshop.no/docs/
     #>
-    [OutputType([object[]])]
+    [OutputType([object])]
     [CmdletBinding(DefaultParameterSetName = 'List')]
     param(
         # The numeric identifier of a specific invoice.

tests/Connect-DomeneshopAccount.Tests.ps1

@@ -102,4 +102,11 @@ Describe 'Connect-DomeneshopAccount' {
 
         Should -Invoke Set-Context -Times 0 -Exactly
     }
+
+    It 'rejects a whitespace-only context name' {
+        { Connect-DomeneshopAccount -Token 'token' -Secret 'secret' -Context ' ' } |
+            Should -Throw '*Context name cannot be empty or whitespace*'
+
+        Should -Invoke Set-Context -Times 0 -Exactly
+    }
 }