Merge pull request #30 from wcarroll/Hackathon

Hackathon

Tests/Public/Get-ZertoAlert.Tests.ps1

@@ -20,10 +20,65 @@ Describe $file.BaseName -Tag 'Unit' {
     Context "$($file.BaseName)::Parameter Unit Tests" {
 
         it "Has a mandatory string parameter for the Alert identifier" {
-            Get-Command $file.BaseName | Should -HaveParameter alertId -Mandatory -Type String[]
+            Get-Command $file.BaseName | Should -HaveParameter alertId
+            Get-Command $file.BaseName | Should -HaveParameter alertId -Mandatory
+            Get-Command $file.BaseName | Should -HaveParameter alertId -Type String[]
         }
 
-    }
+        it "Has a non-mandatory switch parameter for the entities" {
+            Get-Command $file.BaseName | Should -HaveParameter entities
+            Get-Command $file.BaseName | Should -HaveParameter entities -Type switch
+        }
 
+        it "Has a non-mandatory switch parameter for the helpIdentifiers" {
+            Get-Command $file.BaseName | Should -HaveParameter helpIdentifiers
+            Get-Command $file.BaseName | Should -HaveParameter helpIdentifiers -Type switch
+        }
+
+        it "Has a non-mandatory switch parameter for the levels" {
+            Get-Command $file.BaseName | Should -HaveParameter levels
+            Get-Command $file.BaseName | Should -HaveParameter levels -Type switch
+        }
+
+        it "Has a non-mandatory string parameter for the startDate" {
+            Get-Command $file.BaseName | Should -HaveParameter startDate
+            Get-Command $file.BaseName | Should -HaveParameter startDate -Type string
+        }
+
+        it "Has a non-mandatory string parameter for the endDate" {
+            Get-Command $file.BaseName | Should -HaveParameter endDate
+            Get-Command $file.BaseName | Should -HaveParameter endDate -Type string
+        }
+
+        it "Has a non-mandatory string parameter for the vpgIdentifier" {
+            Get-Command $file.BaseName | Should -HaveParameter vpgIdentifier
+            Get-Command $file.BaseName | Should -HaveParameter vpgIdentifier -Type string
+        }
+
+        it "Has a non-mandatory string parameter for the siteIdentifier" {
+            Get-Command $file.BaseName | Should -HaveParameter siteIdentifier
+            Get-Command $file.BaseName | Should -HaveParameter siteIdentifier -Type string
+        }
+
+        it "Has a non-mandatory string parameter for the zorgIdentifier" {
+            Get-Command $file.BaseName | Should -HaveParameter zorgIdentifier
+            Get-Command $file.BaseName | Should -HaveParameter zorgIdentifier -Type string
+        }
+
+        it "Has a non-mandatory string parameter for the level" {
+            Get-Command $file.BaseName | Should -HaveParameter level
+            Get-Command $file.BaseName | Should -HaveParameter level -Type string
+        }
+
+        it "Has a non-mandatory string parameter for the helpIdentifier" {
+            Get-Command $file.BaseName | Should -HaveParameter helpIdentifier
+            Get-Command $file.BaseName | Should -HaveParameter helpIdentifier -Type string
+        }
+
+        it "Has a non-mandatory bool parameter for the isDismissed" {
+            Get-Command $file.BaseName | Should -HaveParameter isDismissed
+            Get-Command $file.BaseName | Should -HaveParameter isDismissed -Type bool
+        }
+    }
 }
 

Tests/Public/Get-ZertoDatastore.Tests.ps1

@@ -16,4 +16,9 @@ Describe $file.BaseName -Tag 'Unit' {
         $null = [System.Management.Automation.PSParser]::Tokenize($contents, [ref]$errors)
         $errors | Should -HaveCount 0
     }
+
+    It "has a non-mandatory string parameter for the datacenterIdentifier" {
+        Get-Command $file.BaseName | Should -HaveParameter datastoreIdentifier
+        Get-Command $file.BaseName | Should -HaveParameter datastoreIdentifier -Type String[]
+    }
 }

Tests/Public/Invoke-ZertoMoveCommit.Tests.ps1

@@ -16,4 +16,13 @@ Describe $file.BaseName -Tag 'Unit' {
         $null = [System.Management.Automation.PSParser]::Tokenize($contents, [ref]$errors)
         $errors | Should -HaveCount 0
     }
+
+    Context "$($file.BaseName)::Parameter Unit Tests" {
+
+        it "has a mandatory string parameter for the vpgName" {
+            Get-Command $file.BaseName | Should -HaveParameter vpgName
+            Get-Command $file.BaseName | Should -HaveParameter vpgName -Type string[]
+            Get-Command $file.BaseName | Should -HaveParameter vpgName -Mandatory
+        }
+    }
 }

Tests/Public/Invoke-ZertoMoveRollback.Tests.ps1

@@ -16,4 +16,13 @@ Describe $file.BaseName -Tag 'Unit' {
         $null = [System.Management.Automation.PSParser]::Tokenize($contents, [ref]$errors)
         $errors | Should -HaveCount 0
     }
+
+    Context "$($file.BaseName)::Parameter Unit Tests" {
+
+        it "has a mandatory string parameter for the vpgName" {
+            Get-Command $file.BaseName | Should -HaveParameter vpgName
+            Get-Command $file.BaseName | Should -HaveParameter vpgName -Type string[]
+            Get-Command $file.BaseName | Should -HaveParameter vpgName -Mandatory
+        }
+    }
 }

ZertoApiWrapper/Public/Get-ZertoAlert.ps1

@@ -43,7 +43,7 @@ function Get-ZertoAlert {
         [string]$endDate,
         [Parameter(
             ParameterSetName = "filter",
-            HelpMessage = "Returns alerts for the specified vraIdentifier"
+            HelpMessage = "Returns alerts for the specified vpgIdentifier"
         )]
         [ValidateNotNullOrEmpty()]
         [Alias("vpgId")]