Merge pull request #560 from kenjis/fix-rector-config

chore: update rector.php

Author: kenjis

.github/workflows/rector.yml

@@ -63,5 +63,5 @@ jobs:
 
       - name: Analyze for refactoring
         run: |
-          composer global require --dev rector/rector:^0.13.3
+          composer global require --dev rector/rector:^0.15.1
           rector process --dry-run --no-progress-bar

.php-cs-fixer.dist.php

@@ -13,7 +13,10 @@
         __DIR__ . '/tests/',
     ])
     ->exclude('build')
-    ->append([__FILE__]);
+    ->append([
+        __FILE__,
+        __DIR__ . '/rector.php',
+    ]);
 
 $overrides = [
     'declare_strict_types' => true,

rector.php

@@ -1,6 +1,9 @@
 <?php
 
+declare(strict_types=1);
+
 use Rector\CodeQuality\Rector\BooleanAnd\SimplifyEmptyArrayCheckRector;
+use Rector\CodeQuality\Rector\Class_\CompleteDynamicPropertiesRector;
 use Rector\CodeQuality\Rector\Expression\InlineIfToExplicitIfRector;
 use Rector\CodeQuality\Rector\For_\ForToForeachRector;
 use Rector\CodeQuality\Rector\Foreach_\UnusedForeachValueToArrayKeysRector;
@@ -30,15 +33,23 @@
 use Rector\Php56\Rector\FunctionLike\AddDefaultValueForUndefinedVariableRector;
 use Rector\Php73\Rector\FuncCall\JsonThrowOnErrorRector;
 use Rector\Php73\Rector\FuncCall\StringifyStrNeedlesRector;
-use Rector\Php74\Rector\Property\TypedPropertyRector;
+use Rector\PHPUnit\Rector\Class_\AnnotationWithValueToAttributeRector;
 use Rector\PHPUnit\Set\PHPUnitSetList;
+use Rector\Privatization\Rector\Property\PrivatizeFinalClassPropertyRector;
 use Rector\PSR4\Rector\FileWithoutNamespace\NormalizeNamespaceByPSR4ComposerAutoloadRector;
 use Rector\Set\ValueObject\LevelSetList;
 use Rector\Set\ValueObject\SetList;
 
 return static function (RectorConfig $rectorConfig): void {
-    $rectorConfig->sets([SetList::DEAD_CODE, LevelSetList::UP_TO_PHP_74, PHPUnitSetList::PHPUNIT_SPECIFIC_METHOD, PHPUnitSetList::PHPUNIT_80]);
+    $rectorConfig->sets([
+        SetList::DEAD_CODE,
+        LevelSetList::UP_TO_PHP_74,
+        PHPUnitSetList::PHPUNIT_SPECIFIC_METHOD,
+        PHPUnitSetList::PHPUNIT_100,
+    ]);
+
     $rectorConfig->parallel();
+
     // The paths to refactor (can also be supplied with CLI arguments)
     $rectorConfig->paths([
         __DIR__ . '/src/',
@@ -74,6 +85,7 @@
 
         // Note: requires php 8
         RemoveUnusedPromotedPropertyRector::class,
+        AnnotationWithValueToAttributeRector::class,
 
         // Ignore tests that might make calls without a result
         RemoveEmptyMethodCallRector::class => [
@@ -97,6 +109,9 @@
             __DIR__ . '/src/Models/UserModel.php',
         ],
     ]);
+    // auto import fully qualified class names
+    $rectorConfig->importNames();
+
     $rectorConfig->rule(SimplifyUselessVariableRector::class);
     $rectorConfig->rule(RemoveAlwaysElseRector::class);
     $rectorConfig->rule(CountArrayToEmptyArrayComparisonRector::class);
@@ -119,8 +134,7 @@
     $rectorConfig->rule(MakeInheritedMethodVisibilitySameAsParentRector::class);
     $rectorConfig->rule(SimplifyEmptyArrayCheckRector::class);
     $rectorConfig->rule(NormalizeNamespaceByPSR4ComposerAutoloadRector::class);
-    $rectorConfig
-        ->ruleWithConfiguration(TypedPropertyRector::class, [
-            TypedPropertyRector::INLINE_PUBLIC => false,
-        ]);
+    $rectorConfig->rule(StringClassNameToClassConstantRector::class);
+    $rectorConfig->rule(PrivatizeFinalClassPropertyRector::class);
+    $rectorConfig->rule(CompleteDynamicPropertiesRector::class);
 };

tests/Authentication/AuthenticationTest.php

@@ -16,7 +16,7 @@
  */
 final class AuthenticationTest extends DatabaseTestCase
 {
-    protected Authentication $auth;
+    private Authentication $auth;
 
     protected function setUp(): void
     {

tests/Authentication/Authenticators/AccessTokenAuthenticatorTest.php

@@ -22,7 +22,7 @@
  */
 final class AccessTokenAuthenticatorTest extends DatabaseTestCase
 {
-    protected AccessTokens $auth;
+    private AccessTokens $auth;
 
     protected function setUp(): void
     {

tests/Authentication/Authenticators/SessionAuthenticatorTest.php

@@ -27,9 +27,9 @@ final class SessionAuthenticatorTest extends TestCase
     use DatabaseTestTrait;
     use FakeUser;
 
-    protected Session $auth;
+    private Session $auth;
     protected $namespace;
-    protected $events;
+    private MockEvents $events;
 
     protected function setUp(): void
     {

tests/Authentication/Filters/AbstractFilterTest.php

@@ -18,6 +18,7 @@ abstract class AbstractFilterTest extends TestCase
     use FeatureTestTrait;
     use AuthenticationTesting;
 
+    protected string $routeFilter;
     protected $namespace;
     protected string $alias;
     protected string $classname;

tests/Authentication/HasAccessTokensTest.php

@@ -15,7 +15,7 @@
  */
 final class HasAccessTokensTest extends DatabaseTestCase
 {
-    protected User $user;
+    private User $user;
 
     protected function setUp(): void
     {

tests/Authorization/AuthorizableTest.php

@@ -23,14 +23,11 @@ final class AuthorizableTest extends TestCase
 
     protected $refresh = true;
     protected $namespace;
-    protected UserModel $model;
 
     protected function setUp(): void
     {
         parent::setUp();
 
-        $this->model = new UserModel();
-
         // Refresh should take care of this....
         db_connect()->table('auth_groups_users')->truncate();
         db_connect()->table('auth_permissions_users')->truncate();

tests/Language/AbstractTranslationTestCase.php

@@ -305,12 +305,12 @@ final public function testAllConfiguredLanguageKeysAreInOrder(string $locale): v
     /**
      * @return string[][]
      */
-    final public function localesProvider(): iterable
+    final public static function localesProvider(): iterable
     {
         $locale = self::$locales[static::class] ?? null;
 
         if (null === $locale) {
-            $this->fail('The locale code should be defined in the $locales property.');
+            static::fail('The locale code should be defined in the $locales property.');
         }
 
         return [$locale => [$locale]];