AdReplicationSite: Adding description attribute, will create integration tests

CHANGELOG.md

@@ -96,6 +96,10 @@
     needs the feature Recycle Bin enabled ([issue #498](https://github.com/PowerShell/xActiveDirectory/issues/498)).
   - Updated integration test to be able to catch when a computer account
     cannot be restored.
+- Changes to ADReplicationSite
+  - Added 'Description' attribute parameter ([issue #500](https://github.com/PowerShell/ActiveDirectoryDsc/issues/500)).
+  - Added Integration testing ([issue #355](https://github.com/PowerShell/ActiveDirectoryDsc/issues/355)).
+  - Correct value returned for RenameDefaultFirstSiteName ([issue #502](https://github.com/PowerShell/ActiveDirectoryDsc/issues/502)).
 
 ## 4.0.0.0
 

DSCResources/MSFT_ADReplicationSite/MSFT_ADReplicationSite.psm1

@@ -27,15 +27,16 @@ function Get-TargetResource
     # Get the replication site filtered by it's name. If the site is not
     # present, the command will return $null.
     Write-Verbose -Message ($script:localizedData.GetReplicationSite -f $Name)
-    $replicationSite = Get-ADReplicationSite -Filter { Name -eq $Name }
+    $replicationSite = Get-ADReplicationSite -Filter { Name -eq $Name } -ErrorAction SilentlyContinue
 
     if ($null -eq $replicationSite)
     {
         Write-Verbose -Message ($script:localizedData.ReplicationSiteAbsent -f $Name)
         $returnValue = @{
             Ensure                     = 'Absent'
             Name                       = $Name
-            RenameDefaultFirstSiteName = ''
+            Description                = $null
+            RenameDefaultFirstSiteName = $false
         }
     }
     else
@@ -44,7 +45,8 @@ function Get-TargetResource
         $returnValue = @{
             Ensure                     = 'Present'
             Name                       = $Name
-            RenameDefaultFirstSiteName = ''
+            Description                = $replicationSite.Description
+            RenameDefaultFirstSiteName = $false
         }
     }
 
@@ -82,12 +84,18 @@ function Set-TargetResource
 
         [Parameter()]
         [System.Boolean]
-        $RenameDefaultFirstSiteName = $false
+        $RenameDefaultFirstSiteName = $false,
+
+        [Parameter()]
+        [System.String]
+        $Description
     )
 
+    $createOrUpdate = Get-TargetResource $Name
+
     if ($Ensure -eq 'Present')
     {
-        $defaultFirstSiteName = Get-ADReplicationSite -Filter { Name -eq 'Default-First-Site-Name' }
+        $defaultFirstSiteName = Get-ADReplicationSite -Filter { Name -eq 'Default-First-Site-Name' } -ErrorAction SilentlyContinue
 
         <#
             Check if the user specified to rename the Default-First-Site-Name
@@ -102,9 +110,28 @@ function Set-TargetResource
         }
         else
         {
-            Write-Verbose -Message ($script:localizedData.AddReplicationSite -f $Name)
+            if ($createOrUpdate.Ensure -eq 'Absent')
+            {
+                Write-Verbose -Message ($script:localizedData.AddReplicationSite -f $Name)
+
+                $newADReplicationSiteParameters = @{
+                    Name        = $Name
+                    ErrorAction = 'Stop'
+                }
+
+                if($PSBoundParameters.ContainsKey('Description'))
+                {
+                    $newADReplicationSiteParameters['Description'] = $Description
+                }
+
+                New-ADReplicationSite @newADReplicationSiteParameters
+            }
+        }
 
-            New-ADReplicationSite -Name $Name -ErrorAction Stop
+        if($PSBoundParameters.ContainsKey('Description') -and ($createOrUpdate.Ensure -eq 'Present' -or $RenameDefaultFirstSiteName))
+        {
+            Write-Verbose -Message ($script:localizedData.UpdateReplicationSite -f $Name)
+            Set-ADReplicationSite -Identity $Name -Description $Description
         }
     }
 
@@ -130,6 +157,11 @@ function Set-TargetResource
     .PARAMETER RenameDefaultFirstSiteName
         Specify if the Default-First-Site-Name should be renamed, if it exists.
         Dafult value is 'false'.
+
+    .PARAMETER Description
+        Specifies a description of the object. This parameter sets the value of
+        the Description property for the object. The LDAP Display Name
+        (ldapDisplayName) for this property is 'description'.
 #>
 function Test-TargetResource
 {
@@ -148,18 +180,53 @@ function Test-TargetResource
 
         [Parameter()]
         [System.Boolean]
-        $RenameDefaultFirstSiteName = $false
+        $RenameDefaultFirstSiteName = $false,
+
+        [Parameter()]
+        [System.String]
+        $Description
     )
 
     $currentConfiguration = Get-TargetResource -Name $Name
+    $configurationCompliant = $true
 
-    if ($currentConfiguration.Ensure -eq $Ensure)
+    if ($currentConfiguration.Ensure -eq 'Absent')
     {
-        Write-Verbose -Message ($script:localizedData.ReplicationSiteInDesiredState -f $Name)
+        # Site doesn't exist
+        if ($currentConfiguration.Ensure -eq $Ensure)
+        {
+            # Site should not exist
+            Write-Verbose -Message ($script:localizedData.ReplicationSiteInDesiredState -f $Name)
+        }
+        else
+        {
+            #Site should exist
+            Write-Verbose -Message ($script:localizedData.ReplicationSiteNotInDesiredState -f $Name)
+            $configurationCompliant = $false
+        }
     }
-    else
-    {
-        Write-Verbose -Message ($script:localizedData.ReplicationSiteNotInDesiredState -f $Name)
+    else {
+        # Site Exists
+        if ($currentConfiguration.Ensure -eq $Ensure)
+        {
+            # Site should exist
+            if ($currentConfiguration.Description -ne $Description)
+            {
+                Write-Verbose -Message ($script:localizedData.ReplicationSiteNotInDesiredState -f $Name)
+                $configurationCompliant = $false
+            }
+            else
+            {
+                Write-Verbose -Message ($script:localizedData.ReplicationSiteInDesiredState -f $Name)
+            }
+        }
+        else
+        {
+            # Site should not exist
+            Write-Verbose -Message ($script:localizedData.ReplicationSiteNotInDesiredState -f $Name)
+            $configurationCompliant = $false
+        }
     }
-    return $currentConfiguration.Ensure -eq $Ensure
+
+    return $configurationCompliant
 }

DSCResources/MSFT_ADReplicationSite/MSFT_ADReplicationSite.schema.mof

@@ -4,4 +4,5 @@ class MSFT_ADReplicationSite : OMI_BaseResource
     [Write, Description("Specifies if the Active Directory replication site should be present or absent. Default value is 'Present'."), ValueMap{"Present", "Absent"}, Values{"Present", "Absent"}] String Ensure;
     [Key, Description("Specifies the name of the Active Directory replication site.")] String Name;
     [Write, Description("Specifies if the Default-First-Site-Name should be renamed if it exists. Default value is $false.")] Boolean RenameDefaultFirstSiteName;
+    [Write, Description("Specifies a description of the object. This parameter sets the value of the Description property for the object. The LDAP Display Name (ldapDisplayName) for this property is 'description'.")] String Description;
 };

DSCResources/MSFT_ADReplicationSite/en-US/MSFT_ADReplicationSite.strings.psd1

@@ -8,4 +8,5 @@ ConvertFrom-StringData @'
     ReplicationSitePresent                 = Replication site '{0}' is present. (ADRS0006)
     ReplicationSiteInDesiredState          = The replication site '{0}' is in the desired state. (ADRS0007)
     ReplicationSiteNotInDesiredState       = The replication site '{0}' is not in the desired state. (ADRS0008)
+    UpdateReplicationSite                  = The replication site '{0}' needs to be updated.
 '@

DSCResources/MSFT_ADReplicationSite/en-US/about_ADReplicationSite.help.txt

@@ -21,6 +21,10 @@
     Write - Boolean
     Specifies if the Default-First-Site-Name should be renamed if it exists. Default value is $false.
 
+.PARAMETER Description
+    Write - String
+    Specifies a description of the object. This parameter sets the value of the Description property for the object. The LDAP Display Name (ldapDisplayName) for this property is 'description'.
+
 .EXAMPLE 1
 
 This configuration will create an Active Directory replication site
@@ -80,4 +84,22 @@ Configuration ADReplicationSite_RemoveADReplicationSite_Config
     }
 }
 
+.EXAMPLE 4
+
+This configuration will set the AD Relication Site description for
+a site called SeattleSite
+
+Configuration ADReplicationSite_CreateADReplicationSite_Config
+{
+    Import-DscResource -Module ActiveDirectoryDsc
 
+    Node localhost
+    {
+        ADReplicationSite 'SeattleSite'
+        {
+            Ensure      = 'Present'
+            Name        = 'Seattle'
+            Description = 'Seattle Head Office'
+        }
+    }
+}