overblog / GraphQLBundle

tests/Functional/Security/QueryMaxDepthTest.php

Indentation -77 removed lines

@@ -9,80 +9,3 @@
 final class QueryMaxDepthTest extends TestCase
 {
     private string $userFriendsWithoutViolationQuery = <<<'EOF'
-        query {
-          user {
-            friends(first:1) {
-              edges {
-                node {
-                  name
-                }
-              }
-            }
-          }
-        }
-        EOF;
-
-    private string $userFriendsWithViolationQuery = <<<'EOF'
-        query {
-          user {
-            friends(first: 1) {
-              edges {
-                node {
-                  name
-                  friends {
-                    edges {
-                      node {
-                        name
-                      }
-                    }
-                  }
-                }
-              }
-            }
-          }
-        }
-        EOF;
-
-    public function testMaxDepthReachLimitation(): void
-    {
-        $expected = [
-            'errors' => [
-                [
-                    'message' => 'Max query depth should be 3 but got 6.',
-                ],
-            ],
-        ];
-
-        $this->assertResponse($this->userFriendsWithViolationQuery, $expected, self::ANONYMOUS_USER, 'queryMaxDepth');
-    }
-
-    public function testMaxDepthReachLimitationEnv(): void
-    {
-        $expected = [
-            'errors' => [
-                [
-                    'message' => 'Max query depth should be 3 but got 6.',
-                ],
-            ],
-        ];
-
-        $this->assertResponse($this->userFriendsWithViolationQuery, $expected, self::ANONYMOUS_USER, 'queryMaxDepthEnv');
-    }
-
-    public function testComplexityUnderLimitation(): void
-    {
-        $expected = [
-            'data' => [
-                'user' => [
-                    'friends' => [
-                        'edges' => [
-                            ['node' => ['name' => 'Nick']],
-                        ],
-                    ],
-                ],
-            ],
-        ];
-
-        $this->assertResponse($this->userFriendsWithoutViolationQuery, $expected, self::ANONYMOUS_USER, 'queryMaxDepth');
-    }
-}

Switch Indentation -77 removed lines

@@ -9,80 +9,3 @@
 final class QueryMaxDepthTest extends TestCase
 {
     private string $userFriendsWithoutViolationQuery = <<<'EOF'
-        query {
-          user {
-            friends(first:1) {
-              edges {
-                node {
-                  name
-                }
-              }
-            }
-          }
-        }
-        EOF;
-
-    private string $userFriendsWithViolationQuery = <<<'EOF'
-        query {
-          user {
-            friends(first: 1) {
-              edges {
-                node {
-                  name
-                  friends {
-                    edges {
-                      node {
-                        name
-                      }
-                    }
-                  }
-                }
-              }
-            }
-          }
-        }
-        EOF;
-
-    public function testMaxDepthReachLimitation(): void
-    {
-        $expected = [
-            'errors' => [
-                [
-                    'message' => 'Max query depth should be 3 but got 6.',
-                ],
-            ],
-        ];
-
-        $this->assertResponse($this->userFriendsWithViolationQuery, $expected, self::ANONYMOUS_USER, 'queryMaxDepth');
-    }
-
-    public function testMaxDepthReachLimitationEnv(): void
-    {
-        $expected = [
-            'errors' => [
-                [
-                    'message' => 'Max query depth should be 3 but got 6.',
-                ],
-            ],
-        ];
-
-        $this->assertResponse($this->userFriendsWithViolationQuery, $expected, self::ANONYMOUS_USER, 'queryMaxDepthEnv');
-    }
-
-    public function testComplexityUnderLimitation(): void
-    {
-        $expected = [
-            'data' => [
-                'user' => [
-                    'friends' => [
-                        'edges' => [
-                            ['node' => ['name' => 'Nick']],
-                        ],
-                    ],
-                ],
-            ],
-        ];
-
-        $this->assertResponse($this->userFriendsWithoutViolationQuery, $expected, self::ANONYMOUS_USER, 'queryMaxDepth');
-    }
-}

Spacing -77 removed lines

@@ -9,80 +9,3 @@
 final class QueryMaxDepthTest extends TestCase
 {
     private string $userFriendsWithoutViolationQuery = <<<'EOF'
-        query {
-          user {
-            friends(first:1) {
-              edges {
-                node {
-                  name
-                }
-              }
-            }
-          }
-        }
-        EOF;
-
-    private string $userFriendsWithViolationQuery = <<<'EOF'
-        query {
-          user {
-            friends(first: 1) {
-              edges {
-                node {
-                  name
-                  friends {
-                    edges {
-                      node {
-                        name
-                      }
-                    }
-                  }
-                }
-              }
-            }
-          }
-        }
-        EOF;
-
-    public function testMaxDepthReachLimitation(): void
-    {
-        $expected = [
-            'errors' => [
-                [
-                    'message' => 'Max query depth should be 3 but got 6.',
-                ],
-            ],
-        ];
-
-        $this->assertResponse($this->userFriendsWithViolationQuery, $expected, self::ANONYMOUS_USER, 'queryMaxDepth');
-    }
-
-    public function testMaxDepthReachLimitationEnv(): void
-    {
-        $expected = [
-            'errors' => [
-                [
-                    'message' => 'Max query depth should be 3 but got 6.',
-                ],
-            ],
-        ];
-
-        $this->assertResponse($this->userFriendsWithViolationQuery, $expected, self::ANONYMOUS_USER, 'queryMaxDepthEnv');
-    }
-
-    public function testComplexityUnderLimitation(): void
-    {
-        $expected = [
-            'data' => [
-                'user' => [
-                    'friends' => [
-                        'edges' => [
-                            ['node' => ['name' => 'Nick']],
-                        ],
-                    ],
-                ],
-            ],
-        ];
-
-        $this->assertResponse($this->userFriendsWithoutViolationQuery, $expected, self::ANONYMOUS_USER, 'queryMaxDepth');
-    }
-}

Braces -77 removed lines

@@ -9,80 +9,3 @@
 final class QueryMaxDepthTest extends TestCase
 {
     private string $userFriendsWithoutViolationQuery = <<<'EOF'
-        query {
-          user {
-            friends(first:1) {
-              edges {
-                node {
-                  name
-                }
-              }
-            }
-          }
-        }
-        EOF;
-
-    private string $userFriendsWithViolationQuery = <<<'EOF'
-        query {
-          user {
-            friends(first: 1) {
-              edges {
-                node {
-                  name
-                  friends {
-                    edges {
-                      node {
-                        name
-                      }
-                    }
-                  }
-                }
-              }
-            }
-          }
-        }
-        EOF;
-
-    public function testMaxDepthReachLimitation(): void
-    {
-        $expected = [
-            'errors' => [
-                [
-                    'message' => 'Max query depth should be 3 but got 6.',
-                ],
-            ],
-        ];
-
-        $this->assertResponse($this->userFriendsWithViolationQuery, $expected, self::ANONYMOUS_USER, 'queryMaxDepth');
-    }
-
-    public function testMaxDepthReachLimitationEnv(): void
-    {
-        $expected = [
-            'errors' => [
-                [
-                    'message' => 'Max query depth should be 3 but got 6.',
-                ],
-            ],
-        ];
-
-        $this->assertResponse($this->userFriendsWithViolationQuery, $expected, self::ANONYMOUS_USER, 'queryMaxDepthEnv');
-    }
-
-    public function testComplexityUnderLimitation(): void
-    {
-        $expected = [
-            'data' => [
-                'user' => [
-                    'friends' => [
-                        'edges' => [
-                            ['node' => ['name' => 'Nick']],
-                        ],
-                    ],
-                ],
-            ],
-        ];
-
-        $this->assertResponse($this->userFriendsWithoutViolationQuery, $expected, self::ANONYMOUS_USER, 'queryMaxDepth');
-    }
-}

Upper-Lower-Casing -77 removed lines

@@ -9,80 +9,3 @@
 final class QueryMaxDepthTest extends TestCase
 {
     private string $userFriendsWithoutViolationQuery = <<<'EOF'
-        query {
-          user {
-            friends(first:1) {
-              edges {
-                node {
-                  name
-                }
-              }
-            }
-          }
-        }
-        EOF;
-
-    private string $userFriendsWithViolationQuery = <<<'EOF'
-        query {
-          user {
-            friends(first: 1) {
-              edges {
-                node {
-                  name
-                  friends {
-                    edges {
-                      node {
-                        name
-                      }
-                    }
-                  }
-                }
-              }
-            }
-          }
-        }
-        EOF;
-
-    public function testMaxDepthReachLimitation(): void
-    {
-        $expected = [
-            'errors' => [
-                [
-                    'message' => 'Max query depth should be 3 but got 6.',
-                ],
-            ],
-        ];
-
-        $this->assertResponse($this->userFriendsWithViolationQuery, $expected, self::ANONYMOUS_USER, 'queryMaxDepth');
-    }
-
-    public function testMaxDepthReachLimitationEnv(): void
-    {
-        $expected = [
-            'errors' => [
-                [
-                    'message' => 'Max query depth should be 3 but got 6.',
-                ],
-            ],
-        ];
-
-        $this->assertResponse($this->userFriendsWithViolationQuery, $expected, self::ANONYMOUS_USER, 'queryMaxDepthEnv');
-    }
-
-    public function testComplexityUnderLimitation(): void
-    {
-        $expected = [
-            'data' => [
-                'user' => [
-                    'friends' => [
-                        'edges' => [
-                            ['node' => ['name' => 'Nick']],
-                        ],
-                    ],
-                ],
-            ],
-        ];
-
-        $this->assertResponse($this->userFriendsWithoutViolationQuery, $expected, self::ANONYMOUS_USER, 'queryMaxDepth');
-    }
-}

tests/Functional/Security/DisableIntrospectionTest.php

Indentation -29 removed lines

@@ -9,32 +9,3 @@
 final class DisableIntrospectionTest extends TestCase
 {
     private string $introspectionQuery = <<<'EOF'
-        query {
-          __schema {
-            types {
-              name
-              description
-            }
-          }
-        }
-        EOF;
-
-    public function testIntrospectionDisabled(): void
-    {
-        $expected = [
-            'errors' => [
-                [
-                    'message' => 'GraphQL introspection is not allowed, but the query contained __schema or __type',
-                    'locations' => [
-                        [
-                            'line' => 2,
-                            'column' => 3,
-                        ],
-                    ],
-                ],
-            ],
-        ];
-
-        $this->assertResponse($this->introspectionQuery, $expected, self::ANONYMOUS_USER, 'disableIntrospection');
-    }
-}

Switch Indentation -29 removed lines

@@ -9,32 +9,3 @@
 final class DisableIntrospectionTest extends TestCase
 {
     private string $introspectionQuery = <<<'EOF'
-        query {
-          __schema {
-            types {
-              name
-              description
-            }
-          }
-        }
-        EOF;
-
-    public function testIntrospectionDisabled(): void
-    {
-        $expected = [
-            'errors' => [
-                [
-                    'message' => 'GraphQL introspection is not allowed, but the query contained __schema or __type',
-                    'locations' => [
-                        [
-                            'line' => 2,
-                            'column' => 3,
-                        ],
-                    ],
-                ],
-            ],
-        ];
-
-        $this->assertResponse($this->introspectionQuery, $expected, self::ANONYMOUS_USER, 'disableIntrospection');
-    }
-}

Spacing -29 removed lines

@@ -9,32 +9,3 @@
 final class DisableIntrospectionTest extends TestCase
 {
     private string $introspectionQuery = <<<'EOF'
-        query {
-          __schema {
-            types {
-              name
-              description
-            }
-          }
-        }
-        EOF;
-
-    public function testIntrospectionDisabled(): void
-    {
-        $expected = [
-            'errors' => [
-                [
-                    'message' => 'GraphQL introspection is not allowed, but the query contained __schema or __type',
-                    'locations' => [
-                        [
-                            'line' => 2,
-                            'column' => 3,
-                        ],
-                    ],
-                ],
-            ],
-        ];
-
-        $this->assertResponse($this->introspectionQuery, $expected, self::ANONYMOUS_USER, 'disableIntrospection');
-    }
-}

Braces -29 removed lines

@@ -9,32 +9,3 @@
 final class DisableIntrospectionTest extends TestCase
 {
     private string $introspectionQuery = <<<'EOF'
-        query {
-          __schema {
-            types {
-              name
-              description
-            }
-          }
-        }
-        EOF;
-
-    public function testIntrospectionDisabled(): void
-    {
-        $expected = [
-            'errors' => [
-                [
-                    'message' => 'GraphQL introspection is not allowed, but the query contained __schema or __type',
-                    'locations' => [
-                        [
-                            'line' => 2,
-                            'column' => 3,
-                        ],
-                    ],
-                ],
-            ],
-        ];
-
-        $this->assertResponse($this->introspectionQuery, $expected, self::ANONYMOUS_USER, 'disableIntrospection');
-    }
-}

Upper-Lower-Casing -29 removed lines

@@ -9,32 +9,3 @@
 final class DisableIntrospectionTest extends TestCase
 {
     private string $introspectionQuery = <<<'EOF'
-        query {
-          __schema {
-            types {
-              name
-              description
-            }
-          }
-        }
-        EOF;
-
-    public function testIntrospectionDisabled(): void
-    {
-        $expected = [
-            'errors' => [
-                [
-                    'message' => 'GraphQL introspection is not allowed, but the query contained __schema or __type',
-                    'locations' => [
-                        [
-                            'line' => 2,
-                            'column' => 3,
-                        ],
-                    ],
-                ],
-            ],
-        ];
-
-        $this->assertResponse($this->introspectionQuery, $expected, self::ANONYMOUS_USER, 'disableIntrospection');
-    }
-}

src/Validator/ValidationNode.php

Spacing +1 added lines, -1 removed lines

@@ -46,7 +46,7 @@
     private ?ResolverArgs $__resolverArgs;
 
     public function __construct(
-        ObjectType|InputObjectType $type,
+        ObjectType | InputObjectType $type,
         string $field = null,
         ?ValidationNode $parent = null,
         ?ResolverArgs $resolverArgs = null

src/EventListener/TypeDecoratorListener.php

Spacing +2 added lines, -2 removed lines

@@ -87,7 +87,7 @@ 
         $this->decorateObjectTypeFields($type, $fieldsResolved, $resolverMap);
     }
 
-    private function decorateInterfaceOrUnionType(InterfaceType|UnionType $type, ResolverMapInterface $resolverMap): void
+    private function decorateInterfaceOrUnionType(InterfaceType | UnionType $type, ResolverMapInterface $resolverMap): void
     {
         $this->configTypeMapping($type, ResolverMapInterface::RESOLVE_TYPE, $resolverMap);
         $covered = $resolverMap->covered($type->name);
@@ -188,7 +188,7 @@ 
         $type->config['fields'] = is_callable($fields) ? $decoratedFields : $decoratedFields();
     }
 
-    private function configTypeMapping(ObjectType|InterfaceType|UnionType|CustomScalarType $type, string $fieldName, ResolverMapInterface $resolverMap): void
+    private function configTypeMapping(ObjectType | InterfaceType | UnionType | CustomScalarType $type, string $fieldName, ResolverMapInterface $resolverMap): void
     {
         if ($resolverMap->isResolvable($type->name, $fieldName)) {
             $type->config[substr($fieldName, 2)] = $resolverMap->resolve($type->name, $fieldName);

src/Validator/InputValidator.php

Spacing +4 added lines, -4 removed lines

@@ -62,7 +62,7 @@ 
     /**
      * @throws ArgumentsValidationException
      */
-    public function validate(string|array|null $groups = null, bool $throw = true): ?ConstraintViolationListInterface
+    public function validate(string | array | null $groups = null, bool $throw = true): ?ConstraintViolationListInterface
     {
         $rootNode = new ValidationNode(
             $this->info->parentType,
@@ -206,7 +206,7 @@ 
         return $rootObject;
     }
 
-    private static function isListOfType(GeneratedTypeInterface|ListOfType|NonNull $type): bool
+    private static function isListOfType(GeneratedTypeInterface | ListOfType | NonNull $type): bool
     {
         if ($type instanceof ListOfType || ($type instanceof NonNull && $type->getWrappedType() instanceof ListOfType)) {
             return true;
@@ -215,7 +215,7 @@ 
         return false;
     }
 
-    private function createCollectionNode(array $values, ObjectType|InputObjectType $type, ValidationNode $parent): array
+    private function createCollectionNode(array $values, ObjectType | InputObjectType $type, ValidationNode $parent): array
     {
         $collection = [];
 
@@ -226,7 +226,7 @@ 
         return $collection;
     }
 
-    private function createObjectNode(array $value, ObjectType|InputObjectType $type, ValidationNode $parent): ValidationNode
+    private function createObjectNode(array $value, ObjectType | InputObjectType $type, ValidationNode $parent): ValidationNode
     {
         /** @phpstan-ignore-next-line */
         $classValidation = static::normalizeConfig($type->config['validation'] ?? []);

tests/Resolver/TypeResolverTest.php

Spacing +2 added lines, -2 removed lines

@@ -18,8 +18,8 @@
     protected function getResolverSolutionsMapping(): array
     {
         return [
-            'Toto' => ['factory' => fn () => new ObjectType(['name' => 'Toto', 'fields' => []]), 'aliases' => ['foo']],
-            'Tata' => ['factory' => fn () => new ObjectType(['name' => 'Tata', 'fields' => []]), 'aliases' => ['bar']],
+            'Toto' => ['factory' => fn() => new ObjectType(['name' => 'Toto', 'fields' => []]), 'aliases' => ['foo']],
+            'Tata' => ['factory' => fn() => new ObjectType(['name' => 'Tata', 'fields' => []]), 'aliases' => ['bar']],
         ];
     }
 

tests/EventListener/TypeDecoratorListenerTest.php

Spacing +7 added lines, -7 removed lines

@@ -31,10 +31,10 @@ 
      *
      * @dataProvider specialTypeFieldProvider
      */
-    public function testSpecialField($fieldName, ObjectType|UnionType|InterfaceType|CustomScalarType $typeWithSpecialField, callable $fieldValueRetriever = null, $strict = true): void
+    public function testSpecialField($fieldName, ObjectType | UnionType | InterfaceType | CustomScalarType $typeWithSpecialField, callable $fieldValueRetriever = null, $strict = true): void
     {
         if (null === $fieldValueRetriever) {
-            $fieldValueRetriever = fn (ObjectType|UnionType|InterfaceType|CustomScalarType $type, $fieldName) => $type->config[$fieldName];
+            $fieldValueRetriever = fn(ObjectType | UnionType | InterfaceType | CustomScalarType $type, $fieldName) => $type->config[$fieldName];
         }
         $expected = static function (): void {
         };
@@ -67,8 +67,8 @@ 
                 ];
             },
         ]);
-        $barResolver = static fn () => 'bar';
-        $bazResolver = static fn () => 'baz';
+        $barResolver = static fn() => 'bar';
+        $bazResolver = static fn() => 'baz';
 
         $this->decorate(
             [$objectType->name => $objectType],
@@ -103,7 +103,7 @@ 
             [$objectType->name => $objectType],
             [
                 $objectType->name => [
-                    'bar' => fn ($value, $args) => $args,
+                    'bar' => fn($value, $args) => $args,
                 ],
             ]
         );
@@ -253,7 +253,7 @@ 
             [
                 ResolverMapInterface::RESOLVE_FIELD,
                 $objectWithResolveField,
-                fn (ObjectType $type) => $type->resolveFieldFn,
+                fn(ObjectType $type) => $type->resolveFieldFn,
                 false,
             ],
             [ResolverMapInterface::RESOLVE_FIELD, $objectWithResolveField, null, false],
@@ -263,7 +263,7 @@ 
             // custom scalar
             [ResolverMapInterface::SERIALIZE, new CustomScalarType(['name' => 'Custom', 'scalarType' => Type::string(), 'serialize' => fn(mixed $input): mixed => ''])],
             [ResolverMapInterface::PARSE_VALUE, new CustomScalarType(['name' => 'Custom', 'scalarType' => Type::string(), 'serialize' => fn(mixed $input): mixed => '', 'parseValue' => fn(mixed $input): mixed => ''])],
-            [ResolverMapInterface::PARSE_LITERAL, new CustomScalarType(['name' => 'Custom', 'scalarType' => Type::string(), 'serialize' => fn(mixed $input): mixed => '', 'parseLiteral' => fn(Node $a, array|null $b): mixed => ''])],
+            [ResolverMapInterface::PARSE_LITERAL, new CustomScalarType(['name' => 'Custom', 'scalarType' => Type::string(), 'serialize' => fn(mixed $input): mixed => '', 'parseLiteral' => fn(Node $a, array | null $b): mixed => ''])],
             [ResolverMapInterface::SCALAR_TYPE, new CustomScalarType(['name' => 'Custom', 'scalarType' => Type::string(), 'serialize' => fn(mixed $input): mixed => ''])],
         ];
     }

tests/Definition/Type/CustomScalarTypeTest.php

Spacing +7 added lines, -7 removed lines

@@ -23,12 +23,12 @@ 
     public function testScalarTypeConfig(): void
     {
         $this->assertScalarTypeConfig(new YearScalarType());
-        $this->assertScalarTypeConfig(fn () => new YearScalarType());
+        $this->assertScalarTypeConfig(fn() => new YearScalarType());
     }
 
     public function testWithoutScalarTypeConfig(): void
     {
-        $genericFunc = fn ($value) => $value;
+        $genericFunc = fn($value) => $value;
         /** @phpstan-ignore-next-line */
         $type = new CustomScalarType([
             'serialize' => $genericFunc,
@@ -67,11 +67,11 @@ 
         yield [false, 'false'];
         yield [new stdClass(), 'instance of stdClass'];
         yield [
-            fn () => false,
+            fn() => false,
             'false',
         ];
         yield [
-            fn () => new stdClass(),
+            fn() => new stdClass(),
             'instance of stdClass',
         ];
     }
@@ -85,9 +85,9 @@ 
     {
         $type = new CustomScalarType([
             'scalarType' => $scalarType,
-            'serialize' => fn () => 'serialize',
-            'parseValue' => fn () => 'parseValue',
-            'parseLiteral' => fn () => 'parseLiteral',
+            'serialize' => fn() => 'serialize',
+            'parseValue' => fn() => 'parseValue',
+            'parseLiteral' => fn() => 'parseLiteral',
         ]);
 
         $this->assertSame('50 AC', $type->serialize(50));