overblog / GraphQLBundle

tests/Executor/Promise/Adapter/ReactPromiseAdapterTest.php

Indentation +1 added lines, -3 removed lines

@@ -81,6 +81,4 @@
         $reactAdapter->convertThenable($promise);
 
         $this->assertInstanceOf(Promise::class, $promise);
-        $this->assertInstanceOf(FulfilledPromise::class, $promise->adoptedPromise); // @phpstan-ignore-line
-    }
-}
+        $this->assertInstanceOf(FulfilledPromise::class, $promise->adoptedPromise
\ No newline at end of file

tests/DependencyInjection/Compiler/ConfigParserPassTest.php

Indentation +1 added lines, -1 removed lines

@@ -293,7 +293,7 @@
                                         'type' => 'Int!',
                                         'defaultValue' => 0,
                                     ],
-                                 ],
+                                    ],
                             ],
                         ],
                         'name' => 'foo',

tests/Functional/Relay/Node/NodeTest.php

Indentation +1 added lines, -3 removed lines

@@ -281,6 +281,4 @@
             ],
         ];
 
-        $this->assertGraphQL($query, $expectedData);
-    }
-}
+        $this->assertGraphQL($query, $expectedData
\ No newline at end of file

tests/Functional/Security/AccessTest.php

Indentation -340 removed lines

@@ -26,343 +26,3 @@
     private string $userIsEnabledQuery = 'query ($hasAccess: Boolean = true) { user { isEnabled(hasAccess: $hasAccess) } }';
 
     private string $userFriendsQuery = <<<'QUERY'
-        query {
-          user {
-            friends(first: 2) {
-              edges {
-                node {
-                  name
-                }
-              }
-            }
-          }
-        }
-        QUERY;
-
-    private string $simpleMutationWithThunkQuery = <<<'MUTATION'
-        mutation M {
-          simpleMutationWithThunkFields(input: {inputData: %d, clientMutationId: "bac"}) {
-            result
-            clientMutationId
-          }
-        }
-        MUTATION;
-
-    public function setUp(): void
-    {
-        parent::setUp();
-        // load types
-        $this->loader = function ($class): void {
-            if (preg_match('@^'.preg_quote('Overblog\GraphQLBundle\Access\__DEFINITIONS__\\').'(.*)$@', $class, $matches)) {
-                $file = sys_get_temp_dir().'/OverblogGraphQLBundle/'.Kernel::VERSION.'/access/cache/testaccess/overblog/graphql-bundle/__definitions__/'.$matches[1].'.php';
-                if (file_exists($file)) {
-                    require $file;
-                }
-            }
-        };
-        spl_autoload_register($this->loader);
-    }
-
-    public function testCustomClassLoaderNotRegister(): void
-    {
-        $this->expectException(Error::class);
-        if ((int) phpversion() <= 7) {
-            $this->expectExceptionMessage('Class \'Overblog\GraphQLBundle\Access\__DEFINITIONS__\RootQueryType\' not found');
-        } else {
-            $this->expectExceptionMessage('Class "Overblog\GraphQLBundle\Access\__DEFINITIONS__\RootQueryType" not found');
-        }
-        spl_autoload_unregister($this->loader);
-        $this->assertResponse($this->userNameQuery, [], static::ANONYMOUS_USER, 'access');
-    }
-
-    public function testNotAuthenticatedUserAccessAsPromisedFulfilledTrue(): void
-    {
-        $this->assertResponse(
-            $this->userIsEnabledQuery,
-            ['data' => ['user' => ['isEnabled' => true]]],
-            static::ANONYMOUS_USER,
-            'access'
-        );
-    }
-
-    public function testNotAuthenticatedUserAccessAsPromisedFulfilledFalse(): void
-    {
-        $this->assertResponse(
-            $this->userIsEnabledQuery,
-            [
-                'data' => [
-                    'user' => [
-                        'isEnabled' => null,
-                    ],
-                ],
-                'extensions' => [
-                    'warnings' => [
-                        [
-                            'message' => 'Access denied to this field.',
-                            'locations' => [['line' => 1, 'column' => 45]],
-                            'path' => ['user', 'isEnabled'],
-                        ],
-                    ],
-                ],
-            ],
-            static::ANONYMOUS_USER,
-            'access',
-            '',
-            ['hasAccess' => false]
-        );
-    }
-
-    public function testNotAuthenticatedUserAccessToUserName(): void
-    {
-        $expected = [
-            'data' => [
-                'user' => [
-                    'name' => null,
-                ],
-            ],
-            'extensions' => [
-                'warnings' => [
-                    [
-                        'message' => 'Access denied to this field.',
-                        'locations' => [['line' => 1, 'column' => 16]],
-                        'path' => ['user', 'name'],
-                    ],
-                ],
-            ],
-        ];
-
-        $this->assertResponse($this->userNameQuery, $expected, static::ANONYMOUS_USER, 'access');
-    }
-
-    public function testNonAuthenticatedUserAccessSecuredFieldWhichInitiallyResolvesToArray(): void
-    {
-        $expected = [
-            'data' => [
-                'youShallNotSeeThisUnauthenticated' => null,
-            ],
-            'extensions' => [
-                'warnings' => [
-                    [
-                        'message' => 'Access denied to this field.',
-                        'locations' => [
-                            [
-                                'line' => 2,
-                                'column' => 3,
-                            ],
-                        ],
-                        'path' => ['youShallNotSeeThisUnauthenticated'],
-                    ],
-                ],
-            ],
-        ];
-
-        $query = <<<'QUERY'
-            {
-              youShallNotSeeThisUnauthenticated {
-                secretValue
-                youAreAuthenticated
-              }
-            }
-            QUERY;
-
-        $this->assertResponse($query, $expected, static::ANONYMOUS_USER, 'access');
-    }
-
-    public function testFullyAuthenticatedUserAccessToUserName(): void
-    {
-        $expected = [
-            'data' => [
-                'user' => [
-                    'name' => 'Dan',
-                ],
-            ],
-        ];
-
-        $this->assertResponse($this->userNameQuery, $expected, static::USER_RYAN, 'access');
-    }
-
-    public function testNotAuthenticatedUserAccessToUserRoles(): void
-    {
-        $this->assertResponse($this->userRolesQuery, $this->expectedFailedUserRoles(), static::ANONYMOUS_USER, 'access');
-    }
-
-    public function testAuthenticatedUserAccessToUserRolesWithoutEnoughRights(): void
-    {
-        $this->assertResponse($this->userRolesQuery, $this->expectedFailedUserRoles(), static::USER_RYAN, 'access');
-    }
-
-    public function testUserWithCorrectRightsAccessToUserRoles(): void
-    {
-        $expected = [
-            'data' => [
-                'user' => [
-                    'roles' => ['ROLE_USER'],
-                ],
-            ],
-        ];
-
-        $this->assertResponse($this->userRolesQuery, $expected, static::USER_ADMIN, 'access');
-    }
-
-    public function testUserForbiddenField(): void
-    {
-        $expected = [
-            'data' => [
-                'user' => null,
-            ],
-            'extensions' => [
-                'warnings' => [
-                    [
-                        'message' => 'Access denied to this field.',
-                        'locations' => [
-                            [
-                                'line' => 3,
-                                'column' => 5,
-                            ],
-                        ],
-                        'path' => ['user', 'forbidden'],
-                    ],
-                ],
-            ],
-        ];
-
-        $query = <<<'QUERY'
-            query MyQuery {
-              user {
-                forbidden
-              }
-            }
-            QUERY;
-
-        $this->assertResponse($query, $expected, static::USER_ADMIN, 'access');
-    }
-
-    public function testUserAccessToUserFriends(): void
-    {
-        $expected = [
-            'data' => [
-                'user' => [
-                    'friends' => [
-                        'edges' => [
-                            ['node' => ['name' => 'Nick']],
-                            ['node' => null],
-                        ],
-                    ],
-                ],
-            ],
-        ];
-
-        $this->assertResponse($this->userFriendsQuery, $expected, static::USER_ADMIN, 'access');
-    }
-
-    public function testUserAccessToUserFriendsAsArray(): void
-    {
-        $expected = [
-            'data' => [
-                'user' => [
-                    'friendsAsArray' => [1, null, 3],
-                ],
-            ],
-        ];
-
-        $this->assertResponse('query { user { friendsAsArray } }', $expected, static::USER_ADMIN, 'access');
-    }
-
-    public function testMutationAllowedUser(): void
-    {
-        $result = 123;
-
-        $expected = [
-            'data' => [
-                'simpleMutationWithThunkFields' => [
-                    'result' => $result,
-                    'clientMutationId' => 'bac',
-                ],
-            ],
-        ];
-
-        $this->assertResponse(sprintf($this->simpleMutationWithThunkQuery, $result), $expected, static::USER_ADMIN, 'access');
-        $this->assertTrue(SimpleMutationWithThunkFieldsMutation::hasMutate(true));
-    }
-
-    public function testMutationAllowedButNoRightsToDisplayPayload(): void
-    {
-        $expected = [
-            'data' => [
-                'simpleMutationWithThunkFields' => [
-                    'result' => null,
-                    'clientMutationId' => 'bac',
-                ],
-            ],
-            'extensions' => [
-                'warnings' => [
-                    [
-                        'message' => 'Access denied to this field.',
-                        'locations' => [
-                            [
-                                'line' => 3,
-                                'column' => 5,
-                            ],
-                        ],
-                        'path' => ['simpleMutationWithThunkFields', 'result'],
-                    ],
-                ],
-            ],
-        ];
-
-        $this->assertResponse(sprintf($this->simpleMutationWithThunkQuery, 321), $expected, static::USER_ADMIN, 'access');
-        $this->assertTrue(SimpleMutationWithThunkFieldsMutation::hasMutate(true));
-    }
-
-    public function testMutationNotAllowedUser(): void
-    {
-        $expected = [
-            'errors' => [
-                [
-                    'message' => 'Access denied to this field.',
-                    'locations' => [
-                        [
-                            'line' => 2,
-                            'column' => 3,
-                        ],
-                    ],
-                    'path' => ['simpleMutationWithThunkFields'],
-                ],
-            ],
-            'data' => [
-                'simpleMutationWithThunkFields' => null,
-            ],
-        ];
-
-        $this->assertResponse(sprintf($this->simpleMutationWithThunkQuery, 123), $expected, static::USER_RYAN, 'access');
-        $this->assertFalse(SimpleMutationWithThunkFieldsMutation::hasMutate(true));
-    }
-
-    private function expectedFailedUserRoles(): array
-    {
-        return [
-            'data' => [
-                'user' => [
-                    'roles' => null,
-                ],
-            ],
-            'extensions' => [
-                'warnings' => [
-                    [
-                        'message' => 'Access denied to this field.',
-                        'locations' => [
-                            [
-                                'line' => 1,
-                                'column' => 16,
-                            ],
-                        ],
-                        'path' => [
-                            'user',
-                            'roles',
-                        ],
-                    ],
-                ],
-            ],
-        ];
-    }
-}

tests/Functional/Exception/ExceptionTest.php

Indentation -25 removed lines

@@ -18,28 +18,3 @@
     public function testExceptionIsMappedToAWarning(): void
     {
         $query = <<<'EOF'
-            query ExceptionQuery {
-                test
-            }
-            EOF;
-
-        $expectedData = [
-            'test' => null,
-        ];
-
-        $expectedErrors = [
-            [
-                'message' => 'Invalid argument exception',
-                'locations' => [
-                    [
-                        'line' => 2,
-                        'column' => 5,
-                    ],
-                ],
-                'path' => ['test'],
-            ],
-        ];
-
-        $this->assertGraphQL($query, $expectedData, $expectedErrors);
-    }
-}

tests/Functional/Security/QueryComplexityTest.php

Indentation -70 removed lines

@@ -9,73 +9,3 @@
 final class QueryComplexityTest extends TestCase
 {
     private string $userFriendsWithoutLimitQuery = <<<'EOF'
-        query {
-          user {
-            friends {
-              edges {
-                node {
-                  name
-                }
-              }
-            }
-          }
-        }
-        EOF;
-
-    private string $userFriendsWithLimitQuery = <<<'EOF'
-        query {
-          user {
-            friends(first: 1) {
-              edges {
-                node {
-                  name
-                }
-              }
-            }
-          }
-        }
-        EOF;
-
-    public function testComplexityReachLimitation(): void
-    {
-        $expected = [
-            'errors' => [
-                [
-                    'message' => 'Max query complexity should be 10 but got 54.',
-                ],
-            ],
-        ];
-
-        $this->assertResponse($this->userFriendsWithoutLimitQuery, $expected, self::ANONYMOUS_USER, 'queryComplexity');
-    }
-
-    public function testComplexityReachLimitationEnv(): void
-    {
-        $expected = [
-            'errors' => [
-                [
-                    'message' => 'Max query complexity should be 10 but got 54.',
-                ],
-            ],
-        ];
-
-        $this->assertResponse($this->userFriendsWithoutLimitQuery, $expected, self::ANONYMOUS_USER, 'queryComplexityEnv');
-    }
-
-    public function testComplexityUnderLimitation(): void
-    {
-        $expected = [
-            'data' => [
-                'user' => [
-                    'friends' => [
-                        'edges' => [
-                            ['node' => ['name' => 'Nick']],
-                        ],
-                    ],
-                ],
-            ],
-        ];
-
-        $this->assertResponse($this->userFriendsWithLimitQuery, $expected, self::ANONYMOUS_USER, 'queryComplexity');
-    }
-}

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');
-    }
-}

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');
-    }
-}

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'
-        );
-    }
-}