MQE-1151: Extending a skipped test fails to generate the parent before and after actions

- Added parameter to getHooks to return the hooks if the value is true
- Added a verification test for the topic

Author: aljcalandra

Diff for: dev/tests/verification/Resources/ExtendingSkippedTest.txt

@@ -0,0 +1,66 @@
+<?php
+namespace Magento\AcceptanceTest\_default\Backend;
+
+use Magento\FunctionalTestingFramework\AcceptanceTester;
+use Magento\FunctionalTestingFramework\DataGenerator\Handlers\DataObjectHandler;
+use Magento\FunctionalTestingFramework\DataGenerator\Persist\DataPersistenceHandler;
+use Magento\FunctionalTestingFramework\DataGenerator\Objects\EntityDataObject;
+use Magento\FunctionalTestingFramework\DataGenerator\Handlers\CredentialStore;
+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;
+
+/**
+ * @Title("ChildExtendedTestSkippedParent")
+ * @group Child
+ */
+class ExtendingSkippedTestCest
+{
+	/**
+	  * @param AcceptanceTester $I
+	  * @throws \Exception
+	  */
+	public function _before(AcceptanceTester $I)
+	{
+		$I->amOnPage("/beforeUrl");
+	}
+
+	/**
+	  * @param AcceptanceTester $I
+	  * @throws \Exception
+	  */
+	public function _after(AcceptanceTester $I)
+	{
+		$I->amOnPage("/afterUrl");
+	}
+
+	/**
+	  * @param AcceptanceTester $I
+	  * @throws \Exception
+	  */
+	public function _failed(AcceptanceTester $I)
+	{
+		$I->saveScreenshot();
+	}
+
+	/**
+	 * @Severity(level = SeverityLevel::CRITICAL)
+	 * @Features({"TestModule"})
+	 * @Stories({"Child"})
+	 * @Parameter(name = "AcceptanceTester", value="$I")
+	 * @param AcceptanceTester $I
+	 * @return void
+	 * @throws \Exception
+	 */
+	public function ExtendingSkippedTest(AcceptanceTester $I)
+	{
+		$I->comment("text");
+		$I->comment("child");
+	}
+}

Diff for: dev/tests/verification/TestModule/Test/ExtendedFunctionalTest.xml

@@ -116,7 +116,6 @@
             <remove keyForRemoval="beforeAmOnPageKey"/>
         </before>
     </test>
-
     <test name="ChildExtendedTestNoParent" extends="ThisTestDoesNotExist">
         <annotations>
             <severity value="CRITICAL"/>
@@ -129,4 +128,34 @@
             <remove keyForRemoval="beforeAmOnPageKey"/>
         </before>
     </test>
+    <test name="SkippedParent">
+        <annotations>
+            <severity value="CRITICAL"/>
+            <title value="PARENTSKIPPED"/>
+            <group value="Parent"/>
+            <features value="Parent"/>
+            <stories value="Parent"/>
+            <skip>
+                <issueId value="NONE"/>
+            </skip>
+        </annotations>
+        <before>
+            <amOnPage url="/beforeUrl" stepKey="beforeAmOnPageKey"/>
+        </before>
+        <after>
+            <amOnPage url="/afterUrl" stepKey="afterAmOnPageKey"/>
+        </after>
+        <comment userInput="text" stepKey="keepMe"/>
+        <comment userInput="text" stepKey="replaceMe"/>
+    </test>
+    <test name="ExtendingSkippedTest" extends="SkippedParent">
+        <annotations>
+            <severity value="CRITICAL"/>
+            <title value="ChildExtendedTestSkippedParent"/>
+            <group value="Child"/>
+            <features value="Child"/>
+            <stories value="Child"/>
+        </annotations>
+        <comment userInput="child" stepKey="replaceMe"/>
+    </test>
 </tests>

Diff for: dev/tests/verification/Tests/ExtendedGenerationTest.php

@@ -96,4 +96,15 @@ public function testExtendedTestGenerationNoParent()
     {
         $this->generateAndCompareTest('ChildExtendedTestNoParent');
     }
+
+    /**
+     * Tests extending a skipped test generation.
+     *
+     * @throws \Exception
+     * @throws \Magento\FunctionalTestingFramework\Exceptions\TestReferenceException
+     */
+    public function testExtendingSkippedGeneration()
+    {
+        $this->generateAndCompareTest('ExtendingSkippedTest');
+    }
 }

Diff for: src/Magento/FunctionalTestingFramework/Test/Objects/TestObject.php

@@ -164,12 +164,13 @@ public function getAnnotations()
     /**
      * Returns hooks.
      *
+     * @param boolean $isExtended
      * @return TestHookObject[]
      */
-    public function getHooks()
+    public function getHooks($isExtended = false)
     {
         // if this test is skipped we do not want any before/after actions to generate as the tests will not run
-        if ($this->isSkipped()) {
+        if ($this->isSkipped() && !$isExtended) {
             return [];
         }
         return $this->hooks;

Diff for: src/Magento/FunctionalTestingFramework/Test/Util/ObjectExtensionUtil.php

@@ -146,7 +146,8 @@ public function extendActionGroup($actionGroupObject)
     private function resolveExtendedHooks($testObject, $parentTestObject)
     {
         $testHooks = $testObject->getHooks();
-        $parentHooks = $parentTestObject->getHooks();
+        // Send true to getHooks to return even if test is skipped
+        $parentHooks = $parentTestObject->getHooks(true);
 
         // Get the hooks for each Test merge changes from the child hooks to the parent hooks into the child hooks
         foreach ($testHooks as $key => $hook) {