CS fixes

.scrutinizer.yml

@@ -0,0 +1,3 @@
+filter:
+    excluded_paths:
+        - "src/Mapping/Guess/DoctrineGuesser.php"

src/Mapping/Context/FieldContext.php

@@ -20,7 +20,7 @@ class FieldContext extends ContainerContext
      */
     public function __construct(Field $field, FieldContainer $container, SchemaContainer $schema)
     {
-        $this->field     = $field;
+        $this->field = $field;
         parent::__construct($container, $schema);
     }
 

src/Mapping/Driver/YamlDriver.php

@@ -32,6 +32,11 @@ private function loadFile($path, SchemaContainer $schemaContainer)
     {
         $config = Yaml::parse($this->getFileContent($path));
 
+        if (!is_array($config)) {
+            return;
+        }
+
+        /** @var array $config */
         foreach ($config as $type => $mapping) {
             switch ($type) {
                 case 'query':

src/Mapping/Field.php

@@ -31,7 +31,7 @@ class Field extends AbstractType
     /**
      * @var array
      */
-    private $resolveConfig = array();
+    private $resolveConfig = [];
 
     /**
      * @var Field[]

src/Mapping/Guess/DoctrineGuesser.php

@@ -20,7 +20,7 @@ class DoctrineGuesser implements TypeGuesserInterface, FieldGuesserInterface
     private $metadataFactory;
 
     /**
-     * @param ClassMetadataFactory $metadataFactory
+     * @param ObjectManager $objectManager
      */
     public function __construct(ObjectManager $objectManager)
     {

src/Mapping/Guess/PropertyGuesser.php

@@ -28,7 +28,7 @@ public function guessFieldResolveConfig(FieldContext $fieldContext)
 
     /**
      * @param string       $className
-     * @param FieldContext $field
+     * @param FieldContext $fieldContext
      * @return string|null
      */
     private function getAccessor($className, FieldContext $fieldContext)

src/Mapping/Type.php

@@ -61,7 +61,7 @@ public function toMapping()
         $mapping = parent::toMapping();
 
         $mapping['extends'] = $this->extends;
-        $mapping['values'] = $this->values;
+        $mapping['values']  = $this->values;
 
         return $mapping;
     }

src/Schema/SchemaFactory.php

@@ -160,16 +160,18 @@ private function createType(FieldContainer $type)
             $this->prepareFields($type->getFields());
         }
 
-        switch ($type->getInternalType()) {
+        $internalType = $type->getInternalType();
+
+        switch ($internalType) {
             case 'ObjectType':
-                $type = new GQLDefinition\ObjectType($type->toMapping());
+                return new GQLDefinition\ObjectType($type->toMapping());
                 break;
             case 'EnumType':
-                $type = new GQLDefinition\EnumType($type->toMapping());
+                return new GQLDefinition\EnumType($type->toMapping());
                 break;
+            default:
+                throw new \InvalidArgumentException(sprintf('Undefined internal type "%s"', $internalType));
         }
-
-        return $type;
     }
 
     /**