Raise minimum PHP version to ^8.1 and refactor code (#76)

.github/workflows/bc.yml

@@ -1,13 +1,33 @@
 on:
-  - pull_request
-  - push
+  pull_request:
+    paths-ignore:
+      - 'docs/**'
+      - 'README.md'
+      - 'CHANGELOG.md'
+      - '.gitignore'
+      - '.gitattributes'
+      - 'infection.json.dist'
+      - 'phpunit.xml.dist'
+      - 'psalm.xml'
+  push:
+    branches: ['master']
+    paths-ignore:
+      - 'docs/**'
+      - 'README.md'
+      - 'CHANGELOG.md'
+      - '.gitignore'
+      - '.gitattributes'
+      - 'infection.json.dist'
+      - 'phpunit.xml.dist'
+      - 'psalm.xml'
 
 name: backwards compatibility
+
 jobs:
   roave_bc_check:
     uses: yiisoft/actions/.github/workflows/bc.yml@master
     with:
       os: >-
         ['ubuntu-latest']
       php: >-
-        ['8.0']
+        ['8.1']

.github/workflows/build.yml

@@ -10,6 +10,7 @@ on:
       - 'psalm.xml'
 
   push:
+    branches: ['master']
     paths-ignore:
       - 'docs/**'
       - 'README.md'
@@ -24,8 +25,10 @@ name: build
 jobs:
   phpunit:
     uses: yiisoft/actions/.github/workflows/phpunit.yml@master
+    secrets:
+      codecovToken: ${{ secrets.CODECOV_TOKEN }}
     with:
       os: >-
         ['ubuntu-latest', 'windows-latest']
       php: >-
-        ['8.0', '8.1', '8.2']
+        ['8.1', '8.2', '8.3']

.github/workflows/composer-require-checker.yml

@@ -11,6 +11,7 @@ on:
       - 'psalm.xml'
 
   push:
+    branches: ['master']
     paths-ignore:
       - 'docs/**'
       - 'README.md'
@@ -30,4 +31,4 @@ jobs:
       os: >-
         ['ubuntu-latest']
       php: >-
-        ['8.0', '8.1', '8.2']
+        ['8.1', '8.2', '8.3']

.github/workflows/mutation.yml

@@ -9,6 +9,7 @@ on:
       - 'psalm.xml'
 
   push:
+    branches: ['master']
     paths-ignore:
       - 'docs/**'
       - 'README.md'
@@ -26,6 +27,6 @@ jobs:
       os: >-
         ['ubuntu-latest']
       php: >-
-        ['8.1']
+        ['8.3']
     secrets:
       STRYKER_DASHBOARD_API_KEY: ${{ secrets.STRYKER_DASHBOARD_API_KEY }}

.github/workflows/rector.yml

@@ -1,5 +1,5 @@
 on:
-  pull_request:
+  pull_request_target:
     paths-ignore:
       - 'docs/**'
       - 'README.md'
@@ -14,8 +14,11 @@ name: rector
 jobs:
   rector:
     uses: yiisoft/actions/.github/workflows/rector.yml@master
+    secrets:
+      token: ${{ secrets.YIISOFT_GITHUB_TOKEN }}
     with:
+      repository: ${{ github.event.pull_request.head.repo.full_name }}
       os: >-
         ['ubuntu-latest']
       php: >-
-        ['8.0']
+        ['8.3']

.github/workflows/static.yml

@@ -10,6 +10,7 @@ on:
       - 'phpunit.xml.dist'
 
   push:
+    branches: ['master']
     paths-ignore:
       - 'docs/**'
       - 'README.md'
@@ -28,4 +29,4 @@ jobs:
       os: >-
         ['ubuntu-latest']
       php: >-
-        ['8.0', '8.1', '8.2', '8.3']
+        ['8.1', '8.2', '8.3']

.gitignore

@@ -23,12 +23,10 @@ composer.phar
 # Mac DS_Store Files
 .DS_Store
 
-# phpunit itself is not needed
-phpunit.phar
-# local phpunit config
+# PhpUnit
+/phpunit.phar
 /phpunit.xml
-# phpunit cache
-.phpunit.result.cache
+/.phpunit.cache
 
 # Phan
 analysis.txt

CHANGELOG.md

@@ -3,6 +3,7 @@
 ## 2.1.1 under development
 
 - Enh #65: Minor refactoring of `CallableFactory` and `ListenerCollectionFactory` (@vjik)
+- Enh #76: Raise minimum PHP version to `^8.1` and refactor code (@vjik)
 
 ## 2.1.0 November 04, 2023
 

README.md

@@ -20,7 +20,7 @@ All you need is to use any [PSR-11](https://www.php-fig.org/psr/psr-11/) compati
 
 ## Requirements
 
-- PHP 8.0 or higher.
+- PHP 8.1 or higher.
 
 ## Installation
 

composer.json

@@ -27,19 +27,19 @@
         }
     ],
     "require": {
-        "php": "^8.0",
+        "php": "^8.1",
         "psr/container": "^1.0|^2.0",
         "yiisoft/event-dispatcher": "^1.0",
         "yiisoft/friendly-exception": "^1.0",
         "yiisoft/injector": "^1.0"
     },
     "require-dev": {
         "maglnet/composer-require-checker": "^4.4",
-        "phpunit/phpunit": "^9.5",
-        "rector/rector": "^1.0.0",
+        "phpunit/phpunit": "^10.5",
+        "rector/rector": "^1.2",
         "roave/infection-static-analysis-plugin": "^1.16",
         "spatie/phpunit-watcher": "^1.23",
-        "vimeo/psalm": "^4.30|^5.21",
+        "vimeo/psalm": "^5.26",
         "yiisoft/config": "^1.3",
         "yiisoft/di": "^1.2",
         "yiisoft/test-support": "^3.0"

phpunit.xml.dist

@@ -1,16 +1,18 @@
 <?xml version="1.0" encoding="UTF-8"?>
-
-<phpunit bootstrap="vendor/autoload.php"
-         colors="true"
-         verbose="true"
+<phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+         xsi:noNamespaceSchemaLocation="vendor/phpunit/phpunit/phpunit.xsd"
+         bootstrap="vendor/autoload.php"
+         cacheDirectory=".phpunit.cache"
+         requireCoverageMetadata="false"
+         beStrictAboutCoverageMetadata="true"
+         beStrictAboutOutputDuringTests="true"
+         executionOrder="random"
          failOnRisky="true"
          failOnWarning="true"
-         convertErrorsToExceptions="true"
-         convertNoticesToExceptions="true"
-         convertWarningsToExceptions="true"
          stopOnFailure="false"
-         executionOrder="random"
-         resolveDependencies="true">
+         colors="true"
+         displayDetailsOnPhpunitDeprecations="true"
+>
     <php>
         <ini name="error_reporting" value="-1"/>
     </php>
@@ -21,9 +23,9 @@
         </testsuite>
     </testsuites>
 
-    <coverage>
+    <source>
         <include>
-            <directory>./src</directory>
+            <directory suffix=".php">./src</directory>
         </include>
-    </coverage>
+    </source>
 </phpunit>

rector.php

@@ -6,6 +6,7 @@
 use Rector\Config\RectorConfig;
 use Rector\Php74\Rector\Closure\ClosureToArrowFunctionRector;
 use Rector\Php80\Rector\Ternary\GetDebugTypeRector;
+use Rector\Php81\Rector\Array_\FirstClassCallableRector;
 use Rector\Set\ValueObject\LevelSetList;
 
 return static function (RectorConfig $rectorConfig): void {
@@ -19,11 +20,14 @@
 
     // define sets of rules
     $rectorConfig->sets([
-        LevelSetList::UP_TO_PHP_80,
+        LevelSetList::UP_TO_PHP_81,
     ]);
 
     $rectorConfig->skip([
         ClosureToArrowFunctionRector::class,
         GetDebugTypeRector::class,
+        FirstClassCallableRector::class => [
+            __DIR__ . '/tests/CallableFactoryTest.php',
+        ],
     ]);
 };

src/CallableFactory.php

@@ -19,7 +19,7 @@
 final class CallableFactory
 {
     public function __construct(
-        private ContainerInterface $container
+        private readonly ContainerInterface $container
     ) {
     }
 

src/ListenerCollectionFactory.php

@@ -12,8 +12,8 @@
 final class ListenerCollectionFactory
 {
     public function __construct(
-        private Injector $injector,
-        private CallableFactory $callableFactory,
+        private readonly Injector $injector,
+        private readonly CallableFactory $callableFactory,
     ) {
     }
 

src/ListenerConfigurationChecker.php

@@ -20,7 +20,7 @@
 final class ListenerConfigurationChecker
 {
     public function __construct(
-        private CallableFactory $callableFactory,
+        private readonly CallableFactory $callableFactory,
     ) {
     }
 

tests/CallableFactoryTest.php

@@ -5,6 +5,7 @@
 namespace Yiisoft\Yii\Event\Tests;
 
 use Closure;
+use PHPUnit\Framework\Attributes\DataProvider;
 use PHPUnit\Framework\TestCase;
 use Psr\Container\ContainerInterface;
 use stdClass;
@@ -18,7 +19,7 @@
 
 class CallableFactoryTest extends TestCase
 {
-    public function dataArray(): array
+    public static function dataArray(): array
     {
         return [
             'base' => [[Event::class, 'register']],
@@ -27,27 +28,23 @@ public function dataArray(): array
         ];
     }
 
-    /**
-     * @dataProvider dataArray
-     */
+    #[DataProvider('dataArray')]
     public function testArray($definition): void
     {
         self::assertIsArray(
             $this->createFactory()->create($definition),
         );
     }
 
-    public function dataInvokableObject(): array
+    public static function dataInvokableObject(): array
     {
         return [
             'base' => [new HandlerInvokable()],
             'instantiate' => [HandlerInvokable::class],
         ];
     }
 
-    /**
-     * @dataProvider dataInvokableObject
-     */
+    #[DataProvider('dataInvokableObject')]
     public function testInvokableObject($definition): void
     {
         self::assertInstanceOf(
@@ -56,7 +53,7 @@ public function testInvokableObject($definition): void
         );
     }
 
-    public function dataClosure(): array
+    public static function dataClosure(): array
     {
         return [
             'closure' => [
@@ -67,9 +64,7 @@ static function () {
         ];
     }
 
-    /**
-     * @dataProvider dataClosure
-     */
+    #[DataProvider('dataClosure')]
     public function testClosure($definition): void
     {
         self::assertInstanceOf(
@@ -78,7 +73,7 @@ public function testClosure($definition): void
         );
     }
 
-    public function dataException(): array
+    public static function dataException(): array
     {
         return [
             'non-existent container definition' => [['test', 'register']],
@@ -94,9 +89,7 @@ public function dataException(): array
         ];
     }
 
-    /**
-     * @dataProvider dataException
-     */
+    #[DataProvider('dataException')]
     public function testException($definition): void
     {
         $this->expectException(InvalidListenerConfigurationException::class);

tests/EventConfiguratorTest.php

@@ -98,7 +98,7 @@ private function getEventsConfig(): array
             Event::class => [
                 ['eventAlias', 'register'],
                 [Event::class, 'register'],
-                [Handler::class, 'handleStatic'],
+                Handler::handleStatic(...),
                 [new Event(), 'register'],
                 static function (Event $event) {
                     $event->register(new stdClass());