Add missing final (#164)

.phpstan/ExceptionTypeResolver.php

@@ -8,7 +8,7 @@
 use PHPStan\Rules\Exceptions\DefaultExceptionTypeResolver;
 use PHPStan\Rules\Exceptions\ExceptionTypeResolver as PHPStanExceptionTypeResolver;
 
-class ExceptionTypeResolver implements PHPStanExceptionTypeResolver
+final class ExceptionTypeResolver implements PHPStanExceptionTypeResolver
 {
     public function __construct(private DefaultExceptionTypeResolver $defaultExceptionTypeResolver)
     {

rector.php

@@ -3,6 +3,9 @@
 declare(strict_types=1);
 
 use Rector\Config\RectorConfig;
+use Rector\Privatization\Rector\Class_\FinalizeClassesWithoutChildrenRector;
+use Rector\Privatization\Rector\ClassMethod\PrivatizeFinalClassMethodRector;
+use Rector\Privatization\Rector\Property\PrivatizeFinalClassPropertyRector;
 use Rector\Set\ValueObject\LevelSetList;
 
 return static function (RectorConfig $rectorConfig): void {
@@ -17,6 +20,11 @@
     $rectorConfig->sets([
         LevelSetList::UP_TO_PHP_80,
     ]);
+    $rectorConfig->rules([
+        FinalizeClassesWithoutChildrenRector::class,
+        PrivatizeFinalClassMethodRector::class,
+        PrivatizeFinalClassPropertyRector::class,
+    ]);
 
     $rectorConfig->importNames();
 };

src/Standard/Twig.php

@@ -14,7 +14,7 @@
  *
  * @see https://twig.symfony.com/doc/3.x/coding_standards.html
  */
-class Twig implements StandardInterface
+final class Twig implements StandardInterface
 {
     public function getRules(): array
     {

tests/File/FinderTest.php

@@ -7,7 +7,7 @@
 use PHPUnit\Framework\TestCase;
 use TwigCsFixer\File\Finder;
 
-class FinderTest extends TestCase
+final class FinderTest extends TestCase
 {
     public function testFinder(): void
     {

tests/Report/ViolationIdTest.php

@@ -7,7 +7,7 @@
 use PHPUnit\Framework\TestCase;
 use TwigCsFixer\Report\ViolationId;
 
-class ViolationIdTest extends TestCase
+final class ViolationIdTest extends TestCase
 {
     /**
      * @dataProvider toStringDataProvider

tests/Report/ViolationTest.php

@@ -24,6 +24,19 @@ public function testGetters(): void
         static::assertSame($violationId, $violation->getIdentifier());
     }
 
+    public function testGetterWithoutViolationId(): void
+    {
+        $violation = new Violation(Violation::LEVEL_WARNING, 'message', 'filename', 'name');
+
+        static::assertSame(Violation::LEVEL_WARNING, $violation->getLevel());
+        static::assertSame('message', $violation->getMessage());
+        static::assertSame('filename', $violation->getFilename());
+        static::assertNull($violation->getLine());
+        static::assertNull($violation->getLinePosition());
+        static::assertSame('name', $violation->getRuleName());
+        static::assertNull($violation->getIdentifier());
+    }
+
     /**
      * @dataProvider getLevelAsStringDataProvider
      */

tests/Rules/Fixtures/FakeRule.php

@@ -9,7 +9,7 @@
 /**
  * This rule reports an error for the first token of every line.
  */
-class FakeRule extends AbstractRule
+final class FakeRule extends AbstractRule
 {
     public function process(int $tokenPosition, array $tokens): void
     {