MQE-1959: Static-checks command can be configured

Author: jilu1

src/Magento/FunctionalTestingFramework/Console/StaticChecksCommand.php

@@ -8,22 +8,31 @@
 
 namespace Magento\FunctionalTestingFramework\Console;
 
+use Magento\FunctionalTestingFramework\StaticCheck\StaticCheckInterface;
 use Magento\FunctionalTestingFramework\StaticCheck\StaticChecksList;
-use Magento\FunctionalTestingFramework\StaticCheck\StaticCheckListInterface;
 use Magento\FunctionalTestingFramework\Util\Logger\LoggingUtil;
 use Symfony\Component\Console\Command\Command;
+use Symfony\Component\Console\Exception\InvalidArgumentException;
+use Symfony\Component\Console\Input\InputArgument;
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Output\OutputInterface;
 use Exception;
 
 class StaticChecksCommand extends Command
 {
     /**
-     * Pool of static check scripts to run
+     * Pool of all existing static check objects
      *
-     * @var StaticCheckListInterface
+     * @var StaticCheckInterface[]
      */
-    private $staticChecksList;
+    private $allStaticCheckObjects;
+
+    /**
+     * Static checks to run
+     *
+     * @var StaticCheckInterface[]
+     */
+    private $staticCheckObjects;
 
     /**
      * Configures the current command.
@@ -32,13 +41,22 @@ class StaticChecksCommand extends Command
      */
     protected function configure()
     {
+        $list = new StaticChecksList();
+        $this->allStaticCheckObjects = $list->getStaticChecks();
+        $staticCheckNames = implode(', ', array_keys($this->allStaticCheckObjects));
+        $description = "This command will run all static checks on xml test materials. "
+            . "Available static check scripts are:\n{$staticCheckNames}";
         $this->setName('static-checks')
-            ->setDescription('This command will run all static checks on xml test materials.');
-        $this->staticChecksList = new StaticChecksList();
+            ->setDescription($description)
+            ->addArgument(
+                'names',
+                InputArgument::OPTIONAL | InputArgument::IS_ARRAY,
+                'name(s) of specific static check script(s) to run'
+            );
     }
 
     /**
-     *
+     * Run required static check scripts
      *
      * @param InputInterface $input
      * @param OutputInterface $output
@@ -47,11 +65,23 @@ protected function configure()
      */
     protected function execute(InputInterface $input, OutputInterface $output)
     {
-        $staticCheckObjects = $this->staticChecksList->getStaticChecks();
+        try {
+            $this->validateInputArguments($input, $output);
+        } catch (InvalidArgumentException $e) {
+            LoggingUtil::getInstance()->getLogger(StaticChecksCommand::class)->error($e->getMessage());
+            $output->writeln($e->getMessage() . " Please fix input arguments and rerun.");
+            return 1;
+        }
 
         $errors = [];
+        foreach ($this->staticCheckObjects as $name => $staticCheck) {
+            LoggingUtil::getInstance()->getLogger(get_class($staticCheck))->info(
+                'Running static check script for: ' . $name
+            );
+            $output->writeln(
+                'Running static check script for: ' . $name
+            );
 
-        foreach ($staticCheckObjects as $staticCheck) {
             $staticCheck->execute($input);
 
             $staticOutput = $staticCheck->getOutput();
@@ -66,4 +96,34 @@ protected function execute(InputInterface $input, OutputInterface $output)
             return 1;
         }
     }
+
+    /**
+     * Validate input arguments
+     *
+     * @param InputInterface $input
+     * @return void
+     * @throws InvalidArgumentException
+     */
+    private function validateInputArguments(InputInterface $input)
+    {
+        $this->staticCheckObjects = [];
+        $requiredChecksNames = $input->getArgument('names');
+        // Found user required static check script(s) to run,
+        // If no static check name is supplied, run all static check scripts
+        if (empty($requiredChecksNames)) {
+            $this->staticCheckObjects = $this->allStaticCheckObjects;
+        } else {
+            foreach ($requiredChecksNames as $checksName) {
+                if (in_array($checksName, $this->allStaticCheckObjects)) {
+                    $this->staticCheckObjects[$checksName] = $this->allStaticCheckObjects[$checksName];
+                    unset($requiredChecksNames[$checksName]);
+                }
+            }
+        }
+
+        $invalidCheckNames = implode(', ', $requiredChecksNames);
+        if (!empty($invalidCheckNames)) {
+            throw new InvalidArgumentException("Invalid static check script(s): {$invalidCheckNames}.");
+        }
+    }
 }