Merge branch 'develop' into MQE-1145

Author: jilu1

CHANGELOG.md

@@ -1,5 +1,9 @@
 Magento Functional Testing Framework Changelog
 ================================================
+2.3.12
+-----
+### Enhancements
+* Fetched latest allure-codeception package
 
 2.3.11
 -----

bin/mftf

@@ -29,7 +29,7 @@ try {
 try {
     $application = new Symfony\Component\Console\Application();
     $application->setName('Magento Functional Testing Framework CLI');
-    $application->setVersion('2.3.11');
+    $application->setVersion('2.3.12');
     /** @var \Magento\FunctionalTestingFramework\Console\CommandListInterface $commandList */
     $commandList = new \Magento\FunctionalTestingFramework\Console\CommandList;
     foreach ($commandList->getCommands() as $command) {

composer.json

@@ -2,17 +2,17 @@
     "name": "magento/magento2-functional-testing-framework",
     "description": "Magento2 Functional Testing Framework",
     "type": "library",
-    "version": "2.3.11",
+    "version": "2.3.12",
     "license": "AGPL-3.0",
     "keywords": ["magento", "automation", "functional", "testing"],
     "config": {
         "sort-packages": true
     },
     "require": {
         "php": "7.0.2|7.0.4|~7.0.6|~7.1.0|~7.2.0",
+        "allure-framework/allure-codeception": "~1.3.0",
         "ext-curl": "*",
-        "allure-framework/allure-codeception": "~1.2.6",
-        "codeception/codeception": "~2.3.4",
+        "codeception/codeception": "~2.3.4 || ~2.4.0 ",
         "consolidation/robo": "^1.0.0",
         "epfremme/swagger-php": "^2.0",
         "flow/jsonpath": ">0.2",

composer.lock

@@ -15,7 +15,7 @@ extensions:
         - Magento\FunctionalTestingFramework\Extension\TestContextExtension
         - Magento\FunctionalTestingFramework\Allure\Adapter\MagentoAllureAdapter
     config:
-        Yandex\Allure\Adapter\AllureAdapter:
+        Magento\FunctionalTestingFramework\Allure\Adapter\MagentoAllureAdapter:
             deletePreviousResults: true
             outputDirectory: allure-results
             ignoredAnnotations:

etc/config/codeception.dist.yml

@@ -6,7 +6,7 @@
 namespace Magento\FunctionalTestingFramework\Allure\Adapter;
 
 use Magento\FunctionalTestingFramework\Suite\Handlers\SuiteObjectHandler;
-use Yandex\Allure\Adapter\AllureAdapter;
+use Yandex\Allure\Codeception\AllureCodeception;
 use Yandex\Allure\Adapter\Event\StepStartedEvent;
 use Yandex\Allure\Adapter\Event\StepFinishedEvent;
 use Yandex\Allure\Adapter\Event\StepFailedEvent;
@@ -23,7 +23,7 @@
  * @package Magento\FunctionalTestingFramework\Allure
  */
 
-class MagentoAllureAdapter extends AllureAdapter
+class MagentoAllureAdapter extends AllureCodeception
 {
     /**
      * Array of group values passed to test runner command

src/Magento/FunctionalTestingFramework/Allure/Adapter/MagentoAllureAdapter.php

@@ -131,7 +131,7 @@ private function encryptCredFileContents($credContents)
                 continue;
             }
 
-            list($key, $value) = explode("=", $credValue);
+            list($key, $value) = explode("=", $credValue, 2);
             if (!empty($value)) {
                 $encryptedCreds[$key] = openssl_encrypt(
                     $value,

src/Magento/FunctionalTestingFramework/DataGenerator/Handlers/CredentialStore.php

@@ -16,6 +16,7 @@
 class TestContextExtension extends BaseExtension
 {
     const TEST_PHASE_AFTER = "_after";
+    const CODECEPT_AFTER_VERSION = "2.3.9";
     const TEST_FAILED_FILE = 'failed';
 
     /**
@@ -69,7 +70,7 @@ public function testFail(\Codeception\Event\FailEvent $e)
             $this->runAfterBlock($e, $cest);
         }
     }
-    
+
     /**
      * Codeception event listener function, triggered on test ending (naturally or by error).
      * @param \Codeception\Event\TestEvent $e
@@ -117,13 +118,15 @@ private function runAfterBlock($e, $cest)
         try {
             $actorClass = $e->getTest()->getMetadata()->getCurrent('actor');
             $I = new $actorClass($cest->getScenario());
-            call_user_func(\Closure::bind(
-                function () use ($cest, $I) {
-                    $cest->executeHook($I, 'after');
-                },
-                null,
-                $cest
-            ));
+            if (version_compare(Codeception\Codecept::VERSION, TestContextExtension::CODECEPT_AFTER_VERSION, "<=")) {
+                call_user_func(\Closure::bind(
+                    function () use ($cest, $I) {
+                        $cest->executeHook($I, 'after');
+                    },
+                    null,
+                    $cest
+                ));
+            }
         } catch (\Exception $e) {
             // Do not rethrow Exception
         }