Skip to content

Commit b9fa74c

Browse files
author
Kapil Borle
committed
Add test case for module name extraction fix
1 parent 67c7919 commit b9fa74c

File tree

1 file changed

+18
-3
lines changed

1 file changed

+18
-3
lines changed

Tests/Engine/ModuleDependencyHandler.tests.ps1

Lines changed: 18 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,7 @@ Describe "Resolve DSC Resource Dependency" {
7272
{$moduleHandlerType::new($rsp)} | Should Throw
7373
$rsp.Dispose()
7474
}
75-
75+
7676
It "Extracts 1 module name" {
7777
$sb = @"
7878
{Configuration SomeConfiguration
@@ -84,9 +84,9 @@ Describe "Resolve DSC Resource Dependency" {
8484
$parseError = $null
8585
$ast = [System.Management.Automation.Language.Parser]::ParseInput($sb, [ref]$tokens, [ref]$parseError)
8686
$resultModuleNames = $moduleHandlerType::GetModuleNameFromErrorExtent($parseError[0], $ast).ToArray()
87-
$resultModuleNames[0] | Should Be 'SomeDscModule1'
87+
$resultModuleNames[0] | Should Be 'SomeDscModule1'
8888
}
89-
89+
9090
It "Extracts more than 1 module names" {
9191
$sb = @"
9292
{Configuration SomeConfiguration
@@ -102,6 +102,21 @@ Describe "Resolve DSC Resource Dependency" {
102102
$resultModuleNames[1] | Should Be 'SomeDscModule2'
103103
$resultModuleNames[2] | Should Be 'SomeDscModule3'
104104
}
105+
106+
107+
It "Extracts module names when ModuleName parameter is not the first named parameter" {
108+
$sb = @"
109+
{Configuration SomeConfiguration
110+
{
111+
Import-DscResource -Name SomeName -ModuleName SomeDscModule1
112+
}}
113+
"@
114+
$tokens = $null
115+
$parseError = $null
116+
$ast = [System.Management.Automation.Language.Parser]::ParseInput($sb, [ref]$tokens, [ref]$parseError)
117+
$resultModuleNames = $moduleHandlerType::GetModuleNameFromErrorExtent($parseError[0], $ast).ToArray()
118+
$resultModuleNames[0] | Should Be 'SomeDscModule1'
119+
}
105120
}
106121

107122
Context "Invoke-ScriptAnalyzer without switch" {

0 commit comments

Comments
 (0)