Youshido / GraphQL

Tests/Parser/RequestTest.php

Spacing +3 added lines, -3 removed lines

@@ -18,8 +18,8 @@ 
 
     public function testMethods()
     {
-        $fragment1     = new Fragment('fragmentName1', 'test', [], new Location(1,1));
-        $fragment2     = new Fragment('fragmentName2', 'test', [], new Location(1,1));
+        $fragment1     = new Fragment('fragmentName1', 'test', [], new Location(1, 1));
+        $fragment2     = new Fragment('fragmentName2', 'test', [], new Location(1, 1));
         $queriesData   = ['query1', 'query2'];
         $mutationsData = ['mutation1', 'mutation2'];
         $fragmentsData = [$fragment1];
@@ -54,7 +54,7 @@ 
     public function testSetVariableParseJson()
     {
         $variables = '{"foo": "bar"}';
-        $expectedVariableArray = [ 'foo' => 'bar' ];
+        $expectedVariableArray = ['foo' => 'bar'];
 
         $request = new Request([], $variables);
         $this->assertEquals($expectedVariableArray, $request->getVariables());

Tests/Parser/AstTest.php

Spacing +22 added lines, -22 removed lines

@@ -25,7 +25,7 @@ 
 
     public function testArgument()
     {
-        $argument = new Argument('test', new Literal('test', new Location(1,1)), new Location(1,1));
+        $argument = new Argument('test', new Literal('test', new Location(1, 1)), new Location(1, 1));
 
         $this->assertNotNull($argument->getValue());
         $this->assertEquals($argument->getName(), 'test');
@@ -39,7 +39,7 @@ 
 
     public function testField()
     {
-        $field = new Field('field', null, [], new Location(1,1));
+        $field = new Field('field', null, [], new Location(1, 1));
 
         $this->assertEquals($field->getName(), 'field');
         $this->assertEmpty($field->getArguments());
@@ -50,7 +50,7 @@ 
         $this->assertEquals($field->getAlias(), 'alias');
         $this->assertEquals($field->getName(), 'alias');
 
-        $field->addArgument(new Argument('argument', new Literal('argument value', new Location(1,1)), new Location(1,1)));
+        $field->addArgument(new Argument('argument', new Literal('argument value', new Location(1, 1)), new Location(1, 1)));
         $this->assertTrue($field->hasArguments());
         $this->assertEquals(['argument' => 'argument value'], $field->getKeyValueArguments());
     }
@@ -58,10 +58,10 @@ 
     public function testFragment()
     {
         $fields = [
-            new Field('field', null, [], new Location(1,1))
+            new Field('field', null, [], new Location(1, 1))
         ];
 
-        $fragment = new Fragment('shipInfo', 'Ship', $fields, new Location(1,1));
+        $fragment = new Fragment('shipInfo', 'Ship', $fields, new Location(1, 1));
 
         $this->assertEquals('shipInfo', $fragment->getName());
         $this->assertEquals('Ship', $fragment->getModel());
@@ -74,7 +74,7 @@ 
         $this->assertEquals('Boat', $fragment->getModel());
 
         $newField = [
-            new Field('id', null, [], new Location(1,1))
+            new Field('id', null, [], new Location(1, 1))
         ];
         $fragment->setFields($newField);
         $this->assertEquals($newField, $fragment->getFields());
@@ -86,7 +86,7 @@ 
 
     public function testFragmentReference()
     {
-        $reference = new FragmentReference('shipInfo', new Location(1,1));
+        $reference = new FragmentReference('shipInfo', new Location(1, 1));
 
         $this->assertEquals('shipInfo', $reference->getName());
 
@@ -97,10 +97,10 @@ 
     public function testTypedFragmentReference()
     {
         $fields = [
-            new Field('id', null, [], new Location(1,1))
+            new Field('id', null, [], new Location(1, 1))
         ];
 
-        $reference = new TypedFragmentReference('Ship', $fields, new Location(1,1));
+        $reference = new TypedFragmentReference('Ship', $fields, new Location(1, 1));
 
         $this->assertEquals('Ship', $reference->getTypeName());
         $this->assertEquals($fields, $reference->getFields());
@@ -109,8 +109,8 @@ 
         $this->assertEquals('BigBoat', $reference->getTypeName());
 
         $newFields = [
-            new Field('name', null, [], new Location(1,1)),
-            new Field('id', null, [], new Location(1,1))
+            new Field('name', null, [], new Location(1, 1)),
+            new Field('id', null, [], new Location(1, 1))
         ];
 
         $reference->setFields($newFields);
@@ -120,14 +120,14 @@ 
     public function testQuery()
     {
         $arguments = [
-            new Argument('limit', new Literal('10', new Location(1,1)), new Location(1,1))
+            new Argument('limit', new Literal('10', new Location(1, 1)), new Location(1, 1))
         ];
 
         $fields = [
-            new Field('id', null, [], new Location(1,1))
+            new Field('id', null, [], new Location(1, 1))
         ];
 
-        $query = new Query('ships', 'lastShips', $arguments, $fields, new Location(1,1));
+        $query = new Query('ships', 'lastShips', $arguments, $fields, new Location(1, 1));
 
         $this->assertEquals('ships', $query->getName());
         $this->assertEquals('lastShips', $query->getAlias());
@@ -147,23 +147,23 @@ 
         $this->assertFalse($query->hasArguments());
         $this->assertFalse($query->hasFields());
 
-        $query->addArgument(new Argument('offset', new Literal(10, new Location(1,1)), new Location(1,1)));
+        $query->addArgument(new Argument('offset', new Literal(10, new Location(1, 1)), new Location(1, 1)));
         $this->assertTrue($query->hasArguments());
     }
 
     public function testArgumentValues()
     {
-        $list = new InputList(['a', 'b'], new Location(1,1));
+        $list = new InputList(['a', 'b'], new Location(1, 1));
         $this->assertEquals(['a', 'b'], $list->getValue());
         $list->setValue(['a']);
         $this->assertEquals(['a'], $list->getValue());
 
-        $inputObject = new InputObject(['a', 'b'], new Location(1,1));
+        $inputObject = new InputObject(['a', 'b'], new Location(1, 1));
         $this->assertEquals(['a', 'b'], $inputObject->getValue());
         $inputObject->setValue(['a']);
         $this->assertEquals(['a'], $inputObject->getValue());
 
-        $literal = new Literal('text', new Location(1,1));
+        $literal = new Literal('text', new Location(1, 1));
         $this->assertEquals('text', $literal->getValue());
         $literal->setValue('new text');
         $this->assertEquals('new text', $literal->getValue());
@@ -171,7 +171,7 @@ 
 
     public function testVariable()
     {
-        $variable = new Variable('id', 'int', false, false, new Location(1,1));
+        $variable = new Variable('id', 'int', false, false, new Location(1, 1));
 
         $this->assertEquals('id', $variable->getName());
         $this->assertEquals('int', $variable->getTypeName());
@@ -190,8 +190,8 @@ 
         $this->assertTrue($variable->isNullable());
         $this->assertTrue($variable->isArray());
 
-        $variable->setValue(new Literal('text', new Location(1,1)));
-        $this->assertEquals(new Literal('text', new Location(1,1)), $variable->getValue());
+        $variable->setValue(new Literal('text', new Location(1, 1)));
+        $this->assertEquals(new Literal('text', new Location(1, 1)), $variable->getValue());
     }
 
     /**
@@ -199,7 +199,7 @@ 
      */
     public function testVariableLogicException()
     {
-        $variable = new Variable('id', 'int', false, false, new Location(1,1));
+        $variable = new Variable('id', 'int', false, false, new Location(1, 1));
         $variable->getValue();
     }
 }

Tests/Parser/VariableTest.php

Spacing +2 added lines, -2 removed lines

@@ -14,7 +14,7 @@ 
      */
     public function testGetValue($actual, $expected)
     {
-        $var = new Variable('foo', 'bar', false, false, new Location(1,1));
+        $var = new Variable('foo', 'bar', false, false, new Location(1, 1));
         $var->setValue($actual);
         $this->assertEquals($var->getValue(), $expected);
     }
@@ -25,7 +25,7 @@ 
      */
     public function testGetNullValueException()
     {
-        $var = new Variable('foo', 'bar', false, false, new Location(1,1));
+        $var = new Variable('foo', 'bar', false, false, new Location(1, 1));
         $var->getValue();
     }
 

Tests/Library/Validator/RequestValidatorTest.php

Spacing +1 added lines, -1 removed lines

@@ -111,7 +111,7 @@
                             new Argument('test2', new VariableReference('test2', $variable2, new Location(1, 1)), new Location(1, 1)),
                         ], [
                             new Field('test', null, [], new Location(1, 1))
-                        ], new Location(1,1))
+                        ], new Location(1, 1))
                     ],
                     'variables'          => [
                         $variable1,

Tests/Schema/FragmentsTest.php

Unused Use Statements -4 removed lines

@@ -2,14 +2,10 @@
 
 namespace Youshido\Tests\Schema;
 
-use Youshido\GraphQL\Config\Object\InterfaceTypeConfig;
-use Youshido\GraphQL\Config\Object\ObjectTypeConfig;
 use Youshido\GraphQL\Execution\Processor;
 use Youshido\GraphQL\Schema\Schema;
 use Youshido\GraphQL\Type\InterfaceType\AbstractInterfaceType;
-use Youshido\GraphQL\Type\InterfaceType\InterfaceType;
 use Youshido\GraphQL\Type\ListType\ListType;
-use Youshido\GraphQL\Type\NonNullType;
 use Youshido\GraphQL\Type\Object\AbstractObjectType;
 use Youshido\GraphQL\Type\Object\ObjectType;
 use Youshido\GraphQL\Type\Scalar\IdType;

Spacing +2 added lines, -2 removed lines

@@ -103,7 +103,7 @@ 
                 'fields' => [
                     'user' => [
                         'type'    => new UserType(),
-                        'resolve' => function ($args) {
+                        'resolve' => function($args) {
                             return [
                                 'id'           => 'user-id-1',
                                 'fullName'     => 'Alex',
@@ -217,7 +217,7 @@ 
                 'fields' => [
                     'user' => [
                         'type'    => new UserType(),
-                        'resolve' => function ($args) {
+                        'resolve' => function($args) {
                             return [
                                 'id'       => 'user-id-1',
                                 'fullName' => 'Alex',

Tests/Schema/NonNullableTest.php

Spacing +6 added lines, -6 removed lines

@@ -48,13 +48,13 @@ 
                 'fields' => [
                     'nonNullScalar'        => [
                         'type'    => new NonNullType(new IntType()),
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return null;
                         },
                     ],
                     'nonNullList'          => [
                         'type'    => new NonNullType(new ListType(new IntType())),
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return null;
                         }
                     ],
@@ -66,7 +66,7 @@ 
                                 'name' => new StringType(),
                             ]
                         ])),
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return [
                                 'id'   => new uid('6cfb044c-9c0a-4ddd-9ef8-a0b940818db3'),
                                 'name' => 'Alex'
@@ -75,7 +75,7 @@ 
                     ],
                     'nonNullListOfNpnNull' => [
                         'type'    => new NonNullType(new ListType(new NonNullType(new IntType()))),
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return [1, null];
                         }
                     ],
@@ -84,7 +84,7 @@ 
                             'ids' => new NonNullType(new ListType(new IntType()))
                         ],
                         'type'    => new IntType(),
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return 1;
                         }
                     ],
@@ -93,7 +93,7 @@ 
                             'ids' => new NonNullType(new ListType(new NonNullType(new IntType())))
                         ],
                         'type'    => new IntType(),
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return 1;
                         }
                     ],

src/Config/Schema/SchemaConfig.php

Doc Comments +4 added lines, -1 removed lines

@@ -23,6 +23,9 @@ 
      */
     private $typesList;
 
+    /**
+     * @param \Youshido\GraphQL\Schema\AbstractSchema $contextObject
+     */
     public function __construct(array $configData, $contextObject = null, $finalClass = false)
     {
         $this->typesList = new SchemaTypesList();
@@ -78,7 +81,7 @@ 
     }
 
     /**
-     * @param $query AbstractObjectType
+     * @param ObjectType $query AbstractObjectType
      *
      * @return SchemaConfig
      */

src/Field/AbstractField.php

Spacing +2 added lines, -2 removed lines

@@ -26,7 +26,7 @@ 
     }
     protected $isFinal = false;
 
-    private $nameCache            = null;
+    private $nameCache = null;
 
     public function __construct(array $config = [])
     {
@@ -34,7 +34,7 @@ 
             $config['type'] = $this->getType();
             $config['name'] = $this->getName();
             if (empty($config['name'])) {
-                $config['name'] =$this->getAutoName();
+                $config['name'] = $this->getAutoName();
             }
         }
 

src/Type/Enum/AbstractEnumType.php

Braces +3 added lines, -1 removed lines

@@ -48,7 +48,9 @@
      */
     public function isValidValue($value)
     {
-        if (is_null($value)) return true;
+        if (is_null($value)) {
+            return true;
+        }
         foreach ($this->getConfig()->get('values') as $item) {
             if ($value === $item['name'] || $value === $item['value']) {
                 return true;