ENGCOM-2317: Alphabetize Schema Fields#11 #104

Author: Valeriy Naida

dev/tests/integration/testsuite/Magento/Framework/GraphQl/Config/GraphQlReaderTest.php

@@ -191,6 +191,16 @@ enumValues(includeDeprecated: true) {
         $mergedSchemaResponseFields = array_merge($schemaResponseFieldsFirstHalf, $schemaResponseFieldsSecondHalf);
 
         foreach ($expectedOutput as $searchTerm) {
+            $sortFields = ['inputFields', 'fields'];
+            foreach ($sortFields as $sortField) {
+                isset($searchTerm[$sortField]) && is_array($searchTerm[$sortField])
+                    ? usort($searchTerm[$sortField], function ($a, $b) {
+                        $cmpField = 'name';
+                        return isset($a[$cmpField]) && isset($b[$cmpField])
+                            ? strcmp($a[$cmpField], $b[$cmpField]) : 0;
+                    }) : null;
+            }
+
             $this->assertTrue(
                 (in_array($searchTerm, $mergedSchemaResponseFields)),
                 'Missing type in the response'

lib/internal/Magento/Framework/GraphQl/Config.php

@@ -65,12 +65,16 @@ public function getConfigElement(string $configElementName) : ConfigElementInter
         }
 
         $fieldsInQuery = $this->queryFields->getFieldsUsedInQuery();
-        if (isset($data['fields']) && !empty($fieldsInQuery)) {
-            foreach ($data['fields'] as $fieldName => $fieldConfig) {
-                if (!isset($fieldsInQuery[$fieldName])) {
-                    unset($data['fields'][$fieldName]);
+        if (isset($data['fields'])) {
+            if (!empty($fieldsInQuery)) {
+                foreach ($data['fields'] as $fieldName => $fieldConfig) {
+                    if (!isset($fieldsInQuery[$fieldName])) {
+                        unset($data['fields'][$fieldName]);
+                    }
                 }
             }
+
+            ksort($data['fields']);
         }
 
         return $this->configElementFactory->createFromConfigData($data);