refactor: remove default destructor and add deleted copy constructor and assignment operator in TestExecutionImpl

Author: daantimmer

cucumber_cpp/library/engine/FailureHandler.hpp

@@ -13,14 +13,12 @@ namespace cucumber_cpp::library::engine
     {
         virtual ~TestAssertionHandler() = default;
 
-    public:
         virtual void AddAssertionError(const char* file, int line, const std::string& message) = 0;
     };
 
     struct TestAssertionHandlerImpl : TestAssertionHandler
     {
         explicit TestAssertionHandlerImpl(cucumber_cpp::library::engine::ContextManager& contextManager, report::ReportForwarder& reportHandler);
-        virtual ~TestAssertionHandlerImpl() = default;
 
         void AddAssertionError(const char* file, int line, const std::string& message) override;
 

cucumber_cpp/library/engine/TestExecution.hpp

@@ -112,6 +112,10 @@ namespace cucumber_cpp::library::engine
     struct TestExecutionImpl : TestExecution
     {
         TestExecutionImpl(cucumber_cpp::library::engine::ContextManager& contextManager, report::ReportForwarder& reportHandler, HookExecutor& hookExecution, const Policy& executionPolicy = executeRunPolicy);
+
+        TestExecutionImpl(const TestExecutionImpl&) = delete;
+        TestExecutionImpl& operator=(const TestExecutionImpl&) = delete;
+
         virtual ~TestExecutionImpl();
 
         ProgramScope StartRun() override;