Review PR

.gitignore

@@ -0,0 +1 @@
+DSCResource.Tests

DSCResources/Helper.psm1

@@ -0,0 +1,64 @@
+<#
+    .SYNOPSIS
+        Tests the $Target parameter to ensure it contains the RootDSE and adds it if it doesn't.
+
+    .PARAMETER Target
+        The target OU Distinguished Name with or without the RootDSE.
+
+    .PARAMETER Domain
+        Not used. Included to allow @PSBoundParameters in function call.
+
+    .PARAMETER Server
+        The name of the server. Optional.
+
+    .PARAMETER Ensure
+        Not used. Included to allow @PSBoundParameters in function call.
+
+#>
+function Test-TargetDN
+{
+    param
+    (
+        [parameter(Mandatory = $true)]
+        [string]
+        $Target,
+
+        [string]
+        $Domain,
+
+        [string]
+        $Server,
+
+        [ValidateSet('Present','Absent')]
+        [string]
+        $Ensure = 'Present'
+    )
+
+    $getADDomainParams = @{}
+    if ($Server)
+    {
+        $getADDomainParams += @{
+            Server = $Server
+        }
+    }
+    Write-Verbose -Message ("Checking the Domain Distinguished Name is " +
+        "present on the Target Distinguished Name.")
+    $domainDN = (Get-ADDomain @getADDomainParams).DistinguishedName
+    if ($Target -like "*$domainDN")
+    {
+        Write-Verbose -Message "Target has full DN."
+    }
+    else
+    {
+        Write-Verbose -Message "Adding the Domain Distinguished Name to the Target DN."
+        if ($Target.EndsWith(","))
+        {
+            $Target = "$Target$domainDN"
+        }
+        else
+        {
+            $Target = "$Target,$domainDN"
+        }
+    }
+    Write-Output -InputObject $Target
+}

DSCResources/MSFT_GPInheritance/MSFT_GPInheritance.psm1

@@ -0,0 +1,156 @@
+function Get-TargetResource
+{
+    [OutputType([Hashtable])]
+    param
+    (
+        [parameter(Mandatory = $true)]
+        [string]
+        $Target,
+
+        [string]
+        $Domain,
+
+        [string]
+        $Server,
+
+        [ValidateSet('Present','Absent')]
+        [string]
+        $Ensure = 'Present'
+    )
+
+    Import-Module $PSScriptRoot\..\Helper.psm1 -Verbose:$false
+    Import-Module -Name GroupPolicy -Verbose:$false
+    $Target = Test-TargetDN @PSBoundParameters
+    $targetResource =  @{
+        Target = $Target
+        Domain = $null
+        Server = $null
+        Ensure = $null
+    }
+    $getGPInheritanceParams = @{
+        Target = $Target
+    }
+    if ($Domain)
+    {
+        $targetResource.Domain = $Domain
+        $getGPInheritanceParams += @{
+            Domain = $Domain
+        }
+    }
+    if ($Server)
+    {
+        $targetResource.Server = $Server
+        $getGPInheritanceParams += @{
+            Server = $Server
+        }
+    }
+    Write-Verbose -Message 'Getting Group Policy Inheritance'
+    $gpoInheritanceBlocked = (Get-GPInheritance @getGPInheritanceParams).GpoInheritanceBlocked
+    if (!$gpoInheritanceBlocked)
+    {
+        $targetResource.Ensure = 'Present'
+    }
+    else
+    {
+        $targetResource.Ensure = 'Absent'
+    }
+    Write-Output -InputObject $targetResource
+}
+
+function Set-TargetResource
+{
+    param
+    (
+        [parameter(Mandatory = $true)]
+        [string]
+        $Target,
+
+        [string]
+        $Domain,
+
+        [string]
+        $Server,
+
+        [ValidateSet('Present','Absent')]
+        [string]
+        $Ensure = 'Present'
+    )
+
+    if ($Ensure -eq 'Present')
+    {
+        Write-Verbose -Message 'Enabling Group Policy Inheritance'
+        $isBlocked = 'No'
+    }
+    else
+    {
+        Write-Verbose -Message 'Disabling Group Policy Inheritance'
+        $isBlocked = 'Yes'
+    }
+    $Target = Test-TargetDN @PSBoundParameters
+    $setGPInheritanceParams = @{
+        Target = $Target
+        IsBlocked = $IsBlocked
+    }
+    if ($Domain)
+    {
+        $setGPInheritanceParams += @{
+            Domain = $Domain
+        }
+    }
+    if ($Server)
+    {
+        $setGPInheritanceParams += @{
+            Server = $Server
+        }
+    }
+    Import-Module -Name GroupPolicy -Verbose:$false
+    $null = Set-GPInheritance @setGPInheritanceParams
+}
+
+function Test-TargetResource
+{
+[OutputType([Boolean])]
+    param
+    (
+        [parameter(Mandatory = $true)]
+        [string]
+        $Target,
+
+        [string]
+        $Domain,
+
+        [string]
+        $Server,
+
+        [ValidateSet('Present','Absent')]
+        [string]
+        $Ensure = 'Present'
+    )
+
+    $targetResource = Get-TargetResource @PSBoundParameters
+    switch ($Ensure)
+    {
+        Present
+        {
+            if ($targetResource.Ensure -eq 'Present')
+            {
+                Write-Output -InputObject $true
+            }
+            else
+            {
+                Write-Output -InputObject $false
+            }
+        }
+        Absent
+        {
+            if ($targetResource.Ensure -eq 'Absent')
+            {
+                Write-Output -InputObject $true
+            }
+            else
+            {
+                Write-Output -InputObject $false
+            }
+        }
+    }
+}

DSCResources/MSFT_GPInheritance/MSFT_GPInheritance.schema.mof

@@ -0,0 +1,8 @@
+[ClassVersion("1.0.0"), FriendlyName("GPInheritance")]
+class MSFT_GPInheritance :  OMI_BaseResource
+{
+    [Key] String Target;
+    [write] String Domain;
+    [write] String Server;
+    [write,ValueMap{"Present", "Absent"},Values{"Present", "Absent"}] string Ensure;
+};