Youshido / GraphQL

src/Validator/SchemaValidator/SchemaValidator.php

Doc Comments +1 added lines, -1 removed lines

@@ -57,7 +57,7 @@
      * @param Field                 $objField
      * @param AbstractInterfaceType $interface
      *
-     * @return bool
+     * @return boolean|null
      *
      * @throws ConfigurationException
      */

Unused Use Statements -1 removed lines

@@ -8,7 +8,6 @@
 
 namespace Youshido\GraphQL\Validator\SchemaValidator;
 
-use Youshido\GraphQL\Config\AbstractConfig;
 use Youshido\GraphQL\Field\Field;
 use Youshido\GraphQL\Schema\AbstractSchema;
 use Youshido\GraphQL\Type\InterfaceType\AbstractInterfaceType;

Tests/Schema/ProcessorTest.php

Indentation +13 added lines, -13 removed lines

@@ -164,19 +164,19 @@
         $this->assertEquals(['data' => ['me' => ['firstName' => 'JOHN']]], $processor->getResponseData());
 
         $schema->getMutationType()
-               ->addField(new Field([
-                   'name'    => 'increaseCounter',
-                   'type'    => new IntType(),
-                   'resolve' => function ($value, $args, ResolveInfo $info) {
-                       return $this->_counter += $args['amount'];
-                   },
-                   'args'    => [
-                       'amount' => [
-                           'type'    => new IntType(),
-                           'default' => 1
-                       ]
-                   ]
-               ]))->addField(new Field([
+                ->addField(new Field([
+                    'name'    => 'increaseCounter',
+                    'type'    => new IntType(),
+                    'resolve' => function ($value, $args, ResolveInfo $info) {
+                        return $this->_counter += $args['amount'];
+                    },
+                    'args'    => [
+                        'amount' => [
+                            'type'    => new IntType(),
+                            'default' => 1
+                        ]
+                    ]
+                ]))->addField(new Field([
                 'name'    => 'invalidResolveTypeMutation',
                 'type'    => new NonNullType(new IntType()),
                 'resolve' => function () {

Spacing +35 added lines, -35 removed lines

@@ -95,14 +95,14 @@ 
                 'fields' => [
                     'list' => [
                         'type'    => new ListType(new StringType()),
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return null;
                         }
                     ]
                 ]
             ])
         ]));
-        $data      = $processor->processPayload(' { list }')->getResponseData();
+        $data = $processor->processPayload(' { list }')->getResponseData();
         $this->assertEquals(['data' => ['list' => null]], $data);
     }
 
@@ -115,20 +115,20 @@ 
                 'fields' => [
                     'list' => [
                         'type'    => new ListType(new StringType()),
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return null;
                         }
                     ]
                 ]
             ])
         ]));
-        $data      = $processor->processPayload(' { __schema { subscriptionType { name } } }')->getResponseData();
+        $data = $processor->processPayload(' { __schema { subscriptionType { name } } }')->getResponseData();
         $this->assertEquals(['data' => ['__schema' => ['subscriptionType' => null]]], $data);
     }
 
     public function testSchemaOperations()
     {
-        $schema    = new Schema([
+        $schema = new Schema([
             'query' => new ObjectType([
                 'name'   => 'RootQuery',
                 'fields' => [
@@ -141,13 +141,13 @@ 
                                     'args'    => [
                                         'shorten' => new BooleanType()
                                     ],
-                                    'resolve' => function ($value, $args) {
+                                    'resolve' => function($value, $args) {
                                         return empty($args['shorten']) ? $value['firstName'] : $value['firstName'];
                                     }
                                 ],
                                 'id_alias'  => [
                                     'type'    => new IdType(),
-                                    'resolve' => function ($value) {
+                                    'resolve' => function($value) {
                                         return $value['id'];
                                     }
                                 ],
@@ -155,7 +155,7 @@ 
                                 'code'      => new StringType(),
                             ]
                         ]),
-                        'resolve' => function ($value, $args) {
+                        'resolve' => function($value, $args) {
                             $data = ['id' => '123', 'firstName' => 'John', 'code' => '007'];
                             if (!empty($args['upper'])) {
                                 foreach ($data as $key => $value) {
@@ -174,19 +174,19 @@ 
                     ],
                     'randomUser'        => [
                         'type'    => new TestObjectType(),
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return ['invalidField' => 'John'];
                         }
                     ],
                     'invalidValueQuery' => [
                         'type'    => new TestObjectType(),
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return 'stringValue';
                         }
                     ],
                     'labels'            => [
                         'type'    => new ListType(new StringType()),
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return ['one', 'two'];
                         }
                     ]
@@ -217,7 +217,7 @@ 
                ->addField(new Field([
                    'name'    => 'increaseCounter',
                    'type'    => new IntType(),
-                   'resolve' => function ($value, $args, ResolveInfo $info) {
+                   'resolve' => function($value, $args, ResolveInfo $info) {
                        return $this->_counter += $args['amount'];
                    },
                    'args'    => [
@@ -229,13 +229,13 @@ 
                ]))->addField(new Field([
                 'name'    => 'invalidResolveTypeMutation',
                 'type'    => new NonNullType(new IntType()),
-                'resolve' => function () {
+                'resolve' => function() {
                     return null;
                 }
             ]))->addField(new Field([
                 'name'    => 'interfacedMutation',
                 'type'    => new TestInterfaceType(),
-                'resolve' => function () {
+                'resolve' => function() {
                     return ['name' => 'John'];
                 }
             ]));
@@ -304,7 +304,7 @@ 
                             ]))
                         ],
                         'type'    => new StringType(),
-                        'resolve' => function ($value, $args) {
+                        'resolve' => function($value, $args) {
                             return $args['argument1'];
                         }
                     ]
@@ -340,43 +340,43 @@ 
                 'fields' => [
                     'listQuery'                 => [
                         'type'    => new ListType(new TestEnumType()),
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return 'invalid list';
                         }
                     ],
                     'listEnumQuery'             => [
                         'type'    => new ListType(new TestEnumType()),
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return ['invalid enum'];
                         }
                     ],
                     'invalidEnumQuery'          => [
                         'type'    => new TestEnumType(),
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return 'invalid enum';
                         }
                     ],
                     'enumQuery'                 => [
                         'type'    => new TestEnumType(),
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return 1;
                         }
                     ],
                     'invalidNonNullQuery'       => [
                         'type'    => new NonNullType(new IntType()),
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return null;
                         }
                     ],
                     'invalidNonNullInsideQuery' => [
                         'type'    => new NonNullType(new IntType()),
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return 'hello';
                         }
                     ],
                     'objectQuery'               => [
                         'type'    => new TestObjectType(),
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return ['name' => 'John'];
                         }
                     ],
@@ -388,7 +388,7 @@ 
                                 'enum'   => new TestEnumType(),
                             ],
                         ]),
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return [
                                 'object' => [
                                     'name' => 'John'
@@ -462,10 +462,10 @@ 
             ]
         ]);
 
-        $union        = new UnionType([
+        $union = new UnionType([
             'name'        => 'TestUnion',
             'types'       => [$object1, $object2],
-            'resolveType' => function ($object) use ($object1, $object2) {
+            'resolveType' => function($object) use ($object1, $object2) {
                 if (isset($object['id'])) {
                     return $object1;
                 }
@@ -476,11 +476,11 @@ 
         $invalidUnion = new UnionType([
             'name'        => 'TestUnion',
             'types'       => [$object1, $object2],
-            'resolveType' => function ($object) use ($object3) {
+            'resolveType' => function($object) use ($object3) {
                 return $object3;
             }
         ]);
-        $processor    = new Processor(new Schema([
+        $processor = new Processor(new Schema([
             'query' => new ObjectType([
                 'name'   => 'RootQuery',
                 'fields' => [
@@ -490,7 +490,7 @@ 
                             'type' => ['type' => 'string']
                         ],
                         'cost'    => 10,
-                        'resolve' => function ($value, $args) {
+                        'resolve' => function($value, $args) {
                             if ($args['type'] == 'object1') {
                                 return [
                                     'id' => 43
@@ -504,7 +504,7 @@ 
                     ],
                     'invalidUnion' => [
                         'type'    => $invalidUnion,
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return ['name' => 'name resolved'];
                         }
                     ],
@@ -559,7 +559,7 @@ 
                 'fields' => [
                     'currentUser' => [
                         'type'    => new StringType(),
-                        'resolve' => function ($source, $args, ResolveInfo $info) {
+                        'resolve' => function($source, $args, ResolveInfo $info) {
                             return $info->getContainer()->get('user')['name'];
                         }
                     ]
@@ -577,7 +577,7 @@ 
 
     public function testComplexityReducer()
     {
-        $schema    = new Schema(
+        $schema = new Schema(
             [
                 'query' => new ObjectType(
                     [
@@ -593,7 +593,7 @@ 
                                                 'args'    => [
                                                     'shorten' => new BooleanType()
                                                 ],
-                                                'resolve' => function ($value, $args) {
+                                                'resolve' => function($value, $args) {
                                                     return empty($args['shorten']) ? $value : $value;
                                                 }
                                             ],
@@ -602,19 +602,19 @@ 
                                             'likes'     => [
                                                 'type'    => new IntType(),
                                                 'cost'    => 10,
-                                                'resolve' => function () {
+                                                'resolve' => function() {
                                                     return 42;
                                                 }
                                             ]
                                         ]
                                     ]
                                 ),
-                                'cost'    => function ($args, $context, $childCost) {
+                                'cost'    => function($args, $context, $childCost) {
                                     $argsCost = isset($args['cost']) ? $args['cost'] : 1;
 
                                     return 1 + $argsCost * $childCost;
                                 },
-                                'resolve' => function ($value, $args) {
+                                'resolve' => function($value, $args) {
                                     $data = ['firstName' => 'John', 'code' => '007'];
 
                                     return $data;

Tests/Library/Type/ScalarTypeTest.php

Spacing +5 added lines, -5 removed lines

@@ -22,7 +22,7 @@ 
     {
         foreach (TypeFactory::getScalarTypesNames() as $typeName) {
             $scalarType     = TypeFactory::getScalarType($typeName);
-            $testDataMethod = 'get' . $typeName . 'TestData';
+            $testDataMethod = 'get'.$typeName.'TestData';
 
             $this->assertNotEmpty($scalarType->getDescription());
             $this->assertEquals($scalarType->getKind(), TypeMap::KIND_SCALAR);
@@ -38,9 +38,9 @@ 
                 $this->assertParse($scalarType, $data, $serialized);
 
                 if ($isValid) {
-                    $this->assertTrue($scalarType->isValidValue($data), $typeName . ' validation for :' . serialize($data));
+                    $this->assertTrue($scalarType->isValidValue($data), $typeName.' validation for :'.serialize($data));
                 } else {
-                    $this->assertFalse($scalarType->isValidValue($data), $typeName . ' validation for :' . serialize($data));
+                    $this->assertFalse($scalarType->isValidValue($data), $typeName.' validation for :'.serialize($data));
                 }
             }
         }
@@ -52,12 +52,12 @@ 
 
     private function assertSerialization(AbstractScalarType $object, $input, $expected)
     {
-        $this->assertEquals($expected, $object->serialize($input), $object->getName() . ' serialize for: ' . serialize($input));
+        $this->assertEquals($expected, $object->serialize($input), $object->getName().' serialize for: '.serialize($input));
     }
 
     private function assertParse(AbstractScalarType $object, $input, $expected)
     {
-        $this->assertEquals($expected, $object->parseValue($input), $object->getName() . ' serialize for: ' . serialize($input));
+        $this->assertEquals($expected, $object->parseValue($input), $object->getName().' serialize for: '.serialize($input));
     }
 
 }

Tests/Library/Type/InterfaceTypeTest.php

Spacing +1 added lines, -1 removed lines

@@ -43,7 +43,7 @@
             'fields'      => [
                 'name' => new StringType()
             ],
-            'resolveType' => function ($object) {
+            'resolveType' => function($object) {
                 return $object;
             }
         ]);

Tests/Library/Type/UnionTypeTest.php

Spacing +3 added lines, -3 removed lines

@@ -32,7 +32,7 @@ 
                 new TestObjectType(),
                 $object
             ],
-            'resolveType' => function ($type) {
+            'resolveType' => function($type) {
                 return $type;
             }
         ]);
@@ -67,7 +67,7 @@ 
             'types'       => [
                 'test', new IntType()
             ],
-            'resolveType' => function ($type) {
+            'resolveType' => function($type) {
                 return $type;
             }
         ]);
@@ -84,7 +84,7 @@ 
             'types'       => [
                 new IntType()
             ],
-            'resolveType' => function ($type) {
+            'resolveType' => function($type) {
                 return $type;
             }
         ]);

Tests/Library/Type/EnumTypeTest.php

Spacing +1 added lines, -1 removed lines

@@ -102,7 +102,7 @@
                 'value' => 'disable'
             ]
         ];
-        $enumType   = new EnumType([
+        $enumType = new EnumType([
             'name'   => 'BoolEnum',
             'values' => $valuesData
         ]);

Tests/Library/Field/FieldAwareConfigTraitTest.php

Spacing +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
                 'type' => new IntType()
             ]
         ];
-        $config     = new ObjectTypeConfig([
+        $config = new ObjectTypeConfig([
             'name'   => 'UserType',
             'fields' => $fieldsData
         ]);

Tests/Library/Field/FieldTest.php

Spacing +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
         $fieldWithResolve = new Field([
             'name'    => 'title',
             'type'    => new StringType(),
-            'resolve' => function ($value, array $args, ResolveInfo $info) {
+            'resolve' => function($value, array $args, ResolveInfo $info) {
                 return $info->getReturnType()->serialize($value);
             }
         ]);

Tests/Library/Config/FieldConfigTest.php

Spacing +1 added lines, -1 removed lines

@@ -20,7 +20,7 @@
         $fieldConfig = new FieldConfig([
             'name'    => 'FirstName',
             'type'    => new StringType(),
-            'resolve' => function ($value, $args = [], $type = null) {
+            'resolve' => function($value, $args = [], $type = null) {
                 return 'John';
             }
         ]);