fixed issues with php8-named-arguments

Author: n0nag0n

flight/Engine.php

@@ -266,9 +266,9 @@ public function handleException(Throwable $e): void
     /**
      * Registers the container handler
      *
-     * @template T of object
-     *
      * @param ContainerInterface|callable(class-string<T> $id, array<int|string, mixed> $params): ?T $containerHandler Callback function or PSR-11 Container object that sets the container and how it will inject classes
+     *
+     * @template T of object
      */
     public function registerContainerHandler($containerHandler): void
     {

flight/core/Dispatcher.php

@@ -52,11 +52,11 @@ class Dispatcher
     /**
      * Sets the dependency injection container handler.
      *
-     * @template T of object
-     *
      * @param ContainerInterface|(callable(class-string<T> $classString, array<int, mixed> $params): ?T) $containerHandler
      * Dependency injection container.
      *
+     * @template T of object
+     *
      * @throws InvalidArgumentException If $containerHandler is not a `callable` or instance of `Psr\Container\ContainerInterface`.
      */
     public function setContainerHandler($containerHandler): void

tests/FlightTest.php

@@ -134,7 +134,8 @@ public function testStaticRouteGroup()
         Flight::start();
     }
 
-    public function testStaticNestedGroups(): void {
+    public function testStaticNestedGroups(): void
+    {
         Flight::group('/', static function (): void {
             Flight::group('/', static function (): void {
                 Flight::route('GET /', static function (): void {

tests/named-arguments/FlightTest.php

@@ -19,39 +19,39 @@ final class FlightTest extends TestCase
 
     protected function setUp(): void
     {
-        $this->engine = new Engine;
+        $this->engine = new Engine();
         Flight::init();
         Flight::setEngine($this->engine);
     }
 
     //////////////////
     // CORE METHODS //
     //////////////////
-    public function test_path(): void
+    public function testPath(): void
     {
-        Flight::path(dir: __DIR__);
+        Flight::path(dir: __DIR__); // phpcs:ignore
 
         $exampleObject = new ExampleClass();
         self::assertInstanceOf(ExampleClass::class, $exampleObject);
     }
 
-    public function test_stop_with_code(): void
+    public function testStopWithCode(): void
     {
         Flight::stop(code: 500);
 
         self::expectOutputString('');
         self::assertSame(500, Flight::response()->status());
     }
 
-    public function test_halt(): void
+    public function testHalt(): void
     {
         Flight::halt(actuallyExit: false, code: 500, message: 'Test');
 
         self::expectOutputString('Test');
         self::assertSame(500, Flight::response()->status());
     }
 
-    public function test_register(): void
+    public function testRegister(): void
     {
         Flight::register(
             class: stdClass::class,
@@ -71,11 +71,11 @@ class: stdClass::class,
         Flight::unregister(methodName: 'customClass');
     }
 
-    public function test_register_container(): void
+    public function testRegisterContainer(): void
     {
         $dateTime = new DateTimeImmutable();
 
-        $controller = new class($dateTime) {
+        $controller = new class ($dateTime) {
             public function __construct(private DateTimeImmutable $dateTime)
             {
                 //
@@ -108,7 +108,7 @@ public function test(): void
     /////////////////////
     // ROUTING METHODS //
     /////////////////////
-    public function test_static_route(): void
+    public function testStaticRoute(): void
     {
         Flight::request()->url = '/test';