Add phpdoc, minor improvements. (#64)

Author: terabytesoftw

src/DbCache.php

@@ -29,7 +29,7 @@
 /**
  * DbCache stores cache data in a database table.
  *
- * Use {@see DbHelper::ensureTable()} to initialize database schema.
+ * Use {@see DbSchemaManager::ensureTable()} to initialize database schema.
  */
 final class DbCache implements CacheInterface
 {

src/DbSchemaManager.php

@@ -11,13 +11,20 @@
 use Yiisoft\Db\Exception\NotSupportedException;
 use Yiisoft\Db\Schema\SchemaInterface;
 
+/**
+ * Manages the cache table schema in the database.
+ */
 final class DbSchemaManager
 {
     public function __construct(private ConnectionInterface $db)
     {
     }
 
     /**
+     * Ensures that the cache table exists in the database.
+     *
+     * @param string $table The name of the cache table. Defaults to '{{%yii_cache}}'.
+     *
      * @throws Exception
      * @throws NotSupportedException
      * @throws InvalidConfigException
@@ -26,14 +33,13 @@ public function __construct(private ConnectionInterface $db)
     public function ensureTable(string $table = '{{%yii_cache}}'): void
     {
         $schema = $this->db->getSchema();
-        $command = $this->db->createCommand();
         $tableRawName = $schema->getRawTableName($table);
 
         if ($this->hasTable($table)) {
             return;
         }
 
-        $command->createTable(
+        $this->db->createCommand()->createTable(
             $table,
             [
                 'id' => $schema->createColumn(SchemaInterface::TYPE_STRING, 128)->notNull(),
@@ -45,6 +51,10 @@ public function ensureTable(string $table = '{{%yii_cache}}'): void
     }
 
     /**
+     * Ensures that the cache table does not exist in the database.
+     *
+     * @param string $table The name of the cache table. Defaults to '{{%yii_cache}}'.
+     *
      * @throws Exception
      * @throws InvalidConfigException
      * @throws Throwable
@@ -58,6 +68,13 @@ public function ensureNoTable(string $table = '{{%yii_cache}}'): void
         }
     }
 
+    /**
+     * Checks if the given table exists in the database.
+     *
+     * @param string $table The name of the table to check.
+     *
+     * @return bool Whether the table exists or not.
+     */
     private function hasTable(string $table): bool
     {
         return $this->db->getTableSchema($table, true) !== null;

tests/Common/AbstractDbSchemaManagerTest.php

@@ -36,6 +36,8 @@ protected function tearDown(): void
     {
         parent::tearDown();
 
+        $this->db->close();
+
         unset($this->db, $this->dbSchemaManager);
     }
 

tests/Common/AbstractSQLDumpFileTest.php

@@ -35,6 +35,8 @@ protected function tearDown(): void
     {
         parent::tearDown();
 
+        $this->db->close();
+
         unset($this->db, $this->driverName);
     }
 

tests/Common/TestCase.php

@@ -54,7 +54,7 @@ protected function tearDown(): void
         // close db connection
         $this->db->close();
 
-        unset($this->db, $this->dbCache, $this->logger);
+        unset($this->db, $this->dbCache, $this->dbSchemaManager, $this->logger);
     }
 
     protected function getLogger(): Logger