Merge branch 'develop' of https://github.com/magento-gl/magento2-functional-testing-framework into ACQE-3210

Author: Manjusha.S

CHANGELOG.md

@@ -1,6 +1,19 @@
 Magento Functional Testing Framework Changelog
 ================================================
 
+3.11.0
+---------
+### Enhancements
+* Composer updated to 2.4.2 version
+* Static check for duplicate step keys in action group
+
+
+### Fixes
+
+* Fixed incorrect MFTF test dependencies path
+* Removed PHP 7.3 build check from MFTF PR build as PHP 7.3 is no longer supported
+
+
 3.10.3
 ---------
 

composer.json

@@ -2,7 +2,7 @@
     "name": "magento/magento2-functional-testing-framework",
     "description": "Magento2 Functional Testing Framework",
     "type": "library",
-    "version": "3.10.3",
+    "version": "3.11.0",
     "license": "AGPL-3.0",
     "keywords": ["magento", "automation", "functional", "testing"],
     "config": {
@@ -22,7 +22,7 @@
         "codeception/module-asserts": "^1.1",
         "codeception/module-sequence": "^1.0",
         "codeception/module-webdriver": "^1.0",
-        "composer/composer": "^1.9 || ^2.0, !=2.2.16",
+        "composer/composer": "^2.0, !=2.2.16",
         "csharpru/vault-php": "^4.2.1",
         "guzzlehttp/guzzle": "^7.3.0",
         "laminas/laminas-diactoros": "^2.8",

src/Magento/FunctionalTestingFramework/StaticCheck/ActionGroupStandardsCheck.php

@@ -103,7 +103,11 @@ private function findErrorsInFileSet($files)
             $actionGroupReferencesDataArray = [];
             $actionGroupToArguments = [];
             $contents = $filePath->getContents();
-            preg_match_all(self::STEP_KEY_REGEX_PATTERN, $contents, $actionGroupReferences);
+            preg_match_all(
+                self::STEP_KEY_REGEX_PATTERN,
+                preg_replace('/<!--(.|\s)*?-->/', '', $contents),
+                $actionGroupReferences
+            );
             foreach ($actionGroupReferences[0] as $actionGroupReferencesData) {
                 $actionGroupReferencesDataArray[] = trim(
                     str_replace(['stepKey', '='], [""], $actionGroupReferencesData)
@@ -119,7 +123,8 @@ private function findErrorsInFileSet($files)
             );
             unset($actionGroupReferencesDataArray);
             if (isset($duplicateStepKeys) && count($duplicateStepKeys) > 0) {
-                throw new TestFrameworkException('Action group has duplicate step keys');
+                throw new TestFrameworkException('Action group has duplicate step keys '
+                  .implode(",", array_unique($duplicateStepKeys))." File Path ".$filePath);
             }
             /** @var DOMElement $actionGroup */
             $actionGroup = $this->getActionGroupDomElement($contents);