Fixed incorrect variable used for hooks with tags in Invoke-GherkinHo…

…ok. (pester#1400)

Examples/Gherkin/HookTag.Steps.ps1

@@ -0,0 +1,11 @@
+BeforeEachFeature -Tags FeatureHookTag {
+    # NOOP
+}
+
+BeforeEachFeature -Tags ScenarioHookTag {
+    # NOOP
+}
+
+Then "not much is going on" {
+    # NOOP
+}

Examples/Gherkin/HookTag.feature

@@ -0,0 +1,6 @@
+@FeatureHookTag
+Feature: A test feature that has a tag
+
+  @ScenarioHookTag
+  Scenario: A scenario that has another tag
+    Then not much is going on

Functions/Gherkin.Tests.ps1

@@ -698,3 +698,25 @@ Describe "A generated NUnit report" -Tag Gherkin {
     }
 
 }
+
+Describe "When hook has tags no errors should be thrown" -Tag Gherkin {
+    # Calling this in a job so we don't monkey with the active pester state that's already running
+    $job = Start-Job -ArgumentList $scriptRoot -ScriptBlock {
+        param ($scriptRoot)
+        Get-Module Pester | Remove-Module -Force
+        Import-Module $scriptRoot\Pester.psd1 -Force
+
+        New-Object psobject -Property @{
+            Results = Invoke-Gherkin (Join-Path $scriptRoot Examples\Gherkin\HookTag.feature) -PassThru -Show None -ErrorAction Stop
+        }
+    }
+
+    $gherkin = $job | Wait-Job | Receive-Job
+    Remove-Job $job
+
+    It 'Should have results' {
+        $gherkin | Should -Not -BeNullOrEmpty -Because "test were run with ErrorAction = Stop and no errors should be thrown"
+        $gherkin.Results | Should -Not -BeNullOrEmpty -Because "test were run with ErrorAction = Stop and no errors should be thrown"
+    }
+
+}

Functions/Gherkin.ps1

@@ -36,7 +36,7 @@ function Invoke-GherkinHook {
                 :tags foreach ($hookTag in $GherkinHook.Tags) {
                     foreach ($testTag in $Tags) {
                         if ($testTag -match "^($hookTag)$") {
-                            & $hook.Script $Name
+                            & $GherkinHook.Script $Name
                             break :tags
                         }
                     }