Set all tests as final

picamator · picamator · commit f2fdb0eef709 · 2026-02-10T18:51:31.000+01:00
diff --git a/AGENTS.md b/AGENTS.md
@@ -98,6 +98,10 @@ Code Style
   * **validator** classes should have `public` methods prefixed by `validate`
   * methods returning `bool` should be prefixed by `is`
 
+### Tests
+
+- all tests classes should be a `final`
+
 Module Structure
 ----------------
 
diff --git a/tests/integration/Command/DefinitionGeneratorCommandTest.php b/tests/integration/Command/DefinitionGeneratorCommandTest.php
@@ -11,7 +11,7 @@
 use Symfony\Component\Console\Tester\CommandTester;
 
 #[Group('command')]
-class DefinitionGeneratorCommandTest extends TestCase
+final class DefinitionGeneratorCommandTest extends TestCase
 {
     private CommandTester $commandTester;
 
diff --git a/tests/integration/Command/TransferGeneratorBulkCommandTest.php b/tests/integration/Command/TransferGeneratorBulkCommandTest.php
@@ -13,7 +13,7 @@
 use Symfony\Component\Console\Tester\CommandTester;
 
 #[Group('command')]
-class TransferGeneratorBulkCommandTest extends TestCase
+final class TransferGeneratorBulkCommandTest extends TestCase
 {
     use FailedFiberTrait;
 
diff --git a/tests/integration/Command/TransferGeneratorCommandTest.php b/tests/integration/Command/TransferGeneratorCommandTest.php
@@ -14,7 +14,7 @@
 use Symfony\Component\Console\Tester\CommandTester;
 
 #[Group('command')]
-class TransferGeneratorCommandTest extends TestCase
+final class TransferGeneratorCommandTest extends TestCase
 {
     use FailedFiberTrait;
 
diff --git a/tests/integration/DefinitionGenerator/DefinitionGeneratorFacadeTest.php b/tests/integration/DefinitionGenerator/DefinitionGeneratorFacadeTest.php
@@ -25,7 +25,7 @@
 
 #[Large]
 #[Group('definition-generator')]
-class DefinitionGeneratorFacadeTest extends TestCase
+final class DefinitionGeneratorFacadeTest extends TestCase
 {
     use DefinitionGeneratorTrait;
     use TransferGeneratorTrait;
diff --git a/tests/integration/Examples/ExamplesTest.php b/tests/integration/Examples/ExamplesTest.php
@@ -13,7 +13,7 @@
 use Picamator\Tests\Integration\TransferObject\Helper\OutputBufferTrait;
 
 #[Group('examples')]
-class ExamplesTest extends TestCase
+final class ExamplesTest extends TestCase
 {
     use OutputBufferTrait;
 
diff --git a/tests/integration/Transfer/Adapter/DummyTransferAdapterTest.php b/tests/integration/Transfer/Adapter/DummyTransferAdapterTest.php
@@ -10,7 +10,7 @@
 use Picamator\Tests\Integration\TransferObject\Transfer\Advanced\BookmarkData;
 
 #[Group('transfer')]
-class DummyTransferAdapterTest extends TestCase
+final class DummyTransferAdapterTest extends TestCase
 {
     private BookmarkData $bookmarkData;
 
diff --git a/tests/integration/Transfer/Adapter/TransferAdapterTest.php b/tests/integration/Transfer/Adapter/TransferAdapterTest.php
@@ -19,7 +19,7 @@
 use Picamator\Tests\Integration\TransferObject\Transfer\Generated\ReservedAdvancedTransfer;
 
 #[Group('transfer')]
-class TransferAdapterTest extends TestCase
+final class TransferAdapterTest extends TestCase
 {
     #[TestDox('Transformation fromArray() and toArray()')]
     public function testTransformationFromArrayToArray(): void
diff --git a/tests/integration/Transfer/NumberTransferTest.php b/tests/integration/Transfer/NumberTransferTest.php
@@ -16,7 +16,7 @@
 
 #[Group('transfer')]
 #[RequiresPhpExtension('bcmath')]
-class NumberTransferTest extends TestCase
+final class NumberTransferTest extends TestCase
 {
     use TransferGeneratorTrait;
 
diff --git a/tests/integration/Transfer/TransferTest.php b/tests/integration/Transfer/TransferTest.php
@@ -31,7 +31,7 @@
 use TypeError;
 
 #[Group('transfer')]
-class TransferTest extends TestCase
+final class TransferTest extends TestCase
 {
     use TransferGeneratorTrait;
 
diff --git a/tests/integration/TransferGenerator/Config/Loader/ConfigLoaderTest.php b/tests/integration/TransferGenerator/Config/Loader/ConfigLoaderTest.php
@@ -13,7 +13,7 @@
 use Picamator\TransferObject\TransferGenerator\Config\Loader\ConfigLoaderInterface;
 
 #[Group('transfer-generator')]
-class ConfigLoaderTest extends TestCase
+final class ConfigLoaderTest extends TestCase
 {
     private const string CONFIG_PATH = __DIR__ . '/../../data/config/error/';
 
diff --git a/tests/integration/TransferGenerator/TransferGeneratorFacadeErrorTest.php b/tests/integration/TransferGenerator/TransferGeneratorFacadeErrorTest.php
@@ -15,7 +15,7 @@
 use Picamator\TransferObject\TransferGenerator\TransferGeneratorFacade;
 
 #[Group('transfer-generator')]
-class TransferGeneratorFacadeErrorTest extends TestCase
+final class TransferGeneratorFacadeErrorTest extends TestCase
 {
     use TransferGeneratorTrait;
 
diff --git a/tests/integration/TransferGenerator/TransferGeneratorFacadeSuccessTest.php b/tests/integration/TransferGenerator/TransferGeneratorFacadeSuccessTest.php
@@ -10,7 +10,7 @@
 use Picamator\Tests\Integration\TransferObject\Helper\TransferGeneratorTrait;
 
 #[Group('transfer-generator')]
-class TransferGeneratorFacadeSuccessTest extends TestCase
+final class TransferGeneratorFacadeSuccessTest extends TestCase
 {
     use TransferGeneratorTrait;
 
diff --git a/tests/unit/Command/Helper/InputNormalizerTest.php b/tests/unit/Command/Helper/InputNormalizerTest.php
@@ -12,7 +12,7 @@
 use Picamator\TransferObject\Command\Helper\InputNormalizerTrait;
 
 #[Group('command')]
-class InputNormalizerTest extends TestCase
+final class InputNormalizerTest extends TestCase
 {
     private InputNormalizerInterface $inputNormalizer;
 
diff --git a/tests/unit/DefinitionGenerator/Content/DefinitionContentBuilderTest.php b/tests/unit/DefinitionGenerator/Content/DefinitionContentBuilderTest.php
@@ -13,7 +13,7 @@
 use Picamator\TransferObject\DefinitionGenerator\Exception\DefinitionGeneratorException;
 
 #[Group('definition-generator')]
-class DefinitionContentBuilderTest extends TestCase
+final class DefinitionContentBuilderTest extends TestCase
 {
     use FileTrait;
 
diff --git a/tests/unit/DefinitionGenerator/Content/Expander/BuiltInTypeBuilderExpanderTest.php b/tests/unit/DefinitionGenerator/Content/Expander/BuiltInTypeBuilderExpanderTest.php
@@ -16,7 +16,7 @@
 use stdClass;
 
 #[Group('definition-generator')]
-class BuiltInTypeBuilderExpanderTest extends TestCase
+final class BuiltInTypeBuilderExpanderTest extends TestCase
 {
     private BuilderExpanderInterface $expander;
 
diff --git a/tests/unit/DefinitionGenerator/Content/Expander/CollectionTypeBuilderExpanderTest.php b/tests/unit/DefinitionGenerator/Content/Expander/CollectionTypeBuilderExpanderTest.php
@@ -16,7 +16,7 @@
 use ReflectionMethod;
 
 #[Group('definition-generator')]
-class CollectionTypeBuilderExpanderTest extends TestCase
+final class CollectionTypeBuilderExpanderTest extends TestCase
 {
     private BuilderExpanderInterface $expander;
 
diff --git a/tests/unit/DefinitionGenerator/Content/Expander/TransferTypeBuilderExpanderTest.php b/tests/unit/DefinitionGenerator/Content/Expander/TransferTypeBuilderExpanderTest.php
@@ -16,7 +16,7 @@
 use ReflectionMethod;
 
 #[Group('definition-generator')]
-class TransferTypeBuilderExpanderTest extends TestCase
+final class TransferTypeBuilderExpanderTest extends TestCase
 {
     private BuilderExpanderInterface $expander;
 
diff --git a/tests/unit/DefinitionGenerator/Generator/Builder/DefinitionGeneratorBuilderTest.php b/tests/unit/DefinitionGenerator/Generator/Builder/DefinitionGeneratorBuilderTest.php
@@ -19,7 +19,7 @@
 use Picamator\TransferObject\Shared\Validator\PathLocalValidatorInterface;
 
 #[Group('definition-generator')]
-class DefinitionGeneratorBuilderTest extends TestCase
+final class DefinitionGeneratorBuilderTest extends TestCase
 {
     private DefinitionGeneratorBuilderInterface $builder;
 
diff --git a/tests/unit/DefinitionGenerator/Generator/Render/DefinitionRenderTest.php b/tests/unit/DefinitionGenerator/Generator/Render/DefinitionRenderTest.php
@@ -19,7 +19,7 @@
 use Picamator\TransferObject\Generated\DefinitionPropertyTransfer;
 
 #[Group('definition-generator')]
-class DefinitionRenderTest extends TestCase
+final class DefinitionRenderTest extends TestCase
 {
     private const string TEST_CLASS_NAME = 'TestClass';
 
diff --git a/tests/unit/Shared/Filesystem/FileAppenderTest.php b/tests/unit/Shared/Filesystem/FileAppenderTest.php
@@ -13,7 +13,7 @@
 use Picamator\TransferObject\Shared\Filesystem\FileAppender;
 
 #[Group('shared')]
-class FileAppenderTest extends TestCase
+final class FileAppenderTest extends TestCase
 {
     use FileTrait;
 
diff --git a/tests/unit/Shared/Filesystem/FileReaderTest.php b/tests/unit/Shared/Filesystem/FileReaderTest.php
@@ -13,7 +13,7 @@
 use Picamator\TransferObject\Shared\Filesystem\FileReader;
 
 #[Group('shared')]
-class FileReaderTest extends TestCase
+final class FileReaderTest extends TestCase
 {
     use FileTrait;
 
diff --git a/tests/unit/Shared/Reader/FileReaderProgressTest.php b/tests/unit/Shared/Reader/FileReaderProgressTest.php
@@ -16,7 +16,7 @@
 use Picamator\TransferObject\Shared\Reader\FileReaderProgress;
 
 #[Group('shared')]
-class FileReaderProgressTest extends TestCase
+final class FileReaderProgressTest extends TestCase
 {
     private FileReaderProgress&MockObject $fileReaderProgressMock;
 
diff --git a/tests/unit/Shared/Reader/JsonReaderTest.php b/tests/unit/Shared/Reader/JsonReaderTest.php
@@ -15,7 +15,7 @@
 use Picamator\TransferObject\Shared\Reader\JsonReaderInterface;
 
 #[Group('shared')]
-class JsonReaderTest extends TestCase
+final class JsonReaderTest extends TestCase
 {
     private const string FILE_PATH = 'test.json';
 
diff --git a/tests/unit/Shared/Validator/FileSizeValidatorTest.php b/tests/unit/Shared/Validator/FileSizeValidatorTest.php
@@ -9,7 +9,7 @@
 use PHPUnit\Framework\TestCase;
 use Picamator\TransferObject\Shared\Validator\FileSizeValidator;
 
-class FileSizeValidatorTest extends TestCase
+final class FileSizeValidatorTest extends TestCase
 {
     /**
      * @see \Picamator\TransferObject\Shared\Validator\FileSizeValidator::MAX_FILE_SIZE_BYTES
diff --git a/tests/unit/Shared/Validator/NamespaceValidatorTest.php b/tests/unit/Shared/Validator/NamespaceValidatorTest.php
@@ -12,7 +12,7 @@
 use Picamator\TransferObject\Shared\Validator\NamespaceValidatorTrait;
 
 #[Group('shared')]
-class NamespaceValidatorTest extends TestCase
+final class NamespaceValidatorTest extends TestCase
 {
     private NamespaceValidatorInterface $validator;
 
diff --git a/tests/unit/Transfer/Attribute/AttributeTest.php b/tests/unit/Transfer/Attribute/AttributeTest.php
@@ -11,7 +11,7 @@
 use Picamator\TransferObject\Transfer\Exception\AttributeTransferException;
 
 #[Group('transfer')]
-class AttributeTest extends TestCase
+final class AttributeTest extends TestCase
 {
     private AttributeInterface $attribute;
 
diff --git a/tests/unit/TransferGenerator/Config/Config/ConfigProxyTest.php b/tests/unit/TransferGenerator/Config/Config/ConfigProxyTest.php
@@ -12,7 +12,7 @@
 use Picamator\TransferObject\TransferGenerator\Exception\TransferGeneratorConfigNotFoundException;
 
 #[Group('transfer-generator')]
-class ConfigProxyTest extends TestCase
+final class ConfigProxyTest extends TestCase
 {
     private ConfigInterface $proxy;
 
diff --git a/tests/unit/TransferGenerator/Config/Environment/ConfigEnvironmentRenderTest.php b/tests/unit/TransferGenerator/Config/Environment/ConfigEnvironmentRenderTest.php
@@ -11,7 +11,7 @@
 use Picamator\TransferObject\TransferGenerator\Config\Environment\ConfigEnvironmentRender;
 
 #[Group('transfer-generator')]
-class ConfigEnvironmentRenderTest extends TestCase
+final class ConfigEnvironmentRenderTest extends TestCase
 {
     private ConfigEnvironmentRender&MockObject $renderMock;
 
diff --git a/tests/unit/TransferGenerator/Definition/Parser/Expander/Builder/NamespaceBuilderTest.php b/tests/unit/TransferGenerator/Definition/Parser/Expander/Builder/NamespaceBuilderTest.php
@@ -14,7 +14,7 @@
 use Picamator\TransferObject\TransferGenerator\Definition\Parser\Expander\Builder\NamespaceBuilderInterface;
 
 #[Group('transfer-generator')]
-class NamespaceBuilderTest extends TestCase
+final class NamespaceBuilderTest extends TestCase
 {
     private NamespaceBuilderInterface $builder;
 
diff --git a/tests/unit/TransferGenerator/Definition/Validator/Content/Property/NumberTypePropertyValidatorTest.php b/tests/unit/TransferGenerator/Definition/Validator/Content/Property/NumberTypePropertyValidatorTest.php
@@ -16,7 +16,7 @@
 use Picamator\TransferObject\TransferGenerator\Definition\Validator\Content\Property\PropertyValidatorInterface;
 
 #[Group('transfer-generator')]
-class NumberTypePropertyValidatorTest extends TestCase
+final class NumberTypePropertyValidatorTest extends TestCase
 {
     private PropertyValidatorInterface&MockObject $validatorMock;
 
diff --git a/tests/unit/TransferGenerator/Generator/Filesystem/GeneratorFilesystemTest.php b/tests/unit/TransferGenerator/Generator/Filesystem/GeneratorFilesystemTest.php
@@ -17,7 +17,7 @@
 use Picamator\TransferObject\TransferGenerator\Generator\Filesystem\GeneratorFilesystemInterface;
 
 #[Group('transfer-generator')]
-class GeneratorFilesystemTest extends TestCase
+final class GeneratorFilesystemTest extends TestCase
 {
     private GeneratorFilesystemInterface $generatorFilesystem;
 
diff --git a/tests/unit/TransferGenerator/Generator/Generator/Processor/Command/PostProcessCommandTest.php b/tests/unit/TransferGenerator/Generator/Generator/Processor/Command/PostProcessCommandTest.php
@@ -15,7 +15,7 @@
 use Picamator\TransferObject\TransferGenerator\Generator\Generator\Processor\Command\PostProcessCommandInterface;
 
 #[Group('transfer-generator')]
-class PostProcessCommandTest extends TestCase
+final class PostProcessCommandTest extends TestCase
 {
     private PostProcessCommandInterface $command;
 
diff --git a/tests/unit/TransferGenerator/Generator/Generator/Processor/Command/PreProcessCommandTest.php b/tests/unit/TransferGenerator/Generator/Generator/Processor/Command/PreProcessCommandTest.php
@@ -18,7 +18,7 @@
 use Picamator\TransferObject\TransferGenerator\Generator\Generator\Processor\Command\PreProcessCommandInterface;
 
 #[Group('transfer-generator')]
-class PreProcessCommandTest extends TestCase
+final class PreProcessCommandTest extends TestCase
 {
     private PreProcessCommandInterface $command;
 
diff --git a/tests/unit/TransferGenerator/Generator/Generator/ServiceTransferGeneratorTest.php b/tests/unit/TransferGenerator/Generator/Generator/ServiceTransferGeneratorTest.php
@@ -16,7 +16,7 @@
 use Picamator\TransferObject\TransferGenerator\Generator\Generator\Workflow\TransferGeneratorWorkflowInterface;
 
 #[Group('transfer-generator')]
-class ServiceTransferGeneratorTest extends TestCase
+final class ServiceTransferGeneratorTest extends TestCase
 {
     private TransferGeneratorServiceInterface $serviceGenerator;
 
diff --git a/tests/unit/TransferGenerator/Generator/Render/Expander/MetaConstantsTemplateExpanderTest.php b/tests/unit/TransferGenerator/Generator/Render/Expander/MetaConstantsTemplateExpanderTest.php
@@ -14,7 +14,7 @@
 use Picamator\TransferObject\TransferGenerator\Generator\Render\Expander\TemplateExpanderInterface;
 
 #[Group('transfer-generator')]
-class MetaConstantsTemplateExpanderTest extends TestCase
+final class MetaConstantsTemplateExpanderTest extends TestCase
 {
     private TemplateExpanderInterface $expander;
 
diff --git a/tests/unit/TransferGenerator/Generator/Render/Template/TemplateHelperTest.php b/tests/unit/TransferGenerator/Generator/Render/Template/TemplateHelperTest.php
@@ -13,7 +13,7 @@
 use Picamator\TransferObject\TransferGenerator\Generator\Render\Template\Helper\TemplateHelper;
 
 #[Group('transfer-generator')]
-class TemplateHelperTest extends TestCase
+final class TemplateHelperTest extends TestCase
 {
     /**
      * @param array<string,mixed> $templateData
diff --git a/tests/unit/TransferGenerator/Generator/Render/TemplateRenderTest.php b/tests/unit/TransferGenerator/Generator/Render/TemplateRenderTest.php
@@ -19,7 +19,7 @@
 use Picamator\TransferObject\TransferGenerator\Generator\Render\TemplateRenderInterface;
 
 #[Group('transfer-generator')]
-class TemplateRenderTest extends TestCase
+final class TemplateRenderTest extends TestCase
 {
     private TemplateRenderInterface $render;
 

Original file line number	Diff line number	Diff line change
`@@ -11,7 +11,7 @@`
`11`	`11`	`use Symfony\Component\Console\Tester\CommandTester;`
`12`	`12`
`13`	`13`	`#[Group('command')]`
`14`		`-class DefinitionGeneratorCommandTest extends TestCase`
	`14`	`+final class DefinitionGeneratorCommandTest extends TestCase`
`15`	`15`	`{`
`16`	`16`	`private CommandTester $commandTester;`
`17`	`17`
Original file line number	Diff line number	Diff line change
`@@ -13,7 +13,7 @@`
`13`	`13`	`use Symfony\Component\Console\Tester\CommandTester;`
`14`	`14`
`15`	`15`	`#[Group('command')]`
`16`		`-class TransferGeneratorBulkCommandTest extends TestCase`
	`16`	`+final class TransferGeneratorBulkCommandTest extends TestCase`
`17`	`17`	`{`
`18`	`18`	`use FailedFiberTrait;`
`19`	`19`
Original file line number	Diff line number	Diff line change
`@@ -14,7 +14,7 @@`
`14`	`14`	`use Symfony\Component\Console\Tester\CommandTester;`
`15`	`15`
`16`	`16`	`#[Group('command')]`
`17`		`-class TransferGeneratorCommandTest extends TestCase`
	`17`	`+final class TransferGeneratorCommandTest extends TestCase`
`18`	`18`	`{`
`19`	`19`	`use FailedFiberTrait;`
`20`	`20`
Original file line number	Diff line number	Diff line change
`@@ -25,7 +25,7 @@`
`25`	`25`
`26`	`26`	`#[Large]`
`27`	`27`	`#[Group('definition-generator')]`
`28`		`-class DefinitionGeneratorFacadeTest extends TestCase`
	`28`	`+final class DefinitionGeneratorFacadeTest extends TestCase`
`29`	`29`	`{`
`30`	`30`	`use DefinitionGeneratorTrait;`
`31`	`31`	`use TransferGeneratorTrait;`
Original file line number	Diff line number	Diff line change
`@@ -10,7 +10,7 @@`
`10`	`10`	`use Picamator\Tests\Integration\TransferObject\Transfer\Advanced\BookmarkData;`
`11`	`11`
`12`	`12`	`#[Group('transfer')]`
`13`		`-class DummyTransferAdapterTest extends TestCase`
	`13`	`+final class DummyTransferAdapterTest extends TestCase`
`14`	`14`	`{`
`15`	`15`	`private BookmarkData $bookmarkData;`
`16`	`16`
Original file line number	Diff line number	Diff line change
`@@ -19,7 +19,7 @@`
`19`	`19`	`use Picamator\Tests\Integration\TransferObject\Transfer\Generated\ReservedAdvancedTransfer;`
`20`	`20`
`21`	`21`	`#[Group('transfer')]`
`22`		`-class TransferAdapterTest extends TestCase`
	`22`	`+final class TransferAdapterTest extends TestCase`
`23`	`23`	`{`
`24`	`24`	`#[TestDox('Transformation fromArray() and toArray()')]`
`25`	`25`	`public function testTransformationFromArrayToArray(): void`
Original file line number	Diff line number	Diff line change
`@@ -16,7 +16,7 @@`
`16`	`16`
`17`	`17`	`#[Group('transfer')]`
`18`	`18`	`#[RequiresPhpExtension('bcmath')]`
`19`		`-class NumberTransferTest extends TestCase`
	`19`	`+final class NumberTransferTest extends TestCase`
`20`	`20`	`{`
`21`	`21`	`use TransferGeneratorTrait;`
`22`	`22`
Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,7 @@`
`31`	`31`	`use TypeError;`
`32`	`32`
`33`	`33`	`#[Group('transfer')]`
`34`		`-class TransferTest extends TestCase`
	`34`	`+final class TransferTest extends TestCase`
`35`	`35`	`{`
`36`	`36`	`use TransferGeneratorTrait;`
`37`	`37`