overblog / GraphQLBundle

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/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/Definition/Type/CustomScalarTypeTest.php

Spacing +8 added lines, -8 removed lines

@@ -22,12 +22,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,
@@ -57,7 +57,7 @@ 
             ScalarType::class,
             $got
         ));
-        $type = new CustomScalarType(['name' => $name, 'scalarType' => $scalarType, 'serialize' => fn (mixed $input): mixed => '']);
+        $type = new CustomScalarType(['name' => $name, 'scalarType' => $scalarType, 'serialize' => fn(mixed $input): mixed => '']);
         $type->assertValid();
     }
 
@@ -66,11 +66,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',
         ];
     }
@@ -84,9 +84,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));

src/Definition/Type/CustomScalarType.php

Spacing +2 added lines, -2 removed lines

@@ -77,7 +77,7 @@ 
             return parent::$type($value);
         }
 
-        $scalarType = match (true) {
+        $scalarType = match(true) {
             $this->config['scalarType'] instanceof ScalarType => $this->config['scalarType'],
             is_callable($this->config['scalarType']) => $this->config['scalarType'](),
             default => $this->config['scalarType'],
@@ -94,7 +94,7 @@ 
             return;
         }
 
-        $scalarType = match (true) {
+        $scalarType = match(true) {
             $this->config['scalarType'] instanceof ScalarType => $this->config['scalarType'],
             is_callable($this->config['scalarType']) => $this->config['scalarType'](),
             default => $this->config['scalarType'],

tests/Functional/Generator/TypeGeneratorTest.php

Indentation -35 removed lines

@@ -111,38 +111,3 @@
 
         // this is the part is why we must make this test run in separate process
         $query = <<<'EOF'
-            mutation {
-              conflictingValidatorNamespaces(test: "123", test2: "1", test3: "4")
-            }
-            EOF;
-
-        $response = static::query(
-            $query,
-            self::USER_RYAN,
-            'conflictingValidatorNamespaces'
-        )->getResponse()->getContent();
-
-        $this->assertEquals('{"data":{"conflictingValidatorNamespaces":true}}', $response);
-        // end part
-
-        // Validate definition file
-        /** @var string $definitionFile */
-        $definitionFile = file_get_contents($kernel->getCacheDir().'/overblog/graphql-bundle/__definitions__/MutationType.php');
-
-        $this->assertStringContainsString(
-            'use Symfony\Component\Validator\Constraints as SymfonyConstraints;',
-            $definitionFile,
-            'Generated definition file should contain import of Symfony\Component\Validator\Constraints aliased as SymfonyConstraints'
-        );
-        $this->assertStringNotContainsString(
-            'use '.Validator\CustomValidator1\Constraint::class,
-            $definitionFile,
-            'Generated definition file should not contain imports of custom constraints, FQCN should be used instead'
-        );
-        $this->assertStringNotContainsString(
-            'use '.Validator\CustomValidator2\Constraint::class,
-            $definitionFile,
-            'Generated definition file should not contain imports of custom constraints, FQCN should be used instead'
-        );
-    }
-}

Switch Indentation -35 removed lines

@@ -111,38 +111,3 @@
 
         // this is the part is why we must make this test run in separate process
         $query = <<<'EOF'
-            mutation {
-              conflictingValidatorNamespaces(test: "123", test2: "1", test3: "4")
-            }
-            EOF;
-
-        $response = static::query(
-            $query,
-            self::USER_RYAN,
-            'conflictingValidatorNamespaces'
-        )->getResponse()->getContent();
-
-        $this->assertEquals('{"data":{"conflictingValidatorNamespaces":true}}', $response);
-        // end part
-
-        // Validate definition file
-        /** @var string $definitionFile */
-        $definitionFile = file_get_contents($kernel->getCacheDir().'/overblog/graphql-bundle/__definitions__/MutationType.php');
-
-        $this->assertStringContainsString(
-            'use Symfony\Component\Validator\Constraints as SymfonyConstraints;',
-            $definitionFile,
-            'Generated definition file should contain import of Symfony\Component\Validator\Constraints aliased as SymfonyConstraints'
-        );
-        $this->assertStringNotContainsString(
-            'use '.Validator\CustomValidator1\Constraint::class,
-            $definitionFile,
-            'Generated definition file should not contain imports of custom constraints, FQCN should be used instead'
-        );
-        $this->assertStringNotContainsString(
-            'use '.Validator\CustomValidator2\Constraint::class,
-            $definitionFile,
-            'Generated definition file should not contain imports of custom constraints, FQCN should be used instead'
-        );
-    }
-}

Spacing -35 removed lines

@@ -111,38 +111,3 @@
 
         // this is the part is why we must make this test run in separate process
         $query = <<<'EOF'
-            mutation {
-              conflictingValidatorNamespaces(test: "123", test2: "1", test3: "4")
-            }
-            EOF;
-
-        $response = static::query(
-            $query,
-            self::USER_RYAN,
-            'conflictingValidatorNamespaces'
-        )->getResponse()->getContent();
-
-        $this->assertEquals('{"data":{"conflictingValidatorNamespaces":true}}', $response);
-        // end part
-
-        // Validate definition file
-        /** @var string $definitionFile */
-        $definitionFile = file_get_contents($kernel->getCacheDir().'/overblog/graphql-bundle/__definitions__/MutationType.php');
-
-        $this->assertStringContainsString(
-            'use Symfony\Component\Validator\Constraints as SymfonyConstraints;',
-            $definitionFile,
-            'Generated definition file should contain import of Symfony\Component\Validator\Constraints aliased as SymfonyConstraints'
-        );
-        $this->assertStringNotContainsString(
-            'use '.Validator\CustomValidator1\Constraint::class,
-            $definitionFile,
-            'Generated definition file should not contain imports of custom constraints, FQCN should be used instead'
-        );
-        $this->assertStringNotContainsString(
-            'use '.Validator\CustomValidator2\Constraint::class,
-            $definitionFile,
-            'Generated definition file should not contain imports of custom constraints, FQCN should be used instead'
-        );
-    }
-}

Braces -35 removed lines

@@ -111,38 +111,3 @@
 
         // this is the part is why we must make this test run in separate process
         $query = <<<'EOF'
-            mutation {
-              conflictingValidatorNamespaces(test: "123", test2: "1", test3: "4")
-            }
-            EOF;
-
-        $response = static::query(
-            $query,
-            self::USER_RYAN,
-            'conflictingValidatorNamespaces'
-        )->getResponse()->getContent();
-
-        $this->assertEquals('{"data":{"conflictingValidatorNamespaces":true}}', $response);
-        // end part
-
-        // Validate definition file
-        /** @var string $definitionFile */
-        $definitionFile = file_get_contents($kernel->getCacheDir().'/overblog/graphql-bundle/__definitions__/MutationType.php');
-
-        $this->assertStringContainsString(
-            'use Symfony\Component\Validator\Constraints as SymfonyConstraints;',
-            $definitionFile,
-            'Generated definition file should contain import of Symfony\Component\Validator\Constraints aliased as SymfonyConstraints'
-        );
-        $this->assertStringNotContainsString(
-            'use '.Validator\CustomValidator1\Constraint::class,
-            $definitionFile,
-            'Generated definition file should not contain imports of custom constraints, FQCN should be used instead'
-        );
-        $this->assertStringNotContainsString(
-            'use '.Validator\CustomValidator2\Constraint::class,
-            $definitionFile,
-            'Generated definition file should not contain imports of custom constraints, FQCN should be used instead'
-        );
-    }
-}

Upper-Lower-Casing -35 removed lines

@@ -111,38 +111,3 @@
 
         // this is the part is why we must make this test run in separate process
         $query = <<<'EOF'
-            mutation {
-              conflictingValidatorNamespaces(test: "123", test2: "1", test3: "4")
-            }
-            EOF;
-
-        $response = static::query(
-            $query,
-            self::USER_RYAN,
-            'conflictingValidatorNamespaces'
-        )->getResponse()->getContent();
-
-        $this->assertEquals('{"data":{"conflictingValidatorNamespaces":true}}', $response);
-        // end part
-
-        // Validate definition file
-        /** @var string $definitionFile */
-        $definitionFile = file_get_contents($kernel->getCacheDir().'/overblog/graphql-bundle/__definitions__/MutationType.php');
-
-        $this->assertStringContainsString(
-            'use Symfony\Component\Validator\Constraints as SymfonyConstraints;',
-            $definitionFile,
-            'Generated definition file should contain import of Symfony\Component\Validator\Constraints aliased as SymfonyConstraints'
-        );
-        $this->assertStringNotContainsString(
-            'use '.Validator\CustomValidator1\Constraint::class,
-            $definitionFile,
-            'Generated definition file should not contain imports of custom constraints, FQCN should be used instead'
-        );
-        $this->assertStringNotContainsString(
-            'use '.Validator\CustomValidator2\Constraint::class,
-            $definitionFile,
-            'Generated definition file should not contain imports of custom constraints, FQCN should be used instead'
-        );
-    }
-}

tests/Functional/MultipleSchema/MultipleSchemaTest.php

Indentation -64 removed lines

@@ -26,67 +26,3 @@
         $this->assertSame([['node' => ['username' => 'user1']]], $result['data']['users']['edges']);
 
         $query = <<<'EOF'
-            mutation M {
-              addUser(input: {username: "user1"}) {
-                user {
-                  username
-                }
-              }
-            }
-            EOF;
-
-        $expectedData = [
-            'addUser' => [
-                'user' => ['username' => 'user1'],
-            ],
-        ];
-
-        $this->assertGraphQL($query, $expectedData, null, [], 'public');
-    }
-
-    public function testInternalSchema(): void
-    {
-        $result = $this->executeGraphQLRequest('{bar foo}', [], 'internal');
-        $this->assertSame('bar', $result['data']['bar']);
-        $this->assertSame('foo', $result['data']['foo']);
-        $this->assertSchemaQueryTypeName('InternalQuery');
-
-        $result = $this->executeGraphQLRequest('{users{edges{node{username email}}}}', [], 'internal');
-        $this->assertSame([['node' => ['username' => 'user1', 'email' => 'topsecret']]], $result['data']['users']['edges']);
-
-        $query = <<<'EOF'
-            mutation M {
-              addUser(input: {username: "user1"}) {
-                user {
-                  username
-                  email
-                }
-              }
-            }
-            EOF;
-
-        $expectedData = [
-            'addUser' => [
-                'user' => ['username' => 'user1', 'email' => 'email1'],
-            ],
-        ];
-
-        $this->assertGraphQL($query, $expectedData, null, [], 'internal');
-    }
-
-    public function testUnknownTypeShouldThrowAnUnresolvableException(): void
-    {
-        // @phpstan-ignore-next-line
-        $schema = $this->getContainer()->get('overblog_graphql.request_executor')->getSchema('public');
-        $this->expectException(UnresolvableException::class);
-        $this->expectExceptionMessage('Could not find type with alias "unknown". Did you forget to define it?');
-        $schema->getType('unknown');
-    }
-
-    private function assertSchemaQueryTypeName(string $typeName): void
-    {
-        // @phpstan-ignore-next-line
-        $query = $this->getContainer()->get('overblog_graphql.type_resolver')->resolve($typeName);
-        $this->assertSame('Query', $query->name);
-    }
-}

Switch Indentation -64 removed lines

@@ -26,67 +26,3 @@
         $this->assertSame([['node' => ['username' => 'user1']]], $result['data']['users']['edges']);
 
         $query = <<<'EOF'
-            mutation M {
-              addUser(input: {username: "user1"}) {
-                user {
-                  username
-                }
-              }
-            }
-            EOF;
-
-        $expectedData = [
-            'addUser' => [
-                'user' => ['username' => 'user1'],
-            ],
-        ];
-
-        $this->assertGraphQL($query, $expectedData, null, [], 'public');
-    }
-
-    public function testInternalSchema(): void
-    {
-        $result = $this->executeGraphQLRequest('{bar foo}', [], 'internal');
-        $this->assertSame('bar', $result['data']['bar']);
-        $this->assertSame('foo', $result['data']['foo']);
-        $this->assertSchemaQueryTypeName('InternalQuery');
-
-        $result = $this->executeGraphQLRequest('{users{edges{node{username email}}}}', [], 'internal');
-        $this->assertSame([['node' => ['username' => 'user1', 'email' => 'topsecret']]], $result['data']['users']['edges']);
-
-        $query = <<<'EOF'
-            mutation M {
-              addUser(input: {username: "user1"}) {
-                user {
-                  username
-                  email
-                }
-              }
-            }
-            EOF;
-
-        $expectedData = [
-            'addUser' => [
-                'user' => ['username' => 'user1', 'email' => 'email1'],
-            ],
-        ];
-
-        $this->assertGraphQL($query, $expectedData, null, [], 'internal');
-    }
-
-    public function testUnknownTypeShouldThrowAnUnresolvableException(): void
-    {
-        // @phpstan-ignore-next-line
-        $schema = $this->getContainer()->get('overblog_graphql.request_executor')->getSchema('public');
-        $this->expectException(UnresolvableException::class);
-        $this->expectExceptionMessage('Could not find type with alias "unknown". Did you forget to define it?');
-        $schema->getType('unknown');
-    }
-
-    private function assertSchemaQueryTypeName(string $typeName): void
-    {
-        // @phpstan-ignore-next-line
-        $query = $this->getContainer()->get('overblog_graphql.type_resolver')->resolve($typeName);
-        $this->assertSame('Query', $query->name);
-    }
-}

Spacing -64 removed lines

@@ -26,67 +26,3 @@
         $this->assertSame([['node' => ['username' => 'user1']]], $result['data']['users']['edges']);
 
         $query = <<<'EOF'
-            mutation M {
-              addUser(input: {username: "user1"}) {
-                user {
-                  username
-                }
-              }
-            }
-            EOF;
-
-        $expectedData = [
-            'addUser' => [
-                'user' => ['username' => 'user1'],
-            ],
-        ];
-
-        $this->assertGraphQL($query, $expectedData, null, [], 'public');
-    }
-
-    public function testInternalSchema(): void
-    {
-        $result = $this->executeGraphQLRequest('{bar foo}', [], 'internal');
-        $this->assertSame('bar', $result['data']['bar']);
-        $this->assertSame('foo', $result['data']['foo']);
-        $this->assertSchemaQueryTypeName('InternalQuery');
-
-        $result = $this->executeGraphQLRequest('{users{edges{node{username email}}}}', [], 'internal');
-        $this->assertSame([['node' => ['username' => 'user1', 'email' => 'topsecret']]], $result['data']['users']['edges']);
-
-        $query = <<<'EOF'
-            mutation M {
-              addUser(input: {username: "user1"}) {
-                user {
-                  username
-                  email
-                }
-              }
-            }
-            EOF;
-
-        $expectedData = [
-            'addUser' => [
-                'user' => ['username' => 'user1', 'email' => 'email1'],
-            ],
-        ];
-
-        $this->assertGraphQL($query, $expectedData, null, [], 'internal');
-    }
-
-    public function testUnknownTypeShouldThrowAnUnresolvableException(): void
-    {
-        // @phpstan-ignore-next-line
-        $schema = $this->getContainer()->get('overblog_graphql.request_executor')->getSchema('public');
-        $this->expectException(UnresolvableException::class);
-        $this->expectExceptionMessage('Could not find type with alias "unknown". Did you forget to define it?');
-        $schema->getType('unknown');
-    }
-
-    private function assertSchemaQueryTypeName(string $typeName): void
-    {
-        // @phpstan-ignore-next-line
-        $query = $this->getContainer()->get('overblog_graphql.type_resolver')->resolve($typeName);
-        $this->assertSame('Query', $query->name);
-    }
-}

Braces -64 removed lines

@@ -26,67 +26,3 @@
         $this->assertSame([['node' => ['username' => 'user1']]], $result['data']['users']['edges']);
 
         $query = <<<'EOF'
-            mutation M {
-              addUser(input: {username: "user1"}) {
-                user {
-                  username
-                }
-              }
-            }
-            EOF;
-
-        $expectedData = [
-            'addUser' => [
-                'user' => ['username' => 'user1'],
-            ],
-        ];
-
-        $this->assertGraphQL($query, $expectedData, null, [], 'public');
-    }
-
-    public function testInternalSchema(): void
-    {
-        $result = $this->executeGraphQLRequest('{bar foo}', [], 'internal');
-        $this->assertSame('bar', $result['data']['bar']);
-        $this->assertSame('foo', $result['data']['foo']);
-        $this->assertSchemaQueryTypeName('InternalQuery');
-
-        $result = $this->executeGraphQLRequest('{users{edges{node{username email}}}}', [], 'internal');
-        $this->assertSame([['node' => ['username' => 'user1', 'email' => 'topsecret']]], $result['data']['users']['edges']);
-
-        $query = <<<'EOF'
-            mutation M {
-              addUser(input: {username: "user1"}) {
-                user {
-                  username
-                  email
-                }
-              }
-            }
-            EOF;
-
-        $expectedData = [
-            'addUser' => [
-                'user' => ['username' => 'user1', 'email' => 'email1'],
-            ],
-        ];
-
-        $this->assertGraphQL($query, $expectedData, null, [], 'internal');
-    }
-
-    public function testUnknownTypeShouldThrowAnUnresolvableException(): void
-    {
-        // @phpstan-ignore-next-line
-        $schema = $this->getContainer()->get('overblog_graphql.request_executor')->getSchema('public');
-        $this->expectException(UnresolvableException::class);
-        $this->expectExceptionMessage('Could not find type with alias "unknown". Did you forget to define it?');
-        $schema->getType('unknown');
-    }
-
-    private function assertSchemaQueryTypeName(string $typeName): void
-    {
-        // @phpstan-ignore-next-line
-        $query = $this->getContainer()->get('overblog_graphql.type_resolver')->resolve($typeName);
-        $this->assertSame('Query', $query->name);
-    }
-}

Upper-Lower-Casing -64 removed lines

@@ -26,67 +26,3 @@
         $this->assertSame([['node' => ['username' => 'user1']]], $result['data']['users']['edges']);
 
         $query = <<<'EOF'
-            mutation M {
-              addUser(input: {username: "user1"}) {
-                user {
-                  username
-                }
-              }
-            }
-            EOF;
-
-        $expectedData = [
-            'addUser' => [
-                'user' => ['username' => 'user1'],
-            ],
-        ];
-
-        $this->assertGraphQL($query, $expectedData, null, [], 'public');
-    }
-
-    public function testInternalSchema(): void
-    {
-        $result = $this->executeGraphQLRequest('{bar foo}', [], 'internal');
-        $this->assertSame('bar', $result['data']['bar']);
-        $this->assertSame('foo', $result['data']['foo']);
-        $this->assertSchemaQueryTypeName('InternalQuery');
-
-        $result = $this->executeGraphQLRequest('{users{edges{node{username email}}}}', [], 'internal');
-        $this->assertSame([['node' => ['username' => 'user1', 'email' => 'topsecret']]], $result['data']['users']['edges']);
-
-        $query = <<<'EOF'
-            mutation M {
-              addUser(input: {username: "user1"}) {
-                user {
-                  username
-                  email
-                }
-              }
-            }
-            EOF;
-
-        $expectedData = [
-            'addUser' => [
-                'user' => ['username' => 'user1', 'email' => 'email1'],
-            ],
-        ];
-
-        $this->assertGraphQL($query, $expectedData, null, [], 'internal');
-    }
-
-    public function testUnknownTypeShouldThrowAnUnresolvableException(): void
-    {
-        // @phpstan-ignore-next-line
-        $schema = $this->getContainer()->get('overblog_graphql.request_executor')->getSchema('public');
-        $this->expectException(UnresolvableException::class);
-        $this->expectExceptionMessage('Could not find type with alias "unknown". Did you forget to define it?');
-        $schema->getType('unknown');
-    }
-
-    private function assertSchemaQueryTypeName(string $typeName): void
-    {
-        // @phpstan-ignore-next-line
-        $query = $this->getContainer()->get('overblog_graphql.type_resolver')->resolve($typeName);
-        $this->assertSame('Query', $query->name);
-    }
-}

src/EventListener/TypeDecoratorListener.php

Spacing +3 added lines, -3 removed lines

@@ -88,7 +88,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);
@@ -138,7 +138,7 @@ 
     {
         $values = $type->config['values'];
 
-        $decoratedValues = function() use ($type, $resolverMap, $values) {
+        $decoratedValues = function () use ($type, $resolverMap, $values) {
             if (is_callable($values)) {
                 $values = $values();
             }
@@ -210,7 +210,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);