MQE-434: Create operation api unit tests

- fix static code sniff failures
- add static checks for unit and verification test

Author: imeron2433

bin/static-checks

@@ -3,6 +3,8 @@
 
 echo "===============================PHP CODE SNIFFER REPORT==============================="
 vendor/bin/phpcs ./src --standard=./dev/tests/static/Magento
+vendor/bin/phpcs ./dev/tests/unit --standard=./dev/tests/static/Magento
+vendor/bin/phpcs ./dev/tests/verification --standard=./dev/tests/static/Magento
 
 echo "===============================COPY PASTE DETECTOR REPORT==============================="
 vendor/bin/phpcpd ./src

dev/tests/unit/Magento/FunctionalTestFramework/Page/Objects/SectionObjectTest.php

@@ -18,7 +18,8 @@ class SectionObjectTest extends TestCase
     /**
      * Assert that the section object has an element
      */
-    public function testHasElement() {
+    public function testHasElement()
+    {
         $element1 = new ElementObject('element1', 'type', '#selector', null, '41', false);
         $element2 = new ElementObject('element2', 'type', '#selector', null, '42', true);
         $elements = [
@@ -32,7 +33,8 @@ public function testHasElement() {
     /**
      * Assert that the section object doesn't have an element
      */
-    public function testDoesntHaveElement() {
+    public function testDoesntHaveElement()
+    {
         $element2 = new ElementObject('element2', 'type', '#selector', null, '42', true);
         $elements = [
             'element2' => $element2
@@ -44,7 +46,8 @@ public function testDoesntHaveElement() {
     /**
      * Assert that an element object is returned
      */
-    public function testGetElement() {
+    public function testGetElement()
+    {
         $element1 = new ElementObject('element1', 'type', '#selector', null, '41', false);
         $element2 = new ElementObject('element2', 'type', '#selector', null, '42', true);
         $elements = [
@@ -60,7 +63,8 @@ public function testGetElement() {
     /**
      * Assert that null is returned if no such element
      */
-    public function testNullGetElement() {
+    public function testNullGetElement()
+    {
         $element1 = new ElementObject('element1', 'type', '#selector', null, '41', false);
         $elements = [
             'element1' => $element1

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

@@ -130,7 +130,6 @@ public function testResolveActionStepEntityData()
         $actionName = "myAction";
         $actionType = "myCustomType";
 
-
         // Set up mock data object
         $mockData = [$dataFieldName => $dataFieldValue];
         $mockDataObject = new EntityDataObject($dataObjectName, $dataObjectType, $mockData, null, null, null);

dev/tests/unit/Util/EntityDataObjectBuilder.php

@@ -113,5 +113,4 @@ public function withLinkedEntities($linkedEntities)
         $this->linkedEntities = $linkedEntities;
         return $this;
     }
-
-}
+}

dev/tests/unit/Util/OperationDefinitionBuilder.php

@@ -112,9 +112,10 @@ public function withMetadata($metadata)
             }
         }
 
-        $this->metadata = array_merge($this->metadata, OperationElementBuilder::buildOperationElementFields($primitives));
+        $this->metadata = array_merge(
+            $this->metadata,
+            OperationElementBuilder::buildOperationElementFields($primitives)
+        );
         return $this;
     }
-
-
-}
+}

dev/tests/unit/Util/OperationElementBuilder.php

@@ -61,7 +61,6 @@ class OperationElementBuilder
      */
     private $nestedElements;
 
-
     /**
      * Build function which takes params defined by the user and returns a new Operation Element.
      *
@@ -189,4 +188,4 @@ public static function buildOperationElementFields($fields)
 
         return $operationElements;
     }
-}
+}

dev/tests/verification/Tests/ActionGroupMergeGenerationTest.php

@@ -32,7 +32,6 @@ public function testActionGroupFunctionalCest()
         $this->runComparisonTest(self::ACTION_GROUP_FUNCTIONAL_CEST);
     }
 
-
     /**
      * Generate a Cest by name and assert that it equals the corresponding .txt source of truth
      *
@@ -51,7 +50,6 @@ private function runComparisonTest($cestName)
 
         $this->assertTrue(file_exists($cestFile));
 
-
         $this->assertFileEquals(
             self::RESOURCES_PATH . DIRECTORY_SEPARATOR . $cestName . ".txt",
             $cestFile

dev/tests/verification/Tests/ReferenceReplacementGenerationTest.php

@@ -67,7 +67,6 @@ private function runComparisonTest($cestName)
 
         $this->assertTrue(file_exists($cestFile));
 
-
         $this->assertFileEquals(
             self::RESOURCES_PATH . DIRECTORY_SEPARATOR . $cestName . ".txt",
             $cestFile

dev/tests/verification/Tests/SuiteGenerationTest.php

@@ -19,7 +19,6 @@ class SuiteGenerationTest extends TestCase
     private static $YML_EXISTS_FLAG = false;
     private static $TEST_GROUPS = [];
 
-
     public static function setUpBeforeClass()
     {
         if (file_exists(self::CONFIG_YML_FILE)) {