fix bats-acceptance tests

Author: daantimmer

.github/workflows/static-analysis.yml

@@ -44,7 +44,7 @@ jobs:
 
       - name: Run acceptance tests
         run: |
-          bats --formatter junit cucumber_cpp/acceptance_test/coverage.bats | tee test-report.xml
+          bats --formatter junit cucumber_cpp/acceptance_test/test.bats | tee test-report.xml
 
       - name: Collect coverage
         run: |

compatibility/compatibility.cpp

@@ -2,10 +2,11 @@
 #include "cucumber_cpp/CucumberCpp.hpp"
 #include "cucumber_cpp/library/api/RunCucumber.hpp"
 #include "cucumber_cpp/library/cucumber_expression/ParameterRegistry.hpp"
+#include "cucumber_cpp/library/support/Duration.hpp"
 #include "cucumber_cpp/library/support/Timestamp.hpp"
 #include "cucumber_cpp/library/support/Types.hpp"
+#include "cucumber_cpp/library/tag_expression/Parser.hpp"
 #include "cucumber_cpp/library/util/Broadcaster.hpp"
-#include "library/support/Duration.hpp"
 #include "nlohmann/json.hpp"
 #include "nlohmann/json_fwd.hpp"
 #include "gmock/gmock.h"
@@ -248,7 +249,7 @@ namespace compatibility
             cucumber_cpp::library::support::RunOptions runOptions{
                 .sources = {
                     .paths = devkit.paths,
-                    .tagExpression = devkit.tagExpression,
+                    .tagExpression = cucumber_cpp::library::tag_expression::Parse(devkit.tagExpression),
                     .ordering = isReversed ? cucumber_cpp::library::support::RunOptions::Ordering::reverse : cucumber_cpp::library::support::RunOptions::Ordering::defined,
                 },
                 .runtime = {

cucumber_cpp/acceptance_test/coverage.bats

@@ -1,5 +1,6 @@
 #include "cucumber_cpp/CucumberCpp.hpp"
 #include "gmock/gmock.h"
+#include <gtest/gtest.h>
 #include <iostream>
 #include <string>
 
@@ -8,7 +9,7 @@ HOOK_BEFORE_ALL()
     std::cout << "HOOK_BEFORE_ALL\n";
 
     if (context.Contains("--failprogramhook") && context.Get<bool>("--failprogramhook"))
-        ASSERT_THAT(false, testing::IsTrue());
+        FAIL();
 }
 
 HOOK_AFTER_ALL()
@@ -38,12 +39,12 @@ HOOK_AFTER_STEP("@stephook and @bats")
 
 HOOK_BEFORE_SCENARIO("@fail_scenariohook_before")
 {
-    ASSERT_THAT(false, testing::IsTrue());
+    FAIL();
 }
 
 HOOK_AFTER_SCENARIO("@fail_scenariohook_after")
 {
-    ASSERT_THAT(false, testing::IsTrue());
+    FAIL();
 }
 
 HOOK_BEFORE_SCENARIO("@throw_scenariohook")