Refactor StoreTest to use attributes for test annotations, update PHPUnit configuration and improve code type hints. Adjust .gitignore for PHPUnit cache directory and add ext-mongodb requirement in composer.json.

Author: DevquasarX9

.gitignore

@@ -1,3 +1,3 @@
 /vendor/
 composer.lock
-.phpunit.result.cache
+.phpunit.cache

composer.json

@@ -9,7 +9,8 @@
         "mongodb/laravel-mongodb": "^4.1"
     },
     "require-dev": {
-        "orchestra/testbench": "^8.0"
+        "orchestra/testbench": "^8.0",
+        "ext-mongodb": "*"
     },
     "license": "MIT",
     "authors": [

phpunit.xml

@@ -1,17 +1,8 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
-         bootstrap="vendor/autoload.php"
-         backupGlobals="false"
-         backupStaticAttributes="false"
-         colors="true"
-         verbose="true"
-         convertErrorsToExceptions="true"
-         convertNoticesToExceptions="true"
-         convertWarningsToExceptions="true"
-         processIsolation="false"
-         stopOnFailure="false"
-         xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/9.3/phpunit.xsd"
->
+<phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" bootstrap="vendor/autoload.php" backupGlobals="false"
+         colors="true" processIsolation="false" stopOnFailure="false"
+         xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/10.5/phpunit.xsd" cacheDirectory=".phpunit.cache"
+         backupStaticProperties="false">
     <testsuites>
         <testsuite name="MongoDB Cache Test Suite">
             <directory>tests</directory>

tests/StoreTest.php

@@ -5,11 +5,12 @@
 use ForFit\Mongodb\Cache\MongoTaggedCache;
 use ForFit\Mongodb\Cache\Store;
 use Illuminate\Support\Carbon;
+use PHPUnit\Framework\Attributes\Test;
 use Tests\Models\Cache;
 
 class StoreTest extends TestCase
 {
-    protected $store;
+    protected Store $store;
 
     protected function setUp(): void
     {
@@ -21,8 +22,8 @@ protected function setUp(): void
         Carbon::setTestNow(now());
     }
 
-    /** @test */
-    public function it_stores_an_item_in_the_cache_for_given_time()
+    #[Test]
+    public function it_stores_an_item_in_the_cache_for_given_time(): void
     {
         // Act
         $sut = $this->store->put('test-key', 'test-value', 3);
@@ -37,8 +38,8 @@ public function it_stores_an_item_in_the_cache_for_given_time()
         ]);
     }
 
-    /** @test */
-    public function it_updates_an_item_in_the_cache_for_given_time()
+    #[Test]
+    public function it_updates_an_item_in_the_cache_for_given_time(): void
     {
         Cache::create([
             'key' => 'test-key',
@@ -58,8 +59,8 @@ public function it_updates_an_item_in_the_cache_for_given_time()
         ]);
     }
 
-    /** @test */
-    public function it_retrieves_value_from_the_cache_by_given_key()
+    #[Test]
+    public function it_retrieves_value_from_the_cache_by_given_key(): void
     {
         // Arrange
         Cache::create([
@@ -75,8 +76,8 @@ public function it_retrieves_value_from_the_cache_by_given_key()
         $this->assertEquals('test-value', $sut);
     }
 
-    /** @test */
-    public function it_returns_null_if_key_does_not_exist()
+    #[Test]
+    public function it_returns_null_if_key_does_not_exist(): void
     {
         // Act
         $sut = $this->store->get('test-key');
@@ -85,8 +86,8 @@ public function it_returns_null_if_key_does_not_exist()
         $this->assertNull($sut);
     }
 
-    /** @test */
-    public function it_sets_the_tags_to_be_used()
+    #[Test]
+    public function it_sets_the_tags_to_be_used(): void
     {
         // Act
         $sut = $this->store->tags(['tag1', 'tag2']);
@@ -96,8 +97,8 @@ public function it_sets_the_tags_to_be_used()
         $this->assertPropertySame(['tag1', 'tag2'], 'tags', $sut);
     }
 
-    /** @test */
-    public function it_deletes_all_records_with_the_given_tag()
+    #[Test]
+    public function it_deletes_all_records_with_the_given_tag(): void
     {
         // Arrange
         Cache::create([
@@ -126,8 +127,8 @@ public function it_deletes_all_records_with_the_given_tag()
         ]);
     }
 
-    /** @test */
-    public function it_retrieves_an_items_expiration_time_by_given_key()
+    #[Test]
+    public function it_retrieves_an_items_expiration_time_by_given_key(): void
     {
         // Arrange
         Cache::create([

tests/TestCase.php

@@ -10,7 +10,7 @@
 
 abstract class TestCase extends Orchestra
 {
-    private $table = 'cache_test';
+    private string $table = 'cache_test';
     private $connectionInterface;
 
     /**
@@ -41,7 +41,7 @@ protected function getPackageProviders($app): array
      *
      * @param \Illuminate\Foundation\Application $app
      */
-    protected function getEnvironmentSetUp($app)
+    protected function getEnvironmentSetUp($app): void
     {
         $app['config']->set('cache.stores.mongodb', [
             'driver' => 'mongodb',
@@ -78,7 +78,7 @@ protected function table(): string
      *
      * @param \Illuminate\Foundation\Application $app
      */
-    protected function setUpDatabase($app)
+    protected function setUpDatabase($app): void
     {
         $app['db']->connection()->getSchemaBuilder()->create($this->table, function (Blueprint $table) {
             $table->increments('_id');
@@ -112,11 +112,10 @@ protected function initializeConnection()
      * @return void
      * @throws \ReflectionException
      */
-    protected function assertPropertySame($expected, $property, $object, string $message = '')
+    protected function assertPropertySame($expected, $property, $object, string $message = ''): void
     {
         $reflectedClass = new \ReflectionClass($object);
         $reflection = $reflectedClass->getProperty($property);
-        $reflection->setAccessible(true);
 
         $this->assertSame($expected, $reflection->getValue($object), $message);
     }