test cases typo fixed & missing PHPDocs added.

src/Column.php

@@ -51,6 +51,9 @@ protected function buildCommentString(): string
         return ' COMMENT ' . (string) (new Quoter('`', '`'))->quoteValue($this->getComment());
     }
 
+    /**
+     * @return string
+     */
     public function asString(): string
     {
         $format = match ($this->getTypeCategory()) {

src/Command.php

@@ -12,6 +12,17 @@
  */
 final class Command extends AbstractPdoCommand
 {
+    /**
+     * @param string $table
+     * @param array $columns
+     * @return bool|array
+     * @throws \Throwable
+     * @throws \Yiisoft\Db\Exception\Exception
+     * @throws \Yiisoft\Db\Exception\InvalidArgumentException
+     * @throws \Yiisoft\Db\Exception\InvalidCallException
+     * @throws \Yiisoft\Db\Exception\InvalidConfigException
+     * @throws \Yiisoft\Db\Exception\NotSupportedException
+     */
     public function insertWithReturningPks(string $table, array $columns): bool|array
     {
         $params = [];
@@ -29,7 +40,7 @@
 
         foreach ($tablePrimaryKeys as $name) {
             if ($tableSchema?->getColumn($name)?->isAutoIncrement()) {
-                $result[$name] = $this->db->getLastInsertID((string) $tableSchema?->getSequenceName());
+                $result[$name] = $this->db->getLastInsertID((string)$tableSchema?->getSequenceName());
                 continue;
             }
 
@@ -40,6 +51,11 @@
         return $result;
     }
 
+    /**
+     * @return array
+     * @throws \Throwable
+     * @throws \Yiisoft\Db\Exception\Exception
+     */
     public function showDatabases(): array
     {
         $sql = <<<SQL

src/Connection.php

@@ -20,6 +20,9 @@
  */
 final class Connection extends AbstractPdoConnection
 {
+    /**
+     * @return void
+     */
     public function close(): void
     {
         if ($this->pdo !== null) {
@@ -39,6 +42,11 @@ public function close(): void
         }
     }
 
+    /**
+     * @param string|null $sql
+     * @param array $params
+     * @return PdoCommandInterface
+     */
     public function createCommand(string $sql = null, array $params = []): PdoCommandInterface
     {
         $command = new Command($this);
@@ -63,6 +71,9 @@ public function createTransaction(): TransactionInterface
         return new Transaction($this);
     }
 
+    /**
+     * @return QueryBuilderInterface
+     */
     public function getQueryBuilder(): QueryBuilderInterface
     {
         if ($this->queryBuilder === null) {
@@ -75,6 +86,9 @@ public function getQueryBuilder(): QueryBuilderInterface
         return $this->queryBuilder;
     }
 
+    /**
+     * @return QuoterInterface
+     */
     public function getQuoter(): QuoterInterface
     {
         if ($this->quoter === null) {
@@ -84,6 +98,9 @@ public function getQuoter(): QuoterInterface
         return $this->quoter;
     }
 
+    /**
+     * @return SchemaInterface
+     */
     public function getSchema(): SchemaInterface
     {
         if ($this->schema === null) {

src/Driver.php

@@ -14,6 +14,9 @@
  */
 final class Driver extends AbstractPdoDriver
 {
+    /**
+     * @return PDO
+     */
     public function createConnection(): PDO
     {
         $this->attributes += [PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION];
@@ -35,6 +38,9 @@ public function createConnection(): PDO
         return $pdo;
     }
 
+    /**
+     * @return string
+     */
     public function getDriverName(): string
     {
         return 'mysql';

src/Quoter.php

@@ -14,6 +14,10 @@
  */
 final class Quoter extends BaseQuoter
 {
+    /**
+     * @param mixed $value
+     * @return mixed
+     */
     public function quoteValue(mixed $value): mixed
     {
         if (!is_string($value)) {

tests/ColumnSchemaTest.php

@@ -65,6 +65,14 @@ public function testPhpTypeCastJson(): void
         $this->assertSame(['a' => 1], $columnSchema->phpTypeCast('{"a":1}'));
     }
 
+    /**
+     * @return void
+     * @throws Exception
+     * @throws Throwable
+     * @throws \Yiisoft\Db\Exception\InvalidArgumentException
+     * @throws \Yiisoft\Db\Exception\InvalidConfigException
+     * @throws \Yiisoft\Db\Exception\NotSupportedException
+     */
     public function testPhpTypeCast(): void
     {
         $db = $this->getConnection(true);

tests/PdoConnectionTest.php

@@ -9,6 +9,7 @@
 use Yiisoft\Db\Exception\Exception;
 use Yiisoft\Db\Exception\InvalidCallException;
 use Yiisoft\Db\Exception\InvalidConfigException;
+use Yiisoft\Db\Exception\NotSupportedException;
 use Yiisoft\Db\Mysql\Tests\Support\TestTrait;
 use Yiisoft\Db\Tests\Common\CommonPdoConnectionTest;
 
@@ -93,18 +94,23 @@ public function testGetLastInsertID(): void
         $this->assertSame('0', $db->getLastInsertID());
     }
 
-    public function testTransactionAutocommit()
+    /**
+     * @return void
+     * @throws Throwable
+     */
+    public function testTransactionAutocommit(): void
     {
         $db = $this->getConnection(true);
-        $db->transaction(function (PdoConnectionInterface $db) {
-            $this->assertTrue($db->getTransaction()->isActive());
-
-            // create table will cause the transaction to be implicitly committed
-            // (see https://dev.mysql.com/doc/refman/8.0/en/implicit-commit.html)
-            $name = 'test_implicit_transaction_table';
-            $db->createCommand()->createTable($name, ['id' => 'pk'])->execute();
-            $db->createCommand()->dropTable($name)->execute();
-        });
+        $db->transaction(
+            function (PdoConnectionInterface $db) {
+                $this->assertTrue($db->getTransaction()->isActive());
+
+                // create table will cause the transaction to be implicitly committed
+                // (see https://dev.mysql.com/doc/refman/8.0/en/implicit-commit.html)
+                $name = 'test_implicit_transaction_table';
+                $db->createCommand()->createTable($name, ['id' => 'pk'])->execute();
+                $db->createCommand()->dropTable($name)->execute();
+            });
         // If we made it this far without an error, then everything's working
     }
 }

tests/Provider/ColumnSchemaBuilderProvider.php

@@ -10,6 +10,9 @@ final class ColumnSchemaBuilderProvider extends \Yiisoft\Db\Tests\Provider\Colum
 {
     protected static string $driverName = 'mysql';
 
+    /**
+     * @return array
+     */
     public static function types(): array
     {
         $types = parent::types();
@@ -39,6 +42,9 @@ public static function types(): array
         );
     }
 
+    /**
+     * @return array
+     */
     public static function createColumnTypes(): array
     {
         $types = parent::createColumnTypes();

tests/Provider/SchemaProvider.php

@@ -11,6 +11,9 @@ final class SchemaProvider extends \Yiisoft\Db\Tests\Provider\SchemaProvider
 {
     use TestTrait;
 
+    /**
+     * @return array
+     */
     public static function columns(): array
     {
         return [
@@ -375,6 +378,9 @@ public static function columnsTypeBit(): array
         ];
     }
 
+    /**
+     * @return array
+     */
     public static function constraints(): array
     {
         $constraints = parent::constraints();
@@ -388,6 +394,9 @@ public static function constraints(): array
         return $constraints;
     }
 
+    /**
+     * @return array[]
+     */
     public static function tableSchemaWithDbSchemes(): array
     {
         return [

tests/QueryBuilderJsonTest.php

@@ -19,7 +19,10 @@ final class QueryBuilderJsonTest extends TestCase
 {
     use TestTrait;
 
-    public function testAlterColumn()
+    /**
+     * @return void
+     */
+    public function testAlterColumn(): void
     {
         $qb = $this->getConnection()->getQueryBuilder();
         $columnSchemaBuilder = new Column(SchemaInterface::TYPE_JSON);
@@ -33,7 +36,10 @@ public function testAlterColumn()
         );
     }
 
-    public function testAddColumn()
+    /**
+     * @return void
+     */
+    public function testAddColumn(): void
     {
         $qb = $this->getConnection()->getQueryBuilder();
         $columnSchemaBuilder = new Column(SchemaInterface::TYPE_JSON);
@@ -47,7 +53,10 @@ public function testAddColumn()
         );
     }
 
-    public function testCreateTable()
+    /**
+     * @return void
+     */
+    public function testCreateTable(): void
     {
         $qb = $this->getConnection()->getQueryBuilder();
         $columnSchemaBuilder = new Column(SchemaInterface::TYPE_JSON);
@@ -63,7 +72,14 @@ public function testCreateTable()
         );
     }
 
-    public function testInsertAndSelect()
+    /**
+     * @return void
+     * @throws \Yiisoft\Db\Exception\Exception
+     * @throws \Yiisoft\Db\Exception\InvalidArgumentException
+     * @throws \Yiisoft\Db\Exception\InvalidConfigException
+     * @throws \Yiisoft\Db\Exception\NotSupportedException
+     */
+    public function testInsertAndSelect(): void
     {
         $db = $this->getConnection(true);
 
@@ -77,7 +93,14 @@ public function testInsertAndSelect()
         );
     }
 
-    public function testInsertJsonExpresionAndSelect()
+    /**
+     * @return void
+     * @throws \Yiisoft\Db\Exception\Exception
+     * @throws \Yiisoft\Db\Exception\InvalidArgumentException
+     * @throws \Yiisoft\Db\Exception\InvalidConfigException
+     * @throws \Yiisoft\Db\Exception\NotSupportedException
+     */
+    public function testInsertJsonExpressionAndSelect(): void
     {
         $db = $this->getConnection(true);
 

tests/QueryBuilderTest.php

@@ -27,7 +27,10 @@ final class QueryBuilderTest extends CommonQueryBuilderTest
 {
     use TestTrait;
 
-    public function testAddcheck(): void
+    /**
+     * @return void
+     */
+    public function testAddCheck(): void
     {
         $db = $this->getConnection();
 
@@ -40,7 +43,9 @@ public function testAddcheck(): void
     }
 
     /**
+     * @return void
      * @throws Exception
+     * @throws InvalidConfigException
      */
     public function testAddCommentOnColumn(): void
     {
@@ -65,6 +70,7 @@ public function testAddCommentOnColumn(): void
     }
 
     /**
+     * @return void
      * @throws Exception
      */
     public function testAddCommentOnTable(): void
@@ -83,7 +89,9 @@ public function testAddCommentOnTable(): void
     }
 
     /**
+     * @return void
      * @throws Exception
+     * @throws NotSupportedException
      */
     public function testAddDefaultValue(): void
     {

tests/QueryTest.php

@@ -71,7 +71,7 @@ public function testLimitOffsetWithExpression(): void
 
         $query = (new Query($db))->from('customer')->select('id')->orderBy('id');
 
-        /* In MySQL limit and offset arguments must both be non negative integer constant */
+        /* In MySQL limit and offset arguments must both be non-negative integer constant */
         $query->limit(new Expression('2'))->offset(new Expression('1'));
 
         $result = $query->column();
@@ -82,7 +82,12 @@ public function testLimitOffsetWithExpression(): void
         $this->assertNotContains('1', $result);
     }
 
-    public function testWithQuery()
+    /**
+     * @return void
+     * @throws Exception
+     * @throws InvalidConfigException
+     */
+    public function testWithQuery(): void
     {
         $db = $this->getConnection();
 
@@ -96,7 +101,12 @@ public function testWithQuery()
         parent::testWithQuery();
     }
 
-    public function testWithQueryRecursive()
+    /**
+     * @return void
+     * @throws Exception
+     * @throws InvalidConfigException
+     */
+    public function testWithQueryRecursive():  void
     {
         $db = $this->getConnection();
 

tests/QuoterTest.php

@@ -51,6 +51,9 @@ public function testQuoteTableName(string $tableName, string $expected): void
         parent::testQuoteTableName($tableName, $expected);
     }
 
+    /**
+     * @return void
+     */
     public function testQuoteValue(): void
     {
         $db = $this->getConnection();