Proposed fix for Issue #413

CHANGELOG.md

@@ -5,6 +5,12 @@
 - Added Comment Based Help for `New-NotImplementedException` common
   function - fixes [Issue #411](https://github.com/PowerShell/NetworkingDsc/issues/411).
 
+- Added common function 'Format-Win32NetworkADapterFilterByNetConnectionID'
+  to properly accept wild cards for Win32_NetworkAdapter filters.
+
+- Updated MSFT_Netbios to use 'Format-Win32NetworkADapterFilterByNetConnectionID'
+  - fixes [Issue #413](https://github.com/PowerShell/NetworkingDsc/issues/413).
+
 ## 7.3.0.0
 
 - DnsClientGlobalSettings:

DSCResources/MSFT_NetBios/MSFT_NetBios.psm1

@@ -31,7 +31,7 @@ catch
     Returns the current state of the Net Bios on an interface.
 
 .PARAMETER InterfaceAlias
-    Specifies the alias of a network interface. Supports the use of '*'.
+    Specifies the alias of a network interface. Supports the use of '*' and '%'.
 
 .PARAMETER Setting
     Specifies if NetBIOS should be enabled or disabled or obtained from
@@ -57,9 +57,11 @@ function Get-TargetResource
 
     Write-Verbose -Message ($script:localizedData.GettingNetBiosSettingMessage -f $InterfaceAlias)
 
+    $Win32NetworkADapterFilter = Format-Win32NetworkADapterFilterByNetConnectionID -InterfaceAlias $InterfaceAlias
+
     $netAdapter = Get-CimInstance `
         -ClassName Win32_NetworkAdapter `
-        -Filter ('NetConnectionID="{0}"' -f $InterfaceAlias)
+        -Filter $Win32NetworkADapterFilter
 
     if ($netAdapter)
     {
@@ -99,7 +101,7 @@ function Get-TargetResource
     Sets the state of the Net Bios on an interface.
 
 .PARAMETER InterfaceAlias
-    Specifies the alias of a network interface. Supports the use of '*'.
+    Specifies the alias of a network interface. Supports the use of '*' and '%'.
 
 .PARAMETER Setting
     Specifies if NetBIOS should be enabled or disabled or obtained from
@@ -122,9 +124,11 @@ function Set-TargetResource
 
     Write-Verbose -Message ($script:localizedData.SettingNetBiosSettingMessage -f $InterfaceAlias)
 
+    $Win32NetworkADapterFilter = Format-Win32NetworkADapterFilterByNetConnectionID -InterfaceAlias $InterfaceAlias
+
     $netAdapter = Get-CimInstance `
         -ClassName Win32_NetworkAdapter `
-        -Filter ('NetConnectionID="{0}"' -f $InterfaceAlias)
+        -Filter $Win32NetworkADapterFilter
 
     if ($netAdapter)
     {
@@ -177,7 +181,7 @@ function Set-TargetResource
     Tests the current state the Net Bios on an interface.
 
 .PARAMETER InterfaceAlias
-    Specifies the alias of a network interface. Supports the use of '*'.
+    Specifies the alias of a network interface. Supports the use of '*' and '%'.
 
 .PARAMETER Setting
     Specifies if NetBIOS should be enabled or disabled or obtained from
@@ -201,9 +205,11 @@ function Test-TargetResource
 
     Write-Verbose -Message ($script:localizedData.TestingNetBiosSettingMessage -f $InterfaceAlias)
 
+    $Win32NetworkADapterFilter = Format-Win32NetworkADapterFilterByNetConnectionID -InterfaceAlias $InterfaceAlias
+
     $netAdapter = Get-CimInstance `
         -ClassName Win32_NetworkAdapter `
-        -Filter ('NetConnectionID="{0}"' -f $InterfaceAlias)
+        -Filter $Win32NetworkADapterFilter
 
     if ($netAdapter)
     {

DSCResources/MSFT_NetBios/MSFT_NetBios.schema.mof

@@ -2,6 +2,6 @@
 [ClassVersion("1.0.0.0"), FriendlyName("NetBios")]
 class MSFT_NetBios : OMI_BaseResource
 {
-    [Key, Description("Specifies the alias of a network interface.")] String InterfaceAlias;
+    [Key, Description("Specifies the alias of a network interface. Supports the use of '*' and '%'")] String InterfaceAlias;
     [Required, Description("Specifies if NetBIOS should be enabled or disabled or obtained from the DHCP server (Default). If static IP, Enable NetBIOS."), ValueMap{"Default","Enable","Disable"}, Values{"Default","Enable","Disable"}] String Setting;
 };

Examples/Resources/NetBios/2-NetBios_DisableNetBios_Config_Wildcard.ps1

@@ -0,0 +1,36 @@
+<#PSScriptInfo
+.VERSION 1.0.0
+.GUID b43f8fac-cc5a-45ec-ab7d-344dff0b31e7
+.AUTHOR Mike Kletz
+.COMPANYNAME
+.COPYRIGHT
+.TAGS DSCConfiguration
+.LICENSEURI https://github.com/PowerShell/NetworkingDsc/blob/master/LICENSE
+.PROJECTURI https://github.com/PowerShell/NetworkingDsc
+.ICONURI
+.EXTERNALMODULEDEPENDENCIES
+.REQUIREDSCRIPTS
+.EXTERNALSCRIPTDEPENDENCIES
+.RELEASENOTES First version.
+.PRIVATEDATA 2016-Datacenter,2016-Datacenter-Server-Core
+#>
+
+#Requires -module NetworkingDsc
+
+<#
+    .DESCRIPTION
+    Disable NetBios on all adapters.
+#>
+Configuration NetBios_DisableNetBios_Config_Wildcard
+{
+    Import-DscResource -ModuleName NetworkingDsc
+
+    Node localhost
+    {
+        NetBios DisableNetBios
+        {
+            InterfaceAlias = '*'
+            Setting        = 'Disable'
+        }
+    }
+}

Modules/NetworkingDsc.Common/NetworkingDsc.Common.psm1

@@ -1365,6 +1365,43 @@ function ConvertTo-HashTable
     }
 }
 
+<#
+.SYNOPSIS
+    Returns a filter string for the net adapter CIM instances. Will detect wild cards and replace them and the operator accordingly.
+
+.PARAMETER InterfaceAlias
+    Specifies the alias of a network interface. Supports the use of '*' or '%'.
+#>
+function Format-Win32NetworkADapterFilterByNetConnectionID
+{
+[CmdletBinding()]
+    param
+    (
+        [Parameter(Mandatory = $true)]
+        [System.String]
+        $InterfaceAlias
+    )
+
+    process
+    {
+        if($InterfaceAlias.contains('*'))
+        {
+            $InterfaceAlias = $InterfaceAlias.replace('*','%')
+        }
+
+        if($InterfaceAlias.contains('%'))
+        {
+            $operator = ' LIKE '
+        }
+        else
+        {
+            $operator = '='
+        }
+
+        'NetConnectionID{0}"{1}"' -f $operator,$InterfaceAlias
+    }
+}
+
 # Import Localization Strings
 $script:localizedData = Get-LocalizedData `
     -ResourceName 'NetworkingDsc.Common' `
@@ -1388,5 +1425,6 @@ Export-ModuleMember -Function @(
     'Test-DscParameterState',
     'Test-DscObjectHasProperty'
     'ConvertTo-HashTable',
-    'ConvertTo-CimInstance'
+    'ConvertTo-CimInstance',
+    'Format-Win32NetworkADapterFilterByNetConnectionID'
 )

Tests/Unit/NetworkingDsc.Common.tests.ps1

@@ -2240,6 +2240,51 @@ try
                 }
             }
         }
+
+        Describe 'NetworkingDsc.Common\Format-Win32NetworkADapterFilterByNetConnectionID'{
+
+            Context 'When interface alias has an ''*''' {
+
+                $interfaceAlias = "Ether*"
+
+                It 'Should convert the ''*'' to a ''%''' {
+                    (Format-Win32NetworkADapterFilterByNetConnectionID -InterfaceAlias $interfaceAlias).contains('%') -eq $True -and
+                    (Format-Win32NetworkADapterFilterByNetConnectionID -InterfaceAlias $interfaceAlias).contains('*') -eq $False | Should -Be $True
+                }
+
+                It 'Should change the operator to ''LIKE''' {
+                    (Format-Win32NetworkADapterFilterByNetConnectionID -InterfaceAlias $interfaceAlias) -like "NetConnectionID LIKE *" | Should -Be $True
+                }
+
+                It 'Looks like a usable filter' {
+                    Format-Win32NetworkADapterFilterByNetConnectionID -InterfaceAlias $interfaceAlias | Should -Be 'NetConnectionID LIKE "Ether%"'
+                }
+
+            }
+
+            Context 'When interface alias has a ''%''' {
+
+                $interfaceAlias = "Ether%"
+
+                It 'Should change the operator to ''LIKE''' {
+                    (Format-Win32NetworkADapterFilterByNetConnectionID -InterfaceAlias $interfaceAlias) -like "NetConnectionID LIKE *" | Should -Be $True
+                }
+
+                It 'Looks like a usable filter' {
+                    Format-Win32NetworkADapterFilterByNetConnectionID -InterfaceAlias $interfaceAlias | Should -Be 'NetConnectionID LIKE "Ether%"'
+                }
+
+            }
+
+            Context 'When interface alias has no wildcards' {
+
+                $interfaceAlias = "Ethernet"
+
+                It 'Looks like a usable filter' {
+                    Format-Win32NetworkADapterFilterByNetConnectionID -InterfaceAlias $interfaceAlias | Should -Be 'NetConnectionID="Ethernet"'
+                }
+            }
+        }
     }
 }
 finally