Fixes & cs

Author: Vincz

phpstan-baseline.neon

@@ -443,22 +443,22 @@ parameters:
 		-
 			message: "#^Method Overblog\\\\GraphQLBundle\\\\Tests\\\\Functional\\\\TestCase\\:\\:createKernel\\(\\) should return Symfony\\\\Component\\\\HttpKernel\\\\KernelInterface but returns object\\.$#"
 			count: 1
-			path: tests/Functional/TestCase.php
+			path: tests/Functional/BaseTestCase.php
 
 		-
 			message: "#^Missing call to parent\\:\\:tearDown\\(\\) method\\.$#"
 			count: 1
-			path: tests/Functional/TestCase.php
+			path: tests/Functional/BaseTestCase.php
 
 		-
 			message: "#^Parameter \\#1 \\$json of function json_decode expects string, string\\|false given\\.$#"
 			count: 1
-			path: tests/Functional/TestCase.php
+			path: tests/Functional/BaseTestCase.php
 
 		-
 			message: "#^Parameter \\#3 \\$message of static method PHPUnit\\\\Framework\\\\Assert\\:\\:assertSame\\(\\) expects string, string\\|false given\\.$#"
 			count: 1
-			path: tests/Functional/TestCase.php
+			path: tests/Functional/BaseTestCase.php
 
 		-
 			message: "#^Parameter \\#1 \\$json of function json_decode expects string, string\\|false given\\.$#"

src/CacheWarmer/CompileCacheWarmer.php

@@ -27,11 +27,9 @@ public function isOptional(): bool
     }
 
     /**
-     * @param string $cacheDir
-     *
      * @return string[]
      */
-    public function warmUp(string $cacheDir, ?string $buildDir = null): array
+    public function warmUp(string $cacheDir, string $buildDir = null): array
     {
         if ($this->compiled) {
             // use warm up cache dir if type generator cache dir not already explicitly declared

tests/ExpressionLanguage/TestCase.php

@@ -87,6 +87,7 @@ protected function getSecurityIsGrantedWithExpectation($with, $expects = null, $
             $returnValue = $this->returnValue($return);
         }
 
+        // @phpstan-ignore-next-line
         $methodExpectation = $security
             ->expects($expects)
             ->method('isGranted');
@@ -96,6 +97,7 @@ protected function getSecurityIsGrantedWithExpectation($with, $expects = null, $
         // @phpstan-ignore-next-line
         $methodExpectation->will($returnValue);
 
+        // @phpstan-ignore-next-line
         return new Security($security);
     }
 

tests/Functional/App/TestKernel.php

@@ -73,6 +73,7 @@ public function registerContainerConfiguration(LoaderInterface $loader): void
             $loader->load(__DIR__.'/config/config.yml');
         }
 
+        // @phpstan-ignore-next-line
         if (Kernel::VERSION < 60200) {
             $loader->load(__DIR__.'/config/config_pre_62.yml');
         }

tests/Functional/BaseTestCase.php

@@ -51,7 +51,7 @@ protected static function createKernel(array $options = []): KernelInterface
 
         $options['test_case'] ??= '';
 
-        $env = $options['environment'] ?? 'test' . strtolower($options['test_case']);
+        $env = $options['environment'] ?? 'test'.strtolower($options['test_case']);
         $debug = $options['debug'] ?? true;
 
         return new static::$class($env, $debug, $options['test_case']);
@@ -63,7 +63,7 @@ protected static function createKernel(array $options = []): KernelInterface
     public static function setUpBeforeClass(): void
     {
         $fs = new Filesystem();
-        $fs->remove(sys_get_temp_dir() . '/OverblogGraphQLBundle/');
+        $fs->remove(sys_get_temp_dir().'/OverblogGraphQLBundle/');
     }
 
     protected function tearDown(): void
@@ -157,7 +157,7 @@ public static function expressionFunctionFromPhp(string $phpFunctionName)
             return call_user_func([ExpressionFunction::class, 'fromPhp'], $phpFunctionName);
         }
 
-        return new ExpressionFunction($phpFunctionName, fn() => sprintf('\%s(%s)', $phpFunctionName, implode(', ', func_get_args())), function (): void {});
+        return new ExpressionFunction($phpFunctionName, fn () => sprintf('\%s(%s)', $phpFunctionName, implode(', ', func_get_args())), function (): void {});
     }
 
     /**