overblog / GraphQLBundle

tests/ExpressionLanguage/TestCase.php

Spacing +4 added lines, -4 removed lines

@@ -106,13 +106,13 @@
     protected function createGraphQLServices(array $services = []): GraphQLServices
     {
         $locateableServices = [
-            'typeResolver' => fn () => $this->createMock(TypeResolver::class),
-            'queryResolver' => fn () => $this->createMock(TypeResolver::class),
-            'mutationResolver' => fn () => $$this->createMock(MutationResolver::class),
+            'typeResolver' => fn() => $this->createMock(TypeResolver::class),
+            'queryResolver' => fn() => $this->createMock(TypeResolver::class),
+            'mutationResolver' => fn() => $$this->createMock(MutationResolver::class),
         ];
 
         foreach ($services as $id => $service) {
-            $locateableServices[$id] = fn () => $service;
+            $locateableServices[$id] = fn() => $service;
         }
 
         return new GraphQLServices($locateableServices);

tests/Definition/Type/CustomScalarTypeTest.php

Spacing +7 added lines, -7 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;
         $type = new CustomScalarType([
             'serialize' => $genericFunc,
             'parseValue' => $genericFunc,
@@ -74,11 +74,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',
         ];
     }
@@ -92,9 +92,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));

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

Switch 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

Spacing +1 added lines, -3 removed lines

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

Braces +1 added lines, -3 removed lines

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

Upper-Lower-Casing +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/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'
-        );
-    }
-}