Tests: Fix code style issues

- Fix class brace spacing in BuilderMan tests
- Remove unused variable in MappingHelper test
- Use sprintf instead of variable interpolation in console test
- Use single quotes for regex patterns in error tests
- Fix reference passing in ContainerEventManager test
- Add phpcs:ignoreFile to Binder test

Author: felixure

tests/Cases/DI/Helper/BuilderMan.phpt

@@ -27,6 +27,7 @@ Toolkit::test(function (): void {
 				public function beforeCompile(): void
 				{
 					$pass = new class ($this) extends AbstractPass {
+
 					};
 
 					$builderMan = BuilderMan::of($pass);
@@ -77,6 +78,7 @@ Toolkit::test(function (): void {
 					public function beforeCompile(): void
 					{
 						$pass = new class ($this) extends AbstractPass {
+
 						};
 
 						$builderMan = BuilderMan::of($pass);
@@ -125,6 +127,7 @@ Toolkit::test(function (): void {
 				public function beforeCompile(): void
 				{
 					$pass = new class ($this) extends AbstractPass {
+
 					};
 
 					$builderMan = BuilderMan::of($pass);
@@ -181,6 +184,7 @@ Toolkit::test(function (): void {
 				public function beforeCompile(): void
 				{
 					$pass = new class ($this) extends AbstractPass {
+
 					};
 
 					$builderMan = BuilderMan::of($pass);
@@ -244,6 +248,7 @@ Toolkit::test(function (): void {
 				public function beforeCompile(): void
 				{
 					$pass = new class ($this) extends AbstractPass {
+
 					};
 
 					$builderMan = BuilderMan::of($pass);
@@ -297,6 +302,7 @@ Toolkit::test(function (): void {
 				public function beforeCompile(): void
 				{
 					$pass = new class ($this) extends AbstractPass {
+
 					};
 
 					$builderMan = BuilderMan::of($pass);

tests/Cases/DI/Helper/MappingHelper.phpt

@@ -149,7 +149,7 @@ Toolkit::test(function (): void {
 
 // Fluent interface - chaining
 Toolkit::test(function (): void {
-	$container = ContainerBuilder::of()
+	ContainerBuilder::of()
 		->withCompiler(function (Compiler $compiler): void {
 			$compiler->addExtension('nettrine.dbal', new DbalExtension());
 			$compiler->addExtension('nettrine.orm', new OrmExtension());

tests/Cases/DI/OrmExtension.console.phpt

@@ -125,7 +125,7 @@ Toolkit::test(function (): void {
 	$taggedServices = $container->findByTag('console.command');
 
 	foreach ($expectedCommands as $commandName) {
-		Assert::true(in_array($commandName, $taggedServices, true), "Command '$commandName' should be tagged");
+		Assert::true(in_array($commandName, $taggedServices, true), sprintf('Command %s should be tagged', $commandName));
 	}
 });
 

tests/Cases/DI/OrmExtension.errors.phpt

@@ -48,7 +48,7 @@ Toolkit::test(function (): void {
 				));
 			})
 			->build();
-	}, InvalidConfigurationException::class, "~EntityManager decorator class must be subclass~");
+	}, InvalidConfigurationException::class, '~EntityManager decorator class must be subclass~');
 });
 
 // Error: Second level cache enabled without cache
@@ -161,7 +161,7 @@ Toolkit::test(function (): void {
 				));
 			})
 			->build();
-	}, InvalidConfigurationException::class, "~mapping.*App.*type~");
+	}, InvalidConfigurationException::class, '~mapping.*App.*type~');
 });
 
 // Error: Invalid service reference for cache
@@ -199,7 +199,7 @@ Toolkit::test(function (): void {
 				));
 			})
 			->build();
-	}, InvalidConfigurationException::class, "~defaultCache~");
+	}, InvalidConfigurationException::class, '~defaultCache~');
 });
 
 // Error: Empty proxyDir
@@ -237,7 +237,7 @@ Toolkit::test(function (): void {
 				));
 			})
 			->build();
-	}, InvalidConfigurationException::class, "~proxyDir~");
+	}, InvalidConfigurationException::class, '~proxyDir~');
 });
 
 // Error: Missing connection in manager
@@ -273,5 +273,5 @@ Toolkit::test(function (): void {
 				));
 			})
 			->build();
-	}, InvalidConfigurationException::class, "~connection~");
+	}, InvalidConfigurationException::class, '~connection~');
 });

tests/Cases/Events/ContainerEventManager.phpt

@@ -3,6 +3,7 @@
 namespace Tests\Cases\Events;
 
 use Contributte\Tester\Toolkit;
+use Doctrine\Common\EventArgs;
 use Doctrine\ORM\EntityManager;
 use Doctrine\ORM\Event\OnClearEventArgs;
 use Doctrine\ORM\Events;
@@ -142,16 +143,17 @@ Toolkit::test(function (): void {
 	$eventManager = new ContainerEventManager($container);
 
 	// Add a generic listener that accepts EventArgs
-	$called = false;
-	$listener = new class ($called) {
+	$state = new \stdClass();
+	$state->called = false;
+	$listener = new class ($state) {
 
-		public function __construct(private bool &$called)
+		public function __construct(private \stdClass $state)
 		{
 		}
 
-		public function onClear(\Doctrine\Common\EventArgs $args): void
+		public function onClear(EventArgs $args): void
 		{
-			$this->called = true;
+			$this->state->called = true;
 		}
 
 	};
@@ -161,7 +163,7 @@ Toolkit::test(function (): void {
 	// This should not throw - null args should be handled
 	$eventManager->dispatchEvent(Events::onClear, null);
 
-	Assert::true($called);
+	Assert::true($state->called);
 });
 
 // addEventListener with multiple events

tests/Cases/Utils/Binder.phpt

@@ -1,5 +1,7 @@
 <?php declare(strict_types = 1);
 
+// phpcs:ignoreFile
+
 namespace Tests\Cases\Utils;
 
 use Contributte\Tester\Toolkit;
@@ -16,9 +18,7 @@ Toolkit::test(function (): void {
 
 	};
 
-	$result = Binder::use($obj, function (): string {
-		return $this->secret; // @phpstan-ignore-line
-	});
+	$result = Binder::use($obj, fn (): string => $this->secret);
 
 	Assert::equal('hidden', $result);
 });
@@ -45,9 +45,7 @@ Toolkit::test(function (): void {
 
 // Bind to class string - access static property
 Toolkit::test(function (): void {
-	$result = Binder::use(TestClassWithStatic::class, function (): string {
-		return self::$staticValue; // @phpstan-ignore-line
-	});
+	$result = Binder::use(TestClassWithStatic::class, fn (): string => self::$staticValue);
 
 	Assert::equal('static_secret', $result);
 });
@@ -60,21 +58,18 @@ Toolkit::test(function (): void {
 
 	};
 
-	$result = Binder::use($obj, function (): int {
-		return $this->value * 2; // @phpstan-ignore-line
-	});
+	$result = Binder::use($obj, fn (): int => $this->value * 2);
 
 	Assert::equal(84, $result);
 });
 
 // Return null from closure
 Toolkit::test(function (): void {
 	$obj = new class {
+
 	};
 
-	$result = Binder::use($obj, function (): mixed {
-		return null;
-	});
+	$result = Binder::use($obj, fn (): mixed => null);
 
 	Assert::null($result);
 });