pass pester tests

alan-turing-institute · Aug 29, 2023 · d8e42fa · d8e42fa
1 parent c596614
commit d8e42fa
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 6 deletions.
diff --git a/deployment/administration/SHM_Delete_Unassigned_Users.ps1 b/deployment/administration/SHM_Delete_Unassigned_Users.ps1
@@ -21,12 +21,12 @@ $null = Set-AzContext -SubscriptionId $config.subscriptionName -ErrorAction Stop
 $script = "remote/Delete_Unassigned_Users.ps1"
 
 # Passing a param to a remote script requires it to be a string
-if ($dryRun.IsPresent){
+if ($dryRun.IsPresent) {
     Add-LogMessage -Level Info "Listing users not assigned to any security group from $($config.dc.vmName)..."
     $params = @{dryRun = "yes" }
 } else {
     Add-LogMessage -Level Info "Deleting users not assigned to any security group from $($config.dc.vmName)..."
-    $params = @{dryRun = "no"}
+    $params = @{dryRun = "no" }
 }
 $result = Invoke-RemoteScript -Shell "PowerShell" -ScriptPath $script -VMName $config.dc.vmName -ResourceGroupName $config.dc.rg -Parameter $params
 

diff --git a/deployment/administration/remote/Delete_Unassigned_Users.ps1 b/deployment/administration/remote/Delete_Unassigned_Users.ps1
@@ -25,11 +25,9 @@ if ($dryRun -eq "no") {
     try {
         Import-Module -Name "C:\Program Files\Microsoft Azure AD Sync\Bin\ADSync" -ErrorAction Stop
         Start-ADSyncSyncCycle -PolicyType Delta
-    }
-    catch [System.IO.FileNotFoundException] {
+    } catch [System.IO.FileNotFoundException] {
         Write-Output "Skipping as Azure AD Sync is not installed"
-    }
-    catch {
+    } catch {
         Write-Output "Unable to run Azure Active Directory synchronisation!"
     }
 }