Merge pull request #38 from dataplat/28-WorkspaceName-validation

Fixed validation for WorkspaceName in Get-FabricWorkspace cmdlet #28

Author: jpomfret

source/Public/Workspace/Get-FabricWorkspace.ps1

@@ -38,7 +38,7 @@ function Get-FabricWorkspace {
 
         [Parameter(Mandatory = $false)]
         [ValidateNotNullOrEmpty()]
-        [ValidatePattern('^[a-zA-Z0-9_ ]*$')]
+        [ValidatePattern('^[a-zA-Z0-9_\- ]*$')]
         [string]$WorkspaceName
     )
 
@@ -140,4 +140,4 @@ function Get-FabricWorkspace {
         $errorDetails = $_.Exception.Message
         Write-Message -Message "Failed to retrieve workspace. Error: $errorDetails" -Level Error
     }
-}
+}

tests/Unit/Get-FabricWorkspace.Tests.ps1

@@ -16,7 +16,7 @@ param(
                 "OutVariable"
                 "OutBuffer"
                 "PipelineVariable"
-                
+
     )
 )
 
@@ -43,5 +43,17 @@ Describe "Get-FabricWorkspace" -Tag "UnitTests" {
             Compare-Object -ReferenceObject $expected -DifferenceObject $hasparms | Should -BeNullOrEmpty
         }
     }
-}
 
+    Context "WorkspaceName parameter validation" {
+        It "Throws error when WorkspaceName does not match ValidatePattern" {
+            # Assuming the ValidatePattern allows only alphanumeric, underscore, space and hyphen
+            { Get-FabricWorkspace -WorkspaceName "InvalidName!" } | Should -Throw
+            { Get-FabricWorkspace -WorkspaceName "Another@Invalid" } | Should -Throw
+        }
+
+        It "Does not throw when WorkspaceName matches ValidatePattern" {
+            { Get-FabricWorkspace -WorkspaceName "Valid_Name-123" } | Should -Not -Throw
+            { Get-FabricWorkspace -WorkspaceName "Another Valid Name" } | Should -Not -Throw
+        }
+    }
+}