Fixes

composer.json

@@ -26,7 +26,7 @@
   },
   "require-dev": {
     "phpstan/phpstan": "^1.10",
-    "phpunit/phpunit": "^10.5"
+    "phpunit/phpunit": "^11.0"
   },
   "suggest": {
     "ext-apcu": "Required to use the APCu cache driver.",

src/Storages/FileStorage.php

@@ -13,9 +13,9 @@
 use RobiNN\Cache\CacheInterface;
 
 class FileStorage implements CacheInterface {
-    private string $path;
+    private readonly string $path;
 
-    private ?string $secret;
+    private readonly ?string $secret;
 
     /**
      * @param array<string, mixed> $config
@@ -167,7 +167,7 @@ private function isExpired(string $key): bool {
             $expired = time() - (int) $data['time'] > (int) $data['expire'];
         }
 
-        if ($expired === true) {
+        if ($expired) {
             $this->delete($key);
         }
 

src/Storages/RedisStorage.php

@@ -14,7 +14,7 @@
 use RobiNN\Cache\CacheInterface;
 
 class RedisStorage implements CacheInterface {
-    private Redis $redis;
+    private readonly Redis $redis;
 
     private bool $connection = true;
 

tests/CacheTestCase.php

@@ -8,6 +8,7 @@
 
 namespace RobiNN\Cache\Tests;
 
+use Iterator;
 use PHPUnit\Framework\Attributes\DataProvider;
 use PHPUnit\Framework\TestCase;
 use RobiNN\Cache\Cache;
@@ -32,18 +33,13 @@ public function testSetterGetter(): void {
         $this->cache->delete($key);
     }
 
-    /**
-     * @return array<int, mixed>
-     */
-    public static function keysProvider(): array {
-        return [
-            ['string', 'Cache'],
-            ['int', 23],
-            ['float', 23.99],
-            ['bool', true],
-            ['null', null],
-            ['array', ['key1', 'key2']],
-        ];
+    public static function keysProvider(): Iterator {
+        yield ['string', 'Cache'];
+        yield ['int', 23];
+        yield ['float', 23.99];
+        yield ['bool', true];
+        yield ['null', null];
+        yield ['array', ['key1', 'key2']];
     }
 
     #[DataProvider('keysProvider')]