MQE-2669: bin/mftf run:failed process used a lot of RAM during MTSv1 build

Author: okolesnyk

src/Magento/FunctionalTestingFramework/Console/GenerateTestFailedCommand.php

@@ -74,7 +74,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
         $testConfiguration = $this->getFailedTestList();
 
         if ($testConfiguration === null) {
-            // no failed tests found, run is a success
+            // No failed tests found, no tests generated
             return 0;
         }
 
@@ -89,7 +89,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
         ];
         $command->run(new ArrayInput($args), $output);
         $output->writeln("Test Failed Generated, now run:failed command");
-        return 1;
+        return 0;
     }
 
     /**

src/Magento/FunctionalTestingFramework/Console/RunTestFailedCommand.php

@@ -18,11 +18,6 @@
 
 class RunTestFailedCommand extends BaseGenerateCommand
 {
-    /**
-     * Default Test group to signify not in suite
-     */
-    const DEFAULT_TEST_GROUP = 'default';
-
     /**
      * @var string
      */
@@ -61,7 +56,6 @@ protected function execute(InputInterface $input, OutputInterface $output): int
     {
         $this->testsFailedFile = $this->getTestsOutputDir() . self::FAILED_FILE;
         $this->testsReRunFile = $this->getTestsOutputDir() . "rerun_tests";
-        $failedTestList = $this->readFailedTestFile($this->testsFailedFile);
 
         $this->testsManifestFile= FilePathFormatter::format(TESTS_MODULE_PATH) .
             "_generated" .
@@ -71,9 +65,6 @@ protected function execute(InputInterface $input, OutputInterface $output): int
         $testManifestList = $this->readTestManifestFile();
         $returnCode = 0;
         for ($i = 0; $i < count($testManifestList); $i++) {
-            if (in_array($testManifestList[$i], $failedTestList) === false) {
-                continue;
-            }
             if ($this->pauseEnabled()) {
                 $codeceptionCommand = self::CODECEPT_RUN_FUNCTIONAL . $testManifestList[$i] . ' --debug ';
                 if ($i !== count($testManifestList) - 1) {
@@ -129,14 +120,7 @@ private function readTestManifestFile()
      */
     private function readFailedTestFile($filePath)
     {
-        $failedTests = file($filePath, FILE_IGNORE_NEW_LINES);
-        if ($failedTests !== false) {
-            foreach ($failedTests as $key => $failedTest) {
-                list($filePath) = explode(":", $failedTest);
-                $failedTests[$key] = $filePath;
-            }
-        }
-        return $failedTests;
+        return file($filePath, FILE_IGNORE_NEW_LINES);
     }
 
     /**