Youshido / GraphQL

src/Type/NonNullType.php

Doc Comments +3 added lines

@@ -50,6 +50,9 @@
         return $value;
     }
 
+    /**
+     * @param null|Scalar\StringType $value
+     */
     public function isValidValue($value)
     {
         return $value !== null;

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
      */

Tests/DataProvider/TestField.php

Doc Comments +4 added lines, -1 removed lines

@@ -16,13 +16,16 @@
 {
 
     /**
-     * @return AbstractObjectType
+     * @return IntType
      */
     public function getType()
     {
         return new IntType();
     }
 
+    /**
+     * @param string $value
+     */
     public function resolve($value, array $args, ResolveInfo $info)
     {
         return $value;

Tests/DataProvider/TestInputField.php

Doc Comments +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
 {
 
     /**
-     * @return AbstractInputObjectType
+     * @return IntType
      */
     public function getType()
     {

Tests/StarWars/Schema/QueryType.php

Unused Use Statements -1 removed lines

@@ -9,7 +9,6 @@
 
 
 use Youshido\GraphQL\Field\Field;
-use Youshido\GraphQL\Field\FieldFactory;
 use Youshido\GraphQL\Type\Object\AbstractObjectType;
 use Youshido\GraphQL\Type\Scalar\IdType;
 

Spacing +3 added lines, -3 removed lines

@@ -36,7 +36,7 @@ 
                 'args'    => [
                     'episode' => ['type' => new EpisodeEnum()]
                 ],
-                'resolve' => function ($root, $args) {
+                'resolve' => function($root, $args) {
                     return StarWarsData::getHero(isset($args['episode']) ? $args['episode'] : null);
                 },
             ])
@@ -46,7 +46,7 @@ 
                 'args'    => [
                     'id' => new IdType()
                 ],
-                'resolve' => function ($value = null, $args = []) {
+                'resolve' => function($value = null, $args = []) {
                     $humans = StarWarsData::humans();
 
                     return isset($humans[$args['id']]) ? $humans[$args['id']] : null;
@@ -58,7 +58,7 @@ 
                 'args'    => [
                     'id' => new IdType()
                 ],
-                'resolve' => function ($value = null, $args = []) {
+                'resolve' => function($value = null, $args = []) {
                     $droids = StarWarsData::droids();
 
                     return isset($droids[$args['id']]) ? $droids[$args['id']] : null;

Tests/Schema/IntrospectionTest.php

Spacing +7 added lines, -7 removed lines

@@ -135,7 +135,7 @@ 
             'description'       => 'latest description',
             'deprecationReason' => 'for test',
             'isDeprecated'      => true,
-            'resolve'           => function () {
+            'resolve'           => function() {
                 return [
                     'id'   => 1,
                     'name' => 'Alex'
@@ -197,8 +197,8 @@ 
                                 ['name' => 'String', 'fields' => null],
                                 ['name' => '__Schema', 'fields' => [['name' => 'queryType'], ['name' => 'mutationType'], ['name' => 'subscriptionType'], ['name' => 'types'], ['name' => 'directives']]],
                                 ['name' => '__Type', 'fields' => [['name' => 'name'], ['name' => 'kind'], ['name' => 'description'], ['name' => 'ofType'], ['name' => 'inputFields'], ['name' => 'enumValues'], ['name' => 'fields'], ['name' => 'interfaces'], ['name' => 'possibleTypes']]],
-                                ['name' => '__InputValue', 'fields' => [['name' => 'name'], ['name' => 'description'], ['name' => 'type'], ['name' => 'defaultValue'],]],
-                                ['name' => '__EnumValue', 'fields' => [['name' => 'name'], ['name' => 'description'], ['name' => 'deprecationReason'], ['name' => 'isDeprecated'],]],
+                                ['name' => '__InputValue', 'fields' => [['name' => 'name'], ['name' => 'description'], ['name' => 'type'], ['name' => 'defaultValue'], ]],
+                                ['name' => '__EnumValue', 'fields' => [['name' => 'name'], ['name' => 'description'], ['name' => 'deprecationReason'], ['name' => 'isDeprecated'], ]],
                                 ['name' => 'Boolean', 'fields' => null],
                                 ['name' => '__Field', 'fields' => [['name' => 'name'], ['name' => 'description'], ['name' => 'isDeprecated'], ['name' => 'deprecationReason'], ['name' => 'type'], ['name' => 'args']]],
                                 ['name' => '__Subscription', 'fields' => [['name' => 'name']]],
@@ -287,7 +287,7 @@ 
                 'id'   => ['type' => new IntType()],
                 'name' => ['type' => new IntType()],
             ],
-            'resolveType' => function ($type) {
+            'resolveType' => function($type) {
 
             }
         ]);
@@ -315,7 +315,7 @@ 
         $unionType = new UnionType([
             'name'        => 'UnionType',
             'types'       => [$object1, $object2],
-            'resolveType' => function () {
+            'resolveType' => function() {
 
             }
         ]);
@@ -326,7 +326,7 @@ 
             'args'    => [
                 'id' => ['type' => TypeMap::TYPE_INT]
             ],
-            'resolve' => function () {
+            'resolve' => function() {
                 return [
                     'id'   => 1,
                     'name' => 'Alex'
@@ -352,7 +352,7 @@ 
                     ]
                 ])
             ],
-            'resolve' => function () {
+            'resolve' => function() {
                 return null;
             }
         ]));

Tests/Schema/SchemaTest.php

Spacing +2 added lines, -2 removed lines

@@ -26,7 +26,7 @@ 
             'fields' => [
                 'currentTime' => [
                     'type'    => new StringType(),
-                    'resolve' => function ($value, $args, $type) {
+                    'resolve' => function($value, $args, $type) {
                         return 'May 5, 9:00am';
                     },
                     'args'    => [
@@ -55,7 +55,7 @@ 
 
         $this->assertEquals(1, count($schema->getMutationType()->getFields()));
 
-        $schema->addMutationField('changeUser', ['type' => new TestObjectType(), 'resolve' => function () { }]);
+        $schema->addMutationField('changeUser', ['type' => new TestObjectType(), 'resolve' => function() { }]);
         $this->assertEquals(2, count($schema->getMutationType()->getFields()));
 
     }

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;
             }
         ]);