MQE-1181: Fatal error while test running

- reintroduced functional.suite.dist.yml restart: true
- MagentoWebDriver overwrites parent _after function and remaps it (to be called in TestContextExtension

Author: KevinBKozan

CHANGELOG.md

@@ -4,7 +4,7 @@ Magento Functional Testing Framework Changelog
 2.3.4
 -----
 ### Fixes
-* Config option `restart: true` removed from `etc/config/functional.suite.dist.yml`.
+* MagentoWebDriver overrides `parent::_after()` function and remaps to `runAfter()`, necessary to solve compatibility issues in Codeception `2.3.x`.
 
 2.3.3
 -----

etc/config/functional.suite.dist.yml

@@ -28,6 +28,7 @@ modules:
             url: "%MAGENTO_BASE_URL%"
             backend_name: "%MAGENTO_BACKEND_NAME%"
             browser: 'chrome'
+            restart: true
             window_size: 1280x1024
             username: "%MAGENTO_ADMIN_USERNAME%"
             password: "%MAGENTO_ADMIN_PASSWORD%"

src/Magento/FunctionalTestingFramework/Extension/TestContextExtension.php

@@ -8,6 +8,7 @@
 
 use \Codeception\Events;
 use Magento\FunctionalTestingFramework\Extension\ErrorLogger;
+use Magento\FunctionalTestingFramework\Module\MagentoWebDriver;
 
 /**
  * Class TestContextExtension
@@ -89,6 +90,8 @@ function () use ($cest, $I) {
                 null,
                 $cest
             ));
+            // Reset Session and Cookies, workaround due to functional.suite.yml restart: true
+            $this->getModule(MagentoWebDriver::class)->_runAfter($e->getTest());
         } catch (\Exception $e) {
             // Do not rethrow Exception
         }

src/Magento/FunctionalTestingFramework/Module/MagentoWebDriver.php

@@ -125,6 +125,25 @@ public function _resetConfig()
         $this->config = ConfigSanitizerUtil::sanitizeWebDriverConfig($this->config);
     }
 
+    /**
+     * Remap parent::_after, called in TestContextExtension
+     * @param TestInterface $test
+     * @return void
+     */
+    public function _runAfter(TestInterface $test)
+    {
+        parent::_after($test); // TODO: Change the autogenerated stub
+    }
+
+    /**
+     * Override parent::_after to do nothing.
+     * @return void
+     */
+    public function _after()
+    {
+        // DO NOT RESET SESSIONS
+    }
+
     /**
      * Returns URL of a host.
      *