Fixes SiteName beeing required (#126)

* Adds unit test for this scenario
* Add chack if sitename was specified or not
* Fixes #119

DSCResources/MSFT_xADDomainController/MSFT_xADDomainController.psm1

@@ -147,7 +147,7 @@ function Set-TargetResource
     elseif ($targetResource.Ensure)
     {
         ## Node is a domain controller. We check if other properties are in desired state
-        if ($targetResource.SiteName -ne $SiteName)
+        if ($PSBoundParameters["SiteName"] -and $targetResource.SiteName -ne $SiteName)
         {
             ## DC is not in correct site. Move it.
             Write-Verbose "Moving Domain Controller from '$($targetResource.SiteName)' to '$SiteName'"

README.md

@@ -243,6 +243,7 @@ Setting an ODJ Request file path for a configuration that creates a computer acc
 ## Versions
 
 ### Unreleased
+* xAdDomainController: Fixes SiteName being required field.
 
 ### 2.14.0.0
 * xADDomainController: Adds Site option.

Tests/Unit/MSFT_xADDomainController.Tests.ps1

@@ -217,6 +217,21 @@ try
 
             Assert-MockCalled Move-ADDirectoryServer -Times 0 @commonAssertParams
         }
+
+        It 'Does not call "Move-ADDirectoryServer" when "SiteName" is not specified' {
+            Mock Get-TargetResource {
+                return $stubTargetResource = @{
+                    Ensure = $true
+                    SiteName = 'PresentSite'
+                }
+            } @commonMockParams
+
+            Mock Move-ADDirectoryServer {} @commonMockParams
+
+            Set-TargetResource @testDefaultParams -DomainName $correctDomainName
+
+            Assert-MockCalled Move-ADDirectoryServer -Times 0 @commonAssertParams
+        }
     }
     #endregion
 }