MQE-1676: Add a static-check that ensures action groups do not have unused arguments

added exception catch for incorrect action group definition

Author: soumyau

docs/commands/mftf.md

@@ -494,12 +494,15 @@ vendor/bin/mftf static-checks testDependencies
 ```bash
 vendor/bin/mftf static-checks actionGroupArguments
 ```
+```bash
+vendor/bin/mftf static-checks testDependencies actionGroupArguments
+```
 
 #### Existing static checks
 
 * Test Dependency: Checks that test dependencies do not violate Magento module's composer dependencies.
-* Unused Arguments: Checks that action groups do not have unused arguments.
-
+* Action Group Unused Arguments: Checks that action groups do not have unused arguments.
+    
 ### `upgrade:tests`
 
 Applies all the MFTF major version upgrade scripts to test components in the given path (`test.xml`, `data.xml`, etc).

src/Magento/FunctionalTestingFramework/StaticCheck/ActionGroupArgumentsCheck.php

@@ -7,6 +7,7 @@
 namespace Magento\FunctionalTestingFramework\StaticCheck;
 
 use Magento\FunctionalTestingFramework\Config\MftfApplicationConfig;
+use Magento\FunctionalTestingFramework\Exceptions\XmlException;
 use Magento\FunctionalTestingFramework\Test\Handlers\ActionGroupObjectHandler;
 use Magento\FunctionalTestingFramework\Test\Objects\ActionGroupObject;
 use Symfony\Component\Console\Input\InputInterface;
@@ -138,9 +139,10 @@ private function findUnusedArguments($actionGroupXml)
 
         preg_match_all(self::ACTIONGROUP_ARGUMENT_REGEX_PATTERN, $actionGroupXml, $arguments);
         preg_match(self::ACTIONGROUP_NAME_REGEX_PATTERN, $actionGroupXml, $actionGroupName);
-
-        $actionGroup = ActionGroupObjectHandler::getInstance()->getObject($actionGroupName[1]);
-
+        try {
+            $actionGroup = ActionGroupObjectHandler::getInstance()->getObject($actionGroupName[1]);
+        } catch (XmlException $e){
+        }
         foreach ($arguments[1] as $argument) {
             //pattern to match all argument references
             $patterns = [