Youshido / GraphQL

src/Execution/Visitor/MaxComplexityQueryVisitor.php

Spacing +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
     }
     $this->memo += $cost ?: $this->defaultScore;
     if ($this->memo > $this->maxScore) {
-      throw new \Exception('query exceeded max allowed complexity of ' . $this->maxScore);
+      throw new \Exception('query exceeded max allowed complexity of '.$this->maxScore);
     }
     return $this->memo;
   }

Indentation +17 added lines, -17 removed lines

@@ -16,34 +16,34 @@
 
 class MaxComplexityQueryVisitor extends AbstractQueryVisitor {
 
-  public $maxScore;
+    public $maxScore;
 
-  protected $defaultScore = 1;
+    protected $defaultScore = 1;
 
-  public function __construct($max) {
+    public function __construct($max) {
     parent::__construct();
 
     $this->maxScore = $max;
-  }
-
-  /**
-   * @param array       $args
-   * @param FieldConfig $fieldConfig
-   * @param int         $childScore
-   *
-   * @return int|null
-   * @throws \Exception
-   */
-  public function visit(array $args, FieldConfig $fieldConfig, $childScore = 0) {
+    }
+
+    /**
+     * @param array       $args
+     * @param FieldConfig $fieldConfig
+     * @param int         $childScore
+     *
+     * @return int|null
+     * @throws \Exception
+     */
+    public function visit(array $args, FieldConfig $fieldConfig, $childScore = 0) {
     $cost = $fieldConfig->get('cost');
     if (is_callable($cost)) {
-      $cost = $cost($args, $fieldConfig, $childScore);
+        $cost = $cost($args, $fieldConfig, $childScore);
     }
     $cost = $cost ?: $this->defaultScore;
     $this->memo += $cost;
     if ($this->memo > $this->maxScore) {
-      throw new \Exception('query exceeded max allowed complexity of ' . $this->maxScore);
+        throw new \Exception('query exceeded max allowed complexity of ' . $this->maxScore);
     }
     return $cost;
-  }
+    }
 }
\ No newline at end of file

src/Execution/Processor.php

Doc Comments +6 added lines, -3 removed lines

@@ -109,7 +109,7 @@ 
     }
 
     /**
-     * @param Query|Field        $query
+     * @param Query        $query
      * @param AbstractObjectType $currentLevelSchema
      * @return array|bool|mixed
      */
@@ -312,7 +312,7 @@ 
     }
 
     /**
-     * @param $query         Query|FragmentInterface
+     * @param Query $query         Query|FragmentInterface
      * @param $queryType     AbstractObjectType|TypeInterface|Field|AbstractType
      * @param $resolvedValue mixed
      * @param $value         array
@@ -415,6 +415,9 @@ 
         return $result;
     }
 
+    /**
+     * @param integer $max
+     */
     public function setMaxComplexity($max) {
         $this->maxComplexity = $max;
     }
@@ -470,7 +473,7 @@ 
      *
      * childScore costs are accumulated via values sent into the coroutine.
      *
-     * @param Query|Field|FragmentInterface $queryNode
+     * @param Query $queryNode
      * @param AbstractField                 $currentLevelAST
      *
      * @return \Generator

Braces +3 added lines, -1 removed lines

@@ -168,7 +168,9 @@
         $value = [];
 
         if ($fieldType->getKind() == TypeMap::KIND_LIST) {
-            if (!$this->resolveValidator->hasArrayAccess($resolvedValue)) return null;
+            if (!$this->resolveValidator->hasArrayAccess($resolvedValue)) {
+                return null;
+            }
             foreach ($resolvedValue as $resolvedValueItem) {
                 $value[] = [];
                 $index   = count($value) - 1;

src/Execution/Visitor/AbstractQueryVisitor.php

Indentation +15 added lines, -15 removed lines

@@ -13,24 +13,24 @@
 
 abstract class AbstractQueryVisitor {
 
-  protected $initialValue = 0;
+    protected $initialValue = 0;
 
-  protected $memo;
+    protected $memo;
 
-  public function __construct() {
+    public function __construct() {
     $this->memo = $this->initialValue;
-  }
+    }
 
-  public function getMemo() {
+    public function getMemo() {
     return $this->memo;
-  }
-
-  /**
-   * @param array       $args
-   * @param FieldConfig $fieldConfig
-   * @param int         $childScore
-   *
-   * @return int|null
-   */
-  abstract public function visit(array $args, FieldConfig $fieldConfig, $childScore = 0);
+    }
+
+    /**
+     * @param array       $args
+     * @param FieldConfig $fieldConfig
+     * @param int         $childScore
+     *
+     * @return int|null
+     */
+    abstract public function visit(array $args, FieldConfig $fieldConfig, $childScore = 0);
 }
\ No newline at end of file

Tests/Schema/ProcessorTest.php

Indentation +76 added lines, -76 removed lines

@@ -212,19 +212,19 @@ 
         $this->assertEquals(['data' => ['labels' => ['one', 'two']]], $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 () {
@@ -482,82 +482,82 @@ 
     }
 
     public function testComplexityReducer() {
-      $schema = new Schema(
-          [
-              'query' => new ObjectType(
-                  [
-                      'name'   => 'RootQuery',
-                      'fields' => [
-                          'me' => [
-                              'type'    => new ObjectType(
-                                  [
-                                      'name'   => 'User',
-                                      'fields' => [
-                                          'firstName' => [
-                                              'type'    => new StringType(),
-                                              'args'    => [
-                                                  'shorten' => new BooleanType()
-                                              ],
-                                              'resolve' => function ($value, $args) {
+        $schema = new Schema(
+            [
+                'query' => new ObjectType(
+                    [
+                        'name'   => 'RootQuery',
+                        'fields' => [
+                            'me' => [
+                                'type'    => new ObjectType(
+                                    [
+                                        'name'   => 'User',
+                                        'fields' => [
+                                            'firstName' => [
+                                                'type'    => new StringType(),
+                                                'args'    => [
+                                                    'shorten' => new BooleanType()
+                                                ],
+                                                'resolve' => function ($value, $args) {
                                                 return empty($args['shorten']) ? $value : $value;
-                                              }
-                                          ],
-                                          'lastName'  => new StringType(),
-                                          'code'      => new StringType(),
-                                          'likes'     => [
-                                              'type'    => new IntType(),
-                                              'cost'    => 10,
-                                              'resolve' => function () {
+                                                }
+                                            ],
+                                            'lastName'  => new StringType(),
+                                            'code'      => new StringType(),
+                                            'likes'     => [
+                                                'type'    => new IntType(),
+                                                'cost'    => 10,
+                                                '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;
-                              },
-                              'args'    => [
-                                  'cost' => [
-                                      'type'    => new IntType(),
-                                      'default' => 1
-                                  ]
-                              ]
-                          ]
-                      ]
-                  ]
-              )
-          ]
-      );
-      $processor = new Processor($schema);
-
-      $processor->setMaxComplexity(10);
-
-      $processor->processPayload('{ me { firstName, lastName } }');
-      $this->assertArrayNotHasKey('error', $processor->getResponseData());
-
-      $processor->processPayload('{ me { firstName, likes } }');
-      $this->assertEquals(['errors' => [['message' => 'query exceeded max allowed complexity of 10']]], $processor->getResponseData());
-
-      // don't let complexity reducer affect query errors
-      $processor->processPayload('{ me { badfield } }');
-      $this->assertArraySubset(['errors' => [['message' => 'Field "badfield" not found in type "User"']]], $processor->getResponseData());
+                                },
+                                'args'    => [
+                                    'cost' => [
+                                        'type'    => new IntType(),
+                                        'default' => 1
+                                    ]
+                                ]
+                            ]
+                        ]
+                    ]
+                )
+            ]
+        );
+        $processor = new Processor($schema);
 
-      foreach (range(1,5) as $cost_multiplier) {
+        $processor->setMaxComplexity(10);
+
+        $processor->processPayload('{ me { firstName, lastName } }');
+        $this->assertArrayNotHasKey('error', $processor->getResponseData());
+
+        $processor->processPayload('{ me { firstName, likes } }');
+        $this->assertEquals(['errors' => [['message' => 'query exceeded max allowed complexity of 10']]], $processor->getResponseData());
+
+        // don't let complexity reducer affect query errors
+        $processor->processPayload('{ me { badfield } }');
+        $this->assertArraySubset(['errors' => [['message' => 'Field "badfield" not found in type "User"']]], $processor->getResponseData());
+
+        foreach (range(1,5) as $cost_multiplier) {
         $visitor = new \Youshido\GraphQL\Execution\Visitor\MaxComplexityQueryVisitor(1000); // arbitrarily high cost
         $processor->processPayload("{ me (cost: $cost_multiplier) { firstName, lastName, code, likes } }", ['cost' => $cost_multiplier], [$visitor]);
         $expected = 1 + 13 * (1 + $cost_multiplier);
         $this->assertEquals($expected, $visitor->getMemo());
-      }
+        }
 
-      // TODO, variables not yet supported
-      /*$query = 'query costQuery ($cost: Int) { me (cost: $cost) { firstName, lastName, code, likes } }';
+        // TODO, variables not yet supported
+        /*$query = 'query costQuery ($cost: Int) { me (cost: $cost) { firstName, lastName, code, likes } }';
       foreach (range(1,5) as $cost_multiplier) {
         $visitor = new \Youshido\GraphQL\Execution\Visitor\MaxComplexityQueryVisitor(1000); // arbitrarily high cost
         $processor->processPayload($query, ['cost' => $cost_multiplier], [$visitor]);

Spacing +33 added lines, -33 removed lines

@@ -93,14 +93,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);
     }
 
@@ -113,20 +113,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' => [
@@ -139,13 +139,13 @@ 
                                     'args'    => [
                                         'shorten' => new BooleanType()
                                     ],
-                                    'resolve' => function ($value, $args) {
+                                    'resolve' => function($value, $args) {
                                         return empty($args['shorten']) ? $value : $value;
                                     }
                                 ],
                                 'id_alias'  => [
                                     'type'    => new IdType(),
-                                    'resolve' => function ($value) {
+                                    'resolve' => function($value) {
                                         return $value['id'];
                                     }
                                 ],
@@ -153,7 +153,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) {
@@ -172,19 +172,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'];
                         }
                     ]
@@ -215,7 +215,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'    => [
@@ -227,13 +227,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'];
                 }
             ]));
@@ -279,43 +279,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'];
                         }
                     ],
@@ -327,7 +327,7 @@ 
                                 'enum'   => new TestEnumType(),
                             ],
                         ]),
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return [
                                 'object' => [
                                     'name' => 'John'
@@ -396,10 +396,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;
                 }
@@ -410,11 +410,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' => [
@@ -424,7 +424,7 @@ 
                             'type' => ['type' => 'string']
                         ],
                         'cost' => 10,
-                        'resolve' => function ($value, $args) {
+                        'resolve' => function($value, $args) {
                             if ($args['type'] == 'object1') {
                                 return [
                                     'id' => 43
@@ -438,7 +438,7 @@ 
                     ],
                     'invalidUnion' => [
                         'type'    => $invalidUnion,
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return ['name' => 'name resolved'];
                         }
                     ],
@@ -498,7 +498,7 @@ 
                                               'args'    => [
                                                   'shorten' => new BooleanType()
                                               ],
-                                              'resolve' => function ($value, $args) {
+                                              'resolve' => function($value, $args) {
                                                 return empty($args['shorten']) ? $value : $value;
                                               }
                                           ],
@@ -507,18 +507,18 @@ 
                                           '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;
@@ -549,7 +549,7 @@ 
       $processor->processPayload('{ me { badfield } }');
       $this->assertArraySubset(['errors' => [['message' => 'Field "badfield" not found in type "User"']]], $processor->getResponseData());
 
-      foreach (range(1,5) as $cost_multiplier) {
+      foreach (range(1, 5) as $cost_multiplier) {
         $visitor = new \Youshido\GraphQL\Execution\Visitor\MaxComplexityQueryVisitor(1000); // arbitrarily high cost
         $processor->processPayload("{ me (cost: $cost_multiplier) { firstName, lastName, code, likes } }", ['cost' => $cost_multiplier], [$visitor]);
         $expected = 1 + 13 * (1 + $cost_multiplier);