Fixing static test failures

Author: pdohogne-magento

lib/internal/Magento/Framework/GraphQl/Type/Definition/FloatType.php

@@ -45,18 +45,18 @@ public function parseValue($value): float
     public function parseLiteral(Node $valueNode, ?array $variables = null): float
     {
         try {
-            if (
-                $valueNode instanceof ValueNode
+            if ($valueNode instanceof ValueNode
                 && !($valueNode instanceof FloatValueNode)
                 && !($valueNode instanceof IntValueNode)
-                && isset($valueNode->value)
-            ) {
+                && isset($valueNode->value))
+            {
                 $valueNode = new FloatValueNode([
                     'value' => (string)$this->parseValue($valueNode->value),
                     'loc' => $valueNode->loc
                 ]);
             }
         } catch (Exception $e) {
+            // If parsing fails let it go through the regular method to throw the expected error
         }
         return parent::parseLiteral($valueNode, $variables);
     }

lib/internal/Magento/Framework/GraphQl/Type/Definition/IntType.php

@@ -16,7 +16,7 @@
 /**
  * Replacement for the IntType definition that can typecast non-numeric values for backwards compatibility
  */
-class IntType extends \GraphQl\Type\Definition\IntType
+class IntType extends \GraphQL\Type\Definition\IntType
 {
     /**
      * Try to typecast valid values before running the native validations
@@ -44,17 +44,17 @@ public function parseValue($value): int
     public function parseLiteral(Node $valueNode, ?array $variables = null): int
     {
         try {
-            if (
-                $valueNode instanceof ValueNode
+            if ($valueNode instanceof ValueNode
                 && !($valueNode instanceof IntValueNode)
-                && isset($valueNode->value)
-            ) {
+                && isset($valueNode->value))
+            {
                 $valueNode = new IntValueNode([
                     'value' => (string)$this->parseValue($valueNode->value),
                     'loc' => $valueNode->loc
                 ]);
             }
         } catch (Exception $e) {
+            // If parsing fails let it go through the regular method to throw the expected error
         }
         return parent::parseLiteral($valueNode, $variables);
     }

lib/internal/Magento/Framework/GraphQl/Type/Definition/StringType.php

@@ -16,7 +16,7 @@
 /**
  * Replacement for the StringType definition that can typecast non-string values for backwards compatibility
  */
-class StringType extends \GraphQl\Type\Definition\StringType
+class StringType extends \GraphQL\Type\Definition\StringType
 {
     /**
      * Try to typecast valid values before running the native validations
@@ -44,17 +44,17 @@ public function parseValue($value): string
     public function parseLiteral(Node $valueNode, ?array $variables = null): string
     {
         try {
-            if (
-                $valueNode instanceof ValueNode
+            if ($valueNode instanceof ValueNode
                 && !($valueNode instanceof StringValueNode)
-                && isset($valueNode->value)
-            ) {
+                && isset($valueNode->value))
+            {
                 $valueNode = new StringValueNode([
                     'value' => $this->parseValue($valueNode->value),
                     'loc' => $valueNode->loc
                 ]);
             }
         } catch (Exception $e) {
+            // If parsing fails let it go through the regular method to throw the expected error
         }
         return parent::parseLiteral($valueNode, $variables);
     }

lib/internal/Magento/Framework/GraphQlSchemaStitching/GraphQlReader.php

@@ -350,7 +350,6 @@ private function addModuleNameToTypes(array $source, string $filePath): array
         return $source;
     }
 
-
     /**
      * Replace the standard type definitions with ones that know how to cast input values
      */