MQE-483: Sanitize test naming to prevent Jenkins build failure

- add new validation util
- update test object extractor to use validation util
- add new unit test

Author: imeron2433

dev/tests/unit/Magento/FunctionalTestFramework/Test/Util/TestNameValidationUtilTest.php

@@ -0,0 +1,73 @@
+<?php
+/**
+ * Copyright © Magento, Inc. All rights reserved.
+ * See COPYING.txt for license details.
+ */
+
+namespace tests\unit\Magento\FunctionalTestFramework\Test\Util;
+
+use Magento\FunctionalTestingFramework\Exceptions\XmlException;
+use Magento\FunctionalTestingFramework\Test\Util\TestNameValidationUtil;
+use PHPUnit\Framework\TestCase;
+
+class TestNameValidationUtilTest extends TestCase
+{
+    /**
+     * Validate name with curly braces throws exception
+     */
+    public function testCurlyBracesInTestName()
+    {
+        $this->validateBlacklistedTestName("{{curlyBraces}}");
+    }
+
+    /**
+     * Validate name with quotation marks throws exception
+     */
+    public function testQuotesInTestName()
+    {
+        $this->validateBlacklistedTestName("\"quotes\"");
+    }
+
+    /**
+     * Validate name with single quotes throws exception
+     */
+    public function testSingleQuotesInTestName()
+    {
+        $this->validateBlacklistedTestName("'singleQuotes'");
+    }
+
+    /**
+     * Validate name with parenthesis throws execption
+     */
+    public function testParenthesesInTestName()
+    {
+        $this->validateBlacklistedTestName("(parenthesis)");
+    }
+
+    /**
+     * Validate name with dollar signs throws exception
+     */
+    public function testDollarSignInTestName()
+    {
+        $this->validateBlacklistedTestName("\$dollarSign\$");
+    }
+
+    /**
+     * Validate name with spaces throws exception
+     */
+    public function testSpacesInTestName()
+    {
+        $this->validateBlacklistedTestName("Test Name With Spaces");
+    }
+
+    /**
+     * Method which takes the name of the test expecting an invalid char. Runs the validation method against name.
+     *
+     * @param string $testName
+     */
+    private function validateBlacklistedTestName($testName)
+    {
+        $this->expectException(XmlException::class);
+        TestNameValidationUtil::validateName($testName);
+    }
+}

src/Magento/FunctionalTestingFramework/Test/Util/TestNameValidationUtil.php

@@ -0,0 +1,32 @@
+<?php
+/**
+ * Copyright © Magento, Inc. All rights reserved.
+ * See COPYING.txt for license details.
+ */
+
+namespace Magento\FunctionalTestingFramework\Test\Util;
+
+use Magento\FunctionalTestingFramework\Exceptions\XmlException;
+
+class TestNameValidationUtil
+{
+    const BLACKLISTED_CHAR = [" ", ",", "'", "\"" , "{", "}", "$", "(", ")" ];
+
+    /**
+     * Function which runs a validation against the blacklisted char defined in this class. Validation occurs to insure
+     * allure report does not error/future devOps builds do not error against illegal char.
+     *
+     * @param string $testName
+     * @return void
+     * @throws XmlException
+     */
+    public static function validateName($testName)
+    {
+        $testChars = str_split($testName);
+
+        $diff = array_diff($testChars, self::BLACKLISTED_CHAR);
+        if (count($diff) != count($testChars)) {
+            throw new XmlException("Test name ${testName} contains illegal characters, please fix and re-run");
+        }
+    }
+}

src/Magento/FunctionalTestingFramework/Test/Util/TestObjectExtractor.php

@@ -63,6 +63,9 @@ public function extractTestData($cestTestData)
                 continue;
             }
 
+            // validate the test name for blacklisted char (will cause allure report issues) MQE-483
+            TestNameValidationUtil::validateName($testName);
+
             $testAnnotations = [];
             $testActions = $this->stripDescriptorTags(
                 $testData,