Cleanup codestyle in tests

tests/CodeHelper/TernaryOperatorTest.php

@@ -61,8 +61,6 @@ public function testComplexMultiLine(): void
 			]),
 		);
 
-		echo FlattenSource::source($t->getSourceArray());
-
 		$this->assertSame('$classTest ? $classTest->success(
 	\'test1\',
 	\'test2\',

tests/Renderer/AssertResultTrait.php

@@ -6,9 +6,9 @@
 
 trait AssertResultTrait
 {
-	public function assertSourceResult(array|string $source, string $result)
+	public function assertSourceResult(array|string $source, string $result): void
 	{
-		$file = __DIR__ . '/resources/' . $result .'.result';
+		$file = __DIR__ . '/resources/' . $result . '.result';
 		$flattenSource = is_array($source) ? FlattenSource::source($source) : $source;
 		if (!file_exists($file)) {
 			echo $flattenSource . PHP_EOL;

tests/Renderer/PhpClassTest.php

@@ -393,7 +393,7 @@ public function testStripEmptyTypesFromUses(): void
 		}
 	}
 
-	public function testVariableAttributeFromCtorNotPropegateToSetter(): void
+	public function testVariableAttributeFromCtorNotPropagateToSetter(): void
 	{
 		$class = new PhpClass(Identifier::fromString('Test'));
 
@@ -407,6 +407,9 @@ public function testVariableAttributeFromCtorNotPropegateToSetter(): void
 		$class->addMethod($ctor);
 
 		$renderer = new Php8Renderer();
-		$this->assertSourceResult($renderer->render($class), 'PhpClassTest.testVariableAttributeFromCtorNotPropegateToSetter');
+		$this->assertSourceResult(
+			$renderer->render($class),
+			'PhpClassTest.testVariableAttributeFromCtorNotPropegateToSetter',
+		);
 	}
 }

tests/Renderer/PhpConstantTest.php

@@ -8,7 +8,7 @@
 
 final class PhpConstantTest extends TestCase
 {
-	public function testSimpleConstantWithoutValue()
+	public function testSimpleConstantWithoutValue(): void
 	{
 		$const = PhpConstant::private('test');
 
@@ -17,7 +17,7 @@ public function testSimpleConstantWithoutValue()
 		$this->assertSame(['private const TEST = \'test\';'], $render->renderConstant($const));
 	}
 
-	public function testArrayConstant()
+	public function testArrayConstant(): void
 	{
 		$const = PhpConstant::protected('test', [
 			'test' => 1,
@@ -36,7 +36,7 @@ public function testArrayConstant()
 		], $render->renderConstant($const));
 	}
 
-	public function testLowerCase()
+	public function testLowerCase(): void
 	{
 		$const = new PhpConstant(
 			access: PhpConstant::PROTECTED_ACCESS,
@@ -48,7 +48,7 @@ public function testLowerCase()
 		$this->assertSame(['protected const test = \'test\';'], $render->renderConstant($const));
 	}
 
-	public function testNoCase()
+	public function testNoCase(): void
 	{
 		$const = new PhpConstant(
 			access: PhpConstant::PUBLIC_ACCESS,
@@ -60,15 +60,15 @@ public function testNoCase()
 		$this->assertSame(['public const testCase = \'testCase\';'], $render->renderConstant($const));
 	}
 
-	public function testUpperCaseNoTransform()
+	public function testUpperCaseNoTransform(): void
 	{
 		$const = PhpConstant::public(identifier: 'TEST_CASE');
 		$render = new Php7Renderer();
 
 		$this->assertSame(['public const TEST_CASE = \'TEST_CASE\';'], $render->renderConstant($const));
 	}
 
-	public function testChangeCase()
+	public function testChangeCase(): void
 	{
 		$const = new PhpConstant(
 			access: PhpConstant::PROTECTED_ACCESS,
@@ -84,7 +84,7 @@ public function testChangeCase()
 		$this->assertSame(['protected const TEST = \'test\';'], $render->renderConstant($const));
 	}
 
-	public function testChangeValue()
+	public function testChangeValue(): void
 	{
 		$const = PhpConstant::public(identifier: 'TEST_CASE');
 		$const->setValue('test_value');

tests/Renderer/PhpDocCommentTest.php

@@ -12,15 +12,15 @@ final class PhpDocCommentTest extends TestCase
 {
 	use AssertResultTrait;
 
-	public function testVarBlock()
+	public function testVarBlock(): void
 	{
 		$comment = PhpDocComment::var(Type::fromString('string'));
 		$renderer = new Php7Renderer();
 
 		$this->assertSame(['/** @var string */'], $renderer->renderComment($comment));
 	}
 
-	public function testWithDescription()
+	public function testWithDescription(): void
 	{
 		$comment = new PhpDocComment('Test Description');
 		$comment->setParams(PhpDocElementFactory::getParam('int', '$param', 'test desc'));
@@ -30,7 +30,7 @@ public function testWithDescription()
 		$this->assertSourceResult($renderer->renderComment($comment), 'PhpDocCommentTest.' . __FUNCTION__);
 	}
 
-	public function testWithoutDescription()
+	public function testWithoutDescription(): void
 	{
 		$comment = new PhpDocComment();
 		$comment->setParams(PhpDocElementFactory::getParam('int', '$param', 'test desc'));
@@ -40,23 +40,23 @@ public function testWithoutDescription()
 		$this->assertSourceResult($renderer->renderComment($comment), 'PhpDocCommentTest.' . __FUNCTION__);
 	}
 
-	public function testOnlyDescription()
+	public function testOnlyDescription(): void
 	{
 		$comment = new PhpDocComment('test');
 
 		$renderer = new Php7Renderer();
 		$this->assertSourceResult($renderer->renderComment($comment), 'PhpDocCommentTest.' . __FUNCTION__);
 	}
 
-	public function testEmpty()
+	public function testEmpty(): void
 	{
 		$comment = new PhpDocComment();
 		$renderer = new Php7Renderer();
 
 		$this->assertCount(0, $renderer->renderComment($comment));
 	}
 
-	public function testDeprecation()
+	public function testDeprecation(): void
 	{
 		$deprecated = PhpDocElementFactory::getDeprecated('use X instead');
 		$comment = new PhpDocComment();
@@ -66,7 +66,7 @@ public function testDeprecation()
 		$this->assertSourceResult($renderer->renderComment($comment), 'PhpDocCommentTest.' . __FUNCTION__);
 	}
 
-	public function testLicense()
+	public function testLicense(): void
 	{
 		$info = 'https://github.com/stefnadev/log/blob/develop/LICENSE.md MIT Licence';
 		$licence = PhpDocElementFactory::getLicence($info);
@@ -77,7 +77,7 @@ public function testLicense()
 		$this->assertSourceResult($renderer->renderComment($comment), 'PhpDocCommentTest.' . __FUNCTION__);
 	}
 
-	public function testGenerated()
+	public function testGenerated(): void
 	{
 		$licence = PhpDocElementFactory::getGenerated('From X');
 		$comment = new PhpDocComment();
@@ -91,7 +91,7 @@ public function testGenerated()
 		], $renderer->renderComment($comment));
 	}
 
-	public function testCheckForParamInCommentAndRemove()
+	public function testCheckForParamInCommentAndRemove(): void
 	{
 		$comment = new PhpDocComment('Test Description');
 		$comment->setParams(

tests/Renderer/PhpEnumTest.php

@@ -2,6 +2,7 @@
 
 namespace Stefna\PhpCodeBuilder\Tests\Renderer;
 
+use PHPUnit\Framework\Attributes\DataProvider;
 use PHPUnit\Framework\TestCase;
 use Stefna\PhpCodeBuilder\PhpEnum;
 use Stefna\PhpCodeBuilder\Renderer\Php74Renderer;
@@ -15,19 +16,15 @@ class PhpEnumTest extends TestCase
 {
 	use AssertResultTrait;
 
-	/**
-	 * @dataProvider enumTypes
-	 */
+	#[DataProvider('enumTypes')]
 	public function testPhp7(PhpEnum $enum, string $expectedFile): void
 	{
 		$renderer = new Php74Renderer();
 
 		$this->assertSourceResult($renderer->render($enum), 'PhpEnumTest.' . __FUNCTION__ . '.' . $expectedFile);
 	}
 
-	/**
-	 * @dataProvider enumTypes
-	 */
+	#[DataProvider('enumTypes')]
 	public function testPhp81(PhpEnum $enum, string $expectedFile): void
 	{
 		$renderer = new Php81Renderer();

tests/Renderer/PhpFileTest.php

@@ -16,7 +16,7 @@ final class PhpFileTest extends TestCase
 {
 	use AssertResultTrait;
 
-	public function testSingleClassFile()
+	public function testSingleClassFile(): void
 	{
 		$class = new PhpClass(
 			Identifier::fromString(Test\TestClass::class),

tests/Renderer/PhpFunctionTest.php

@@ -19,7 +19,7 @@ final class PhpFunctionTest extends TestCase
 {
 	use AssertResultTrait;
 
-	public function testSimpleFunction()
+	public function testSimpleFunction(): void
 	{
 		$func = new PhpFunction(
 			'testFunction',
@@ -39,7 +39,7 @@ public function testSimpleFunction()
 		$this->assertSourceResult($renderer->renderFunction($func), 'PhpFunctionTest.' . __FUNCTION__);
 	}
 
-	public function testFunctionWithMoreThan3Params()
+	public function testFunctionWithMoreThan3Params(): void
 	{
 		$param3 = new PhpParam('test3', Type::fromString('string'));
 		$func = (new PhpFunction(
@@ -61,7 +61,7 @@ public function testFunctionWithMoreThan3Params()
 		$this->assertSourceResult($renderer->render($func), 'PhpFunctionTest.' . __FUNCTION__);
 	}
 
-	public function testFunctionModifyParam()
+	public function testFunctionModifyParam(): void
 	{
 		$func = new PhpFunction(
 			'testFunction',
@@ -87,7 +87,7 @@ public function testFunctionModifyParam()
 		$this->assertSourceResult($renderer->renderFunction($func), 'PhpFunctionTest.' . __FUNCTION__);
 	}
 
-	public function testCloneHandlingParams()
+	public function testCloneHandlingParams(): void
 	{
 		$func = new PhpFunction('testFunction', [
 			new PhpParam('test', Type::fromString('string')),
@@ -102,7 +102,7 @@ public function testCloneHandlingParams()
 		);
 	}
 
-	public function testLegacyFunctionTestSimple()
+	public function testLegacyFunctionTestSimple(): void
 	{
 		$func = new PhpFunction('test', [
 			new PhpParam('foo', Type::fromString('string|int')),
@@ -123,7 +123,7 @@ public function testLegacyFunctionTestSimple()
 		], $renderer->renderFunction($func));
 	}
 
-	public function testLegacyFunctionTestWithoutReturnType()
+	public function testLegacyFunctionTestWithoutReturnType(): void
 	{
 		$func = new PhpFunction('test', [
 			new PhpParam('foo', Type::fromString('string|int')),
@@ -142,7 +142,7 @@ public function testLegacyFunctionTestWithoutReturnType()
 		], $renderer->renderFunction($func));
 	}
 
-	public function testLegacyFunctionTestComplexSource()
+	public function testLegacyFunctionTestComplexSource(): void
 	{
 		$func = new PhpFunction('test', [
 			new PhpParam('foo', Type::fromString('string|int')),
@@ -169,7 +169,7 @@ function test($foo): int
 ', FlattenSource::source($renderer->renderFunction($func)));
 	}
 
-	public function testLegacyFunctionTestMultilineParams()
+	public function testLegacyFunctionTestMultilineParams(): void
 	{
 		$func = new PhpFunction('test', [
 			new PhpParam('fooIpsumLong', Type::fromString('string')),

tests/Renderer/PhpInterfaceTest.php

@@ -2,6 +2,7 @@
 
 namespace Stefna\PhpCodeBuilder\Tests\Renderer;
 
+use PHPUnit\Framework\Attributes\DataProvider;
 use PHPUnit\Framework\TestCase;
 use Stefna\PhpCodeBuilder\PhpClass;
 use Stefna\PhpCodeBuilder\PhpConstant;
@@ -19,7 +20,7 @@ final class PhpInterfaceTest extends TestCase
 {
 	use AssertResultTrait;
 
-	public function testSimpleInterface()
+	public function testSimpleInterface(): void
 	{
 		$interface = new PhpInterface(Identifier::fromString(\Test\TestInterface::class));
 		$interface->addMethod(PhpMethod::public('testMethod', [], []));
@@ -29,7 +30,7 @@ public function testSimpleInterface()
 		$this->assertSourceResult($renderer->renderInterface($interface), 'PhpInterfaceTest.' . __FUNCTION__);
 	}
 
-	public function testExtendSingleInterface()
+	public function testExtendSingleInterface(): void
 	{
 		$interface = new PhpInterface(Identifier::fromString(\Test\TestInterface::class));
 		$interface->addMethod(PhpMethod::public('testMethod', [], []));
@@ -40,7 +41,7 @@ public function testExtendSingleInterface()
 		$this->assertSourceResult($renderer->renderInterface($interface), 'PhpInterfaceTest.' . __FUNCTION__);
 	}
 
-	public function testExtendMultipleInterface()
+	public function testExtendMultipleInterface(): void
 	{
 		$interface = new PhpInterface(Identifier::fromString(\Test\TestInterface::class));
 		$interface->addMethod(PhpMethod::public('testMethod', [], []));
@@ -53,7 +54,7 @@ public function testExtendMultipleInterface()
 		$this->assertSourceResult($renderer->renderInterface($interface), 'PhpInterfaceTest.' . __FUNCTION__);
 	}
 
-	public function testInterfaceWithEverything()
+	public function testInterfaceWithEverything(): void
 	{
 		$interface = new PhpInterface(Identifier::fromString(\Test\TestInterface::class));
 		$interface->addMethod(PhpMethod::public('testMethod', [], []));
@@ -71,12 +72,10 @@ public function testInterfaceWithEverything()
 		$this->assertSourceResult($renderer->render($interface), 'PhpInterfaceTest.' . __FUNCTION__);
 	}
 
-	/**
-	 * @dataProvider privateProtectedStuff
-	 */
+	#[DataProvider('privateProtectedStuff')]
 	public function testAddingPrivateStuffToInterface(
 		PhpVariable|PhpConstant|PhpMethod $stuff,
-	) {
+	): void {
 		$interface = new PhpInterface(Identifier::fromString(\Test\TestInterface::class));
 
 		$this->expectException(\BadMethodCallException::class);
@@ -92,9 +91,7 @@ public function testAddingPrivateStuffToInterface(
 		}
 	}
 
-	/**
-	 * @dataProvider privateProtectedStuff
-	 */
+	#[DataProvider('privateProtectedStuff')]
 	public function testAddingPrivateStuffToInterfaceWithConvert(
 		PhpVariable|PhpConstant|PhpMethod $stuff,
 	): void {