Tests for AvoidPositionalParameters rule fixes.

Author: kalgiz

Tests/Rules/AvoidPositionalParameters.tests.ps1

@@ -21,26 +21,28 @@ Describe "AvoidPositionalParameters" {
         It "returns no violations" {
             $noViolations.Count | Should -Be 0
         }
-        
+
         It "returns no violations for DSC configuration" {
             $noViolationsDSC.Count | Should -Be 0
         }
     }
 
     Context "Function defined and called in script, which has 3 or more positional parameters triggers rule." {
-        $sb=
-        {
-            Function Foo {
-             param(
-                [Parameter(Mandatory=$true,Position=1)] $A,
-                [Parameter(Position=2)]$B,
-                [Parameter(Position=3)]$C)
+        It "returns avoid positional parameters violation" {
+            $sb=
+            {
+                Function Foo {
+                param(
+                    [Parameter(Mandatory=$true,Position=1)] $A,
+                    [Parameter(Position=2)]$B,
+                    [Parameter(Position=3)]$C)
+                }
+                Foo "a" "b" "c"
             }
-            Foo "a" "b" "c"
+            $warnings = Invoke-ScriptAnalyzer -ScriptDefinition "$sb"
+            $warnings.Count | Should -BeGreaterThan 0
+            $warnings.RuleName | Should -Contain "PSAvoidUsingPositionalParameters"
+            $warnings.Message | Should -Contain "Cmdlet 'Foo' has positional parameter. Please use named parameters instead of positional parameters when calling a command."
         }
-        $warnings = Invoke-ScriptAnalyzer -ScriptDefinition "$sb"
-        $warnings.Count | Should -BeGreaterThan 0
-        $warning.RuleName | Should -Contain "PSAvoidUsingPositionalParameters"
-        $warnings.Message | Should -Contain "Cmdlet 'Foo' has positional parameter. Please use named parameters instead of positional parameters when calling a command."
     }
 }