UPDATE Tests

Existing pester tests ammended for Get-PASAccountGroup, Get-PASDirectory, Get-PASPSMConnectionParameter, New-PASSession. Relates to #109
pspete · Oct 31, 2018 · a503893 · a503893
1 parent e6d8ccb
commit a503893
Show file tree

Hide file tree

Showing 4 changed files with 62 additions and 5 deletions.
diff --git a/Tests/Get-PASAccountGroup.Tests.ps1 b/Tests/Get-PASAccountGroup.Tests.ps1
@@ -64,7 +64,7 @@ Describe $FunctionName {
 
 		}
 
-		$response = $InputObj | Get-PASAccountGroup -verbose
+		$response = $InputObj | Get-PASAccountGroup -UseV9API -verbose
 
 		Context "Input" {
 
@@ -100,6 +100,22 @@ Describe $FunctionName {
 				{$InputObj | Get-PASAccountGroup -ExternalVersion "1.0"} | Should Throw
 			}
 
+			It "sends request to expected endpoint - V10 ParameterSet" {
+
+				$InputObj | Get-PASAccountGroup -safe "SomeSafe" -verbose
+
+				Assert-MockCalled Invoke-PASRestMethod -ParameterFilter {
+
+					$URI -eq "$($InputObj.BaseURI)/$($InputObj.PVWAAppName)/API/Safes/SomeSafe/AccountGroups"
+
+				} -Times 1 -Exactly -Scope It
+
+			}
+
+			It "throws error if version requirement not met" {
+				{$InputObj | Get-PASAccountGroup -safe "SomeSafe" -ExternalVersion 1.1} | Should Throw
+			}
+
 		}
 
 		Context "Output" {

diff --git a/Tests/Get-PASDirectory.Tests.ps1 b/Tests/Get-PASDirectory.Tests.ps1
@@ -62,7 +62,7 @@ Describe $FunctionName {
 
 		}
 
-		$response = $InputObj | Get-PASDirectory
+		$response = $InputObj | Get-PASDirectory -id SomeDir
 
 		Context "Input" {
 
@@ -76,7 +76,7 @@ Describe $FunctionName {
 
 				Assert-MockCalled Invoke-PASRestMethod -ParameterFilter {
 
-					$URI -eq "$($InputObj.BaseURI)/$($InputObj.PVWAAppName)/api/Configuration/LDAP/Directories"
+					$URI -eq "$($InputObj.BaseURI)/$($InputObj.PVWAAppName)/api/Configuration/LDAP/Directories/SomeDir/"
 
 				} -Times 1 -Exactly -Scope Describe
 

diff --git a/Tests/Get-PASPSMConnectionParameter.Tests.ps1 b/Tests/Get-PASPSMConnectionParameter.Tests.ps1
@@ -49,6 +49,19 @@ Describe $FunctionName {
 
 		}
 
+		$AdHocObj = [pscustomobject]@{
+			"sessionToken"        = @{"Authorization" = "P_AuthValue"}
+			"WebSession"          = New-Object Microsoft.PowerShell.Commands.WebRequestSession
+			"BaseURI"             = "https://P_URI"
+			"PVWAAppName"         = "P_App"
+			"ConnectionComponent" = "SomeConnectionComponent"
+			"UserName"            = "SomeUser"
+			"secret"              = "SomeSecret" | ConvertTo-SecureString -AsPlainText -Force
+			"address"             = "Some.Address"
+			"platformID"          = "SomePlatform"
+
+		}
+
 		Context "Mandatory Parameters" {
 
 			$Parameters = @{Parameter = 'BaseURI'},
@@ -60,7 +73,7 @@ Describe $FunctionName {
 
 				param($Parameter)
 
-				(Get-Command Get-PASPSMConnectionParameter).Parameters["$Parameter"].Attributes.Mandatory | Should Be $true
+				(Get-Command Get-PASPSMConnectionParameter).Parameters["$Parameter"].Attributes.Mandatory | Select-Object -Unique | Should Be $true
 
 			}
 
@@ -76,7 +89,7 @@ Describe $FunctionName {
 
 			}
 
-			It "sends request to expected endpoint" {
+			It "sends request to expected endpoint for PSMConnect" {
 
 				Assert-MockCalled Invoke-PASRestMethod -ParameterFilter {
 
@@ -136,6 +149,21 @@ Describe $FunctionName {
 				{$InputObj | Get-PASPSMConnectionParameter -ConnectionMethod PSMGW -ExternalVersion "9.10"} | Should Throw
 			}
 
+			It "sends request to expected endpoint for AdHocConnect" {
+
+				$AdHocObj | Get-PASPSMConnectionParameter -ConnectionMethod RDP
+				Assert-MockCalled Invoke-PASRestMethod -ParameterFilter {
+
+					$URI -eq "$($InputObj.BaseURI)/$($InputObj.PVWAAppName)/API/Accounts/AdHocConnect"
+
+				} -Times 1 -Exactly -Scope It
+
+			}
+
+			It "throws error if version requirement not met for AdHocConnect" {
+				{$AdHocObj | Get-PASPSMConnectionParameter -ConnectionMethod PSMGW -ExternalVersion "10.4"} | Should Throw
+			}
+
 		}
 
 		Context "Output" {

diff --git a/Tests/New-PASSession.Tests.ps1 b/Tests/New-PASSession.Tests.ps1
@@ -173,6 +173,17 @@ Describe $FunctionName {
 
 			}
 
+			It "sends request to expected v10 URL for WINDOWS Authentication" {
+
+				$response = New-PASSession -BaseURI "https://P_URI" -UseDefaultCredentials
+				Assert-MockCalled Invoke-PASRestMethod -ParameterFilter {
+
+					$URI -eq "https://P_URI/PasswordVault/api/Auth/Windows/Logon"
+
+				} -Times 1 -Exactly -Scope It
+
+			}
+
 			It "calls Get-PASServer" {
 
 				$response = $Credentials | New-PASSession -BaseURI "https://P_URI" -type LDAP
@@ -187,6 +198,8 @@ Describe $FunctionName {
 
 			}
 
+
+
 		}
 
 		Context "Output" {