Merge pull request #338 from magento-gl/ACQE-5547

KevinBKozan · web-flow · commit e5a2d3ed1c34 · 2023-10-27T10:41:31.000-05:00
[ACQE-5547] Calling generateTestgroupmembership from GenerateTestsCommand.php
diff --git a/src/Magento/FunctionalTestingFramework/Console/GenerateTestsCommand.php b/src/Magento/FunctionalTestingFramework/Console/GenerateTestsCommand.php
@@ -231,9 +231,11 @@ protected function execute(InputInterface $input, OutputInterface $output)
                 $testManifest->createTestGroups($configNumber);
             }
 
+            SuiteGenerator::getInstance()->generateAllSuites($testManifest);
+
             $testManifest->generate();
 
-            SuiteGenerator::getInstance()->generateAllSuites($testManifest);
+            SuiteGenerator::getInstance()->generateTestgroupmembership($testManifest);
         } catch (\Exception $e) {
             if (!empty(GenerationErrorHandler::getInstance()->getAllErrors())) {
                 GenerationErrorHandler::getInstance()->printErrorSummary();
diff --git a/src/Magento/FunctionalTestingFramework/Suite/SuiteGenerator.php b/src/Magento/FunctionalTestingFramework/Suite/SuiteGenerator.php
@@ -95,7 +95,6 @@ public static function getInstance(): SuiteGenerator
      */
     public function generateAllSuites($testManifest)
     {
-        $this->generateTestgroupmembership($testManifest);
         $suites = $testManifest->getSuiteConfig();
 
         foreach ($suites as $suiteName => $suiteContent) {

Original file line number	Diff line number	Diff line change
`@@ -95,7 +95,6 @@ public static function getInstance(): SuiteGenerator`
`95`	`95`	`*/`
`96`	`96`	`public function generateAllSuites($testManifest)`
`97`	`97`	`{`
`98`		`- $this->generateTestgroupmembership($testManifest);`
`99`	`98`	`$suites = $testManifest->getSuiteConfig();`
`100`	`99`
`101`	`100`	`foreach ($suites as $suiteName => $suiteContent) {`