Youshido / GraphQL

src/Execution/Visitor/MaxComplexityQueryVisitor.php

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