chore: add tests for failing hooks

Author: daantimmer

cucumber_cpp/acceptance_test/features/test_hooks.feature

@@ -1,23 +1,37 @@
 @bats
 Feature: Test scenario and step hook bindings
 
-  Background:
-    Given a background step
-
-  @scenariohook @stephook
-  Scenario: Run Scenario and Step hooks
+  @fail_scenariohook_before
+  Scenario: Run failing Scenario hooks before
     Given a given step
-    When a when step
-    Then a then step
 
-  @scenariohook
-  Scenario: Run only Scenario hooks
+  @fail_scenariohook_after
+  Scenario: Run failing Scenario hooks after
     Given a given step
-    When a when step
-    Then a then step
 
-  @stephook
-  Scenario: Run only Step hooks
+  @throw_scenariohook
+  Scenario: Run throwing Scenario hooks
     Given a given step
-    When a when step
-    Then a then step
+
+  Rule: These scenarios have a background
+
+    Background:
+      Given a background step
+
+    @scenariohook @stephook
+    Scenario: Run Scenario and Step hooks
+      Given a given step
+      When a when step
+      Then a then step
+
+    @scenariohook
+    Scenario: Run only Scenario hooks
+      Given a given step
+      When a when step
+      Then a then step
+
+    @stephook
+    Scenario: Run only Step hooks
+      Given a given step
+      When a when step
+      Then a then step

cucumber_cpp/acceptance_test/hooks/Hooks.cpp

@@ -1,4 +1,5 @@
 #include "cucumber_cpp/library/Hooks.hpp"
+#include "gmock/gmock.h"
 #include <iostream>
 
 HOOK_BEFORE_ALL()
@@ -30,3 +31,18 @@ HOOK_AFTER_STEP("@stephook and @bats")
 {
     std::cout << "HOOK_AFTER_STEP\n";
 }
+
+HOOK_BEFORE_SCENARIO("@fail_scenariohook_before")
+{
+    ASSERT_THAT(false, testing::IsTrue());
+}
+
+HOOK_AFTER_SCENARIO("@fail_scenariohook_after")
+{
+    ASSERT_THAT(false, testing::IsTrue());
+}
+
+HOOK_BEFORE_SCENARIO("@throw_scenariohook")
+{
+    throw "error";
+}

cucumber_cpp/acceptance_test/test.bats

@@ -154,3 +154,25 @@ teardown() {
     assert_failure
     assert_output --partial "tests   : 1/2 passed"
 }
+
+@test "Test failing hook before results in error" {
+    run .build/Host/cucumber_cpp/acceptance_test/Debug/cucumber_cpp.acceptance_test run --feature cucumber_cpp/acceptance_test/features  --tag "@fail_scenariohook_before" --report console
+    assert_failure
+    assert_output --partial "skipped Given a given step"
+    assert_output --partial "tests   : 0/1 passed"
+}
+
+@test "Test failing hook after results in error" {
+    run .build/Host/cucumber_cpp/acceptance_test/Debug/cucumber_cpp.acceptance_test run --feature cucumber_cpp/acceptance_test/features  --tag "@fail_scenariohook_after" --report console
+    assert_failure
+    assert_output --partial "Given a given step"
+    assert_output --partial "-> done"
+    assert_output --partial "tests   : 0/1 passed"
+}
+
+@test "Test throwing hook results in error" {
+    run .build/Host/cucumber_cpp/acceptance_test/Debug/cucumber_cpp.acceptance_test run --feature cucumber_cpp/acceptance_test/features  --tag "@throw_scenariohook" --report console
+    assert_failure
+    assert_output --partial "skipped Given a given step"
+    assert_output --partial "tests   : 0/1 passed"
+}