MQE-980: [Annotations] <features> tag defaults to module

- CR fixes, test updates

Author: KevinBKozan

dev/tests/unit/Magento/FunctionalTestFramework/Test/Handlers/TestObjectHandlerTest.php

@@ -141,6 +141,44 @@ public function testGetTestsByGroup()
         $this->assertArrayNotHasKey('excludeTest', $tests);
     }
 
+    /**
+     * Tests the function used to parse and determine a test's Module (used in allure Features annotation)
+     *
+     * @throws \Exception
+     */
+    public function testGetTestWithModuleName()
+    {
+        // set up Test Data
+        $moduleExpected = "SomeTestModule";
+        $filepath = DIRECTORY_SEPARATOR .
+            "user" .
+            "magento2ce" . DIRECTORY_SEPARATOR .
+            "dev" . DIRECTORY_SEPARATOR .
+            "tests" . DIRECTORY_SEPARATOR .
+            "acceptance" . DIRECTORY_SEPARATOR .
+            "tests" . DIRECTORY_SEPARATOR .
+            $moduleExpected . DIRECTORY_SEPARATOR .
+            "Tests" . DIRECTORY_SEPARATOR .
+            "text.xml";
+        // set up mock data
+        $testDataArrayBuilder = new TestDataArrayBuilder();
+        $mockData = $testDataArrayBuilder
+            ->withAnnotations()
+            ->withFailedHook()
+            ->withAfterHook()
+            ->withBeforeHook()
+            ->withTestActions()
+            ->withFileName($filepath)
+            ->build();
+        $this->setMockParserOutput(['tests' => $mockData]);
+        // Execute Test Method
+        $toh = TestObjectHandler::getInstance();
+        $actualTestObject = $toh->getObject($testDataArrayBuilder->testName);
+        $moduleName = $actualTestObject->getAnnotations()["features"][0];
+        //performAsserts
+        $this->assertEquals($moduleExpected, $moduleName);
+    }
+
     /**
      * Function used to set mock for parser return and force init method to run between tests.
      *

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

@@ -134,11 +134,13 @@ public function extractTestData($testData)
      */
     private function extractModuleName($path)
     {
-        if ($path === "") {
+        if (empty($path)) {
             return "NO MODULE DETECTED";
         }
         $paths = explode(DIRECTORY_SEPARATOR, $path);
-        if ($paths[count($paths)-3] == "Mftf") {
+        if (!isset($paths[count($paths)-3])) {
+            return "NO MODULE DETECTED";
+        } elseif ($paths[count($paths)-3] == "Mftf") {
             // app/code/Magento/[Analytics]/Test/Mftf/Test/SomeText.xml
             return $paths[count($paths)-5];
         }