MQE-2463: Mftf test extends from a skipped parent should be skipped

Author: jilu1

src/Magento/FunctionalTestingFramework/Test/Util/ObjectExtensionUtil.php

@@ -48,7 +48,7 @@ public function extendTest($testObject)
                     ["parent" => $testObject->getParentName(), "test" => $testObject->getName()]
                 );
             }
-            $skippedTest = $this->skipTest($testObject);
+            $skippedTest = $this->skipTest($testObject, 'ParentTestDoesNotExist');
             return $skippedTest;
         }
 
@@ -64,6 +64,11 @@ public function extendTest($testObject)
                 ->debug("extending test", ["parent" => $parentTest->getName(), "test" => $testObject->getName()]);
         }
 
+        // Skip test if parent is skipped
+        if ($parentTest->isSkipped()) {
+            return $this->skipTest($testObject);
+        }
+
         // Get steps for both the parent and the child tests
         $referencedTestSteps = $parentTest->getUnresolvedSteps();
         $newSteps = $this->processRemoveActions(array_merge($referencedTestSteps, $testObject->getUnresolvedSteps()));
@@ -207,17 +212,19 @@ private function processRemoveActions($actions)
      * This method returns a skipped form of the Test Object
      *
      * @param TestObject $testObject
+     * @param string     $skipReason
      * @return TestObject
      */
-    public function skipTest($testObject)
+    public function skipTest($testObject, $skipReason = null)
     {
         $annotations = $testObject->getAnnotations();
+        $skipReason = $skipReason ?? 'ParentTestIsSkipped';
 
         // Add skip to the group array if it doesn't already exist
         if (array_key_exists('skip', $annotations)) {
             return $testObject;
         } elseif (!array_key_exists('skip', $annotations)) {
-            $annotations['skip'] = ['issueId' => "ParentTestDoesNotExist"];
+            $annotations['skip'] = ['issueId' => $skipReason];
         }
 
         $skippedTest = new TestObject(
@@ -229,6 +236,10 @@ public function skipTest($testObject)
             $testObject->getParentName()
         );
 
+        LoggingUtil::getInstance()->getLogger(ObjectExtensionUtil::class)->info(
+            "\nMQE-2463 LOGGING: {$testObject->getName()} is skipped due to {$skipReason}\n"
+        );
+
         return $skippedTest;
     }
 }