MQE-1222: Selector Fails To Resolve Correctly in Action Group

- Added support for passing in data into arguments as `{{mydata.value}}`

Author: KevinBKozan

dev/tests/verification/Resources/ActionGroupWithSectionAndDataAsArguments.txt

@@ -0,0 +1,32 @@
+<?php
+namespace Magento\AcceptanceTest\_default\Backend;
+
+use Magento\FunctionalTestingFramework\AcceptanceTester;
+use Magento\FunctionalTestingFramework\DataGenerator\Handlers\CredentialStore;
+use Magento\FunctionalTestingFramework\DataGenerator\Handlers\PersistedObjectHandler;
+use \Codeception\Util\Locator;
+use Yandex\Allure\Adapter\Annotation\Features;
+use Yandex\Allure\Adapter\Annotation\Stories;
+use Yandex\Allure\Adapter\Annotation\Title;
+use Yandex\Allure\Adapter\Annotation\Description;
+use Yandex\Allure\Adapter\Annotation\Parameter;
+use Yandex\Allure\Adapter\Annotation\Severity;
+use Yandex\Allure\Adapter\Model\SeverityLevel;
+use Yandex\Allure\Adapter\Annotation\TestCaseId;
+
+/**
+ */
+class ActionGroupWithSectionAndDataAsArgumentsCest
+{
+	/**
+	 * @Features({"TestModule"})
+	 * @Parameter(name = "AcceptanceTester", value="$I")
+	 * @param AcceptanceTester $I
+	 * @return void
+	 * @throws \Exception
+	 */
+	public function ActionGroupWithSectionAndDataAsArguments(AcceptanceTester $I)
+	{
+		$I->waitForElementVisible("#element .John");
+	}
+}

dev/tests/verification/TestModule/ActionGroup/BasicActionGroup.xml

@@ -118,4 +118,12 @@
     <actionGroup name="actionGroupWithSkipReadinessActions">
         <comment userInput="ActionGroupSkipReadiness" stepKey="skip" skipReadiness="true"/>
     </actionGroup>
+
+    <actionGroup name="actionGroupWithSectionAndData">
+        <arguments>
+            <argument name="content" type="string"/>
+            <argument name="section"/>
+        </arguments>
+        <waitForElementVisible selector="{{section.oneParamElement(content)}}" stepKey="arg1"/>
+    </actionGroup>
 </actionGroups>

dev/tests/verification/TestModule/Test/ActionGroupTest.xml

@@ -158,6 +158,14 @@
             <argument name="sameStepKeyAsArg" value="arg1"/>
         </actionGroup>
     </test>
+
+    <test name="ActionGroupWithSectionAndDataAsArguments">
+        <actionGroup ref="actionGroupWithSectionAndData" stepKey="actionGroup">
+            <argument name="content" value="{{simpleData.firstname}}"/>
+            <argument name="section" value="SampleSection"/>
+        </actionGroup>
+    </test>
+
     <test name="ActionGroupWithParameterizedElementWithHyphen">
         <actionGroup ref="SectionArgumentWithParameterizedSelector" stepKey="actionGroup">
             <argument name="section" value="SampleSection"/>

dev/tests/verification/Tests/ActionGroupGenerationTest.php

@@ -196,6 +196,17 @@ public function testActionGroupWithSkipReadiness()
         $this->generateAndCompareTest('ActionGroupSkipReadiness');
     }
 
+    /**
+     * Test an action group with an arg containing stepKey text
+     *
+     * @throws \Exception
+     * @throws \Magento\FunctionalTestingFramework\Exceptions\TestReferenceException
+     */
+    public function testActionGroupWithSectionAndDataArguments()
+    {
+        $this->generateAndCompareTest('ActionGroupWithSectionAndDataAsArguments');
+    }
+
     /**
      * Test an action group with an arg that resolves into section.element with a hyphen in the parameter
      *

src/Magento/FunctionalTestingFramework/Test/Objects/ArgumentObject.php

@@ -104,8 +104,8 @@ public function getResolvedValue($isInnerArgument)
      * Takes in boolean to determine if the replacement is being done with an inner argument (as in if it's a parameter)
      *
      * Example Type     Non Inner           Inner
-     * {{XML.DATA}}:    {{XML.DATA}}        XML.DATA
-     * $TEST.DATA$:     $TEST.DATA$         $TEST.DATA$
+     * {{XML.DATA}}     {{XML.DATA}}        XML.DATA
+     * $TEST.DATA$      $TEST.DATA$         $TEST.DATA$
      * stringLiteral    stringLiteral       'stringLiteral'
      *
      * @param boolean $isInnerArgument
@@ -114,6 +114,11 @@ public function getResolvedValue($isInnerArgument)
     private function resolveStringArgument($isInnerArgument)
     {
         if ($isInnerArgument) {
+            if (preg_match('/{{[\w.\[\]]+}}/', $this->value)) {
+                return ltrim(rtrim($this->value, "}"), "{");
+            } elseif (preg_match('/\${1,2}[\w.\[\]]+\${1,2}/', $this->value)) {
+                return $this->value;
+            }
             return "'" . $this->value . "'";
         } else {
             return $this->value;