MQE-2047: Static Check Options

- Added docs
- Fixed static check errors

Author: KevinBKozan

docs/commands/mftf.md

@@ -430,8 +430,11 @@ The example parameters are taken from the `etc/config/.env.example` file.
 
 ### `static-checks`
 
-Runs all or specific MFTF static-checks on the test codebase that MFTF is currently attached to. 
-If no script name argument is specified, all existing static check scripts will run.
+Runs all or specific MFTF static-checks on the test codebase that MFTF is currently attached to.
+Behavior for determining what tests to runs is as follows:
+* If test names are specified, runs only those tests
+* If no test names are specified, run tests according to `staticRuleset.json`
+* If no `staticRuleset.json` is found, run all tests.
 
 #### Usage
 
@@ -489,6 +492,27 @@ vendor/bin/mftf static-checks testDependencies actionGroupArguments
 |`actionGroupArguments` | Checks that action groups do not have unused arguments.|
 |`deprecatedEntityUsage`| Checks that deprecated test entities are not being referenced.|
          
+#### Defining ruleset
+
+The `static-checks` command will look for a `staticRuleset.json` file under either:
+* `dev/tests/acceptance/staticRuleset.json` if embedded with Magento2
+* `dev/staticRuleset.json` if standalone
+
+This file works as a default configuration to easily allow for integration of `static-checks` in a CI environment.
+
+Currently, the ruleset only defines the tests to run. Here is an example of the expected format:
+
+```json
+{
+  "tests": [
+    "actionGroupArguments",
+    "anotherTest"
+  ]
+}
+
+```       
+         
+         
 ### `upgrade:tests`
 
 When the path argument is specified, this `upgrade` command applies all the major version MFTF upgrade scripts to a `Test Module` in the given path.

src/Magento/FunctionalTestingFramework/Console/StaticChecksCommand.php

@@ -140,28 +140,14 @@ private function validateInput(InputInterface $input)
     {
         $this->staticCheckObjects = [];
         $requiredChecksNames = $input->getArgument('names');
-        $invalidCheckNames = [];
         // Build list of static check names to run.
         if (empty($requiredChecksNames) && isset($this->ruleSet['tests'])) {
             $requiredChecksNames = $this->ruleSet['tests'];
         }
         if (empty($requiredChecksNames)) {
             $this->staticCheckObjects = $this->allStaticCheckObjects;
         } else {
-            for ($index = 0; $index < count($requiredChecksNames); $index++) {
-                if (in_array($requiredChecksNames[$index], array_keys($this->allStaticCheckObjects))) {
-                    $this->staticCheckObjects[$requiredChecksNames[$index]] =
-                        $this->allStaticCheckObjects[$requiredChecksNames[$index]];
-                } else {
-                    $invalidCheckNames[] = $requiredChecksNames[$index];
-                }
-            }
-        }
-
-        if (!empty($invalidCheckNames)) {
-            throw new InvalidArgumentException(
-                'Invalid static check script(s): ' . implode(', ', $invalidCheckNames) . '.'
-            );
+            $this->validateTestNames($requiredChecksNames);
         }
 
         if ($input->getOption('path')) {
@@ -175,6 +161,30 @@ private function validateInput(InputInterface $input)
         }
     }
 
+    /**
+     * Validates that all passed in static-check names match an existing static check
+     * @param string[] $requiredChecksNames
+     * @return void
+     */
+    private function validateTestNames($requiredChecksNames)
+    {
+        $invalidCheckNames = [];
+        for ($index = 0; $index < count($requiredChecksNames); $index++) {
+            if (in_array($requiredChecksNames[$index], array_keys($this->allStaticCheckObjects))) {
+                $this->staticCheckObjects[$requiredChecksNames[$index]] =
+                    $this->allStaticCheckObjects[$requiredChecksNames[$index]];
+            } else {
+                $invalidCheckNames[] = $requiredChecksNames[$index];
+            }
+        }
+
+        if (!empty($invalidCheckNames)) {
+            throw new InvalidArgumentException(
+                'Invalid static check script(s): ' . implode(', ', $invalidCheckNames) . '.'
+            );
+        }
+    }
+
     /**
      * Parses and sets local ruleSet. If not found, simply returns and lets script continue.
      * @return void;