Merge branch '2.11.x' into 3.0.x

phpcs.xml.dist

@@ -9,8 +9,8 @@
     <arg name="cache" value=".phpcs-cache"/>
     <arg name="colors"/>
 
-    <!-- Ignore warnings, show progress of the run and show sniff names -->
-    <arg value="nps"/>
+    <!-- Show progress of the run and show sniff names -->
+    <arg value="ps"/>
 
     <file>bin</file>
     <file>src</file>
@@ -39,6 +39,16 @@
         </properties>
     </rule>
 
+    <rule ref="PSR2.Classes.PropertyDeclaration.Underscore">
+        <exclude-pattern>*/src/Configuration.php</exclude-pattern>
+        <exclude-pattern>*/src/Connection.php</exclude-pattern>
+        <exclude-pattern>*/src/Driver</exclude-pattern>
+        <exclude-pattern>*/src/Event/Listeners</exclude-pattern>
+        <exclude-pattern>*/src/Platforms</exclude-pattern>
+        <exclude-pattern>*/src/Schema</exclude-pattern>
+        <exclude-pattern>*/src/Tools/Console/Helper/ConnectionHelper.php</exclude-pattern>
+    </rule>
+
     <rule ref="SlevomatCodingStandard.TypeHints.ParameterTypeHint.MissingNativeTypeHint">
         <exclude-pattern>*/src/*</exclude-pattern>
     </rule>

phpstan.neon.dist

@@ -2,8 +2,6 @@ parameters:
     level: 7
     paths:
         - %currentWorkingDirectory%/src
-    scanFiles:
-        - %currentWorkingDirectory%/tests/phpstan-polyfill.php
     treatPhpDocTypesAsCertain: false
     reportUnmatchedIgnoredErrors: false
     checkMissingIterableValueType: false

src/Connection.php

@@ -864,7 +864,8 @@ class Connection
     }
 
     /**
-     * Prepares and executes an SQL query and returns the result as an iterator over rows represented as associative arrays.
+     * Prepares and executes an SQL query and returns the result as an iterator over rows represented
+     * as associative arrays.
      *
      * @param string                                           $query  The SQL query.
      * @param array<int, mixed>|array<string, mixed>           $params The query parameters.

src/Connections/PrimaryReadReplicaConnection.php

@@ -68,7 +68,8 @@ use function count;
  *    )
  * ));
  *
- * You can also pass 'driverOptions' and any other documented option to each of this drivers to pass additional information.
+ * You can also pass 'driverOptions' and any other documented option to each of this drivers
+ * to pass additional information.
  */
 class PrimaryReadReplicaConnection extends Connection
 {
@@ -97,8 +98,12 @@ class PrimaryReadReplicaConnection extends Connection
      * @throws DBALException
      * @throws InvalidArgumentException
      */
-    public function __construct(array $params, Driver $driver, ?Configuration $config = null, ?EventManager $eventManager = null)
-    {
+    public function __construct(
+        array $params,
+        Driver $driver,
+        ?Configuration $config = null,
+        ?EventManager $eventManager = null
+    ) {
         if (! isset($params['replica'], $params['primary'])) {
             throw new InvalidArgumentException('primary or replica configuration missing');
         }
@@ -133,7 +138,10 @@ class PrimaryReadReplicaConnection extends Connection
     public function connect($connectionName = null)
     {
         if ($connectionName !== null) {
-            throw new InvalidArgumentException('Passing a connection name as first argument is not supported anymore. Use ensureConnectedToPrimary()/ensureConnectedToReplica() instead.');
+            throw new InvalidArgumentException(
+                'Passing a connection name as first argument is not supported anymore.'
+                    . ' Use ensureConnectedToPrimary()/ensureConnectedToReplica() instead.'
+            );
         }
 
         return $this->performConnect();

src/DBALException.php

@@ -123,7 +123,9 @@ class DBALException extends Exception
      */
     public static function invalidDriverClass($driverClass)
     {
-        return new self("The given 'driverClass' " . $driverClass . ' has to implement the ' . Driver::class . ' interface.');
+        return new self(
+            "The given 'driverClass' " . $driverClass . ' has to implement the ' . Driver::class . ' interface.'
+        );
     }
 
     /**
@@ -182,7 +184,9 @@ class DBALException extends Exception
 
     public static function typeNotRegistered(Type $type): self
     {
-        return new self(sprintf('Type of the class %s@%s is not registered.', get_class($type), spl_object_hash($type)));
+        return new self(
+            sprintf('Type of the class %s@%s is not registered.', get_class($type), spl_object_hash($type))
+        );
     }
 
     public static function typeAlreadyRegistered(Type $type): self

src/Driver/OCI8/ConvertPositionalToNamedPlaceholders.php

@@ -74,7 +74,8 @@ final class ConvertPositionalToNamedPlaceholders
      * @param string[]    $fragments               Fragments of the original statement not containing placeholders
      * @param string|null $currentLiteralDelimiter The delimiter of the current string literal
      *                                             or NULL if not currently in a literal
-     * @param string[]    $paramMap                Mapping of the original parameter positions to their named replacements
+     * @param string[]    $paramMap                Mapping of the original parameter positions
+     *                                             to their named replacements
      *
      * @return bool Whether the token was found
      */

src/DriverManager.php

@@ -197,7 +197,10 @@ final class DriverManager
             throw DBALException::unknownDriver($params['driver'], array_keys(self::$_driverMap));
         }
 
-        if (isset($params['driverClass']) && ! in_array(Driver::class, class_implements($params['driverClass']), true)) {
+        if (
+            isset($params['driverClass'])
+            && ! in_array(Driver::class, class_implements($params['driverClass']), true)
+        ) {
             throw DBALException::invalidDriverClass($params['driverClass']);
         }
     }

src/Event/SchemaAlterTableAddColumnEventArgs.php

@@ -11,7 +11,7 @@ use function func_get_args;
 use function is_array;
 
 /**
- * Event Arguments used when SQL queries for adding table columns are generated inside Doctrine\DBAL\Platform\*Platform.
+ * Event Arguments used when SQL queries for adding table columns are generated inside {@link AbstractPlatform}.
  */
 class SchemaAlterTableAddColumnEventArgs extends SchemaEventArgs
 {

src/Event/SchemaAlterTableChangeColumnEventArgs.php

@@ -11,7 +11,7 @@ use function func_get_args;
 use function is_array;
 
 /**
- * Event Arguments used when SQL queries for changing table columns are generated inside Doctrine\DBAL\Platform\*Platform.
+ * Event Arguments used when SQL queries for changing table columns are generated inside {@link AbstractPlatform}.
  */
 class SchemaAlterTableChangeColumnEventArgs extends SchemaEventArgs
 {

src/Event/SchemaAlterTableEventArgs.php

@@ -10,7 +10,7 @@ use function func_get_args;
 use function is_array;
 
 /**
- * Event Arguments used when SQL queries for creating tables are generated inside Doctrine\DBAL\Platform\*Platform.
+ * Event Arguments used when SQL queries for creating tables are generated inside {@link AbstractPlatform}.
  */
 class SchemaAlterTableEventArgs extends SchemaEventArgs
 {

src/Event/SchemaAlterTableRemoveColumnEventArgs.php

@@ -11,7 +11,7 @@ use function func_get_args;
 use function is_array;
 
 /**
- * Event Arguments used when SQL queries for removing table columns are generated inside Doctrine\DBAL\Platform\*Platform.
+ * Event Arguments used when SQL queries for removing table columns are generated inside {@link AbstractPlatform}.
  */
 class SchemaAlterTableRemoveColumnEventArgs extends SchemaEventArgs
 {

src/Event/SchemaAlterTableRenameColumnEventArgs.php

@@ -11,7 +11,7 @@ use function func_get_args;
 use function is_array;
 
 /**
- * Event Arguments used when SQL queries for renaming table columns are generated inside Doctrine\DBAL\Platform\*Platform.
+ * Event Arguments used when SQL queries for renaming table columns are generated inside {@link AbstractPlatform}.
  */
 class SchemaAlterTableRenameColumnEventArgs extends SchemaEventArgs
 {

src/Event/SchemaColumnDefinitionEventArgs.php

@@ -6,7 +6,7 @@ use Doctrine\DBAL\Connection;
 use Doctrine\DBAL\Schema\Column;
 
 /**
- * Event Arguments used when the portable column definition is generated inside Doctrine\DBAL\Schema\AbstractSchemaManager.
+ * Event Arguments used when the portable column definition is generated inside {@link AbstractPlatform}.
  */
 class SchemaColumnDefinitionEventArgs extends SchemaEventArgs
 {

src/Event/SchemaCreateTableColumnEventArgs.php

@@ -11,7 +11,7 @@ use function func_get_args;
 use function is_array;
 
 /**
- * Event Arguments used when SQL queries for creating table columns are generated inside Doctrine\DBAL\Platform\AbstractPlatform.
+ * Event Arguments used when SQL queries for creating table columns are generated inside {@link AbstractPlatform}.
  */
 class SchemaCreateTableColumnEventArgs extends SchemaEventArgs
 {

src/Event/SchemaCreateTableEventArgs.php

@@ -10,7 +10,7 @@ use function func_get_args;
 use function is_array;
 
 /**
- * Event Arguments used when SQL queries for creating tables are generated inside Doctrine\DBAL\Platform\AbstractPlatform.
+ * Event Arguments used when SQL queries for creating tables are generated inside {@link AbstractPlatform}.
  */
 class SchemaCreateTableEventArgs extends SchemaEventArgs
 {

src/Event/SchemaDropTableEventArgs.php

@@ -7,7 +7,7 @@ use Doctrine\DBAL\Schema\Table;
 use InvalidArgumentException;
 
 /**
- * Event Arguments used when the SQL query for dropping tables are generated inside Doctrine\DBAL\Platform\AbstractPlatform.
+ * Event Arguments used when the SQL query for dropping tables are generated inside {@link AbstractPlatform}.
  */
 class SchemaDropTableEventArgs extends SchemaEventArgs
 {

src/Event/SchemaIndexDefinitionEventArgs.php

@@ -6,7 +6,7 @@ use Doctrine\DBAL\Connection;
 use Doctrine\DBAL\Schema\Index;
 
 /**
- * Event Arguments used when the portable index definition is generated inside Doctrine\DBAL\Schema\AbstractSchemaManager.
+ * Event Arguments used when the portable index definition is generated inside {@link AbstractSchemaManager}.
  */
 class SchemaIndexDefinitionEventArgs extends SchemaEventArgs
 {

src/Id/TableGenerator.php

@@ -148,7 +148,11 @@ class TableGenerator
         } catch (Throwable $e) {
             $this->conn->rollBack();
 
-            throw new DBALException('Error occurred while generating ID with TableGenerator, aborted generation: ' . $e->getMessage(), 0, $e);
+            throw new DBALException(
+                'Error occurred while generating ID with TableGenerator, aborted generation: ' . $e->getMessage(),
+                0,
+                $e
+            );
         }
 
         return $value;

src/Logging/DebugStack.php

@@ -38,8 +38,14 @@ class DebugStack implements SQLLogger
             return;
         }
 
-        $this->start                          = microtime(true);
-        $this->queries[++$this->currentQuery] = ['sql' => $sql, 'params' => $params, 'types' => $types, 'executionMS' => 0];
+        $this->start = microtime(true);
+
+        $this->queries[++$this->currentQuery] = [
+            'sql' => $sql,
+            'params' => $params,
+            'types' => $types,
+            'executionMS' => 0,
+        ];
     }
 
     /**

src/Platforms/AbstractPlatform.php

@@ -231,7 +231,8 @@ abstract class AbstractPlatform
         if ($column['length'] > $maxLength) {
             if ($maxLength > 0) {
                 @trigger_error(sprintf(
-                    'Binary column length %d is greater than supported by the platform (%d). Reduce the column length or use a BLOB column instead.',
+                    'Binary column length %d is greater than supported by the platform (%d).'
+                        . ' Reduce the column length or use a BLOB column instead.',
                     $column['length'],
                     $maxLength
                 ), E_USER_DEPRECATED);
@@ -379,7 +380,9 @@ abstract class AbstractPlatform
         $dbType = strtolower($dbType);
 
         if (! isset($this->doctrineTypeMapping[$dbType])) {
-            throw new DBALException('Unknown database type ' . $dbType . ' requested, ' . static::class . ' may not support it.');
+            throw new DBALException(
+                'Unknown database type ' . $dbType . ' requested, ' . static::class . ' may not support it.'
+            );
         }
 
         return $this->doctrineTypeMapping[$dbType];
@@ -1286,7 +1289,8 @@ abstract class AbstractPlatform
     }
 
     /**
-     * Honors that some SQL vendors such as MsSql use table hints for locking instead of the ANSI SQL FOR UPDATE specification.
+     * Honors that some SQL vendors such as MsSql use table hints for locking instead of the
+     * ANSI SQL FOR UPDATE specification.
      *
      * @param string   $fromClause The FROM clause to append the hint for the given lock mode to.
      * @param int|null $lockMode   One of the Doctrine\DBAL\LockMode::* constants. If null is given, nothing will
@@ -1354,7 +1358,9 @@ abstract class AbstractPlatform
         }
 
         if (! is_string($table)) {
-            throw new InvalidArgumentException('getDropTableSQL() expects $table parameter to be string or \Doctrine\DBAL\Schema\Table.');
+            throw new InvalidArgumentException(
+                __METHOD__ . '() expects $table parameter to be string or ' . Table::class . '.'
+            );
         }
 
         if ($this->_eventManager !== null && $this->_eventManager->hasListeners(Events::onSchemaDropTable)) {
@@ -1402,7 +1408,9 @@ abstract class AbstractPlatform
         if ($index instanceof Index) {
             $index = $index->getQuotedName($this);
         } elseif (! is_string($index)) {
-            throw new InvalidArgumentException('AbstractPlatform::getDropIndexSQL() expects $index parameter to be string or \Doctrine\DBAL\Schema\Index.');
+            throw new InvalidArgumentException(
+                __METHOD__ . '() expects $index parameter to be string or ' . Index::class . '.'
+            );
         }
 
         return 'DROP INDEX ' . $index;
@@ -1470,7 +1478,9 @@ abstract class AbstractPlatform
     public function getCreateTableSQL(Table $table, $createFlags = self::CREATE_INDEXES)
     {
         if (! is_int($createFlags)) {
-            throw new InvalidArgumentException('Second argument of AbstractPlatform::getCreateTableSQL() has to be integer.');
+            throw new InvalidArgumentException(
+                'Second argument of AbstractPlatform::getCreateTableSQL() has to be integer.'
+            );
         }
 
         if (count($table->getColumns()) === 0) {
@@ -1498,7 +1508,10 @@ abstract class AbstractPlatform
         $columns   = [];
 
         foreach ($table->getColumns() as $column) {
-            if ($this->_eventManager !== null && $this->_eventManager->hasListeners(Events::onSchemaCreateTableColumn)) {
+            if (
+                $this->_eventManager !== null
+                && $this->_eventManager->hasListeners(Events::onSchemaCreateTableColumn)
+            ) {
                 $eventArgs = new SchemaCreateTableColumnEventArgs($column, $table, $this);
                 $this->_eventManager->dispatchEvent(Events::onSchemaCreateTableColumn, $eventArgs);
 
@@ -1509,10 +1522,11 @@ abstract class AbstractPlatform
                 }
             }
 
-            $columnData            = $column->toArray();
-            $columnData['name']    = $column->getQuotedName($this);
-            $columnData['version'] = $column->hasPlatformOption('version') ? $column->getPlatformOption('version') : false;
-            $columnData['comment'] = $this->getColumnComment($column);
+            $columnData = array_merge($column->toArray(), [
+                'name' => $column->getQuotedName($this),
+                'version' => $column->hasPlatformOption('version') ? $column->getPlatformOption('version') : false,
+                'comment' => $this->getColumnComment($column),
+            ]);
 
             if ($columnData['type'] instanceof Types\StringType && $columnData['length'] === null) {
                 $columnData['length'] = 255;

src/Platforms/DB2Platform.php

@@ -628,8 +628,12 @@ class DB2Platform extends AbstractPlatform
      * @param string[]   $sql        The sequence of table alteration statements to fill.
      * @param mixed[]    $queryParts The sequence of column alteration clauses to fill.
      */
-    private function gatherAlterColumnSQL(Identifier $table, ColumnDiff $columnDiff, array &$sql, array &$queryParts): void
-    {
+    private function gatherAlterColumnSQL(
+        Identifier $table,
+        ColumnDiff $columnDiff,
+        array &$sql,
+        array &$queryParts
+    ): void {
         $alterColumnClauses = $this->getAlterColumnClausesSQL($columnDiff);
 
         if (empty($alterColumnClauses)) {

src/Platforms/MySqlPlatform.php

@@ -194,10 +194,9 @@ class MySqlPlatform extends AbstractPlatform
 
         $databaseNameSql = $database ?? 'DATABASE()';
 
-        $sql .= ' AND k.table_schema = ' . $databaseNameSql . ' /*!50116 AND c.constraint_schema = ' . $databaseNameSql . ' */';
-        $sql .= ' AND k.`REFERENCED_COLUMN_NAME` is not NULL';
-
-        return $sql;
+        return $sql . ' AND k.table_schema = ' . $databaseNameSql
+            . ' /*!50116 AND c.constraint_schema = ' . $databaseNameSql . ' */'
+            . ' AND k.`REFERENCED_COLUMN_NAME` is not NULL';
     }
 
     /**
@@ -548,9 +547,11 @@ SQL
                 continue;
             }
 
-            $columnArray            = $column->toArray();
-            $columnArray['comment'] = $this->getColumnComment($column);
-            $queryParts[]           = 'ADD ' . $this->getColumnDeclarationSQL($column->getQuotedName($this), $columnArray);
+            $columnArray = array_merge($column->toArray(), [
+                'comment' => $this->getColumnComment($column),
+            ]);
+
+            $queryParts[] = 'ADD ' . $this->getColumnDeclarationSQL($column->getQuotedName($this), $columnArray);
         }
 
         foreach ($diff->removedColumns as $column) {
@@ -617,7 +618,8 @@ SQL
 
         if (! $this->onSchemaAlterTable($diff, $tableSql)) {
             if (count($queryParts) > 0) {
-                $sql[] = 'ALTER TABLE ' . $diff->getName($this)->getQuotedName($this) . ' ' . implode(', ', $queryParts);
+                $sql[] = 'ALTER TABLE ' . $diff->getName($this)->getQuotedName($this) . ' '
+                    . implode(', ', $queryParts);
             }
 
             $sql = array_merge(
@@ -992,13 +994,17 @@ SQL
         } elseif (is_string($index)) {
             $indexName = $index;
         } else {
-            throw new InvalidArgumentException('MysqlPlatform::getDropIndexSQL() expects $index parameter to be string or \Doctrine\DBAL\Schema\Index.');
+            throw new InvalidArgumentException(
+                __METHOD__ . '() expects $index parameter to be string or ' . Index::class . '.'
+            );
         }
 
         if ($table instanceof Table) {
             $table = $table->getQuotedName($this);
         } elseif (! is_string($table)) {
-            throw new InvalidArgumentException('MysqlPlatform::getDropIndexSQL() expects $table parameter to be string or \Doctrine\DBAL\Schema\Table.');
+            throw new InvalidArgumentException(
+                __METHOD__ . '() expects $table parameter to be string or ' . Table::class . '.'
+            );
         }
 
         if ($index instanceof Index && $index->isPrimary()) {
@@ -1118,7 +1124,9 @@ SQL
         if ($table instanceof Table) {
             $table = $table->getQuotedName($this);
         } elseif (! is_string($table)) {
-            throw new InvalidArgumentException('getDropTemporaryTableSQL() expects $table parameter to be string or \Doctrine\DBAL\Schema\Table.');
+            throw new InvalidArgumentException(
+                __METHOD__ . '() expects $table parameter to be string or ' . Table::class . '.'
+            );
         }
 
         return 'DROP TEMPORARY TABLE ' . $table;

src/Platforms/OraclePlatform.php

@@ -503,14 +503,15 @@ class OraclePlatform extends AbstractPlatform
 
         $idx = new Index($autoincrementIdentifierName, [$quotedName], true, true);
 
-        $sql[] = 'DECLARE
+        $sql[] = "DECLARE
   constraints_Count NUMBER;
 BEGIN
-  SELECT COUNT(CONSTRAINT_NAME) INTO constraints_Count FROM USER_CONSTRAINTS WHERE TABLE_NAME = \'' . $unquotedTableName . '\' AND CONSTRAINT_TYPE = \'P\';
-  IF constraints_Count = 0 OR constraints_Count = \'\' THEN
-    EXECUTE IMMEDIATE \'' . $this->getCreateConstraintSQL($idx, $quotedTableName) . '\';
+  SELECT COUNT(CONSTRAINT_NAME) INTO constraints_Count FROM USER_CONSTRAINTS WHERE TABLE_NAME = '" . $unquotedTableName
+            . "' AND CONSTRAINT_TYPE = 'P';
+  IF constraints_Count = 0 OR constraints_Count = '' THEN
+    EXECUTE IMMEDIATE '" . $this->getCreateConstraintSQL($idx, $quotedTableName) . "';
   END IF;
-END;';
+END;";
 
         $sequenceName = $this->getIdentitySequenceName(
             $tableIdentifier->isQuoted() ? $quotedTableName : $unquotedTableName,
@@ -818,7 +819,8 @@ SQL
         }
 
         if (count($fields) > 0) {
-            $sql[] = 'ALTER TABLE ' . $diff->getName($this)->getQuotedName($this) . ' ADD (' . implode(', ', $fields) . ')';
+            $sql[] = 'ALTER TABLE ' . $diff->getName($this)->getQuotedName($this)
+                . ' ADD (' . implode(', ', $fields) . ')';
         }
 
         $fields = [];
@@ -867,7 +869,8 @@ SQL
         }
 
         if (count($fields) > 0) {
-            $sql[] = 'ALTER TABLE ' . $diff->getName($this)->getQuotedName($this) . ' MODIFY (' . implode(', ', $fields) . ')';
+            $sql[] = 'ALTER TABLE ' . $diff->getName($this)->getQuotedName($this)
+                . ' MODIFY (' . implode(', ', $fields) . ')';
         }
 
         foreach ($diff->renamedColumns as $oldColumnName => $column) {
@@ -891,7 +894,8 @@ SQL
         }
 
         if (count($fields) > 0) {
-            $sql[] = 'ALTER TABLE ' . $diff->getName($this)->getQuotedName($this) . ' DROP (' . implode(', ', $fields) . ')';
+            $sql[] = 'ALTER TABLE ' . $diff->getName($this)->getQuotedName($this)
+                . ' DROP (' . implode(', ', $fields) . ')';
         }
 
         $tableSql = [];
@@ -1203,7 +1207,9 @@ SQL
 
         if ($database !== null && $database !== '/') {
             $tableCommentsName = 'all_tab_comments';
-            $ownerCondition    = ' AND owner = ' . $this->quoteStringLiteral($this->normalizeIdentifier($database)->getName());
+            $ownerCondition    = ' AND owner = ' . $this->quoteStringLiteral(
+                $this->normalizeIdentifier($database)->getName()
+            );
         }
 
         return sprintf(

src/Platforms/PostgreSQL94Platform.php

@@ -112,7 +112,8 @@ class PostgreSQL94Platform extends AbstractPlatform
         if ($startPos !== false) {
             $str = $this->getSubstringExpression($str, $startPos);
 
-            return 'CASE WHEN (POSITION(' . $substr . ' IN ' . $str . ') = 0) THEN 0 ELSE (POSITION(' . $substr . ' IN ' . $str . ') + ' . ($startPos - 1) . ') END';
+            return 'CASE WHEN (POSITION(' . $substr . ' IN ' . $str . ') = 0) THEN 0'
+                . ' ELSE (POSITION(' . $substr . ' IN ' . $str . ') + ' . ($startPos - 1) . ') END';
         }
 
         return 'POSITION(' . $substr . ' IN ' . $str . ')';
@@ -358,7 +359,8 @@ SQL
                  WHERE oid IN (
                     SELECT indexrelid
                     FROM pg_index si, pg_class sc, pg_namespace sn
-                    WHERE ' . $this->getTableWhereClause($table, 'sc', 'sn') . ' AND sc.oid=si.indrelid AND sc.relnamespace = sn.oid
+                    WHERE ' . $this->getTableWhereClause($table, 'sc', 'sn') . '
+                    AND sc.oid=si.indrelid AND sc.relnamespace = sn.oid
                  ) AND pg_index.indexrelid = oid';
     }
 
@@ -548,7 +550,12 @@ SQL
             $oldColumnName = $columnDiff->getOldColumnName()->getQuotedName($this);
             $column        = $columnDiff->column;
 
-            if ($columnDiff->hasChanged('type') || $columnDiff->hasChanged('precision') || $columnDiff->hasChanged('scale') || $columnDiff->hasChanged('fixed')) {
+            if (
+                $columnDiff->hasChanged('type')
+                || $columnDiff->hasChanged('precision')
+                || $columnDiff->hasChanged('scale')
+                || $columnDiff->hasChanged('fixed')
+            ) {
                 $type = $column->getType();
 
                 // SERIAL/BIGSERIAL are not "real" types and we can't alter a column to that type
@@ -579,7 +586,8 @@ SQL
                     $seqName = $this->getIdentitySequenceName($diff->name, $oldColumnName);
 
                     $sql[] = 'CREATE SEQUENCE ' . $seqName;
-                    $sql[] = "SELECT setval('" . $seqName . "', (SELECT MAX(" . $oldColumnName . ') FROM ' . $diff->getName($this)->getQuotedName($this) . '))';
+                    $sql[] = "SELECT setval('" . $seqName . "', (SELECT MAX(" . $oldColumnName . ') FROM '
+                        . $diff->getName($this)->getQuotedName($this) . '))';
                     $query = 'ALTER ' . $oldColumnName . " SET DEFAULT nextval('" . $seqName . "')";
                     $sql[] = 'ALTER TABLE ' . $diff->getName($this)->getQuotedName($this) . ' ' . $query;
                 } else {
@@ -592,7 +600,10 @@ SQL
             $newComment = $this->getColumnComment($column);
             $oldComment = $this->getOldColumnComment($columnDiff);
 
-            if ($columnDiff->hasChanged('comment') || ($columnDiff->fromColumn !== null && $oldComment !== $newComment)) {
+            if (
+                $columnDiff->hasChanged('comment')
+                || ($columnDiff->fromColumn !== null && $oldComment !== $newComment)
+            ) {
                 $commentsSQL[] = $this->getCommentOnColumnSQL(
                     $diff->getName($this)->getQuotedName($this),
                     $column->getQuotedName($this),
@@ -604,7 +615,8 @@ SQL
                 continue;
             }
 
-            $query = 'ALTER ' . $oldColumnName . ' TYPE ' . $column->getType()->getSQLDeclaration($column->toArray(), $this);
+            $query = 'ALTER ' . $oldColumnName . ' TYPE '
+                . $column->getType()->getSQLDeclaration($column->toArray(), $this);
             $sql[] = 'ALTER TABLE ' . $diff->getName($this)->getQuotedName($this) . ' ' . $query;
         }
 

src/Platforms/SQLServer2012Platform.php

@@ -266,7 +266,9 @@ class SQLServer2012Platform extends AbstractPlatform
         if ($index instanceof Index) {
             $index = $index->getQuotedName($this);
         } elseif (! is_string($index)) {
-            throw new InvalidArgumentException('AbstractPlatform::getDropIndexSQL() expects $index parameter to be string or \Doctrine\DBAL\Schema\Index.');
+            throw new InvalidArgumentException(
+                __METHOD__ . '() expects $index parameter to be string or ' . Index::class . '.'
+            );
         }
 
         if (! isset($table)) {
@@ -340,7 +342,8 @@ SQL
                 $flags = ' NONCLUSTERED';
             }
 
-            $columnListSql .= ', PRIMARY KEY' . $flags . ' (' . implode(', ', array_unique(array_values($options['primary']))) . ')';
+            $columnListSql .= ', PRIMARY KEY' . $flags
+                . ' (' . implode(', ', array_unique(array_values($options['primary']))) . ')';
         }
 
         $query = 'CREATE TABLE ' . $tableName . ' (' . $columnListSql;
@@ -608,7 +611,10 @@ SQL
             $queryParts[] = 'ALTER COLUMN ' .
                     $this->getColumnDeclarationSQL($column->getQuotedName($this), $columnDef);
 
-            if (! isset($columnDef['default']) || (! $requireDropDefaultConstraint && ! $columnDiff->hasChanged('default'))) {
+            if (
+                ! isset($columnDef['default'])
+                || (! $requireDropDefaultConstraint && ! $columnDiff->hasChanged('default'))
+            ) {
                 continue;
             }
 
@@ -952,7 +958,8 @@ SQL
     {
         // "sysdiagrams" table must be ignored as it's internal SQL Server table for Database Diagrams
         // Category 2 must be ignored as it is "MS SQL Server 'pseudo-system' object[s]" for replication
-        return "SELECT name, SCHEMA_NAME (uid) AS schema_name FROM sysobjects WHERE type = 'U' AND name != 'sysdiagrams' AND category != 2 ORDER BY name";
+        return 'SELECT name, SCHEMA_NAME (uid) AS schema_name FROM sysobjects'
+            . " WHERE type = 'U' AND name != 'sysdiagrams' AND category != 2 ORDER BY name";
     }
 
     /**
@@ -1125,15 +1132,6 @@ SQL
             return $trimFn . '(' . $str . ')';
         }
 
-        /** Original query used to get those expressions
-          declare @c varchar(100) = 'xxxBarxxx', @trim_char char(1) = 'x';
-          declare @pat varchar(10) = '%[^' + @trim_char + ']%';
-          select @c as string
-          , @trim_char as trim_char
-          , stuff(@c, 1, patindex(@pat, @c) - 1, null) as trim_leading
-          , reverse(stuff(reverse(@c), 1, patindex(@pat, reverse(@c)) - 1, null)) as trim_trailing
-          , reverse(stuff(reverse(stuff(@c, 1, patindex(@pat, @c) - 1, null)), 1, patindex(@pat, reverse(stuff(@c, 1, patindex(@pat, @c) - 1, null))) - 1, null)) as trim_both;
-         */
         $pattern = "'%[^' + " . $char . " + ']%'";
 
         if ($pos === TrimMode::LEADING) {
@@ -1141,10 +1139,13 @@ SQL
         }
 
         if ($pos === TrimMode::TRAILING) {
-            return 'reverse(stuff(reverse(' . $str . '), 1, patindex(' . $pattern . ', reverse(' . $str . ')) - 1, null))';
+            return 'reverse(stuff(reverse(' . $str . '), 1, '
+                . 'patindex(' . $pattern . ', reverse(' . $str . ')) - 1, null))';
         }
 
-        return 'reverse(stuff(reverse(stuff(' . $str . ', 1, patindex(' . $pattern . ', ' . $str . ') - 1, null)), 1, patindex(' . $pattern . ', reverse(stuff(' . $str . ', 1, patindex(' . $pattern . ', ' . $str . ') - 1, null))) - 1, null))';
+        return 'reverse(stuff(reverse(stuff(' . $str . ', 1, patindex(' . $pattern . ', ' . $str . ') - 1, null)), 1, '
+            . 'patindex(' . $pattern . ', reverse(stuff(' . $str . ', 1, patindex(' . $pattern . ', ' . $str
+            . ') - 1, null))) - 1, null))';
     }
 
     /**

src/Platforms/SqlitePlatform.php

@@ -472,9 +472,13 @@ class SqlitePlatform extends AbstractPlatform
      */
     public function getListTablesSQL()
     {
-        return "SELECT name FROM sqlite_master WHERE type = 'table' AND name != 'sqlite_sequence' AND name != 'geometry_columns' AND name != 'spatial_ref_sys' "
-             . 'UNION ALL SELECT name FROM sqlite_temp_master '
-             . "WHERE type = 'table' ORDER BY name";
+        return 'SELECT name FROM sqlite_master'
+            . " WHERE type = 'table'"
+            . " AND name != 'sqlite_sequence'"
+            . " AND name != 'geometry_columns'"
+            . " AND name != 'spatial_ref_sys'"
+            . ' UNION ALL SELECT name FROM sqlite_temp_master'
+            . " WHERE type = 'table' ORDER BY name";
     }
 
     /**
@@ -508,8 +512,18 @@ class SqlitePlatform extends AbstractPlatform
     {
         $query = parent::getAdvancedForeignKeyOptionsSQL($foreignKey);
 
-        $query .= ($foreignKey->hasOption('deferrable') && $foreignKey->getOption('deferrable') !== false ? ' ' : ' NOT ') . 'DEFERRABLE';
-        $query .= ' INITIALLY ' . ($foreignKey->hasOption('deferred') && $foreignKey->getOption('deferred') !== false ? 'DEFERRED' : 'IMMEDIATE');
+        if (! $foreignKey->hasOption('deferrable') || $foreignKey->getOption('deferrable') === false) {
+            $query .= ' NOT';
+        }
+
+        $query .= ' DEFERRABLE';
+        $query .= ' INITIALLY';
+
+        if ($foreignKey->hasOption('deferred') && $foreignKey->getOption('deferred') !== false) {
+            $query .= ' DEFERRED';
+        } else {
+            $query .= ' IMMEDIATE';
+        }
 
         return $query;
     }
@@ -682,7 +696,9 @@ class SqlitePlatform extends AbstractPlatform
     protected function getPreAlterTableIndexForeignKeySQL(TableDiff $diff)
     {
         if (! $diff->fromTable instanceof Table) {
-            throw new DBALException('Sqlite platform requires for alter table the table diff with reference to original table schema');
+            throw new DBALException(
+                'Sqlite platform requires for alter table the table diff with reference to original table schema'
+            );
         }
 
         $sql = [];
@@ -703,7 +719,9 @@ class SqlitePlatform extends AbstractPlatform
     protected function getPostAlterTableIndexForeignKeySQL(TableDiff $diff)
     {
         if (! $diff->fromTable instanceof Table) {
-            throw new DBALException('Sqlite platform requires for alter table the table diff with reference to original table schema');
+            throw new DBALException(
+                'Sqlite platform requires for alter table the table diff with reference to original table schema'
+            );
         }
 
         $sql       = [];
@@ -794,7 +812,10 @@ class SqlitePlatform extends AbstractPlatform
      */
     public function getCreateForeignKeySQL(ForeignKeyConstraint $foreignKey, $table)
     {
-        throw new DBALException('Sqlite platform does not support alter foreign key, the table must be fully recreated using getAlterTableSQL.');
+        throw new DBALException(
+            'Sqlite platform does not support alter foreign key, '
+            . 'the table must be fully recreated using getAlterTableSQL.'
+        );
     }
 
     /**
@@ -802,7 +823,10 @@ class SqlitePlatform extends AbstractPlatform
      */
     public function getDropForeignKeySQL($foreignKey, $table)
     {
-        throw new DBALException('Sqlite platform does not support alter foreign key, the table must be fully recreated using getAlterTableSQL.');
+        throw new DBALException(
+            'Sqlite platform does not support alter foreign key, '
+            . 'the table must be fully recreated using getAlterTableSQL.'
+        );
     }
 
     /**
@@ -850,7 +874,9 @@ class SqlitePlatform extends AbstractPlatform
 
         $fromTable = $diff->fromTable;
         if (! $fromTable instanceof Table) {
-            throw new DBALException('Sqlite platform requires for alter table the table diff with reference to original table schema');
+            throw new DBALException(
+                'Sqlite platform requires for alter table the table diff with reference to original table schema'
+            );
         }
 
         $table = clone $fromTable;
@@ -933,16 +959,34 @@ class SqlitePlatform extends AbstractPlatform
         if (! $this->onSchemaAlterTable($diff, $tableSql)) {
             $dataTable = new Table('__temp__' . $table->getName());
 
-            $newTable = new Table($table->getQuotedName($this), $columns, $this->getPrimaryIndexInAlteredTable($diff), $this->getForeignKeysInAlteredTable($diff), 0, $table->getOptions());
+            $newTable = new Table(
+                $table->getQuotedName($this),
+                $columns,
+                $this->getPrimaryIndexInAlteredTable($diff),
+                $this->getForeignKeysInAlteredTable($diff),
+                0,
+                $table->getOptions()
+            );
             $newTable->addOption('alter', true);
 
             $sql = $this->getPreAlterTableIndexForeignKeySQL($diff);
-            //$sql = array_merge($sql, $this->getCreateTableSQL($dataTable, 0));
-            $sql[] = sprintf('CREATE TEMPORARY TABLE %s AS SELECT %s FROM %s', $dataTable->getQuotedName($this), implode(', ', $oldColumnNames), $table->getQuotedName($this));
+
+            $sql[] = sprintf(
+                'CREATE TEMPORARY TABLE %s AS SELECT %s FROM %s',
+                $dataTable->getQuotedName($this),
+                implode(', ', $oldColumnNames),
+                $table->getQuotedName($this)
+            );
             $sql[] = $this->getDropTableSQL($fromTable);
 
             $sql   = array_merge($sql, $this->getCreateTableSQL($newTable));
-            $sql[] = sprintf('INSERT INTO %s (%s) SELECT %s FROM %s', $newTable->getQuotedName($this), implode(', ', $newColumnNames), implode(', ', $oldColumnNames), $dataTable->getQuotedName($this));
+            $sql[] = sprintf(
+                'INSERT INTO %s (%s) SELECT %s FROM %s',
+                $newTable->getQuotedName($this),
+                implode(', ', $newColumnNames),
+                implode(', ', $oldColumnNames),
+                $dataTable->getQuotedName($this)
+            );
             $sql[] = $this->getDropTableSQL($dataTable);
 
             $newName = $diff->getNewName();
@@ -994,10 +1038,16 @@ class SqlitePlatform extends AbstractPlatform
         }
 
         if (
-            ! empty($diff->renamedColumns) || ! empty($diff->addedForeignKeys) || ! empty($diff->addedIndexes)
-                || ! empty($diff->changedColumns) || ! empty($diff->changedForeignKeys) || ! empty($diff->changedIndexes)
-                || ! empty($diff->removedColumns) || ! empty($diff->removedForeignKeys) || ! empty($diff->removedIndexes)
-                || ! empty($diff->renamedIndexes)
+            ! empty($diff->renamedColumns)
+            || ! empty($diff->addedForeignKeys)
+            || ! empty($diff->addedIndexes)
+            || ! empty($diff->changedColumns)
+            || ! empty($diff->changedForeignKeys)
+            || ! empty($diff->changedIndexes)
+            || ! empty($diff->removedColumns)
+            || ! empty($diff->removedForeignKeys)
+            || ! empty($diff->removedIndexes)
+            || ! empty($diff->renamedIndexes)
         ) {
             return false;
         }
@@ -1013,8 +1063,13 @@ class SqlitePlatform extends AbstractPlatform
                 continue;
             }
 
-            $definition = array_merge(['unique' => null, 'autoincrement' => null, 'default' => null], $column->toArray());
-            $type       = $definition['type'];
+            $definition = array_merge([
+                'unique' => null,
+                'autoincrement' => null,
+                'default' => null,
+            ], $column->toArray());
+
+            $type = $definition['type'];
 
             switch (true) {
                 case isset($definition['columnDefinition']) || $definition['autoincrement'] || $definition['unique']:
@@ -1029,13 +1084,16 @@ class SqlitePlatform extends AbstractPlatform
                 $definition['length'] = 255;
             }
 
-            $sql[] = 'ALTER TABLE ' . $table->getQuotedName($this) . ' ADD COLUMN ' . $this->getColumnDeclarationSQL($definition['name'], $definition);
+            $sql[] = 'ALTER TABLE ' . $table->getQuotedName($this) . ' ADD COLUMN '
+                . $this->getColumnDeclarationSQL($definition['name'], $definition);
         }
 
         if (! $this->onSchemaAlterTable($diff, $tableSql)) {
             if ($diff->newName !== false) {
                 $newTable = new Identifier($diff->newName);
-                $sql[]    = 'ALTER TABLE ' . $table->getQuotedName($this) . ' RENAME TO ' . $newTable->getQuotedName($this);
+
+                $sql[] = 'ALTER TABLE ' . $table->getQuotedName($this) . ' RENAME TO '
+                    . $newTable->getQuotedName($this);
             }
         }
 
@@ -1120,7 +1178,13 @@ class SqlitePlatform extends AbstractPlatform
                 continue;
             }
 
-            $indexes[$key] = new Index($index->getName(), $indexColumns, $index->isUnique(), $index->isPrimary(), $index->getFlags());
+            $indexes[$key] = new Index(
+                $index->getName(),
+                $indexColumns,
+                $index->isUnique(),
+                $index->isPrimary(),
+                $index->getFlags()
+            );
         }
 
         foreach ($diff->removedIndexes as $index) {
@@ -1174,7 +1238,13 @@ class SqlitePlatform extends AbstractPlatform
                 continue;
             }
 
-            $foreignKeys[$key] = new ForeignKeyConstraint($localColumns, $constraint->getForeignTableName(), $constraint->getForeignColumns(), $constraint->getName(), $constraint->getOptions());
+            $foreignKeys[$key] = new ForeignKeyConstraint(
+                $localColumns,
+                $constraint->getForeignTableName(),
+                $constraint->getForeignColumns(),
+                $constraint->getName(),
+                $constraint->getOptions()
+            );
         }
 
         foreach ($diff->removedForeignKeys as $constraint) {

src/Query/QueryBuilder.php

@@ -429,7 +429,12 @@ class QueryBuilder
         $this->state = self::STATE_DIRTY;
 
         if ($append) {
-            if ($sqlPartName === 'orderBy' || $sqlPartName === 'groupBy' || $sqlPartName === 'select' || $sqlPartName === 'set') {
+            if (
+                $sqlPartName === 'orderBy'
+                || $sqlPartName === 'groupBy'
+                || $sqlPartName === 'select'
+                || $sqlPartName === 'set'
+            ) {
                 foreach ($sqlPart as $part) {
                     $this->sqlParts[$sqlPartName][] = $part;
                 }
@@ -1228,7 +1233,8 @@ class QueryBuilder
      */
     private function getSQLForUpdate()
     {
-        $table = $this->sqlParts['from']['table'] . ($this->sqlParts['from']['alias'] ? ' ' . $this->sqlParts['from']['alias'] : '');
+        $table = $this->sqlParts['from']['table']
+            . ($this->sqlParts['from']['alias'] ? ' ' . $this->sqlParts['from']['alias'] : '');
 
         return 'UPDATE ' . $table
             . ' SET ' . implode(', ', $this->sqlParts['set'])
@@ -1242,9 +1248,11 @@ class QueryBuilder
      */
     private function getSQLForDelete()
     {
-        $table = $this->sqlParts['from']['table'] . ($this->sqlParts['from']['alias'] ? ' ' . $this->sqlParts['from']['alias'] : '');
+        $table = $this->sqlParts['from']['table']
+            . ($this->sqlParts['from']['alias'] ? ' ' . $this->sqlParts['from']['alias'] : '');
 
-        return 'DELETE FROM ' . $table . ($this->sqlParts['where'] !== null ? ' WHERE ' . ((string) $this->sqlParts['where']) : '');
+        return 'DELETE FROM ' . $table
+            . ($this->sqlParts['where'] !== null ? ' WHERE ' . ((string) $this->sqlParts['where']) : '');
     }
 
     /**

src/SQLParserUtils.php

@@ -67,7 +67,12 @@ class SQLParserUtils
             $statement,
             ':',
             self::NAMED_TOKEN,
-            static function (string $placeholder, int $placeholderPosition, int $fragmentPosition, array &$carry): void {
+            static function (
+                string $placeholder,
+                int $placeholderPosition,
+                int $fragmentPosition,
+                array &$carry
+            ): void {
                 $carry[$placeholderPosition + $fragmentPosition] = substr($placeholder, 1);
             }
         );
@@ -76,8 +81,12 @@ class SQLParserUtils
     /**
      * @return mixed[]
      */
-    private static function collectPlaceholders(string $statement, string $match, string $token, callable $collector): array
-    {
+    private static function collectPlaceholders(
+        string $statement,
+        string $match,
+        string $token,
+        callable $collector
+    ): array {
         if (strpos($statement, $match) === false) {
             return [];
         }

src/SQLParserUtilsException.php

@@ -18,7 +18,9 @@ class SQLParserUtilsException extends DBALException
      */
     public static function missingParam($paramName)
     {
-        return new self(sprintf('Value for :%1$s not found in params array. Params array key should be "%1$s"', $paramName));
+        return new self(
+            sprintf('Value for :%1$s not found in params array. Params array key should be "%1$s"', $paramName)
+        );
     }
 
     /**
@@ -28,6 +30,8 @@ class SQLParserUtilsException extends DBALException
      */
     public static function missingType($typeName)
     {
-        return new self(sprintf('Value for :%1$s not found in types array. Types array key should be "%1$s"', $typeName));
+        return new self(
+            sprintf('Value for :%1$s not found in types array. Types array key should be "%1$s"', $typeName)
+        );
     }
 }

src/Schema/AbstractSchemaManager.php

@@ -566,7 +566,8 @@ abstract class AbstractSchemaManager
     /**
      * Drops and creates a new foreign key.
      *
-     * @param ForeignKeyConstraint $foreignKey An associative array that defines properties of the foreign key to be created.
+     * @param ForeignKeyConstraint $foreignKey An associative array that defines properties
+     *                                         of the foreign key to be created.
      * @param Table|string         $table      The name of the table on which the foreign key is to be created.
      *
      * @return void
@@ -691,7 +692,8 @@ abstract class AbstractSchemaManager
     /**
      * Converts a list of namespace names from the native DBMS data definition to a portable Doctrine definition.
      *
-     * @param array<int, array<string, mixed>> $namespaces The list of namespace names in the native DBMS data definition.
+     * @param array<int, array<string, mixed>> $namespaces The list of namespace names
+     *                                                     in the native DBMS data definition.
      *
      * @return string[]
      */
@@ -905,7 +907,14 @@ abstract class AbstractSchemaManager
             }
 
             if (! $defaultPrevented) {
-                $index = new Index($data['name'], $data['columns'], $data['unique'], $data['primary'], $data['flags'], $data['options']);
+                $index = new Index(
+                    $data['name'],
+                    $data['columns'],
+                    $data['unique'],
+                    $data['primary'],
+                    $data['flags'],
+                    $data['options']
+                );
             }
 
             if ($index === null) {

src/Schema/ColumnDiff.php

@@ -25,8 +25,12 @@ class ColumnDiff
      * @param string   $oldColumnName
      * @param string[] $changedProperties
      */
-    public function __construct($oldColumnName, Column $column, array $changedProperties = [], ?Column $fromColumn = null)
-    {
+    public function __construct(
+        $oldColumnName,
+        Column $column,
+        array $changedProperties = [],
+        ?Column $fromColumn = null
+    ) {
         $this->oldColumnName     = $oldColumnName;
         $this->column            = $column;
         $this->changedProperties = $changedProperties;

src/Schema/Comparator.php

@@ -71,7 +71,11 @@ class Comparator
             if (! $fromSchema->hasTable($tableName)) {
                 $diff->newTables[$tableName] = $toSchema->getTable($tableName);
             } else {
-                $tableDifferences = $this->diffTable($fromSchema->getTable($tableName), $toSchema->getTable($tableName));
+                $tableDifferences = $this->diffTable(
+                    $fromSchema->getTable($tableName),
+                    $toSchema->getTable($tableName)
+                );
+
                 if ($tableDifferences !== false) {
                     $diff->changedTables[$tableName] = $tableDifferences;
                 }
@@ -230,7 +234,8 @@ class Comparator
                 continue;
             }
 
-            $columnDiff                                           = new ColumnDiff($column->getName(), $table2->getColumn($columnName), $changedProperties);
+            $columnDiff = new ColumnDiff($column->getName(), $table2->getColumn($columnName), $changedProperties);
+
             $columnDiff->fromColumn                               = $column;
             $tableDifferences->changedColumns[$column->getName()] = $columnDiff;
             $changes++;
@@ -399,11 +404,17 @@ class Comparator
      */
     public function diffForeignKey(ForeignKeyConstraint $key1, ForeignKeyConstraint $key2)
     {
-        if (array_map('strtolower', $key1->getUnquotedLocalColumns()) !== array_map('strtolower', $key2->getUnquotedLocalColumns())) {
+        if (
+            array_map('strtolower', $key1->getUnquotedLocalColumns())
+            !== array_map('strtolower', $key2->getUnquotedLocalColumns())
+        ) {
             return true;
         }
 
-        if (array_map('strtolower', $key1->getUnquotedForeignColumns()) !== array_map('strtolower', $key2->getUnquotedForeignColumns())) {
+        if (
+            array_map('strtolower', $key1->getUnquotedForeignColumns())
+            !== array_map('strtolower', $key2->getUnquotedForeignColumns())
+        ) {
             return true;
         }
 

src/Schema/ForeignKeyConstraint.php

@@ -62,8 +62,13 @@ class ForeignKeyConstraint extends AbstractAsset implements Constraint
      * @param string|null  $name               Name of the foreign key constraint.
      * @param mixed[]      $options            Options associated with the foreign key constraint.
      */
-    public function __construct(array $localColumnNames, $foreignTableName, array $foreignColumnNames, $name = null, array $options = [])
-    {
+    public function __construct(
+        array $localColumnNames,
+        $foreignTableName,
+        array $foreignColumnNames,
+        $name = null,
+        array $options = []
+    ) {
         if ($name !== null) {
             $this->_setName($name);
         }

src/Schema/Index.php

@@ -54,8 +54,14 @@ class Index extends AbstractAsset implements Constraint
      * @param string[] $flags
      * @param mixed[]  $options
      */
-    public function __construct($name, array $columns, $isUnique = false, $isPrimary = false, array $flags = [], array $options = [])
-    {
+    public function __construct(
+        $name,
+        array $columns,
+        $isUnique = false,
+        $isPrimary = false,
+        array $flags = [],
+        array $options = []
+    ) {
         $isUnique = $isUnique || $isPrimary;
 
         $this->_setName($name);
@@ -183,7 +189,10 @@ class Index extends AbstractAsset implements Constraint
         $sameColumns     = true;
 
         for ($i = 0; $i < $numberOfColumns; $i++) {
-            if (isset($columnNames[$i]) && $this->trimQuotes(strtolower($columns[$i])) === $this->trimQuotes(strtolower($columnNames[$i]))) {
+            if (
+                isset($columnNames[$i])
+                && $this->trimQuotes(strtolower($columns[$i])) === $this->trimQuotes(strtolower($columnNames[$i]))
+            ) {
                 continue;
             }
 
@@ -251,7 +260,9 @@ class Index extends AbstractAsset implements Constraint
             return false;
         }
 
-        return $this->spansColumns($other->getColumns()) && ($this->isPrimary() || $this->isUnique()) && $this->samePartialIndex($other);
+        return $this->spansColumns($other->getColumns())
+            && ($this->isPrimary() || $this->isUnique())
+            && $this->samePartialIndex($other);
     }
 
     /**
@@ -339,7 +350,11 @@ class Index extends AbstractAsset implements Constraint
      */
     private function samePartialIndex(Index $other)
     {
-        if ($this->hasOption('where') && $other->hasOption('where') && $this->getOption('where') === $other->getOption('where')) {
+        if (
+            $this->hasOption('where')
+            && $other->hasOption('where')
+            && $this->getOption('where') === $other->getOption('where')
+        ) {
             return true;
         }
 

src/Schema/PostgreSqlSchemaManager.php

@@ -45,7 +45,9 @@ class PostgreSqlSchemaManager extends AbstractSchemaManager
      */
     public function getSchemaNames()
     {
-        return $this->_conn->fetchFirstColumn("SELECT nspname FROM pg_namespace WHERE nspname !~ '^pg_.*' AND nspname != 'information_schema'");
+        return $this->_conn->fetchFirstColumn(
+            "SELECT nspname FROM pg_namespace WHERE nspname !~ '^pg_.*' AND nspname != 'information_schema'"
+        );
     }
 
     /**
@@ -315,7 +317,9 @@ class PostgreSqlSchemaManager extends AbstractSchemaManager
 
         if (! isset($sequence['increment_by'], $sequence['min_value'])) {
             /** @var string[] $data */
-            $data = $this->_conn->fetchAssociative('SELECT min_value, increment_by FROM ' . $this->_platform->quoteIdentifier($sequenceName));
+            $data = $this->_conn->fetchAssociative(
+                'SELECT min_value, increment_by FROM ' . $this->_platform->quoteIdentifier($sequenceName)
+            );
 
             $sequence += $data;
         }

src/Schema/SQLServerSchemaManager.php

@@ -158,20 +158,22 @@ class SQLServerSchemaManager extends AbstractSchemaManager
         $foreignKeys = [];
 
         foreach ($tableForeignKeys as $tableForeignKey) {
-            if (! isset($foreignKeys[$tableForeignKey['ForeignKey']])) {
-                $foreignKeys[$tableForeignKey['ForeignKey']] = [
+            $name = $tableForeignKey['ForeignKey'];
+
+            if (! isset($foreignKeys[$name])) {
+                $foreignKeys[$name] = [
                     'local_columns' => [$tableForeignKey['ColumnName']],
                     'foreign_table' => $tableForeignKey['ReferenceTableName'],
                     'foreign_columns' => [$tableForeignKey['ReferenceColumnName']],
-                    'name' => $tableForeignKey['ForeignKey'],
+                    'name' => $name,
                     'options' => [
                         'onUpdate' => str_replace('_', ' ', $tableForeignKey['update_referential_action_desc']),
                         'onDelete' => str_replace('_', ' ', $tableForeignKey['delete_referential_action_desc']),
                     ],
                 ];
             } else {
-                $foreignKeys[$tableForeignKey['ForeignKey']]['local_columns'][]   = $tableForeignKey['ColumnName'];
-                $foreignKeys[$tableForeignKey['ForeignKey']]['foreign_columns'][] = $tableForeignKey['ReferenceColumnName'];
+                $foreignKeys[$name]['local_columns'][]   = $tableForeignKey['ColumnName'];
+                $foreignKeys[$name]['foreign_columns'][] = $tableForeignKey['ReferenceColumnName'];
             }
         }
 

src/Schema/SchemaException.php

@@ -168,8 +168,8 @@ class SchemaException extends DBALException
         return new self(
             'The performed schema operation on ' . $localTable->getName() . ' requires a named foreign key, ' .
             'but the given foreign key from (' . implode(', ', $foreignKey->getColumns()) . ') onto foreign table ' .
-            "'" . $foreignKey->getForeignTableName() . "' (" . implode(', ', $foreignKey->getForeignColumns()) . ') is currently ' .
-            'unnamed.'
+            "'" . $foreignKey->getForeignTableName() . "' (" . implode(', ', $foreignKey->getForeignColumns()) . ')' .
+            ' is currently unnamed.'
         );
     }
 

src/Schema/SqliteSchemaManager.php

@@ -10,6 +10,7 @@ use Doctrine\DBAL\Types\Type;
 
 use function array_change_key_case;
 use function array_map;
+use function array_merge;
 use function array_reverse;
 use function array_values;
 use function explode;
@@ -144,10 +145,13 @@ class SqliteSchemaManager extends AbstractSchemaManager
             }
 
             foreach ($tableForeignKeys as $key => $value) {
-                $id                                        = $value['id'];
-                $tableForeignKeys[$key]['constraint_name'] = isset($names[$id]) && $names[$id] !== '' ? $names[$id] : $id;
-                $tableForeignKeys[$key]['deferrable']      = isset($deferrable[$id]) && strtolower($deferrable[$id]) === 'deferrable';
-                $tableForeignKeys[$key]['deferred']        = isset($deferred[$id]) && strtolower($deferred[$id]) === 'deferred';
+                $id = $value['id'];
+
+                $tableForeignKeys[$key] = array_merge($tableForeignKeys[$key], [
+                    'constraint_name' => isset($names[$id]) && $names[$id] !== '' ? $names[$id] : $id,
+                    'deferrable'      => isset($deferrable[$id]) && strtolower($deferrable[$id]) === 'deferrable',
+                    'deferred'        => isset($deferred[$id]) && strtolower($deferred[$id]) === 'deferred',
+                ]);
             }
         }
 
@@ -444,7 +448,9 @@ class SqliteSchemaManager extends AbstractSchemaManager
             $tableDetails = $this->tryMethod('listTableDetails', $table);
 
             if ($tableDetails === false) {
-                throw new DBALException(sprintf('Sqlite schema manager requires to modify foreign keys table definition "%s".', $table));
+                throw new DBALException(
+                    sprintf('Sqlite schema manager requires to modify foreign keys table definition "%s".', $table)
+                );
             }
 
             $table = $tableDetails;
@@ -458,7 +464,8 @@ class SqliteSchemaManager extends AbstractSchemaManager
 
     private function parseColumnCollationFromSQL(string $column, string $sql): ?string
     {
-        $pattern = '{(?:\W' . preg_quote($column) . '\W|\W' . preg_quote($this->_platform->quoteSingleIdentifier($column))
+        $pattern = '{(?:\W' . preg_quote($column) . '\W|\W'
+            . preg_quote($this->_platform->quoteSingleIdentifier($column))
             . '\W)[^,(]+(?:\([^()]+\)[^,]*)?(?:(?:DEFAULT|CHECK)\s*(?:\(.*?\))?[^,]*)*COLLATE\s+["\']?([^\s,"\')]+)}is';
 
         if (preg_match($pattern, $sql, $match) !== 1) {
@@ -489,8 +496,8 @@ CREATE\sTABLE # Match "CREATE TABLE"
 
     private function parseColumnCommentFromSQL(string $column, string $sql): ?string
     {
-        $pattern = '{[\s(,](?:\W' . preg_quote($this->_platform->quoteSingleIdentifier($column)) . '\W|\W' . preg_quote($column)
-            . '\W)(?:\([^)]*?\)|[^,(])*?,?((?:(?!\n))(?:\s*--[^\n]*\n?)+)}i';
+        $pattern = '{[\s(,](?:\W' . preg_quote($this->_platform->quoteSingleIdentifier($column))
+            . '\W|\W' . preg_quote($column) . '\W)(?:\([^)]*?\)|[^,(])*?,?((?:(?!\n))(?:\s*--[^\n]*\n?)+)}i';
 
         if (preg_match($pattern, $sql, $match) !== 1) {
             return null;

src/Schema/Table.php

@@ -54,8 +54,14 @@ class Table extends AbstractAsset
      *
      * @throws SchemaException
      */
-    public function __construct($name, array $columns = [], array $indexes = [], array $fkConstraints = [], $idGeneratorType = 0, array $options = [])
-    {
+    public function __construct(
+        $name,
+        array $columns = [],
+        array $indexes = [],
+        array $fkConstraints = [],
+        $idGeneratorType = 0,
+        array $options = []
+    ) {
         if (strlen($name) === 0) {
             throw InvalidTableName::new($name);
         }
@@ -280,8 +286,14 @@ class Table extends AbstractAsset
      *
      * @throws SchemaException
      */
-    private function _createIndex(array $columnNames, $indexName, $isUnique, $isPrimary, array $flags = [], array $options = [])
-    {
+    private function _createIndex(
+        array $columnNames,
+        $indexName,
+        $isUnique,
+        $isPrimary,
+        array $flags = [],
+        array $options = []
+    ) {
         if (preg_match('(([^a-zA-Z0-9_]+))', $this->normalizeIdentifier($indexName)) === 1) {
             throw SchemaException::indexNameInvalid($indexName);
         }
@@ -361,10 +373,19 @@ class Table extends AbstractAsset
      *
      * @throws SchemaException
      */
-    public function addForeignKeyConstraint($foreignTable, array $localColumnNames, array $foreignColumnNames, array $options = [], $name = null)
-    {
+    public function addForeignKeyConstraint(
+        $foreignTable,
+        array $localColumnNames,
+        array $foreignColumnNames,
+        array $options = [],
+        $name = null
+    ) {
         if ($name === null) {
-            $name = $this->_generateIdentifierName(array_merge((array) $this->getName(), $localColumnNames), 'fk', $this->_getMaxIdentifierLength());
+            $name = $this->_generateIdentifierName(
+                array_merge((array) $this->getName(), $localColumnNames),
+                'fk',
+                $this->_getMaxIdentifierLength()
+            );
         }
 
         if ($foreignTable instanceof Table) {
@@ -487,9 +508,11 @@ class Table extends AbstractAsset
 
         $this->_fkConstraints[$name] = $constraint;
 
-        // add an explicit index on the foreign key columns. If there is already an index that fulfils this requirements drop the request.
-        // In the case of __construct calling this method during hydration from schema-details all the explicitly added indexes
-        // lead to duplicates. This creates computation overhead in this case, however no duplicate indexes are ever added (based on columns).
+        // Add an explicit index on the foreign key columns.
+        // If there is already an index that fulfils this requirements drop the request.
+        // In the case of __construct calling this method during hydration from schema-details
+        // all the explicitly added indexes lead to duplicates. This creates computation overhead in this case,
+        // however no duplicate indexes are ever added (based on columns).
         $indexName      = $this->_generateIdentifierName(
             array_merge([$this->getName()], $constraint->getColumns()),
             'idx',

src/Schema/Visitor/Graphviz.php

@@ -71,17 +71,23 @@ class Graphviz extends AbstractVisitor
         $label = '<<TABLE CELLSPACING="0" BORDER="1" ALIGN="LEFT">';
 
         // The title
-        $label .= '<TR><TD BORDER="1" COLSPAN="3" ALIGN="CENTER" BGCOLOR="#fcaf3e"><FONT COLOR="#2e3436" FACE="Helvetica" POINT-SIZE="12">' . $table->getName() . '</FONT></TD></TR>';
+        $label .= '<TR><TD BORDER="1" COLSPAN="3" ALIGN="CENTER" BGCOLOR="#fcaf3e">'
+            . '<FONT COLOR="#2e3436" FACE="Helvetica" POINT-SIZE="12">' . $table->getName() . '</FONT></TD></TR>';
 
         // The attributes block
         foreach ($table->getColumns() as $column) {
             $columnLabel = $column->getName();
 
-            $label .= '<TR>';
-            $label .= '<TD BORDER="0" ALIGN="LEFT" BGCOLOR="#eeeeec">';
-            $label .= '<FONT COLOR="#2e3436" FACE="Helvetica" POINT-SIZE="12">' . $columnLabel . '</FONT>';
-            $label .= '</TD><TD BORDER="0" ALIGN="LEFT" BGCOLOR="#eeeeec"><FONT COLOR="#2e3436" FACE="Helvetica" POINT-SIZE="10">' . strtolower($column->getType()->getName()) . '</FONT></TD>';
-            $label .= '<TD BORDER="0" ALIGN="RIGHT" BGCOLOR="#eeeeec" PORT="col' . $column->getName() . '">';
+            $label .= '<TR>'
+                . '<TD BORDER="0" ALIGN="LEFT" BGCOLOR="#eeeeec">'
+                . '<FONT COLOR="#2e3436" FACE="Helvetica" POINT-SIZE="12">' . $columnLabel . '</FONT>'
+                . '</TD>'
+                . '<TD BORDER="0" ALIGN="LEFT" BGCOLOR="#eeeeec">'
+                . '<FONT COLOR="#2e3436" FACE="Helvetica" POINT-SIZE="10">'
+                . strtolower($column->getType()->getName())
+                . '</FONT>'
+                . '</TD>'
+                . '<TD BORDER="0" ALIGN="RIGHT" BGCOLOR="#eeeeec" PORT="col' . $column->getName() . '">';
 
             $primaryKey = $table->getPrimaryKey();
 

src/Tools/Console/Command/ReservedWordsCommand.php

@@ -139,7 +139,10 @@ EOT
             $keywords[] = new $class();
         }
 
-        $output->write('Checking keyword violations for <comment>' . implode(', ', $keywordLists) . '</comment>...', true);
+        $output->write(
+            'Checking keyword violations for <comment>' . implode(', ', $keywordLists) . '</comment>...',
+            true
+        );
 
         $schema  = $conn->getSchemaManager()->createSchema();
         $visitor = new ReservedKeywordsValidator($keywords);
@@ -147,7 +150,12 @@ EOT
 
         $violations = $visitor->getViolations();
         if (count($violations) !== 0) {
-            $output->write('There are <error>' . count($violations) . '</error> reserved keyword violations in your database schema:', true);
+            $output->write(
+                'There are <error>' . count($violations) . '</error> reserved keyword violations'
+                . ' in your database schema:',
+                true
+            );
+
             foreach ($violations as $violation) {
                 $output->write('  - ' . $violation, true);
             }

src/Types/DateTimeType.php

@@ -61,7 +61,11 @@ class DateTimeType extends Type implements PhpDateTimeMappingType
         }
 
         if ($val === false) {
-            throw ConversionException::conversionFailedFormat($value, $this->getName(), $platform->getDateTimeFormatString());
+            throw ConversionException::conversionFailedFormat(
+                $value,
+                $this->getName(),
+                $platform->getDateTimeFormatString()
+            );
         }
 
         return $val;

src/Types/DateTimeTzType.php

@@ -53,7 +53,11 @@ class DateTimeTzType extends Type implements PhpDateTimeMappingType
             return $value->format($platform->getDateTimeTzFormatString());
         }
 
-        throw ConversionException::conversionFailedInvalidType($value, $this->getName(), ['null', 'DateTime']);
+        throw ConversionException::conversionFailedInvalidType(
+            $value,
+            $this->getName(),
+            ['null', 'DateTime']
+        );
     }
 
     /**
@@ -67,7 +71,11 @@ class DateTimeTzType extends Type implements PhpDateTimeMappingType
 
         $val = DateTime::createFromFormat($platform->getDateTimeTzFormatString(), $value);
         if ($val === false) {
-            throw ConversionException::conversionFailedFormat($value, $this->getName(), $platform->getDateTimeTzFormatString());
+            throw ConversionException::conversionFailedFormat(
+                $value,
+                $this->getName(),
+                $platform->getDateTimeTzFormatString()
+            );
         }
 
         return $val;

src/Types/DateType.php

@@ -54,7 +54,11 @@ class DateType extends Type
 
         $val = DateTime::createFromFormat('!' . $platform->getDateFormatString(), $value);
         if ($val === false) {
-            throw ConversionException::conversionFailedFormat($value, $this->getName(), $platform->getDateFormatString());
+            throw ConversionException::conversionFailedFormat(
+                $value,
+                $this->getName(),
+                $platform->getDateFormatString()
+            );
         }
 
         return $val;

src/Types/TimeType.php

@@ -54,7 +54,11 @@ class TimeType extends Type
 
         $val = DateTime::createFromFormat('!' . $platform->getTimeFormatString(), $value);
         if ($val === false) {
-            throw ConversionException::conversionFailedFormat($value, $this->getName(), $platform->getTimeFormatString());
+            throw ConversionException::conversionFailedFormat(
+                $value,
+                $this->getName(),
+                $platform->getTimeFormatString()
+            );
         }
 
         return $val;

tests/ConnectionTest.php

@@ -159,7 +159,12 @@ class ConnectionTest extends TestCase
     public function testDriverExceptionIsWrapped(callable $callback): void
     {
         $this->expectException(DBALException::class);
-        $this->expectExceptionMessage("An exception occurred while executing 'MUUHAAAAHAAAA':\n\nSQLSTATE[HY000]: General error: 1 near \"MUUHAAAAHAAAA\"");
+        $this->expectExceptionMessage(<<<EOF
+An exception occurred while executing 'MUUHAAAAHAAAA':
+
+SQLSTATE[HY000]: General error: 1 near "MUUHAAAAHAAAA"
+EOF
+        );
 
         $connection = DriverManager::getConnection([
             'driver' => 'pdo_sqlite',

tests/DBALExceptionTest.php

@@ -31,7 +31,8 @@ class DBALExceptionTest extends TestCase
         $exception = DBALException::invalidPlatformType(new stdClass());
 
         self::assertSame(
-            "Option 'platform' must be a subtype of 'Doctrine\DBAL\Platforms\AbstractPlatform', instance of 'stdClass' given",
+            "Option 'platform' must be a subtype of 'Doctrine\DBAL\Platforms\AbstractPlatform', "
+                . "instance of 'stdClass' given",
             $exception->getMessage()
         );
     }
@@ -41,7 +42,8 @@ class DBALExceptionTest extends TestCase
         $exception = DBALException::invalidPlatformType('some string');
 
         self::assertSame(
-            "Option 'platform' must be an object and subtype of 'Doctrine\DBAL\Platforms\AbstractPlatform'. Got 'string'",
+            "Option 'platform' must be an object and subtype of 'Doctrine\DBAL\Platforms\AbstractPlatform'. "
+                . "Got 'string'",
             $exception->getMessage()
         );
     }

tests/Driver/AbstractOracleDriver/EasyConnectStringTest.php

@@ -45,7 +45,8 @@ class EasyConnectStringTest extends TestCase
                     'service' => true,
                     'servicename' => 'BILLING',
                 ],
-                '(DESCRIPTION=(ADDRESS=(PROTOCOL=TCP)(HOST=localhost)(PORT=1521))(CONNECT_DATA=(SERVICE_NAME=BILLING)))',
+                '(DESCRIPTION=(ADDRESS=(PROTOCOL=TCP)(HOST=localhost)(PORT=1521))'
+                    . '(CONNECT_DATA=(SERVICE_NAME=BILLING)))',
             ],
             'advanced-params' => [
                 [
@@ -55,7 +56,8 @@ class EasyConnectStringTest extends TestCase
                     'instancename' => 'SALES',
                     'pooled' => true,
                 ],
-                '(DESCRIPTION=(ADDRESS=(PROTOCOL=TCP)(HOST=localhost)(PORT=41521))(CONNECT_DATA=(SID=XE)(INSTANCE_NAME=SALES)(SERVER=POOLED)))',
+                '(DESCRIPTION=(ADDRESS=(PROTOCOL=TCP)(HOST=localhost)(PORT=41521))'
+                    . '(CONNECT_DATA=(SID=XE)(INSTANCE_NAME=SALES)(SERVER=POOLED)))',
             ],
         ];
     }

tests/Driver/OCI8/ConvertPositionalToNamedPlaceholdersTest.php

@@ -23,8 +23,11 @@ class ConvertPositionalToNamedPlaceholdersTest extends TestCase
      *
      * @dataProvider positionalToNamedPlaceholdersProvider
      */
-    public function testConvertPositionalToNamedParameters(string $inputSQL, string $expectedOutputSQL, array $expectedOutputParamsMap): void
-    {
+    public function testConvertPositionalToNamedParameters(
+        string $inputSQL,
+        string $expectedOutputSQL,
+        array $expectedOutputParamsMap
+    ): void {
         [$statement, $params] = ($this->convertPositionalToNamedPlaceholders)($inputSQL);
 
         self::assertEquals($expectedOutputSQL, $statement);

tests/Functional/BlobTest.php

@@ -140,7 +140,9 @@ class BlobTest extends FunctionalTestCase
             self::markTestIncomplete('The oci8 driver does not support stream resources as parameters');
         }
 
-        $stmt = $this->connection->prepare("INSERT INTO blob_table(id, clobcolumn, blobcolumn) VALUES (1, 'ignored', ?)");
+        $stmt = $this->connection->prepare(
+            "INSERT INTO blob_table(id, clobcolumn, blobcolumn) VALUES (1, 'ignored', ?)"
+        );
 
         $stream = null;
         $stmt->bindParam(1, $stream, ParameterType::LARGE_OBJECT);

tests/Functional/ConnectionTest.php

@@ -67,8 +67,6 @@ class ConnectionTest extends FunctionalTestCase
                 self::assertEquals(2, $this->connection->getTransactionNestingLevel());
 
                 throw new Exception();
-
-                $this->connection->commit(); // never reached
             } catch (Throwable $e) {
                 $this->connection->rollBack();
                 self::assertEquals(1, $this->connection->getTransactionNestingLevel());
@@ -140,8 +138,6 @@ class ConnectionTest extends FunctionalTestCase
                 self::assertEquals(2, $this->connection->getTransactionNestingLevel());
 
                 throw new Exception();
-
-                $this->connection->commit(); // never reached
             } catch (Throwable $e) {
                 $this->connection->rollBack();
                 self::assertEquals(1, $this->connection->getTransactionNestingLevel());
@@ -151,7 +147,7 @@ class ConnectionTest extends FunctionalTestCase
             self::assertFalse($this->connection->isRollbackOnly());
             try {
                 $this->connection->setNestTransactionsWithSavepoints(false);
-                self::fail('Should not be able to disable savepoints in usage for nested transactions inside an open transaction.');
+                self::fail('Should not be able to disable savepoints in usage inside a nested open transaction.');
             } catch (ConnectionException $e) {
                 self::assertTrue($this->connection->getNestTransactionsWithSavepoints());
             }
@@ -228,8 +224,6 @@ class ConnectionTest extends FunctionalTestCase
             self::assertEquals(1, $this->connection->getTransactionNestingLevel());
 
             throw new Exception();
-
-            $this->connection->commit(); // never reached
         } catch (Throwable $e) {
             self::assertEquals(1, $this->connection->getTransactionNestingLevel());
             $this->connection->rollBack();

tests/Functional/DataAccessTest.php

@@ -43,7 +43,12 @@ class DataAccessTest extends FunctionalTestCase
         $sm = $this->connection->getSchemaManager();
         $sm->createTable($table);
 
-        $this->connection->insert('fetch_table', ['test_int' => 1, 'test_string' => 'foo', 'test_datetime' => '2010-01-01 10:10:10']);
+        $this->connection->insert('fetch_table', [
+            'test_int' => 1,
+            'test_string' => 'foo',
+            'test_datetime' => '2010-01-01 10:10:10',
+        ]);
+
         self::$generated = true;
     }
 
@@ -186,7 +191,7 @@ class DataAccessTest extends FunctionalTestCase
     public function testFetchNoResult(): void
     {
         self::assertFalse(
-            $this->connection->executeQuery('SELECT test_int FROM fetch_table WHERE test_int = ?', [-1])->fetchAssociative()
+            $this->connection->fetchAssociative('SELECT test_int FROM fetch_table WHERE test_int = ?', [-1])
         );
     }
 
@@ -329,7 +334,11 @@ class DataAccessTest extends FunctionalTestCase
     public function testNativeArrayListSupport(): void
     {
         for ($i = 100; $i < 110; $i++) {
-            $this->connection->insert('fetch_table', ['test_int' => $i, 'test_string' => 'foo' . $i, 'test_datetime' => '2010-01-01 10:10:10']);
+            $this->connection->insert('fetch_table', [
+                'test_int' => $i,
+                'test_string' => 'foo' . $i,
+                'test_datetime' => '2010-01-01 10:10:10',
+            ]);
         }
 
         $result = $this->connection->executeQuery(
@@ -440,22 +449,22 @@ class DataAccessTest extends FunctionalTestCase
         $row = $this->connection->fetchAssociative($sql);
         $row = array_change_key_case($row, CASE_LOWER);
 
-        self::assertEquals('2010-01-01 10:10:11', date('Y-m-d H:i:s', strtotime($row['add_seconds'])), 'Adding second should end up on 2010-01-01 10:10:11');
-        self::assertEquals('2010-01-01 10:10:09', date('Y-m-d H:i:s', strtotime($row['sub_seconds'])), 'Subtracting second should end up on 2010-01-01 10:10:09');
-        self::assertEquals('2010-01-01 10:15:10', date('Y-m-d H:i:s', strtotime($row['add_minutes'])), 'Adding minutes should end up on 2010-01-01 10:15:10');
-        self::assertEquals('2010-01-01 10:05:10', date('Y-m-d H:i:s', strtotime($row['sub_minutes'])), 'Subtracting minutes should end up on 2010-01-01 10:05:10');
-        self::assertEquals('2010-01-01 13:10', date('Y-m-d H:i', strtotime($row['add_hour'])), 'Adding date should end up on 2010-01-01 13:10');
-        self::assertEquals('2010-01-01 07:10', date('Y-m-d H:i', strtotime($row['sub_hour'])), 'Subtracting date should end up on 2010-01-01 07:10');
-        self::assertEquals('2010-01-11', date('Y-m-d', strtotime($row['add_days'])), 'Adding date should end up on 2010-01-11');
-        self::assertEquals('2009-12-22', date('Y-m-d', strtotime($row['sub_days'])), 'Subtracting date should end up on 2009-12-22');
-        self::assertEquals('2010-01-08', date('Y-m-d', strtotime($row['add_weeks'])), 'Adding week should end up on 2010-01-08');
-        self::assertEquals('2009-12-25', date('Y-m-d', strtotime($row['sub_weeks'])), 'Subtracting week should end up on 2009-12-25');
-        self::assertEquals('2010-03-01', date('Y-m-d', strtotime($row['add_month'])), 'Adding month should end up on 2010-03-01');
-        self::assertEquals('2009-11-01', date('Y-m-d', strtotime($row['sub_month'])), 'Subtracting month should end up on 2009-11-01');
-        self::assertEquals('2010-10-01', date('Y-m-d', strtotime($row['add_quarters'])), 'Adding quarters should end up on 2010-04-01');
-        self::assertEquals('2009-04-01', date('Y-m-d', strtotime($row['sub_quarters'])), 'Subtracting quarters should end up on 2009-10-01');
-        self::assertEquals('2016-01-01', date('Y-m-d', strtotime($row['add_years'])), 'Adding years should end up on 2016-01-01');
-        self::assertEquals('2004-01-01', date('Y-m-d', strtotime($row['sub_years'])), 'Subtracting years should end up on 2004-01-01');
+        self::assertEquals('2010-01-01 10:10:11', date('Y-m-d H:i:s', strtotime($row['add_seconds'])));
+        self::assertEquals('2010-01-01 10:10:09', date('Y-m-d H:i:s', strtotime($row['sub_seconds'])));
+        self::assertEquals('2010-01-01 10:15:10', date('Y-m-d H:i:s', strtotime($row['add_minutes'])));
+        self::assertEquals('2010-01-01 10:05:10', date('Y-m-d H:i:s', strtotime($row['sub_minutes'])));
+        self::assertEquals('2010-01-01 13:10', date('Y-m-d H:i', strtotime($row['add_hour'])));
+        self::assertEquals('2010-01-01 07:10', date('Y-m-d H:i', strtotime($row['sub_hour'])));
+        self::assertEquals('2010-01-11', date('Y-m-d', strtotime($row['add_days'])));
+        self::assertEquals('2009-12-22', date('Y-m-d', strtotime($row['sub_days'])));
+        self::assertEquals('2010-01-08', date('Y-m-d', strtotime($row['add_weeks'])));
+        self::assertEquals('2009-12-25', date('Y-m-d', strtotime($row['sub_weeks'])));
+        self::assertEquals('2010-03-01', date('Y-m-d', strtotime($row['add_month'])));
+        self::assertEquals('2009-11-01', date('Y-m-d', strtotime($row['sub_month'])));
+        self::assertEquals('2010-10-01', date('Y-m-d', strtotime($row['add_quarters'])));
+        self::assertEquals('2009-04-01', date('Y-m-d', strtotime($row['sub_quarters'])));
+        self::assertEquals('2016-01-01', date('Y-m-d', strtotime($row['add_years'])));
+        self::assertEquals('2004-01-01', date('Y-m-d', strtotime($row['sub_years'])));
     }
 
     public function testSqliteDateArithmeticWithDynamicInterval(): void

tests/Functional/Driver/Mysqli/ConnectionTest.php

@@ -28,11 +28,6 @@ class ConnectionTest extends FunctionalTestCase
         self::markTestSkipped('MySQLi only test.');
     }
 
-    protected function tearDown(): void
-    {
-        parent::tearDown();
-    }
-
     public function testSupportedDriverOptions(): void
     {
         $this->expectNotToPerformAssertions();

tests/Functional/Driver/PDO/PgSQL/DriverTest.php

@@ -31,8 +31,11 @@ class DriverTest extends AbstractDriverTest
     /**
      * @dataProvider getDatabaseParameter
      */
-    public function testDatabaseParameters(?string $databaseName, ?string $defaultDatabaseName, ?string $expectedDatabaseName): void
-    {
+    public function testDatabaseParameters(
+        ?string $databaseName,
+        ?string $defaultDatabaseName,
+        ?string $expectedDatabaseName
+    ): void {
         $params                   = $this->connection->getParams();
         $params['dbname']         = $databaseName;
         $params['default_dbname'] = $defaultDatabaseName;

tests/Functional/ModifyLimitQueryTest.php

@@ -72,7 +72,11 @@ class ModifyLimitQueryTest extends FunctionalTestCase
         $this->connection->insert('modify_limit_table2', ['test_int' => 2]);
         $this->connection->insert('modify_limit_table2', ['test_int' => 2]);
 
-        $sql = 'SELECT modify_limit_table.test_int FROM modify_limit_table INNER JOIN modify_limit_table2 ON modify_limit_table.test_int = modify_limit_table2.test_int ORDER BY modify_limit_table.test_int DESC';
+        $sql = 'SELECT modify_limit_table.test_int'
+            . ' FROM modify_limit_table'
+            . ' INNER JOIN modify_limit_table2'
+            . ' ON modify_limit_table.test_int = modify_limit_table2.test_int'
+            . ' ORDER BY modify_limit_table.test_int DESC';
 
         $this->assertLimitResult([2, 2, 1, 1, 1], $sql, 10, 0);
         $this->assertLimitResult([1, 1, 1], $sql, 3, 2);
@@ -120,7 +124,8 @@ class ModifyLimitQueryTest extends FunctionalTestCase
         $this->connection->insert('modify_limit_table', ['test_int' => 3]);
         $this->connection->insert('modify_limit_table', ['test_int' => 4]);
 
-        $sql = 'SELECT modify_limit_table.*, (SELECT COUNT(*) FROM modify_limit_table) AS cnt FROM modify_limit_table ORDER BY test_int DESC';
+        $sql = 'SELECT modify_limit_table.*, (SELECT COUNT(*) FROM modify_limit_table) AS cnt'
+            . ' FROM modify_limit_table ORDER BY test_int DESC';
 
         $this->assertLimitResult([4, 3, 2, 1], $sql, 10, 0);
         $this->assertLimitResult([4, 3], $sql, 2, 0);
@@ -174,8 +179,13 @@ SQL;
     /**
      * @param array<int, int> $expectedResults
      */
-    private function assertLimitResult(array $expectedResults, string $sql, ?int $limit, int $offset, bool $deterministic = true): void
-    {
+    private function assertLimitResult(
+        array $expectedResults,
+        string $sql,
+        ?int $limit,
+        int $offset,
+        bool $deterministic = true
+    ): void {
         $p    = $this->connection->getDatabasePlatform();
         $data = [];
         foreach ($this->connection->fetchAllAssociative($p->modifyLimitQuery($sql, $limit, $offset)) as $row) {

tests/Functional/PortabilityTest.php

@@ -34,8 +34,17 @@ class PortabilityTest extends FunctionalTestCase
             $sm = $this->connection->getSchemaManager();
             $sm->createTable($table);
 
-            $this->connection->insert('portability_table', ['Test_Int' => 1, 'Test_String' => 'foo', 'Test_Null' => '']);
-            $this->connection->insert('portability_table', ['Test_Int' => 2, 'Test_String' => 'foo  ', 'Test_Null' => null]);
+            $this->connection->insert('portability_table', [
+                'Test_Int' => 1,
+                'Test_String' => 'foo',
+                'Test_Null' => '',
+            ]);
+
+            $this->connection->insert('portability_table', [
+                'Test_Int' => 2,
+                'Test_String' => 'foo  ',
+                'Test_Null' => null,
+            ]);
         } catch (Throwable $e) {
         }
     }
@@ -105,7 +114,7 @@ class PortabilityTest extends FunctionalTestCase
         ));
 
         self::assertArrayHasKey('test_string', $row, 'Case should be lowered.');
-        self::assertEquals(3, strlen($row['test_string']), 'test_string should be rtrimed to length of three for CHAR(32) column.');
+        self::assertEquals(3, strlen($row['test_string']));
         self::assertNull($row['test_null']);
         self::assertArrayNotHasKey(0, $row, 'The row should not contain numerical keys.');
     }

tests/Functional/ResultCacheTest.php

@@ -19,7 +19,11 @@ use const CASE_LOWER;
 class ResultCacheTest extends FunctionalTestCase
 {
     /** @var list<array{test_int: int, test_string: string}> */
-    private $expectedResult = [['test_int' => 100, 'test_string' => 'foo'], ['test_int' => 200, 'test_string' => 'bar'], ['test_int' => 300, 'test_string' => 'baz']];
+    private $expectedResult = [
+        ['test_int' => 100, 'test_string' => 'foo'],
+        ['test_int' => 200, 'test_string' => 'bar'],
+        ['test_int' => 300, 'test_string' => 'baz'],
+    ];
 
     /** @var DebugStack */
     private $sqlLogger;
@@ -101,7 +105,12 @@ class ResultCacheTest extends FunctionalTestCase
             $numExpectedResult[] = array_values($v);
         }
 
-        $stmt = $this->connection->executeQuery('SELECT * FROM caching ORDER BY test_int ASC', [], [], new QueryCacheProfile(0, 'testcachekey'));
+        $stmt = $this->connection->executeQuery(
+            'SELECT * FROM caching ORDER BY test_int ASC',
+            [],
+            [],
+            new QueryCacheProfile(0, 'testcachekey')
+        );
 
         $data = $this->hydrateViaFetchAll($stmt, static function (Result $result) {
             return $result->fetchAllAssociative();
@@ -109,7 +118,12 @@ class ResultCacheTest extends FunctionalTestCase
 
         self::assertEquals($this->expectedResult, $data);
 
-        $stmt = $this->connection->executeQuery('SELECT * FROM caching ORDER BY test_int ASC', [], [], new QueryCacheProfile(0, 'testcachekey'));
+        $stmt = $this->connection->executeQuery(
+            'SELECT * FROM caching ORDER BY test_int ASC',
+            [],
+            [],
+            new QueryCacheProfile(0, 'testcachekey')
+        );
 
         $data = $this->hydrateViaFetchAll($stmt, static function (Result $result) {
             return $result->fetchAllNumeric();
@@ -123,10 +137,22 @@ class ResultCacheTest extends FunctionalTestCase
      */
     public function testFetchViaIteration(callable $fetch, callable $fetchAll): void
     {
-        $stmt = $this->connection->executeQuery('SELECT * FROM caching ORDER BY test_int ASC', [], [], new QueryCacheProfile(0, 'testcachekey'));
+        $stmt = $this->connection->executeQuery(
+            'SELECT * FROM caching ORDER BY test_int ASC',
+            [],
+            [],
+            new QueryCacheProfile(0, 'testcachekey')
+        );
+
         $data = $this->hydrateViaFetchAll($stmt, $fetchAll);
 
-        $stmt         = $this->connection->executeQuery('SELECT * FROM caching ORDER BY test_int ASC', [], [], new QueryCacheProfile(0, 'testcachekey'));
+        $stmt = $this->connection->executeQuery(
+            'SELECT * FROM caching ORDER BY test_int ASC',
+            [],
+            [],
+            new QueryCacheProfile(0, 'testcachekey')
+        );
+
         $dataIterator = $this->hydrateViaIteration($stmt, $fetch);
 
         self::assertEquals($data, $dataIterator);
@@ -134,13 +160,23 @@ class ResultCacheTest extends FunctionalTestCase
 
     public function testFetchAndFinishSavesCache(): void
     {
-        $result = $this->connection->executeQuery('SELECT * FROM caching ORDER BY test_int ASC', [], [], new QueryCacheProfile(0, 'testcachekey'));
+        $result = $this->connection->executeQuery(
+            'SELECT * FROM caching ORDER BY test_int ASC',
+            [],
+            [],
+            new QueryCacheProfile(0, 'testcachekey')
+        );
 
         while (($row = $result->fetchAssociative()) !== false) {
             $data[] = $row;
         }
 
-        $result = $this->connection->executeQuery('SELECT * FROM caching ORDER BY test_int ASC', [], [], new QueryCacheProfile(0, 'testcachekey'));
+        $result = $this->connection->executeQuery(
+            'SELECT * FROM caching ORDER BY test_int ASC',
+            [],
+            [],
+            new QueryCacheProfile(0, 'testcachekey')
+        );
 
         while (($row = $result->fetchNumeric()) !== false) {
             $data[] = $row;
@@ -151,11 +187,21 @@ class ResultCacheTest extends FunctionalTestCase
 
     public function testDontFinishNoCache(): void
     {
-        $result = $this->connection->executeQuery('SELECT * FROM caching ORDER BY test_int ASC', [], [], new QueryCacheProfile(0, 'testcachekey'));
+        $result = $this->connection->executeQuery(
+            'SELECT * FROM caching ORDER BY test_int ASC',
+            [],
+            [],
+            new QueryCacheProfile(0, 'testcachekey')
+        );
 
         $result->fetchAssociative();
 
-        $result = $this->connection->executeQuery('SELECT * FROM caching ORDER BY test_int ASC', [], [], new QueryCacheProfile(0, 'testcachekey'));
+        $result = $this->connection->executeQuery(
+            'SELECT * FROM caching ORDER BY test_int ASC',
+            [],
+            [],
+            new QueryCacheProfile(0, 'testcachekey')
+        );
 
         $this->hydrateViaIteration($result, static function (Result $result) {
             return $result->fetchNumeric();
@@ -167,7 +213,8 @@ class ResultCacheTest extends FunctionalTestCase
     public function testFetchAllSavesCache(): void
     {
         $layerCache = new ArrayCache();
-        $result     = $this->connection->executeQuery(
+
+        $result = $this->connection->executeQuery(
             'SELECT * FROM caching WHERE test_int > 500',
             [],
             [],
@@ -198,13 +245,23 @@ class ResultCacheTest extends FunctionalTestCase
      */
     private function assertCacheNonCacheSelectSameFetchModeAreEqual(array $expectedResult, callable $fetchMode): void
     {
-        $stmt = $this->connection->executeQuery('SELECT * FROM caching ORDER BY test_int ASC', [], [], new QueryCacheProfile(0, 'testcachekey'));
+        $stmt = $this->connection->executeQuery(
+            'SELECT * FROM caching ORDER BY test_int ASC',
+            [],
+            [],
+            new QueryCacheProfile(0, 'testcachekey')
+        );
 
         self::assertEquals(2, $stmt->columnCount());
         $data = $this->hydrateViaIteration($stmt, $fetchMode);
         self::assertEquals($expectedResult, $data);
 
-        $stmt = $this->connection->executeQuery('SELECT * FROM caching ORDER BY test_int ASC', [], [], new QueryCacheProfile(0, 'testcachekey'));
+        $stmt = $this->connection->executeQuery(
+            'SELECT * FROM caching ORDER BY test_int ASC',
+            [],
+            [],
+            new QueryCacheProfile(0, 'testcachekey')
+        );
 
         self::assertEquals(2, $stmt->columnCount());
         $data = $this->hydrateViaIteration($stmt, $fetchMode);
@@ -214,12 +271,24 @@ class ResultCacheTest extends FunctionalTestCase
 
     public function testEmptyResultCache(): void
     {
-        $stmt = $this->connection->executeQuery('SELECT * FROM caching WHERE test_int > 500', [], [], new QueryCacheProfile(0, 'emptycachekey'));
+        $stmt = $this->connection->executeQuery(
+            'SELECT * FROM caching WHERE test_int > 500',
+            [],
+            [],
+            new QueryCacheProfile(10, 'emptycachekey')
+        );
+
         $this->hydrateViaIteration($stmt, static function (Result $result) {
             return $result->fetchAssociative();
         });
 
-        $stmt = $this->connection->executeQuery('SELECT * FROM caching WHERE test_int > 500', [], [], new QueryCacheProfile(0, 'emptycachekey'));
+        $stmt = $this->connection->executeQuery(
+            'SELECT * FROM caching WHERE test_int > 500',
+            [],
+            [],
+            new QueryCacheProfile(10, 'emptycachekey')
+        );
+
         $this->hydrateViaIteration($stmt, static function (Result $result) {
             return $result->fetchAssociative();
         });
@@ -229,14 +298,26 @@ class ResultCacheTest extends FunctionalTestCase
 
     public function testChangeCacheImpl(): void
     {
-        $stmt = $this->connection->executeQuery('SELECT * FROM caching WHERE test_int > 500', [], [], new QueryCacheProfile(0, 'emptycachekey'));
+        $stmt = $this->connection->executeQuery(
+            'SELECT * FROM caching WHERE test_int > 500',
+            [],
+            [],
+            new QueryCacheProfile(10, 'emptycachekey')
+        );
+
         $this->hydrateViaIteration($stmt, static function (Result $result) {
             return $result->fetchAssociative();
         });
 
         $secondCache = new ArrayCache();
 
-        $stmt = $this->connection->executeQuery('SELECT * FROM caching WHERE test_int > 500', [], [], new QueryCacheProfile(0, 'emptycachekey', $secondCache));
+        $stmt = $this->connection->executeQuery(
+            'SELECT * FROM caching WHERE test_int > 500',
+            [],
+            [],
+            new QueryCacheProfile(10, 'emptycachekey', $secondCache)
+        );
+
         $this->hydrateViaIteration($stmt, static function (Result $result) {
             return $result->fetchAssociative();
         });

tests/Functional/Schema/MySqlSchemaManagerTest.php

@@ -246,7 +246,10 @@ class MySqlSchemaManagerTest extends SchemaManagerFunctionalTestCase
     public function testColumnCharsetChange(): void
     {
         $table = new Table('test_column_charset_change');
-        $table->addColumn('col_string', 'string')->setLength(100)->setNotnull(true)->setPlatformOption('charset', 'utf8');
+        $table->addColumn('col_string', 'string')
+            ->setLength(100)
+            ->setNotnull(true)
+            ->setPlatformOption('charset', 'utf8');
 
         $diffTable = clone $table;
         $diffTable->getColumn('col_string')->setPlatformOption('charset', 'ascii');
@@ -255,7 +258,11 @@ class MySqlSchemaManagerTest extends SchemaManagerFunctionalTestCase
         $toSchema   = new Schema([$diffTable]);
 
         $diff = $fromSchema->getMigrateToSql($toSchema, $this->connection->getDatabasePlatform());
-        self::assertContains('ALTER TABLE test_column_charset_change CHANGE col_string col_string VARCHAR(100) CHARACTER SET ascii NOT NULL', $diff);
+        self::assertContains(
+            'ALTER TABLE test_column_charset_change CHANGE col_string'
+                . ' col_string VARCHAR(100) CHARACTER SET ascii NOT NULL',
+            $diff
+        );
     }
 
     public function testColumnCollation(): void

tests/Functional/Schema/OracleSchemaManagerTest.php

@@ -243,7 +243,10 @@ class OracleSchemaManagerTest extends SchemaManagerFunctionalTestCase
 
         // Adding a primary key on already indexed columns
         // Oracle will reuse the unique index, which cause a constraint name differing from the index name
-        $this->schemaManager->createConstraint(new Schema\Index('id_pk_id_index', ['id'], true, true), 'list_table_indexes_pk_id_test');
+        $this->schemaManager->createConstraint(
+            new Schema\Index('id_pk_id_index', ['id'], true, true),
+            'list_table_indexes_pk_id_test'
+        );
 
         $tableIndexes = $this->schemaManager->listTableIndexes('list_table_indexes_pk_id_test');
 
@@ -271,6 +274,8 @@ class OracleSchemaManagerTest extends SchemaManagerFunctionalTestCase
 
     public function testCreateAndListSequences(): void
     {
-        self::markTestSkipped("Skipped for uppercase letters are contained in sequences' names. Fix the schema manager in 3.0.");
+        self::markTestSkipped(
+            "Skipped for uppercase letters are contained in sequences' names. Fix the schema manager in 3.0."
+        );
     }
 }

tests/Functional/Schema/PostgreSqlSchemaManagerTest.php

@@ -119,12 +119,15 @@ class PostgreSqlSchemaManagerTest extends SchemaManagerFunctionalTestCase
         self::assertTrue($tableFrom->getColumn('id')->getAutoincrement());
 
         $tableTo = new Table('autoinc_table_drop');
-        $column  = $tableTo->addColumn('id', 'integer');
+        $tableTo->addColumn('id', 'integer');
 
         $c    = new Comparator();
         $diff = $c->diffTable($tableFrom, $tableTo);
-        self::assertInstanceOf(TableDiff::class, $diff, 'There should be a difference and not false being returned from the table comparison');
-        self::assertEquals(['ALTER TABLE autoinc_table_drop ALTER id DROP DEFAULT'], $this->connection->getDatabasePlatform()->getAlterTableSQL($diff));
+        self::assertInstanceOf(TableDiff::class, $diff);
+        self::assertEquals(
+            ['ALTER TABLE autoinc_table_drop ALTER id DROP DEFAULT'],
+            $this->connection->getDatabasePlatform()->getAlterTableSQL($diff)
+        );
 
         $this->schemaManager->alterTable($diff);
         $tableFinal = $this->schemaManager->listTableDetails('autoinc_table_drop');
@@ -164,10 +167,12 @@ class PostgreSqlSchemaManagerTest extends SchemaManagerFunctionalTestCase
 
     public function testReturnQuotedAssets(): void
     {
-        $sql = 'create table dbal91_something ( id integer  CONSTRAINT id_something PRIMARY KEY NOT NULL  ,"table"   integer );';
+        $sql = 'create table dbal91_something'
+            . ' (id integer CONSTRAINT id_something PRIMARY KEY NOT NULL, "table" integer)';
         $this->connection->executeStatement($sql);
 
-        $sql = 'ALTER TABLE dbal91_something ADD CONSTRAINT something_input FOREIGN KEY( "table" ) REFERENCES dbal91_something ON UPDATE CASCADE;';
+        $sql = 'ALTER TABLE dbal91_something ADD CONSTRAINT something_input'
+            . ' FOREIGN KEY( "table" ) REFERENCES dbal91_something ON UPDATE CASCADE;';
         $this->connection->executeStatement($sql);
 
         $table = $this->schemaManager->listTableDetails('dbal91_something');
@@ -231,13 +236,14 @@ class PostgreSqlSchemaManagerTest extends SchemaManagerFunctionalTestCase
         }
 
         $fkeys = $this->schemaManager->listTableForeignKeys('test_create_fk1');
-        self::assertEquals(count($foreignKeys), count($fkeys), "Table 'test_create_fk1' has to have " . count($foreignKeys) . ' foreign keys.');
+        self::assertEquals(count($foreignKeys), count($fkeys));
+
         for ($i = 0; $i < count($fkeys); $i++) {
             self::assertEquals(['foreign_key_test' . $i], array_map('strtolower', $fkeys[$i]->getLocalColumns()));
             self::assertEquals(['id'], array_map('strtolower', $fkeys[$i]->getForeignColumns()));
             self::assertEquals('test_create_fk2', strtolower($fkeys[0]->getForeignTableName()));
             if ($foreignKeys[$i]->getOption('onDelete') === 'NO ACTION') {
-                self::assertFalse($fkeys[$i]->hasOption('onDelete'), 'Unexpected option: ' . $fkeys[$i]->getOption('onDelete'));
+                self::assertFalse($fkeys[$i]->hasOption('onDelete'));
             } else {
                 self::assertEquals($foreignKeys[$i]->getOption('onDelete'), $fkeys[$i]->getOption('onDelete'));
             }
@@ -474,8 +480,11 @@ class PostgreSqlSchemaManagerTest extends SchemaManagerFunctionalTestCase
 
         $c    = new Comparator();
         $diff = $c->diffTable($tableFrom, $tableTo);
-        self::assertInstanceOf(TableDiff::class, $diff, 'There should be a difference and not false being returned from the table comparison');
-        self::assertSame(['ALTER TABLE autoinc_type_modification ALTER id TYPE ' . $expected], $this->connection->getDatabasePlatform()->getAlterTableSQL($diff));
+        self::assertInstanceOf(TableDiff::class, $diff);
+        self::assertSame(
+            ['ALTER TABLE autoinc_type_modification ALTER id TYPE ' . $expected],
+            $this->connection->getDatabasePlatform()->getAlterTableSQL($diff)
+        );
 
         $this->schemaManager->alterTable($diff);
         $tableFinal = $this->schemaManager->listTableDetails('autoinc_type_modification');

tests/Functional/Schema/SQLServerSchemaManagerTest.php

@@ -40,7 +40,8 @@ class SQLServerSchemaManagerTest extends SchemaManagerFunctionalTestCase
         $this->schemaManager->dropAndCreateTable($table);
         $columns = $this->schemaManager->listTableColumns($tableName);
 
-        self::assertTrue($columns[$columnName]->hasPlatformOption('collation')); // SQL Server should report a default collation on the column
+        // SQL Server should report a default collation on the column
+        self::assertTrue($columns[$columnName]->hasPlatformOption('collation'));
 
         $column->setPlatformOption('collation', $collation = 'Icelandic_CS_AS');
 
@@ -89,9 +90,15 @@ class SQLServerSchemaManagerTest extends SchemaManagerFunctionalTestCase
                 ),
                 'df_string_3' => new ColumnDiff(
                     'df_string_3',
-                    new Column('df_string_3', Type::getType('string'), ['length' => 50, 'default' => 'another default value']),
+                    new Column('df_string_3', Type::getType('string'), [
+                        'length' => 50,
+                        'default' => 'another default value',
+                    ]),
                     ['length'],
-                    new Column('df_string_3', Type::getType('string'), ['length' => 50, 'default' => 'another default value'])
+                    new Column('df_string_3', Type::getType('string'), [
+                        'length' => 50,
+                        'default' => 'another default value',
+                    ])
                 ),
                 'df_boolean' => new ColumnDiff(
                     'df_boolean',
@@ -163,7 +170,11 @@ class SQLServerSchemaManagerTest extends SchemaManagerFunctionalTestCase
         $table->addColumn('comment_float_0', 'integer', ['comment' => 0.0]);
         $table->addColumn('comment_string_0', 'integer', ['comment' => '0']);
         $table->addColumn('comment', 'integer', ['comment' => 'Doctrine 0wnz you!']);
-        $table->addColumn('`comment_quoted`', 'integer', ['comment' => 'Doctrine 0wnz comments for explicitly quoted columns!']);
+        $table->addColumn(
+            '`comment_quoted`',
+            'integer',
+            ['comment' => 'Doctrine 0wnz comments for explicitly quoted columns!']
+        );
         $table->addColumn('create', 'integer', ['comment' => 'Doctrine 0wnz comments for reserved keyword columns!']);
         $table->addColumn('commented_type', 'object');
         $table->addColumn('commented_type_with_comment', 'array', ['comment' => 'Doctrine array type.']);
@@ -181,27 +192,55 @@ class SQLServerSchemaManagerTest extends SchemaManagerFunctionalTestCase
         self::assertEquals('0', $columns['comment_float_0']->getComment());
         self::assertEquals('0', $columns['comment_string_0']->getComment());
         self::assertEquals('Doctrine 0wnz you!', $columns['comment']->getComment());
-        self::assertEquals('Doctrine 0wnz comments for explicitly quoted columns!', $columns['comment_quoted']->getComment());
+        self::assertEquals(
+            'Doctrine 0wnz comments for explicitly quoted columns!',
+            $columns['comment_quoted']->getComment()
+        );
         self::assertEquals('Doctrine 0wnz comments for reserved keyword columns!', $columns['[create]']->getComment());
         self::assertNull($columns['commented_type']->getComment());
         self::assertEquals('Doctrine array type.', $columns['commented_type_with_comment']->getComment());
 
-        $tableDiff                                                    = new TableDiff('sqlsrv_column_comment');
-        $tableDiff->fromTable                                         = $table;
-        $tableDiff->addedColumns['added_comment_none']                = new Column('added_comment_none', Type::getType('integer'));
-        $tableDiff->addedColumns['added_comment_null']                = new Column('added_comment_null', Type::getType('integer'), ['comment' => null]);
-        $tableDiff->addedColumns['added_comment_false']               = new Column('added_comment_false', Type::getType('integer'), ['comment' => false]);
-        $tableDiff->addedColumns['added_comment_empty_string']        = new Column('added_comment_empty_string', Type::getType('integer'), ['comment' => '']);
-        $tableDiff->addedColumns['added_comment_integer_0']           = new Column('added_comment_integer_0', Type::getType('integer'), ['comment' => 0]);
-        $tableDiff->addedColumns['added_comment_float_0']             = new Column('added_comment_float_0', Type::getType('integer'), ['comment' => 0.0]);
-        $tableDiff->addedColumns['added_comment_string_0']            = new Column('added_comment_string_0', Type::getType('integer'), ['comment' => '0']);
-        $tableDiff->addedColumns['added_comment']                     = new Column('added_comment', Type::getType('integer'), ['comment' => 'Doctrine']);
-        $tableDiff->addedColumns['`added_comment_quoted`']            = new Column('`added_comment_quoted`', Type::getType('integer'), ['comment' => 'rulez']);
-        $tableDiff->addedColumns['select']                            = new Column('select', Type::getType('integer'), ['comment' => '666']);
-        $tableDiff->addedColumns['added_commented_type']              = new Column('added_commented_type', Type::getType('object'));
-        $tableDiff->addedColumns['added_commented_type_with_comment'] = new Column('added_commented_type_with_comment', Type::getType('array'), ['comment' => '666']);
-
-        $tableDiff->renamedColumns['comment_float_0'] = new Column('comment_double_0', Type::getType('decimal'), ['comment' => 'Double for real!']);
+        $tableDiff            = new TableDiff('sqlsrv_column_comment');
+        $tableDiff->fromTable = $table;
+
+        $tableDiff->addedColumns['added_comment_none']
+            = new Column('added_comment_none', Type::getType('integer'));
+
+        $tableDiff->addedColumns['added_comment_null']
+            = new Column('added_comment_null', Type::getType('integer'), ['comment' => null]);
+
+        $tableDiff->addedColumns['added_comment_false']
+            = new Column('added_comment_false', Type::getType('integer'), ['comment' => false]);
+
+        $tableDiff->addedColumns['added_comment_empty_string']
+            = new Column('added_comment_empty_string', Type::getType('integer'), ['comment' => '']);
+
+        $tableDiff->addedColumns['added_comment_integer_0']
+            = new Column('added_comment_integer_0', Type::getType('integer'), ['comment' => 0]);
+
+        $tableDiff->addedColumns['added_comment_float_0']
+            = new Column('added_comment_float_0', Type::getType('integer'), ['comment' => 0.0]);
+
+        $tableDiff->addedColumns['added_comment_string_0']
+            = new Column('added_comment_string_0', Type::getType('integer'), ['comment' => '0']);
+
+        $tableDiff->addedColumns['added_comment']
+            = new Column('added_comment', Type::getType('integer'), ['comment' => 'Doctrine']);
+
+        $tableDiff->addedColumns['`added_comment_quoted`']
+            = new Column('`added_comment_quoted`', Type::getType('integer'), ['comment' => 'rulez']);
+
+        $tableDiff->addedColumns['select']
+            = new Column('select', Type::getType('integer'), ['comment' => '666']);
+
+        $tableDiff->addedColumns['added_commented_type']
+            = new Column('added_commented_type', Type::getType('object'));
+
+        $tableDiff->addedColumns['added_commented_type_with_comment']
+            = new Column('added_commented_type_with_comment', Type::getType('array'), ['comment' => '666']);
+
+        $tableDiff->renamedColumns['comment_float_0']
+            = new Column('comment_double_0', Type::getType('decimal'), ['comment' => 'Double for real!']);
 
         // Add comment to non-commented column.
         $tableDiff->changedColumns['id'] = new ColumnDiff(
@@ -264,7 +303,11 @@ class SQLServerSchemaManagerTest extends SchemaManagerFunctionalTestCase
             'create',
             new Column('create', Type::getType('object')),
             ['comment', 'type'],
-            new Column('create', Type::getType('integer'), ['comment' => 'Doctrine 0wnz comments for reserved keyword columns!'])
+            new Column(
+                'create',
+                Type::getType('integer'),
+                ['comment' => 'Doctrine 0wnz comments for reserved keyword columns!']
+            )
         );
 
         // Add comment and change custom type to regular type from non-commented column.
@@ -283,7 +326,8 @@ class SQLServerSchemaManagerTest extends SchemaManagerFunctionalTestCase
             new Column('commented_type_with_comment', Type::getType('array'), ['comment' => 'Doctrine array type.'])
         );
 
-        $tableDiff->removedColumns['comment_integer_0'] = new Column('comment_integer_0', Type::getType('integer'), ['comment' => 0]);
+        $tableDiff->removedColumns['comment_integer_0']
+            = new Column('comment_integer_0', Type::getType('integer'), ['comment' => 0]);
 
         $this->schemaManager->alterTable($tableDiff);
 

tests/Functional/Schema/SchemaManagerFunctionalTestCase.php

@@ -210,7 +210,9 @@ abstract class SchemaManagerFunctionalTestCase extends FunctionalTestCase
         $namespaces = array_map('strtolower', $namespaces);
 
         if (! in_array('test_create_schema', $namespaces, true)) {
-            $this->connection->executeStatement($this->schemaManager->getDatabasePlatform()->getCreateSchemaSQL('test_create_schema'));
+            $this->connection->executeStatement(
+                $this->schemaManager->getDatabasePlatform()->getCreateSchemaSQL('test_create_schema')
+            );
 
             $namespaces = $this->schemaManager->listNamespaceNames();
             $namespaces = array_map('strtolower', $namespaces);
@@ -398,7 +400,9 @@ abstract class SchemaManagerFunctionalTestCase extends FunctionalTestCase
     public function testDiffListTableColumns(): void
     {
         if ($this->schemaManager->getDatabasePlatform()->getName() === 'oracle') {
-            self::markTestSkipped('Does not work with Oracle, since it cannot detect DateTime, Date and Time differenecs (at the moment).');
+            self::markTestSkipped(
+                'Does not work with Oracle, since it cannot detect DateTime, Date and Time differenecs (at the moment).'
+            );
         }
 
         $offlineTable = $this->createListTableColumns();
@@ -480,7 +484,7 @@ abstract class SchemaManagerFunctionalTestCase extends FunctionalTestCase
         self::assertEquals(['foreign_key_test'], array_map('strtolower', $fkConstraint->getColumns()));
         self::assertEquals(['id'], array_map('strtolower', $fkConstraint->getForeignColumns()));
 
-        self::assertTrue($fkTable->columnsAreIndexed($fkConstraint->getColumns()), 'The columns of a foreign key constraint should always be indexed.');
+        self::assertTrue($fkTable->columnsAreIndexed($fkConstraint->getColumns()));
     }
 
     public function testListForeignKeys(): void
@@ -580,7 +584,10 @@ abstract class SchemaManagerFunctionalTestCase extends FunctionalTestCase
         $table = $this->schemaManager->listTableDetails('alter_table');
         self::assertEquals(2, count($table->getIndexes()));
         self::assertTrue($table->hasIndex('foo_idx'));
-        self::assertEquals(['foo', 'foreign_key_test'], array_map('strtolower', $table->getIndex('foo_idx')->getColumns()));
+        self::assertEquals(
+            ['foo', 'foreign_key_test'],
+            array_map('strtolower', $table->getIndex('foo_idx')->getColumns())
+        );
 
         $tableDiff                            = new TableDiff('alter_table');
         $tableDiff->fromTable                 = $table;
@@ -592,7 +599,10 @@ abstract class SchemaManagerFunctionalTestCase extends FunctionalTestCase
         self::assertEquals(2, count($table->getIndexes()));
         self::assertTrue($table->hasIndex('bar_idx'));
         self::assertFalse($table->hasIndex('foo_idx'));
-        self::assertEquals(['foo', 'foreign_key_test'], array_map('strtolower', $table->getIndex('bar_idx')->getColumns()));
+        self::assertEquals(
+            ['foo', 'foreign_key_test'],
+            array_map('strtolower', $table->getIndex('bar_idx')->getColumns())
+        );
         self::assertFalse($table->getIndex('bar_idx')->isPrimary());
         self::assertFalse($table->getIndex('bar_idx')->isUnique());
 
@@ -843,10 +853,10 @@ abstract class SchemaManagerFunctionalTestCase extends FunctionalTestCase
         $columns = $this->schemaManager->listTableColumns('column_comment_test2');
         self::assertEquals(3, count($columns));
         self::assertEquals('This is a comment', $columns['id']->getComment());
-        self::assertEquals('This is a comment', $columns['obj']->getComment(), 'The Doctrine2 Typehint should be stripped from comment.');
-        self::assertInstanceOf(ObjectType::class, $columns['obj']->getType(), 'The Doctrine2 should be detected from comment hint.');
-        self::assertEquals('This is a comment', $columns['arr']->getComment(), 'The Doctrine2 Typehint should be stripped from comment.');
-        self::assertInstanceOf(ArrayType::class, $columns['arr']->getType(), 'The Doctrine2 should be detected from comment hint.');
+        self::assertEquals('This is a comment', $columns['obj']->getComment());
+        self::assertInstanceOf(ObjectType::class, $columns['obj']->getType());
+        self::assertEquals('This is a comment', $columns['arr']->getComment());
+        self::assertInstanceOf(ArrayType::class, $columns['arr']->getType());
     }
 
     public function testCommentHintOnDateIntervalTypeColumn(): void
@@ -869,8 +879,8 @@ abstract class SchemaManagerFunctionalTestCase extends FunctionalTestCase
         $columns = $this->schemaManager->listTableColumns('column_dateinterval_comment');
         self::assertEquals(2, count($columns));
         self::assertEquals('This is a comment', $columns['id']->getComment());
-        self::assertEquals('This is a comment', $columns['date_interval']->getComment(), 'The Doctrine2 Typehint should be stripped from comment.');
-        self::assertInstanceOf(DateIntervalType::class, $columns['date_interval']->getType(), 'The Doctrine2 should be detected from comment hint.');
+        self::assertEquals('This is a comment', $columns['date_interval']->getComment());
+        self::assertInstanceOf(DateIntervalType::class, $columns['date_interval']->getType());
     }
 
     public function testChangeColumnsTypeWithDefaultValue(): void
@@ -1150,16 +1160,24 @@ abstract class SchemaManagerFunctionalTestCase extends FunctionalTestCase
             self::markTestSkipped('Database does not support column comments.');
         }
 
-        $options          = [
+        $options = [
             'type' => Type::getType('integer'),
             'default' => 0,
             'notnull' => true,
             'comment' => 'expected+column+comment',
         ];
-        $columnDefinition = substr($this->connection->getDatabasePlatform()->getColumnDeclarationSQL('id', $options), strlen('id') + 1);
+
+        $columnDefinition = substr(
+            $this->connection->getDatabasePlatform()->getColumnDeclarationSQL('id', $options),
+            strlen('id') + 1
+        );
 
         $table = new Table('my_table');
-        $table->addColumn('id', 'integer', ['columnDefinition' => $columnDefinition, 'comment' => 'unexpected_column_comment']);
+        $table->addColumn('id', 'integer', [
+            'columnDefinition' => $columnDefinition,
+            'comment' => 'unexpected_column_comment',
+        ]);
+
         $sql = $this->connection->getDatabasePlatform()->getCreateTableSQL($table);
 
         self::assertStringContainsString('expected+column+comment', $sql[0]);
@@ -1404,7 +1422,9 @@ abstract class SchemaManagerFunctionalTestCase extends FunctionalTestCase
     public function testGenerateAnIndexWithPartialColumnLength(): void
     {
         if (! $this->schemaManager->getDatabasePlatform()->supportsColumnLengthIndexes()) {
-            self::markTestSkipped('This test is only supported on platforms that support indexes with column length definitions.');
+            self::markTestSkipped(
+                'This test is only supported on platforms that support indexes with column length definitions.'
+            );
         }
 
         $table = new Table('test_partial_column_index');
@@ -1466,7 +1486,11 @@ abstract class SchemaManagerFunctionalTestCase extends FunctionalTestCase
         $diff = Comparator::compareSchemas($offlineSchema, $schema);
 
         foreach ($diff->changedTables as $table) {
-            if (count($table->changedForeignKeys) <= 0 && count($table->addedForeignKeys) <= 0 && count($table->removedForeignKeys) <= 0) {
+            if (
+                count($table->changedForeignKeys) <= 0
+                && count($table->addedForeignKeys) <= 0
+                && count($table->removedForeignKeys) <= 0
+            ) {
                 continue;
             }
 

tests/Functional/Schema/SqliteSchemaManagerTest.php

@@ -178,8 +178,11 @@ SQL;
     /**
      * @dataProvider getDiffListIntegerAutoincrementTableColumnsData
      */
-    public function testDiffListIntegerAutoincrementTableColumns(string $integerType, bool $unsigned, bool $expectedComparatorDiff): void
-    {
+    public function testDiffListIntegerAutoincrementTableColumns(
+        string $integerType,
+        bool $unsigned,
+        bool $expectedComparatorDiff
+    ): void {
         $tableName = 'test_int_autoincrement_table';
 
         $offlineTable = new Table($tableName);

tests/Functional/TemporaryTableTest.php

@@ -64,8 +64,8 @@ class TemporaryTableTest extends FunctionalTestCase
 
         $this->connection->rollBack();
 
-        $rows = $this->connection->fetchAllAssociative('SELECT * FROM nontemporary');
-        self::assertEquals([], $rows, 'In an event of an error this result has one row, because of an implicit commit.');
+        // In an event of an error this result has one row, because of an implicit commit
+        self::assertEquals([], $this->connection->fetchAllAssociative('SELECT * FROM nontemporary'));
     }
 
     public function testCreateTemporaryTableNotAutoCommitTransaction(): void
@@ -102,7 +102,7 @@ class TemporaryTableTest extends FunctionalTestCase
         } catch (Throwable $e) {
         }
 
-        $rows = $this->connection->fetchAllAssociative('SELECT * FROM nontemporary');
-        self::assertEquals([], $rows, 'In an event of an error this result has one row, because of an implicit commit.');
+        // In an event of an error this result has one row, because of an implicit commit
+        self::assertEquals([], $this->connection->fetchAllAssociative('SELECT * FROM nontemporary'));
     }
 }

tests/Functional/TypeConversionTest.php

@@ -238,7 +238,10 @@ class TypeConversionTest extends FunctionalTestCase
     {
         $columnName     = 'test_' . $type;
         $typeInstance   = Type::getType($type);
-        $insertionValue = $typeInstance->convertToDatabaseValue($originalValue, $this->connection->getDatabasePlatform());
+        $insertionValue = $typeInstance->convertToDatabaseValue(
+            $originalValue,
+            $this->connection->getDatabasePlatform()
+        );
 
         $this->connection->insert('type_conversion', ['id' => ++self::$typeCounter, $columnName => $insertionValue]);
 

tests/Functional/WriteTest.php

@@ -136,9 +136,23 @@ class WriteTest extends FunctionalTestCase
     {
         $this->insertRows();
 
-        self::assertEquals(1, $this->connection->update('write_table', ['test_string' => 'bar'], ['test_string' => 'foo']));
-        self::assertEquals(2, $this->connection->update('write_table', ['test_string' => 'baz'], ['test_string' => 'bar']));
-        self::assertEquals(0, $this->connection->update('write_table', ['test_string' => 'baz'], ['test_string' => 'bar']));
+        self::assertEquals(1, $this->connection->update(
+            'write_table',
+            ['test_string' => 'bar'],
+            ['test_string' => 'foo']
+        ));
+
+        self::assertEquals(2, $this->connection->update(
+            'write_table',
+            ['test_string' => 'baz'],
+            ['test_string' => 'bar']
+        ));
+
+        self::assertEquals(0, $this->connection->update(
+            'write_table',
+            ['test_string' => 'baz'],
+            ['test_string' => 'bar']
+        ));
     }
 
     public function testLastInsertId(): void
@@ -183,8 +197,13 @@ class WriteTest extends FunctionalTestCase
 
     public function testLastInsertIdNoSequenceGiven(): void
     {
-        if (! $this->connection->getDatabasePlatform()->supportsSequences() || $this->connection->getDatabasePlatform()->supportsIdentityColumns()) {
-            self::markTestSkipped("Test only works consistently on platforms that support sequences and don't support identity columns.");
+        if (
+            ! $this->connection->getDatabasePlatform()->supportsSequences()
+            || $this->connection->getDatabasePlatform()->supportsIdentityColumns()
+        ) {
+            self::markTestSkipped(
+                "Test only works consistently on platforms that support sequences and don't support identity columns."
+            );
         }
 
         self::assertFalse($this->lastInsertId());
@@ -202,7 +221,10 @@ class WriteTest extends FunctionalTestCase
 
         $data = $this->connection->fetchOne('SELECT test_string FROM write_table WHERE test_int = 30');
 
-        self::assertEquals($testString->format($this->connection->getDatabasePlatform()->getDateTimeFormatString()), $data);
+        self::assertEquals(
+            $testString->format($this->connection->getDatabasePlatform()->getDateTimeFormatString()),
+            $data
+        );
     }
 
     public function testUpdateWithKeyValueTypes(): void
@@ -226,7 +248,10 @@ class WriteTest extends FunctionalTestCase
 
         $data = $this->connection->fetchOne('SELECT test_string FROM write_table WHERE test_int = 30');
 
-        self::assertEquals($testString->format($this->connection->getDatabasePlatform()->getDateTimeFormatString()), $data);
+        self::assertEquals(
+            $testString->format($this->connection->getDatabasePlatform()->getDateTimeFormatString()),
+            $data
+        );
     }
 
     public function testDeleteWithKeyValueTypes(): void
@@ -238,7 +263,13 @@ class WriteTest extends FunctionalTestCase
             ['test_string' => 'datetime', 'test_int' => 'integer']
         );
 
-        $this->connection->delete('write_table', ['test_int' => 30, 'test_string' => $val], ['test_string' => 'datetime', 'test_int' => 'integer']);
+        $this->connection->delete('write_table', [
+            'test_int' => 30,
+            'test_string' => $val,
+        ], [
+            'test_string' => 'datetime',
+            'test_int' => 'integer',
+        ]);
 
         $data = $this->connection->fetchOne('SELECT test_string FROM write_table WHERE test_int = 30');
 
@@ -297,7 +328,10 @@ class WriteTest extends FunctionalTestCase
 
         self::assertCount(1, $data);
 
-        $this->connection->update('write_table', ['test_int' => 10], ['test_string' => null], ['test_string' => 'string', 'test_int' => 'integer']);
+        $this->connection->update('write_table', ['test_int' => 10], ['test_string' => null], [
+            'test_string' => 'string',
+            'test_int' => 'integer',
+        ]);
 
         $data = $this->connection->fetchAllAssociative('SELECT * FROM write_table WHERE test_int = 30');
 

tests/FunctionalTestCase.php

@@ -106,7 +106,8 @@ abstract class FunctionalTestCase extends TestCase
                 $traceMsg .= $part['file'] . ':' . $part['line'] . PHP_EOL;
             }
 
-            $message = '[' . get_class($t) . '] ' . $t->getMessage() . PHP_EOL . PHP_EOL . 'With queries:' . PHP_EOL . $queries . PHP_EOL . 'Trace:' . PHP_EOL . $traceMsg;
+            $message = '[' . get_class($t) . '] ' . $t->getMessage() . PHP_EOL . PHP_EOL
+                . 'With queries:' . PHP_EOL . $queries . PHP_EOL . 'Trace:' . PHP_EOL . $traceMsg;
 
             throw new Exception($message, (int) $t->getCode(), $t);
         }

tests/Platforms/AbstractPlatformTestCase.php

@@ -362,7 +362,10 @@ abstract class AbstractPlatformTestCase extends TestCase
             ->method('onSchemaCreateTableColumn');
 
         $eventManager = new EventManager();
-        $eventManager->addEventListener([Events::onSchemaCreateTable, Events::onSchemaCreateTableColumn], $listenerMock);
+        $eventManager->addEventListener([
+            Events::onSchemaCreateTable,
+            Events::onSchemaCreateTableColumn,
+        ], $listenerMock);
 
         $this->platform->setEventManager($eventManager);
 
@@ -627,27 +630,63 @@ abstract class AbstractPlatformTestCase extends TestCase
 
         // Foreign table with reserved keyword as name (needs quotation).
         $foreignTable = new Table('foreign');
-        $foreignTable->addColumn('create', 'string');    // Foreign column with reserved keyword as name (needs quotation).
-        $foreignTable->addColumn('bar', 'string');       // Foreign column with non-reserved keyword as name (does not need quotation).
-        $foreignTable->addColumn('`foo-bar`', 'string'); // Foreign table with special character in name (needs quotation on some platforms, e.g. Sqlite).
 
-        $table->addForeignKeyConstraint($foreignTable, ['create', 'foo', '`bar`'], ['create', 'bar', '`foo-bar`'], [], 'FK_WITH_RESERVED_KEYWORD');
+        // Foreign column with reserved keyword as name (needs quotation).
+        $foreignTable->addColumn('create', 'string');
+
+        // Foreign column with non-reserved keyword as name (does not need quotation).
+        $foreignTable->addColumn('bar', 'string');
+
+        // Foreign table with special character in name (needs quotation on some platforms, e.g. Sqlite).
+        $foreignTable->addColumn('`foo-bar`', 'string');
+
+        $table->addForeignKeyConstraint(
+            $foreignTable,
+            ['create', 'foo', '`bar`'],
+            ['create', 'bar', '`foo-bar`'],
+            [],
+            'FK_WITH_RESERVED_KEYWORD'
+        );
 
         // Foreign table with non-reserved keyword as name (does not need quotation).
         $foreignTable = new Table('foo');
-        $foreignTable->addColumn('create', 'string');    // Foreign column with reserved keyword as name (needs quotation).
-        $foreignTable->addColumn('bar', 'string');       // Foreign column with non-reserved keyword as name (does not need quotation).
-        $foreignTable->addColumn('`foo-bar`', 'string'); // Foreign table with special character in name (needs quotation on some platforms, e.g. Sqlite).
 
-        $table->addForeignKeyConstraint($foreignTable, ['create', 'foo', '`bar`'], ['create', 'bar', '`foo-bar`'], [], 'FK_WITH_NON_RESERVED_KEYWORD');
+        // Foreign column with reserved keyword as name (needs quotation).
+        $foreignTable->addColumn('create', 'string');
+
+        // Foreign column with non-reserved keyword as name (does not need quotation).
+        $foreignTable->addColumn('bar', 'string');
+
+        // Foreign table with special character in name (needs quotation on some platforms, e.g. Sqlite).
+        $foreignTable->addColumn('`foo-bar`', 'string');
+
+        $table->addForeignKeyConstraint(
+            $foreignTable,
+            ['create', 'foo', '`bar`'],
+            ['create', 'bar', '`foo-bar`'],
+            [],
+            'FK_WITH_NON_RESERVED_KEYWORD'
+        );
 
         // Foreign table with special character in name (needs quotation on some platforms, e.g. Sqlite).
         $foreignTable = new Table('`foo-bar`');
-        $foreignTable->addColumn('create', 'string');    // Foreign column with reserved keyword as name (needs quotation).
-        $foreignTable->addColumn('bar', 'string');       // Foreign column with non-reserved keyword as name (does not need quotation).
-        $foreignTable->addColumn('`foo-bar`', 'string'); // Foreign table with special character in name (needs quotation on some platforms, e.g. Sqlite).
 
-        $table->addForeignKeyConstraint($foreignTable, ['create', 'foo', '`bar`'], ['create', 'bar', '`foo-bar`'], [], 'FK_WITH_INTENDED_QUOTATION');
+        // Foreign column with reserved keyword as name (needs quotation).
+        $foreignTable->addColumn('create', 'string');
+
+        // Foreign column with non-reserved keyword as name (does not need quotation).
+        $foreignTable->addColumn('bar', 'string');
+
+        // Foreign table with special character in name (needs quotation on some platforms, e.g. Sqlite).
+        $foreignTable->addColumn('`foo-bar`', 'string');
+
+        $table->addForeignKeyConstraint(
+            $foreignTable,
+            ['create', 'foo', '`bar`'],
+            ['create', 'bar', '`foo-bar`'],
+            [],
+            'FK_WITH_INTENDED_QUOTATION'
+        );
 
         $sql = $this->platform->getCreateTableSQL($table, AbstractPlatform::CREATE_FOREIGNKEYS);
         self::assertEquals($this->getQuotedColumnInForeignKeySQL(), $sql);
@@ -870,17 +909,32 @@ abstract class AbstractPlatformTestCase extends TestCase
 
         $toTable = new Table('mytable');
 
-        $toTable->addColumn('unquoted', 'integer', ['comment' => 'Unquoted 1']); // unquoted -> unquoted
-        $toTable->addColumn('where', 'integer', ['comment' => 'Unquoted 2']); // unquoted -> reserved keyword
-        $toTable->addColumn('`foo`', 'integer', ['comment' => 'Unquoted 3']); // unquoted -> quoted
+        // unquoted -> unquoted
+        $toTable->addColumn('unquoted', 'integer', ['comment' => 'Unquoted 1']);
+
+        // unquoted -> reserved keyword
+        $toTable->addColumn('where', 'integer', ['comment' => 'Unquoted 2']);
+
+        // unquoted -> quoted
+        $toTable->addColumn('`foo`', 'integer', ['comment' => 'Unquoted 3']);
+
+        // reserved keyword -> unquoted
+        $toTable->addColumn('reserved_keyword', 'integer', ['comment' => 'Reserved keyword 1']);
 
-        $toTable->addColumn('reserved_keyword', 'integer', ['comment' => 'Reserved keyword 1']); // reserved keyword -> unquoted
-        $toTable->addColumn('from', 'integer', ['comment' => 'Reserved keyword 2']); // reserved keyword -> reserved keyword
-        $toTable->addColumn('`bar`', 'integer', ['comment' => 'Reserved keyword 3']); // reserved keyword -> quoted
+        // reserved keyword -> reserved keyword
+        $toTable->addColumn('from', 'integer', ['comment' => 'Reserved keyword 2']);
 
-        $toTable->addColumn('quoted', 'integer', ['comment' => 'Quoted 1']); // quoted -> unquoted
-        $toTable->addColumn('and', 'integer', ['comment' => 'Quoted 2']); // quoted -> reserved keyword
-        $toTable->addColumn('`baz`', 'integer', ['comment' => 'Quoted 3']); // quoted -> quoted
+        // reserved keyword -> quoted
+        $toTable->addColumn('`bar`', 'integer', ['comment' => 'Reserved keyword 3']);
+
+        // quoted -> unquoted
+        $toTable->addColumn('quoted', 'integer', ['comment' => 'Quoted 1']);
+
+        // quoted -> reserved keyword
+        $toTable->addColumn('and', 'integer', ['comment' => 'Quoted 2']);
+
+        // quoted -> quoted
+        $toTable->addColumn('`baz`', 'integer', ['comment' => 'Quoted 3']);
 
         $comparator = new Comparator();
 
@@ -1160,7 +1214,9 @@ abstract class AbstractPlatformTestCase extends TestCase
         }
 
         $this->expectException(DBALException::class);
-        $this->expectExceptionMessage("Operation 'Doctrine\\DBAL\\Platforms\\AbstractPlatform::getInlineColumnCommentSQL' is not supported by platform.");
+        $this->expectExceptionMessage(
+            "Operation '" . AbstractPlatform::class . "::getInlineColumnCommentSQL' is not supported by platform."
+        );
         $this->expectExceptionCode(0);
 
         $this->platform->getInlineColumnCommentSQL('unsupported');

tests/Platforms/AbstractPostgreSQLPlatformTestCase.php

@@ -66,7 +66,8 @@ abstract class AbstractPostgreSQLPlatformTestCase extends AbstractPlatformTestCa
 
     protected function getGenerateForeignKeySql(): string
     {
-        return 'ALTER TABLE test ADD FOREIGN KEY (fk_name_id) REFERENCES other_table (id) NOT DEFERRABLE INITIALLY IMMEDIATE';
+        return 'ALTER TABLE test ADD FOREIGN KEY (fk_name_id)'
+            . ' REFERENCES other_table (id) NOT DEFERRABLE INITIALLY IMMEDIATE';
     }
 
     public function testGeneratesForeignKeySqlForNonStandardOptions(): void
@@ -79,7 +80,8 @@ abstract class AbstractPostgreSQLPlatformTestCase extends AbstractPlatformTestCa
             ['onDelete' => 'CASCADE']
         );
         self::assertEquals(
-            'CONSTRAINT my_fk FOREIGN KEY (foreign_id) REFERENCES my_table (id) ON DELETE CASCADE NOT DEFERRABLE INITIALLY IMMEDIATE',
+            'CONSTRAINT my_fk FOREIGN KEY (foreign_id)'
+                . ' REFERENCES my_table (id) ON DELETE CASCADE NOT DEFERRABLE INITIALLY IMMEDIATE',
             $this->platform->getForeignKeyDeclarationSQL($foreignKey)
         );
 
@@ -91,7 +93,8 @@ abstract class AbstractPostgreSQLPlatformTestCase extends AbstractPlatformTestCa
             ['match' => 'full']
         );
         self::assertEquals(
-            'CONSTRAINT my_fk FOREIGN KEY (foreign_id) REFERENCES my_table (id) MATCH full NOT DEFERRABLE INITIALLY IMMEDIATE',
+            'CONSTRAINT my_fk FOREIGN KEY (foreign_id)'
+                . ' REFERENCES my_table (id) MATCH full NOT DEFERRABLE INITIALLY IMMEDIATE',
             $this->platform->getForeignKeyDeclarationSQL($foreignKey)
         );
 
@@ -103,7 +106,8 @@ abstract class AbstractPostgreSQLPlatformTestCase extends AbstractPlatformTestCa
             ['deferrable' => true]
         );
         self::assertEquals(
-            'CONSTRAINT my_fk FOREIGN KEY (foreign_id) REFERENCES my_table (id) DEFERRABLE INITIALLY IMMEDIATE',
+            'CONSTRAINT my_fk FOREIGN KEY (foreign_id)'
+                . ' REFERENCES my_table (id) DEFERRABLE INITIALLY IMMEDIATE',
             $this->platform->getForeignKeyDeclarationSQL($foreignKey)
         );
 
@@ -115,7 +119,8 @@ abstract class AbstractPostgreSQLPlatformTestCase extends AbstractPlatformTestCa
             ['deferred' => true]
         );
         self::assertEquals(
-            'CONSTRAINT my_fk FOREIGN KEY (foreign_id) REFERENCES my_table (id) NOT DEFERRABLE INITIALLY DEFERRED',
+            'CONSTRAINT my_fk FOREIGN KEY (foreign_id)'
+                . ' REFERENCES my_table (id) NOT DEFERRABLE INITIALLY DEFERRED',
             $this->platform->getForeignKeyDeclarationSQL($foreignKey)
         );
 
@@ -127,7 +132,8 @@ abstract class AbstractPostgreSQLPlatformTestCase extends AbstractPlatformTestCa
             ['feferred' => true]
         );
         self::assertEquals(
-            'CONSTRAINT my_fk FOREIGN KEY (foreign_id) REFERENCES my_table (id) NOT DEFERRABLE INITIALLY DEFERRED',
+            'CONSTRAINT my_fk FOREIGN KEY (foreign_id)'
+                . ' REFERENCES my_table (id) NOT DEFERRABLE INITIALLY DEFERRED',
             $this->platform->getForeignKeyDeclarationSQL($foreignKey)
         );
 
@@ -139,18 +145,24 @@ abstract class AbstractPostgreSQLPlatformTestCase extends AbstractPlatformTestCa
             ['deferrable' => true, 'deferred' => true, 'match' => 'full']
         );
         self::assertEquals(
-            'CONSTRAINT my_fk FOREIGN KEY (foreign_id) REFERENCES my_table (id) MATCH full DEFERRABLE INITIALLY DEFERRED',
+            'CONSTRAINT my_fk FOREIGN KEY (foreign_id)'
+                . ' REFERENCES my_table (id) MATCH full DEFERRABLE INITIALLY DEFERRED',
             $this->platform->getForeignKeyDeclarationSQL($foreignKey)
         );
     }
 
     public function testGeneratesSqlSnippets(): void
     {
-        self::assertEquals('SIMILAR TO', $this->platform->getRegexpExpression(), 'Regular expression operator is not correct');
-        self::assertEquals('"', $this->platform->getIdentifierQuoteCharacter(), 'Identifier quote character is not correct');
-        self::assertEquals('column1 || column2 || column3', $this->platform->getConcatExpression('column1', 'column2', 'column3'), 'Concatenation expression is not correct');
-        self::assertEquals('SUBSTRING(column FROM 5)', $this->platform->getSubstringExpression('column', 5), 'Substring expression without length is not correct');
-        self::assertEquals('SUBSTRING(column FROM 1 FOR 5)', $this->platform->getSubstringExpression('column', 1, 5), 'Substring expression with length is not correct');
+        self::assertEquals('SIMILAR TO', $this->platform->getRegexpExpression());
+        self::assertEquals('"', $this->platform->getIdentifierQuoteCharacter());
+
+        self::assertEquals(
+            'column1 || column2 || column3',
+            $this->platform->getConcatExpression('column1', 'column2', 'column3')
+        );
+
+        self::assertEquals('SUBSTRING(column FROM 5)', $this->platform->getSubstringExpression('column', 5));
+        self::assertEquals('SUBSTRING(column FROM 1 FOR 5)', $this->platform->getSubstringExpression('column', 1, 5));
     }
 
     public function testGeneratesTransactionCommands(): void
@@ -186,7 +198,10 @@ abstract class AbstractPostgreSQLPlatformTestCase extends AbstractPlatformTestCa
         $column = $table->addColumn('id', 'integer');
         $column->setAutoincrement(true);
 
-        self::assertEquals(['CREATE TABLE autoinc_table (id SERIAL NOT NULL)'], $this->platform->getCreateTableSQL($table));
+        self::assertEquals(
+            ['CREATE TABLE autoinc_table (id SERIAL NOT NULL)'],
+            $this->platform->getCreateTableSQL($table)
+        );
     }
 
     /**
@@ -274,13 +289,11 @@ abstract class AbstractPostgreSQLPlatformTestCase extends AbstractPlatformTestCa
         );
         self::assertEquals(
             'VARCHAR(50)',
-            $this->platform->getVarcharTypeDeclarationSQL(['length' => 50]),
-            'Variable string declaration is not correct'
+            $this->platform->getVarcharTypeDeclarationSQL(['length' => 50])
         );
         self::assertEquals(
             'VARCHAR(255)',
-            $this->platform->getVarcharTypeDeclarationSQL([]),
-            'Long string declaration is not correct'
+            $this->platform->getVarcharTypeDeclarationSQL([])
         );
     }
 
@@ -419,10 +432,14 @@ abstract class AbstractPostgreSQLPlatformTestCase extends AbstractPlatformTestCa
     protected function getQuotedColumnInForeignKeySQL(): array
     {
         return [
-            'CREATE TABLE "quoted" ("create" VARCHAR(255) NOT NULL, foo VARCHAR(255) NOT NULL, "bar" VARCHAR(255) NOT NULL)',
-            'ALTER TABLE "quoted" ADD CONSTRAINT FK_WITH_RESERVED_KEYWORD FOREIGN KEY ("create", foo, "bar") REFERENCES "foreign" ("create", bar, "foo-bar") NOT DEFERRABLE INITIALLY IMMEDIATE',
-            'ALTER TABLE "quoted" ADD CONSTRAINT FK_WITH_NON_RESERVED_KEYWORD FOREIGN KEY ("create", foo, "bar") REFERENCES foo ("create", bar, "foo-bar") NOT DEFERRABLE INITIALLY IMMEDIATE',
-            'ALTER TABLE "quoted" ADD CONSTRAINT FK_WITH_INTENDED_QUOTATION FOREIGN KEY ("create", foo, "bar") REFERENCES "foo-bar" ("create", bar, "foo-bar") NOT DEFERRABLE INITIALLY IMMEDIATE',
+            'CREATE TABLE "quoted" ("create" VARCHAR(255) NOT NULL, '
+            . 'foo VARCHAR(255) NOT NULL, "bar" VARCHAR(255) NOT NULL)',
+            'ALTER TABLE "quoted" ADD CONSTRAINT FK_WITH_RESERVED_KEYWORD FOREIGN KEY ("create", foo, "bar")'
+                . ' REFERENCES "foreign" ("create", bar, "foo-bar") NOT DEFERRABLE INITIALLY IMMEDIATE',
+            'ALTER TABLE "quoted" ADD CONSTRAINT FK_WITH_NON_RESERVED_KEYWORD FOREIGN KEY ("create", foo, "bar")'
+                . ' REFERENCES foo ("create", bar, "foo-bar") NOT DEFERRABLE INITIALLY IMMEDIATE',
+            'ALTER TABLE "quoted" ADD CONSTRAINT FK_WITH_INTENDED_QUOTATION FOREIGN KEY ("create", foo, "bar")'
+                . ' REFERENCES "foo-bar" ("create", bar, "foo-bar") NOT DEFERRABLE INITIALLY IMMEDIATE',
         ];
     }
 
@@ -484,8 +501,12 @@ abstract class AbstractPostgreSQLPlatformTestCase extends AbstractPlatformTestCa
      *
      * @dataProvider pgBooleanProvider
      */
-    public function testConvertFromBoolean($databaseValue, string $prepareStatementValue, ?int $integerValue, ?bool $booleanValue): void
-    {
+    public function testConvertFromBoolean(
+        $databaseValue,
+        string $prepareStatementValue,
+        ?int $integerValue,
+        ?bool $booleanValue
+    ): void {
         $platform = $this->createPlatform();
 
         self::assertSame($booleanValue, $platform->convertFromBoolean($databaseValue));

tests/Platforms/OraclePlatformTest.php

@@ -111,7 +111,8 @@ class OraclePlatformTest extends AbstractPlatformTestCase
     {
         return [
             'ALTER TABLE mytable ADD (quota NUMBER(10) DEFAULT NULL NULL)',
-            "ALTER TABLE mytable MODIFY (baz VARCHAR2(255) DEFAULT 'def' NOT NULL, bloo NUMBER(1) DEFAULT '0' NOT NULL)",
+            "ALTER TABLE mytable MODIFY (baz VARCHAR2(255) DEFAULT 'def' NOT NULL, "
+                . "bloo NUMBER(1) DEFAULT '0' NOT NULL)",
             'ALTER TABLE mytable DROP (foo)',
             'ALTER TABLE mytable RENAME TO userlist',
         ];
@@ -121,13 +122,16 @@ class OraclePlatformTest extends AbstractPlatformTestCase
     {
         $this->expectException(DBALException::class);
 
-        self::assertEquals('RLIKE', $this->platform->getRegexpExpression(), 'Regular expression operator is not correct');
+        self::assertEquals('RLIKE', $this->platform->getRegexpExpression());
     }
 
     public function testGeneratesSqlSnippets(): void
     {
-        self::assertEquals('"', $this->platform->getIdentifierQuoteCharacter(), 'Identifier quote character is not correct');
-        self::assertEquals('column1 || column2 || column3', $this->platform->getConcatExpression('column1', 'column2', 'column3'), 'Concatenation expression is not correct');
+        self::assertEquals('"', $this->platform->getIdentifierQuoteCharacter());
+        self::assertEquals(
+            'column1 || column2 || column3',
+            $this->platform->getConcatExpression('column1', 'column2', 'column3')
+        );
     }
 
     public function testGeneratesTransactionsCommands(): void
@@ -195,13 +199,11 @@ class OraclePlatformTest extends AbstractPlatformTestCase
         );
         self::assertEquals(
             'VARCHAR2(50)',
-            $this->platform->getVarcharTypeDeclarationSQL(['length' => 50]),
-            'Variable string declaration is not correct'
+            $this->platform->getVarcharTypeDeclarationSQL(['length' => 50])
         );
         self::assertEquals(
             'VARCHAR2(255)',
-            $this->platform->getVarcharTypeDeclarationSQL([]),
-            'Long string declaration is not correct'
+            $this->platform->getVarcharTypeDeclarationSQL([])
         );
     }
 
@@ -296,13 +298,25 @@ class OraclePlatformTest extends AbstractPlatformTestCase
     public function testModifyLimitQueryWithNonEmptyOffset(): void
     {
         $sql = $this->platform->modifyLimitQuery('SELECT * FROM user', 10, 10);
-        self::assertEquals('SELECT * FROM (SELECT a.*, ROWNUM AS doctrine_rownum FROM (SELECT * FROM user) a WHERE ROWNUM <= 20) WHERE doctrine_rownum >= 11', $sql);
+
+        self::assertEquals(
+            'SELECT * FROM ('
+                . 'SELECT a.*, ROWNUM AS doctrine_rownum FROM (SELECT * FROM user) a WHERE ROWNUM <= 20'
+                . ') WHERE doctrine_rownum >= 11',
+            $sql
+        );
     }
 
     public function testModifyLimitQueryWithEmptyLimit(): void
     {
         $sql = $this->platform->modifyLimitQuery('SELECT * FROM user', null, 10);
-        self::assertEquals('SELECT * FROM (SELECT a.*, ROWNUM AS doctrine_rownum FROM (SELECT * FROM user) a) WHERE doctrine_rownum >= 11', $sql);
+
+        self::assertEquals(
+            'SELECT * FROM ('
+                . 'SELECT a.*, ROWNUM AS doctrine_rownum FROM (SELECT * FROM user) a'
+                . ') WHERE doctrine_rownum >= 11',
+            $sql
+        );
     }
 
     public function testModifyLimitQueryWithAscOrderBy(): void
@@ -327,7 +341,17 @@ class OraclePlatformTest extends AbstractPlatformTestCase
         $targets    = [
             sprintf('CREATE TABLE %s (%s NUMBER(10) NOT NULL)', $tableName, $columnName),
             sprintf(
-                "DECLARE constraints_Count NUMBER; BEGIN SELECT COUNT(CONSTRAINT_NAME) INTO constraints_Count FROM USER_CONSTRAINTS WHERE TABLE_NAME = '%s' AND CONSTRAINT_TYPE = 'P'; IF constraints_Count = 0 OR constraints_Count = '' THEN EXECUTE IMMEDIATE 'ALTER TABLE %s ADD CONSTRAINT %s_AI_PK PRIMARY KEY (%s)'; END IF; END;",
+                'DECLARE constraints_Count NUMBER;'
+                    . ' BEGIN'
+                    . ' SELECT COUNT(CONSTRAINT_NAME)'
+                    . ' INTO constraints_Count'
+                    . ' FROM USER_CONSTRAINTS'
+                    . " WHERE TABLE_NAME = '%s' AND CONSTRAINT_TYPE = 'P';"
+                    . " IF constraints_Count = 0 OR constraints_Count = ''"
+                    . ' THEN EXECUTE IMMEDIATE'
+                    . " 'ALTER TABLE %s ADD CONSTRAINT %s_AI_PK PRIMARY KEY (%s)';"
+                    . ' END IF;'
+                    . ' END;',
                 $tableName,
                 $tableName,
                 $tableName,
@@ -335,7 +359,20 @@ class OraclePlatformTest extends AbstractPlatformTestCase
             ),
             sprintf('CREATE SEQUENCE %s_SEQ START WITH 1 MINVALUE 1 INCREMENT BY 1', $tableName),
             sprintf(
-                "CREATE TRIGGER %s_AI_PK BEFORE INSERT ON %s FOR EACH ROW DECLARE last_Sequence NUMBER; last_InsertID NUMBER; BEGIN SELECT %s_SEQ.NEXTVAL INTO :NEW.%s FROM DUAL; IF (:NEW.%s IS NULL OR :NEW.%s = 0) THEN SELECT %s_SEQ.NEXTVAL INTO :NEW.%s FROM DUAL; ELSE SELECT NVL(Last_Number, 0) INTO last_Sequence FROM User_Sequences WHERE Sequence_Name = '%s_SEQ'; SELECT :NEW.%s INTO last_InsertID FROM DUAL; WHILE (last_InsertID > last_Sequence) LOOP SELECT %s_SEQ.NEXTVAL INTO last_Sequence FROM DUAL; END LOOP; END IF; END;",
+                'CREATE TRIGGER %s_AI_PK BEFORE INSERT ON %s FOR EACH ROW DECLARE last_Sequence NUMBER;'
+                . ' last_InsertID NUMBER;'
+                . ' BEGIN SELECT %s_SEQ.NEXTVAL'
+                . ' INTO :NEW.%s FROM DUAL;'
+                . ' IF (:NEW.%s IS NULL OR :NEW.%s = 0)'
+                . ' THEN SELECT %s_SEQ.NEXTVAL INTO :NEW.%s FROM DUAL;'
+                . ' ELSE SELECT NVL(Last_Number, 0) INTO last_Sequence'
+                . " FROM User_Sequences WHERE Sequence_Name = '%s_SEQ';"
+                . ' SELECT :NEW.%s INTO last_InsertID FROM DUAL;'
+                . ' WHILE (last_InsertID > last_Sequence) LOOP'
+                . ' SELECT %s_SEQ.NEXTVAL INTO last_Sequence FROM DUAL;'
+                . ' END LOOP;'
+                . ' END IF;'
+                . ' END;',
                 $tableName,
                 $tableName,
                 $tableName,
@@ -443,10 +480,14 @@ class OraclePlatformTest extends AbstractPlatformTestCase
     protected function getQuotedColumnInForeignKeySQL(): array
     {
         return [
-            'CREATE TABLE "quoted" ("create" VARCHAR2(255) NOT NULL, foo VARCHAR2(255) NOT NULL, "bar" VARCHAR2(255) NOT NULL)',
-            'ALTER TABLE "quoted" ADD CONSTRAINT FK_WITH_RESERVED_KEYWORD FOREIGN KEY ("create", foo, "bar") REFERENCES foreign ("create", bar, "foo-bar")',
-            'ALTER TABLE "quoted" ADD CONSTRAINT FK_WITH_NON_RESERVED_KEYWORD FOREIGN KEY ("create", foo, "bar") REFERENCES foo ("create", bar, "foo-bar")',
-            'ALTER TABLE "quoted" ADD CONSTRAINT FK_WITH_INTENDED_QUOTATION FOREIGN KEY ("create", foo, "bar") REFERENCES "foo-bar" ("create", bar, "foo-bar")',
+            'CREATE TABLE "quoted" ("create" VARCHAR2(255) NOT NULL, foo VARCHAR2(255) NOT NULL, '
+                . '"bar" VARCHAR2(255) NOT NULL)',
+            'ALTER TABLE "quoted" ADD CONSTRAINT FK_WITH_RESERVED_KEYWORD FOREIGN KEY ("create", foo, "bar")'
+                . ' REFERENCES foreign ("create", bar, "foo-bar")',
+            'ALTER TABLE "quoted" ADD CONSTRAINT FK_WITH_NON_RESERVED_KEYWORD FOREIGN KEY ("create", foo, "bar")'
+                . ' REFERENCES foo ("create", bar, "foo-bar")',
+            'ALTER TABLE "quoted" ADD CONSTRAINT FK_WITH_INTENDED_QUOTATION FOREIGN KEY ("create", foo, "bar")'
+                . ' REFERENCES "foo-bar" ("create", bar, "foo-bar")',
         ];
     }
 
@@ -481,7 +522,11 @@ class OraclePlatformTest extends AbstractPlatformTestCase
             ['notnull']
         );
 
-        $expectedSql = ["ALTER TABLE mytable MODIFY (foo VARCHAR2(255) DEFAULT 'bla', baz VARCHAR2(255) DEFAULT 'bla' NOT NULL, metar VARCHAR2(2000) DEFAULT NULL NULL)"];
+        $expectedSql = [
+            "ALTER TABLE mytable MODIFY (foo VARCHAR2(255) DEFAULT 'bla', baz VARCHAR2(255) DEFAULT 'bla' NOT NULL, "
+                . 'metar VARCHAR2(2000) DEFAULT NULL NULL)',
+        ];
+
         self::assertEquals($expectedSql, $this->platform->getAlterTableSQL($tableDiff));
     }
 
@@ -510,7 +555,11 @@ class OraclePlatformTest extends AbstractPlatformTestCase
 
         self::assertSame('RAW(255)', $this->platform->getBinaryTypeDeclarationSQL(['fixed' => true]));
         self::assertSame('RAW(2000)', $this->platform->getBinaryTypeDeclarationSQL(['fixed' => true, 'length' => 0]));
-        self::assertSame('RAW(2000)', $this->platform->getBinaryTypeDeclarationSQL(['fixed' => true, 'length' => 2000]));
+
+        self::assertSame(
+            'RAW(2000)',
+            $this->platform->getBinaryTypeDeclarationSQL(['fixed' => true, 'length' => 2000])
+        );
     }
 
     public function testReturnsBinaryTypeLongerThanMaxDeclarationSQL(): void

tests/Portability/ConverterTest.php

@@ -68,8 +68,13 @@ class ConverterTest extends TestCase
      *
      * @dataProvider convertAssociativeProvider
      */
-    public function testConvertAssociative($row, bool $convertEmptyStringToNull, bool $rightTrimString, ?int $case, $expected): void
-    {
+    public function testConvertAssociative(
+        $row,
+        bool $convertEmptyStringToNull,
+        bool $rightTrimString,
+        ?int $case,
+        $expected
+    ): void {
         self::assertSame(
             $expected,
             $this->createConverter($convertEmptyStringToNull, $rightTrimString, $case)

tests/Query/QueryBuilderTest.php

@@ -177,7 +177,11 @@ class QueryBuilderTest extends TestCase
            ->orWhere('u.name = ?')
            ->andWhere('u.name = ?');
 
-        self::assertEquals('SELECT u.*, p.* FROM users u WHERE (((u.username = ?) AND (u.username = ?)) OR (u.name = ?)) AND (u.name = ?)', (string) $qb);
+        self::assertEquals(
+            'SELECT u.*, p.* FROM users u'
+                . ' WHERE (((u.username = ?) AND (u.username = ?)) OR (u.name = ?)) AND (u.name = ?)',
+            (string) $qb
+        );
     }
 
     public function testSelectGroupBy(): void
@@ -279,7 +283,10 @@ class QueryBuilderTest extends TestCase
            ->having('u.name = ?')
            ->andHaving('u.username = ?');
 
-        self::assertEquals('SELECT u.*, p.* FROM users u GROUP BY u.id HAVING (u.name = ?) AND (u.username = ?)', (string) $qb);
+        self::assertEquals(
+            'SELECT u.*, p.* FROM users u GROUP BY u.id HAVING (u.name = ?) AND (u.username = ?)',
+            (string) $qb
+        );
     }
 
     public function testSelectHavingOrHaving(): void
@@ -293,7 +300,10 @@ class QueryBuilderTest extends TestCase
            ->having('u.name = ?')
            ->orHaving('u.username = ?');
 
-        self::assertEquals('SELECT u.*, p.* FROM users u GROUP BY u.id HAVING (u.name = ?) OR (u.username = ?)', (string) $qb);
+        self::assertEquals(
+            'SELECT u.*, p.* FROM users u GROUP BY u.id HAVING (u.name = ?) OR (u.username = ?)',
+            (string) $qb
+        );
     }
 
     public function testSelectOrHavingOrHaving(): void
@@ -307,7 +317,10 @@ class QueryBuilderTest extends TestCase
            ->orHaving('u.name = ?')
            ->orHaving('u.username = ?');
 
-        self::assertEquals('SELECT u.*, p.* FROM users u GROUP BY u.id HAVING (u.name = ?) OR (u.username = ?)', (string) $qb);
+        self::assertEquals(
+            'SELECT u.*, p.* FROM users u GROUP BY u.id HAVING (u.name = ?) OR (u.username = ?)',
+            (string) $qb
+        );
     }
 
     public function testSelectHavingAndOrHaving(): void
@@ -322,7 +335,10 @@ class QueryBuilderTest extends TestCase
            ->orHaving('u.username = ?')
            ->andHaving('u.username = ?');
 
-        self::assertEquals('SELECT u.*, p.* FROM users u GROUP BY u.id HAVING ((u.name = ?) OR (u.username = ?)) AND (u.username = ?)', (string) $qb);
+        self::assertEquals(
+            'SELECT u.*, p.* FROM users u GROUP BY u.id HAVING ((u.name = ?) OR (u.username = ?)) AND (u.username = ?)',
+            (string) $qb
+        );
     }
 
     public function testSelectOrderBy(): void
@@ -679,7 +695,10 @@ class QueryBuilderTest extends TestCase
             ->where('nt.lang = :lang AND n.deleted != 1');
 
         $this->expectException(QueryException::class);
-        $this->expectExceptionMessage("The given alias 'invalid' is not part of any FROM or JOIN clause table. The currently registered aliases are: news, nv.");
+        $this->expectExceptionMessage(
+            "The given alias 'invalid' is not part of any FROM or JOIN clause table. "
+                . 'The currently registered aliases are: news, nv.'
+        );
         self::assertEquals('', $qb->getSQL());
     }
 
@@ -695,7 +714,13 @@ class QueryBuilderTest extends TestCase
             ->where('nt.lang = ?')
             ->andWhere('n.deleted = 0');
 
-        self::assertEquals("SELECT COUNT(DISTINCT news.id) FROM newspages news INNER JOIN nodeversion nv ON nv.refId = news.id AND nv.refEntityname='Entity\\News' INNER JOIN nodetranslation nt ON nv.nodetranslation = nt.id INNER JOIN node n ON nt.node = n.id WHERE (nt.lang = ?) AND (n.deleted = 0)", $qb->getSQL());
+        self::assertEquals(
+            'SELECT COUNT(DISTINCT news.id) FROM newspages news'
+                . " INNER JOIN nodeversion nv ON nv.refId = news.id AND nv.refEntityname='Entity\\News'"
+                . ' INNER JOIN nodetranslation nt ON nv.nodetranslation = nt.id'
+                . ' INNER JOIN node n ON nt.node = n.id WHERE (nt.lang = ?) AND (n.deleted = 0)',
+            $qb->getSQL()
+        );
     }
 
     public function testSelectWithMultipleFromAndJoins(): void
@@ -710,7 +735,13 @@ class QueryBuilderTest extends TestCase
             ->where('u.id = a.user_id')
             ->andWhere('p.read = 1');
 
-        self::assertEquals('SELECT DISTINCT u.id FROM users u INNER JOIN permissions p ON p.user_id = u.id, articles a INNER JOIN comments c ON c.article_id = a.id WHERE (u.id = a.user_id) AND (p.read = 1)', $qb->getSQL());
+        self::assertEquals(
+            'SELECT DISTINCT u.id FROM users u'
+            . ' INNER JOIN permissions p ON p.user_id = u.id, articles a'
+            . ' INNER JOIN comments c ON c.article_id = a.id'
+            . ' WHERE (u.id = a.user_id) AND (p.read = 1)',
+            $qb->getSQL()
+        );
     }
 
     public function testSelectWithJoinsWithMultipleOnConditionsParseOrder(): void
@@ -814,7 +845,13 @@ class QueryBuilderTest extends TestCase
             ->where('users.id = articles.user_id')
             ->andWhere('p.read = 1');
 
-        self::assertEquals('SELECT DISTINCT users.id FROM users INNER JOIN permissions p ON p.user_id = users.id, articles INNER JOIN comments c ON c.article_id = articles.id WHERE (users.id = articles.user_id) AND (p.read = 1)', $qb->getSQL());
+        self::assertEquals(
+            'SELECT DISTINCT users.id FROM users'
+                . ' INNER JOIN permissions p ON p.user_id = users.id, articles'
+                . ' INNER JOIN comments c ON c.article_id = articles.id'
+                . ' WHERE (users.id = articles.user_id) AND (p.read = 1)',
+            $qb->getSQL()
+        );
     }
 
     public function testComplexSelectWithSomeTableAliases(): void
@@ -827,7 +864,12 @@ class QueryBuilderTest extends TestCase
             ->innerJoin('u', 'permissions', 'p', 'p.user_id = u.id')
             ->innerJoin('articles', 'comments', 'c', 'c.article_id = articles.id');
 
-        self::assertEquals('SELECT u.id FROM users u INNER JOIN permissions p ON p.user_id = u.id, articles INNER JOIN comments c ON c.article_id = articles.id', $qb->getSQL());
+        self::assertEquals(
+            'SELECT u.id FROM users u'
+                . ' INNER JOIN permissions p ON p.user_id = u.id, articles'
+                . ' INNER JOIN comments c ON c.article_id = articles.id',
+            $qb->getSQL()
+        );
     }
 
     public function testSelectAllFromTableWithoutTableAlias(): void
@@ -901,7 +943,9 @@ class QueryBuilderTest extends TestCase
             ->join('a', 'table_b', 'a', 'a.fk_b = a.id');
 
         $this->expectException(QueryException::class);
-        $this->expectExceptionMessage("The given alias 'a' is not unique in FROM and JOIN clause table. The currently registered aliases are: a.");
+        $this->expectExceptionMessage(
+            "The given alias 'a' is not unique in FROM and JOIN clause table. The currently registered aliases are: a."
+        );
 
         $qb->getSQL();
     }

tests/SQLParserUtilsTest.php

@@ -41,9 +41,15 @@ class SQLParserUtilsTest extends TestCase
             ['SELECT "Doctrine\DBAL?" FROM foo WHERE bar = ?', [45]], // Ticket DBAL-558
             ['SELECT `Doctrine\DBAL?` FROM foo WHERE bar = ?', [45]], // Ticket DBAL-558
             ['SELECT [Doctrine\DBAL?] FROM foo WHERE bar = ?', [45]], // Ticket DBAL-558
-            ["SELECT * FROM FOO WHERE bar = 'it\\'s a trap? \\\\' OR bar = ?\nAND baz = \"\\\"quote\\\" me on it? \\\\\" OR baz = ?", [58, 104]],
+            [
+                <<<'SQL'
+SELECT * FROM FOO WHERE bar = 'it\'s a trap? \\' OR bar = ?
+AND baz = "\"quote\" me on it? \\" OR baz = ?
+SQL
+,
+                [58, 104],
+            ],
             ['SELECT * FROM foo WHERE foo = ? AND bar = ?', [1 => 42, 0 => 30]], // explicit keys
-
         ];
     }
 
@@ -64,16 +70,57 @@ class SQLParserUtilsTest extends TestCase
             ['SELECT :foo_id', [7 => 'foo_id']], // Ticket DBAL-231
             ['SELECT @rank := 1', []], // Ticket DBAL-398
             ['SELECT @rank := 1 AS rank, :foo AS foo FROM :bar', [27 => 'foo', 44 => 'bar']], // Ticket DBAL-398
-            ['SELECT * FROM Foo WHERE bar > :start_date AND baz > :start_date', [30 => 'start_date', 52 => 'start_date']], // Ticket GH-113
-            ['SELECT foo::date as date FROM Foo WHERE bar > :start_date AND baz > :start_date', [46 => 'start_date', 68 => 'start_date']], // Ticket GH-259
-            ['SELECT `d.ns:col_name` FROM my_table d WHERE `d.date` >= :param1', [57 => 'param1']], // Ticket DBAL-552
-            ['SELECT [d.ns:col_name] FROM my_table d WHERE [d.date] >= :param1', [57 => 'param1']], // Ticket DBAL-552
+            // Ticket GH-113
+            [
+                'SELECT * FROM Foo WHERE bar > :start_date AND baz > :start_date',
+                [
+                    30 => 'start_date',
+                    52 => 'start_date',
+                ],
+            ],
+
+            // Ticket GH-259
+            [
+                'SELECT foo::date as date FROM Foo WHERE bar > :start_date AND baz > :start_date',
+                [
+                    46 => 'start_date',
+                    68 => 'start_date',
+                ],
+            ],
+
+            // Ticket DBAL-552
+            [
+                'SELECT `d.ns:col_name` FROM my_table d WHERE `d.date` >= :param1',
+                [57 => 'param1'],
+            ],
+
+            // Ticket DBAL-552
+            ['SELECT [d.ns:col_name] FROM my_table d WHERE [d.date] >= :param1', [57 => 'param1']],
             ['SELECT * FROM foo WHERE jsonb_exists_any(foo.bar, ARRAY[:foo])', [56 => 'foo']], // Ticket GH-2295
-            ['SELECT * FROM foo WHERE jsonb_exists_any(foo.bar, array[:foo])', [56 => 'foo']],
-            ['SELECT table.column1, ARRAY[\'3\'] FROM schema.table table WHERE table.f1 = :foo AND ARRAY[\'3\']', [74 => 'foo']],
-            ['SELECT table.column1, ARRAY[\'3\']::integer[] FROM schema.table table WHERE table.f1 = :foo AND ARRAY[\'3\']::integer[]', [85 => 'foo']],
-            ['SELECT table.column1, ARRAY[:foo] FROM schema.table table WHERE table.f1 = :bar AND ARRAY[\'3\']', [28 => 'foo', 75 => 'bar']],
-            ['SELECT table.column1, ARRAY[:foo]::integer[] FROM schema.table table WHERE table.f1 = :bar AND ARRAY[\'3\']::integer[]', [28 => 'foo', 86 => 'bar']],
+            ['SELECT * FROM foo WHERE jsonb_exists_any(foo.bar, array[:foo])', [56 => 'foo']],[
+                'SELECT table.column1, ARRAY[\'3\'] FROM schema.table table WHERE table.f1 = :foo AND ARRAY[\'3\']',
+                [74 => 'foo'],
+            ],
+            [
+                'SELECT table.column1, ARRAY[\'3\']::integer[] FROM schema.table table'
+                    . ' WHERE table.f1 = :foo AND ARRAY[\'3\']::integer[]',
+                [85 => 'foo'],
+            ],
+            [
+                'SELECT table.column1, ARRAY[:foo] FROM schema.table table WHERE table.f1 = :bar AND ARRAY[\'3\']',
+                [
+                    28 => 'foo',
+                    75 => 'bar',
+                ],
+            ],
+            [
+                'SELECT table.column1, ARRAY[:foo]::integer[] FROM schema.table table'
+                    . ' WHERE table.f1 = :bar AND ARRAY[\'3\']::integer[]',
+                [
+                    28 => 'foo',
+                    86 => 'bar',
+                ],
+            ],
             [
                 <<<'SQLDATA'
 SELECT * FROM foo WHERE 
@@ -91,12 +138,60 @@ SQLDATA
                     190 => 'a_param3',
                 ],
             ],
-            ["SELECT data.age AS age, data.id AS id, data.name AS name, data.id AS id FROM test_data data WHERE (data.description LIKE :condition_0 ESCAPE '\\\\') AND (data.description LIKE :condition_1 ESCAPE '\\\\') ORDER BY id ASC", [121 => 'condition_0', 174 => 'condition_1']],
-            ['SELECT data.age AS age, data.id AS id, data.name AS name, data.id AS id FROM test_data data WHERE (data.description LIKE :condition_0 ESCAPE "\\\\") AND (data.description LIKE :condition_1 ESCAPE "\\\\") ORDER BY id ASC', [121 => 'condition_0', 174 => 'condition_1']],
-            ['SELECT data.age AS age, data.id AS id, data.name AS name, data.id AS id FROM test_data data WHERE (data.description LIKE :condition_0 ESCAPE "\\\\") AND (data.description LIKE :condition_1 ESCAPE \'\\\\\') ORDER BY id ASC', [121 => 'condition_0', 174 => 'condition_1']],
-            ['SELECT data.age AS age, data.id AS id, data.name AS name, data.id AS id FROM test_data data WHERE (data.description LIKE :condition_0 ESCAPE `\\\\`) AND (data.description LIKE :condition_1 ESCAPE `\\\\`) ORDER BY id ASC', [121 => 'condition_0', 174 => 'condition_1']],
-            ['SELECT data.age AS age, data.id AS id, data.name AS name, data.id AS id FROM test_data data WHERE (data.description LIKE :condition_0 ESCAPE \'\\\\\') AND (data.description LIKE :condition_1 ESCAPE `\\\\`) ORDER BY id ASC', [121 => 'condition_0', 174 => 'condition_1']],
-            ["SELECT * FROM Foo WHERE (foo.bar LIKE :condition_0 ESCAPE '\') AND (foo.baz = :condition_1) AND (foo.bak LIKE :condition_2 ESCAPE '\')", [38 => 'condition_0', 78 => 'condition_1', 110 => 'condition_2']],
+            [
+                'SELECT data.age AS age, data.id AS id, data.name AS name, data.id AS id FROM test_data data'
+                    . " WHERE (data.description LIKE :condition_0 ESCAPE '\\\\')"
+                    . " AND (data.description LIKE :condition_1 ESCAPE '\\\\') ORDER BY id ASC",
+                [
+                    121 => 'condition_0',
+                    174 => 'condition_1',
+                ],
+            ],
+            [
+                'SELECT data.age AS age, data.id AS id, data.name AS name, data.id AS id FROM test_data data'
+                    . ' WHERE (data.description LIKE :condition_0 ESCAPE "\\\\")'
+                    . ' AND (data.description LIKE :condition_1 ESCAPE "\\\\") ORDER BY id ASC',
+                [
+                    121 => 'condition_0',
+                    174 => 'condition_1',
+                ],
+            ],
+            [
+                'SELECT data.age AS age, data.id AS id, data.name AS name, data.id AS id FROM test_data data'
+                    . ' WHERE (data.description LIKE :condition_0 ESCAPE "\\\\")'
+                    . ' AND (data.description LIKE :condition_1 ESCAPE \'\\\\\') ORDER BY id ASC',
+                [
+                    121 => 'condition_0',
+                    174 => 'condition_1',
+                ],
+            ],
+            [
+                'SELECT data.age AS age, data.id AS id, data.name AS name, data.id AS id FROM test_data data'
+                    . ' WHERE (data.description LIKE :condition_0 ESCAPE `\\\\`)'
+                    . ' AND (data.description LIKE :condition_1 ESCAPE `\\\\`) ORDER BY id ASC',
+                [
+                    121 => 'condition_0',
+                    174 => 'condition_1',
+                ],
+            ],
+            [
+                'SELECT data.age AS age, data.id AS id, data.name AS name, data.id AS id FROM test_data data'
+                    . ' WHERE (data.description LIKE :condition_0 ESCAPE \'\\\\\')'
+                    . ' AND (data.description LIKE :condition_1 ESCAPE `\\\\`) ORDER BY id ASC',
+                [
+                    121 => 'condition_0',
+                    174 => 'condition_1',
+                ],
+            ],
+            [
+                "SELECT * FROM Foo WHERE (foo.bar LIKE :condition_0 ESCAPE '\')"
+                    . " AND (foo.baz = :condition_1) AND (foo.bak LIKE :condition_2 ESCAPE '\')",
+                [
+                    38 => 'condition_0',
+                    78 => 'condition_1',
+                    110 => 'condition_2',
+                ],
+            ],
         ];
     }
 

tests/Schema/ComparatorTest.php

@@ -74,7 +74,8 @@ class ComparatorTest extends TestCase
     public function testCompareMissingTable(): void
     {
         $schemaConfig = new SchemaConfig();
-        $table        = new Table('bugdb', ['integercolumn1' => new Column('integercolumn1', Type::getType('integer'))]);
+
+        $table = new Table('bugdb', ['integercolumn1' => new Column('integercolumn1', Type::getType('integer'))]);
         $table->setSchemaConfig($schemaConfig);
 
         $schema1 = new Schema([$table], [], $schemaConfig);
@@ -88,7 +89,8 @@ class ComparatorTest extends TestCase
     public function testCompareNewTable(): void
     {
         $schemaConfig = new SchemaConfig();
-        $table        = new Table('bugdb', ['integercolumn1' => new Column('integercolumn1', Type::getType('integer'))]);
+
+        $table = new Table('bugdb', ['integercolumn1' => new Column('integercolumn1', Type::getType('integer'))]);
         $table->setSchemaConfig($schemaConfig);
 
         $schema1 = new Schema([], [], $schemaConfig);
@@ -253,12 +255,12 @@ class ComparatorTest extends TestCase
         $c         = new Comparator();
         $tableDiff = $c->diffTable($tableA, $tableB);
 
-        self::assertCount(1, $tableDiff->renamedColumns, 'we should have one rename datecolumn1 => new_datecolumn1.');
-        self::assertArrayHasKey('datecolumn1', $tableDiff->renamedColumns, "'datecolumn1' should be set to be renamed to new_datecolumn1");
-        self::assertCount(1, $tableDiff->addedColumns, "'new_datecolumn2' should be added");
-        self::assertArrayHasKey('new_datecolumn2', $tableDiff->addedColumns, "'new_datecolumn2' should be added, not created through renaming!");
-        self::assertCount(0, $tableDiff->removedColumns, 'Nothing should be removed.');
-        self::assertCount(0, $tableDiff->changedColumns, 'Nothing should be changed as all columns old & new have diff names.');
+        self::assertCount(1, $tableDiff->renamedColumns);
+        self::assertArrayHasKey('datecolumn1', $tableDiff->renamedColumns);
+        self::assertCount(1, $tableDiff->addedColumns);
+        self::assertArrayHasKey('new_datecolumn2', $tableDiff->addedColumns);
+        self::assertCount(0, $tableDiff->removedColumns);
+        self::assertCount(0, $tableDiff->changedColumns);
     }
 
     public function testCompareRemovedIndex(): void
@@ -744,12 +746,12 @@ class ComparatorTest extends TestCase
         $c         = new Comparator();
         $tableDiff = $c->diffTable($tableA, $tableB);
 
-        self::assertCount(1, $tableDiff->addedColumns, "'baz' should be added, not created through renaming!");
-        self::assertArrayHasKey('baz', $tableDiff->addedColumns, "'baz' should be added, not created through renaming!");
-        self::assertCount(2, $tableDiff->removedColumns, "'foo' and 'bar' should both be dropped, an ambiguity exists which one could be renamed to 'baz'.");
-        self::assertArrayHasKey('foo', $tableDiff->removedColumns, "'foo' should be removed.");
-        self::assertArrayHasKey('bar', $tableDiff->removedColumns, "'bar' should be removed.");
-        self::assertCount(0, $tableDiff->renamedColumns, 'no renamings should take place.');
+        self::assertCount(1, $tableDiff->addedColumns);
+        self::assertArrayHasKey('baz', $tableDiff->addedColumns);
+        self::assertCount(2, $tableDiff->removedColumns);
+        self::assertArrayHasKey('foo', $tableDiff->removedColumns);
+        self::assertArrayHasKey('bar', $tableDiff->removedColumns);
+        self::assertCount(0, $tableDiff->renamedColumns);
     }
 
     public function testDetectRenameIndex(): void
@@ -1034,11 +1036,14 @@ class ComparatorTest extends TestCase
         $table     = $newSchema->createTable('foo');
         $table->addColumn('id', 'string');
 
-        $expected                      = new SchemaDiff();
-        $expected->fromSchema          = $oldSchema;
-        $tableDiff                     = $expected->changedTables['foo'] = new TableDiff('foo');
-        $tableDiff->fromTable          = $tableFoo;
-        $columnDiff                    = $tableDiff->changedColumns['id'] = new ColumnDiff('id', $table->getColumn('id'));
+        $expected             = new SchemaDiff();
+        $expected->fromSchema = $oldSchema;
+
+        $tableDiff            = $expected->changedTables['foo'] = new TableDiff('foo');
+        $tableDiff->fromTable = $tableFoo;
+
+        $columnDiff = $tableDiff->changedColumns['id'] = new ColumnDiff('id', $table->getColumn('id'));
+
         $columnDiff->fromColumn        = $tableFoo->getColumn('id');
         $columnDiff->changedProperties = ['type'];
 
@@ -1056,11 +1061,14 @@ class ComparatorTest extends TestCase
         $table     = $newSchema->createTable('foo');
         $table->addColumn('id', 'binary', ['length' => 42, 'fixed' => true]);
 
-        $expected                      = new SchemaDiff();
-        $expected->fromSchema          = $oldSchema;
-        $tableDiff                     = $expected->changedTables['foo'] = new TableDiff('foo');
-        $tableDiff->fromTable          = $tableFoo;
-        $columnDiff                    = $tableDiff->changedColumns['id'] = new ColumnDiff('id', $table->getColumn('id'));
+        $expected             = new SchemaDiff();
+        $expected->fromSchema = $oldSchema;
+
+        $tableDiff            = $expected->changedTables['foo'] = new TableDiff('foo');
+        $tableDiff->fromTable = $tableFoo;
+
+        $columnDiff = $tableDiff->changedColumns['id'] = new ColumnDiff('id', $table->getColumn('id'));
+
         $columnDiff->fromColumn        = $tableFoo->getColumn('id');
         $columnDiff->changedProperties = ['length', 'fixed'];
 
@@ -1078,8 +1086,12 @@ class ComparatorTest extends TestCase
         self::assertFalse($diff);
     }
 
-    public function assertSchemaTableChangeCount(SchemaDiff $diff, int $newTableCount = 0, int $changeTableCount = 0, int $removeTableCount = 0): void
-    {
+    public function assertSchemaTableChangeCount(
+        SchemaDiff $diff,
+        int $newTableCount = 0,
+        int $changeTableCount = 0,
+        int $removeTableCount = 0
+    ): void {
         self::assertCount($newTableCount, $diff->newTables);
         self::assertCount($changeTableCount, $diff->changedTables);
         self::assertCount($removeTableCount, $diff->removedTables);
@@ -1091,16 +1103,34 @@ class ComparatorTest extends TestCase
         int $changeSequenceCount = 0,
         int $removeSequenceCount = 0
     ): void {
-        self::assertCount($newSequenceCount, $diff->newSequences, 'Expected number of new sequences is wrong.');
-        self::assertCount($changeSequenceCount, $diff->changedSequences, 'Expected number of changed sequences is wrong.');
-        self::assertCount($removeSequenceCount, $diff->removedSequences, 'Expected number of removed sequences is wrong.');
+        self::assertCount($newSequenceCount, $diff->newSequences);
+        self::assertCount($changeSequenceCount, $diff->changedSequences);
+        self::assertCount($removeSequenceCount, $diff->removedSequences);
     }
 
     public function testDiffColumnPlatformOptions(): void
     {
-        $column1 = new Column('foo', Type::getType('string'), ['platformOptions' => ['foo' => 'foo', 'bar' => 'bar']]);
-        $column2 = new Column('foo', Type::getType('string'), ['platformOptions' => ['foo' => 'foo', 'foobar' => 'foobar']]);
-        $column3 = new Column('foo', Type::getType('string'), ['platformOptions' => ['foo' => 'foo', 'bar' => 'rab']]);
+        $column1 = new Column('foo', Type::getType('string'), [
+            'platformOptions' => [
+                'foo' => 'foo',
+                'bar' => 'bar',
+            ],
+        ]);
+
+        $column2 = new Column('foo', Type::getType('string'), [
+            'platformOptions' => [
+                'foo' => 'foo',
+                'foobar' => 'foobar',
+            ],
+        ]);
+
+        $column3 = new Column('foo', Type::getType('string'), [
+            'platformOptions' => [
+                'foo' => 'foo',
+                'bar' => 'rab',
+            ],
+        ]);
+
         $column4 = new Column('foo', Type::getType('string'));
 
         $comparator = new Comparator();

tests/Schema/MySqlInheritCharsetTest.php

@@ -39,31 +39,37 @@ class MySqlInheritCharsetTest extends TestCase
 
     public function testTableOptions(): void
     {
-        $eventManager = new EventManager();
-        $driverMock   = $this->createMock(Driver::class);
-        $platform     = new MySqlPlatform();
+        $platform = new MySqlPlatform();
 
         // default, no overrides
         $table = new Table('foobar', [new Column('aa', Type::getType('integer'))]);
         self::assertSame(
-            $platform->getCreateTableSQL($table),
-            ['CREATE TABLE foobar (aa INT NOT NULL) DEFAULT CHARACTER SET utf8 COLLATE `utf8_unicode_ci` ENGINE = InnoDB']
+            [
+                'CREATE TABLE foobar (aa INT NOT NULL)'
+                    . ' DEFAULT CHARACTER SET utf8 COLLATE `utf8_unicode_ci` ENGINE = InnoDB',
+            ],
+            $platform->getCreateTableSQL($table)
         );
 
         // explicit utf8
         $table = new Table('foobar', [new Column('aa', Type::getType('integer'))]);
         $table->addOption('charset', 'utf8');
         self::assertSame(
-            $platform->getCreateTableSQL($table),
-            ['CREATE TABLE foobar (aa INT NOT NULL) DEFAULT CHARACTER SET utf8 COLLATE `utf8_unicode_ci` ENGINE = InnoDB']
+            [
+                'CREATE TABLE foobar (aa INT NOT NULL)'
+                    . ' DEFAULT CHARACTER SET utf8 COLLATE `utf8_unicode_ci` ENGINE = InnoDB',
+            ],
+            $platform->getCreateTableSQL($table)
         );
 
         // explicit utf8mb4
         $table = new Table('foobar', [new Column('aa', Type::getType('integer'))]);
         $table->addOption('charset', 'utf8mb4');
         self::assertSame(
-            $platform->getCreateTableSQL($table),
-            ['CREATE TABLE foobar (aa INT NOT NULL) DEFAULT CHARACTER SET utf8mb4 COLLATE `utf8mb4_unicode_ci` ENGINE = InnoDB']
+            ['CREATE TABLE foobar (aa INT NOT NULL)'
+                    . ' DEFAULT CHARACTER SET utf8mb4 COLLATE `utf8mb4_unicode_ci` ENGINE = InnoDB',
+            ],
+            $platform->getCreateTableSQL($table)
         );
     }
 

tests/Schema/MySqlSchemaManagerTest.php

@@ -41,13 +41,26 @@ class MySqlSchemaManagerTest extends TestCase
 
     public function testCompositeForeignKeys(): void
     {
-        $this->conn->expects(self::once())->method('fetchAllAssociative')->will(self::returnValue($this->getFKDefinition()));
+        $this->conn->expects(self::once())
+            ->method('fetchAllAssociative')
+            ->willReturn($this->getFKDefinition());
+
         $fkeys = $this->manager->listTableForeignKeys('dummy', 'dummy');
         self::assertCount(1, $fkeys, 'Table has to have one foreign key.');
 
         self::assertInstanceOf(ForeignKeyConstraint::class, $fkeys[0]);
-        self::assertEquals(['column_1', 'column_2', 'column_3'], array_map('strtolower', $fkeys[0]->getLocalColumns()));
-        self::assertEquals(['column_1', 'column_2', 'column_3'], array_map('strtolower', $fkeys[0]->getForeignColumns()));
+
+        self::assertEquals([
+            'column_1',
+            'column_2',
+            'column_3',
+        ], array_map('strtolower', $fkeys[0]->getLocalColumns()));
+
+        self::assertEquals([
+            'column_1',
+            'column_2',
+            'column_3',
+        ], array_map('strtolower', $fkeys[0]->getForeignColumns()));
     }
 
     /**

tests/Schema/Platforms/MySQLSchemaTest.php

@@ -55,7 +55,13 @@ class MySQLSchemaTest extends TestCase
             $sqls[] = $this->platform->getCreateForeignKeySQL($fk, $tableOld);
         }
 
-        self::assertEquals(['ALTER TABLE test ADD CONSTRAINT FK_D87F7E0C8E48560F FOREIGN KEY (foo_id) REFERENCES test_foreign (foo_id)'], $sqls);
+        self::assertEquals(
+            [
+                'ALTER TABLE test ADD CONSTRAINT FK_D87F7E0C8E48560F FOREIGN KEY (foo_id)'
+                    . ' REFERENCES test_foreign (foo_id)',
+            ],
+            $sqls
+        );
     }
 
     public function testClobNoAlterTable(): void

tests/Schema/SchemaDiffTest.php

@@ -20,9 +20,17 @@ class SchemaDiffTest extends TestCase
 
         $sql = $diff->toSql($platform);
 
-        $expected = ['create_schema', 'drop_orphan_fk', 'alter_seq', 'drop_seq', 'create_seq', 'create_table', 'create_foreign_key', 'drop_table', 'alter_table'];
-
-        self::assertEquals($expected, $sql);
+        self::assertEquals([
+            'create_schema',
+            'drop_orphan_fk',
+            'alter_seq',
+            'drop_seq',
+            'create_seq',
+            'create_table',
+            'create_foreign_key',
+            'drop_table',
+            'alter_table',
+        ], $sql);
     }
 
     public function testSchemaDiffToSaveSql(): void

tests/Schema/SqliteSchemaManagerTest.php

@@ -30,22 +30,99 @@ class SqliteSchemaManagerTest extends TestCase
     public static function getDataColumnCollation(): iterable
     {
         return [
-            ['RTRIM', 'a', 'CREATE TABLE "a" ("a" text DEFAULT "aa" COLLATE "RTRIM" NOT NULL)'],
-            ['utf-8', 'a', 'CREATE TABLE "a" ("b" text UNIQUE NOT NULL COLLATE NOCASE, "a" text DEFAULT "aa" COLLATE "utf-8" NOT NULL)'],
-            ['NOCASE', 'a', 'CREATE TABLE "a" ("a" text DEFAULT (lower(ltrim(" a") || rtrim("a "))) CHECK ("a") NOT NULL COLLATE NOCASE UNIQUE, "b" text COLLATE RTRIM)'],
-            [null, 'a', 'CREATE TABLE "a" ("a" text CHECK ("a") NOT NULL, "b" text COLLATE RTRIM)'],
-            ['RTRIM', 'a"b', 'CREATE TABLE "a" ("a""b" text COLLATE RTRIM)'],
-            ['BINARY', 'b', 'CREATE TABLE "a" (bb TEXT COLLATE RTRIM, b VARCHAR(42) NOT NULL COLLATE BINARY)'],
-            ['BINARY', 'b', 'CREATE TABLE "a" (bbb TEXT COLLATE NOCASE, bb TEXT COLLATE RTRIM, b VARCHAR(42) NOT NULL COLLATE BINARY)'],
-            ['BINARY', 'b', 'CREATE TABLE "a" (b VARCHAR(42) NOT NULL COLLATE BINARY, bb TEXT COLLATE RTRIM)'],
-            ['utf-8', 'bar#', 'CREATE TABLE dummy_table (id INTEGER NOT NULL, foo VARCHAR(255) COLLATE "utf-8" NOT NULL, "bar#" VARCHAR(255) COLLATE "utf-8" NOT NULL, baz VARCHAR(255) COLLATE "utf-8" NOT NULL, PRIMARY KEY(id))'],
-            [null,    'bar#', 'CREATE TABLE dummy_table (id INTEGER NOT NULL, foo VARCHAR(255) NOT NULL, "bar#" VARCHAR(255) NOT NULL, baz VARCHAR(255) NOT NULL, PRIMARY KEY(id))'],
-            ['utf-8', 'baz',  'CREATE TABLE dummy_table (id INTEGER NOT NULL, foo VARCHAR(255) COLLATE "utf-8" NOT NULL, "bar#" INTEGER NOT NULL, baz VARCHAR(255) COLLATE "utf-8" NOT NULL, PRIMARY KEY(id))'],
-            [null,    'baz',  'CREATE TABLE dummy_table (id INTEGER NOT NULL, foo VARCHAR(255) NOT NULL, "bar#" INTEGER NOT NULL, baz VARCHAR(255) NOT NULL, PRIMARY KEY(id))'],
-            ['utf-8', 'bar/', 'CREATE TABLE dummy_table (id INTEGER NOT NULL, foo VARCHAR(255) COLLATE "utf-8" NOT NULL, "bar/" VARCHAR(255) COLLATE "utf-8" NOT NULL, baz VARCHAR(255) COLLATE "utf-8" NOT NULL, PRIMARY KEY(id))'],
-            [null,    'bar/', 'CREATE TABLE dummy_table (id INTEGER NOT NULL, foo VARCHAR(255) NOT NULL, "bar/" VARCHAR(255) NOT NULL, baz VARCHAR(255) NOT NULL, PRIMARY KEY(id))'],
-            ['utf-8', 'baz',  'CREATE TABLE dummy_table (id INTEGER NOT NULL, foo VARCHAR(255) COLLATE "utf-8" NOT NULL, "bar/" INTEGER NOT NULL, baz VARCHAR(255) COLLATE "utf-8" NOT NULL, PRIMARY KEY(id))'],
-            [null,    'baz',  'CREATE TABLE dummy_table (id INTEGER NOT NULL, foo VARCHAR(255) NOT NULL, "bar/" INTEGER NOT NULL, baz VARCHAR(255) NOT NULL, PRIMARY KEY(id))'],
+            [
+                'RTRIM',
+                'a',
+                'CREATE TABLE "a" ("a" text DEFAULT "aa" COLLATE "RTRIM" NOT NULL)',
+            ],
+            [
+                'utf-8',
+                'a',
+                'CREATE TABLE "a" ("b" text UNIQUE NOT NULL COLLATE NOCASE, '
+                    . '"a" text DEFAULT "aa" COLLATE "utf-8" NOT NULL)',
+            ],
+            [
+                'NOCASE',
+                'a',
+                'CREATE TABLE "a" ("a" text DEFAULT (lower(ltrim(" a") || rtrim("a ")))'
+                    . ' CHECK ("a") NOT NULL COLLATE NOCASE UNIQUE, "b" text COLLATE RTRIM)',
+            ],
+            [
+                null,
+                'a',
+                'CREATE TABLE "a" ("a" text CHECK ("a") NOT NULL, "b" text COLLATE RTRIM)',
+            ],
+            [
+                'RTRIM',
+                'a"b',
+                'CREATE TABLE "a" ("a""b" text COLLATE RTRIM)',
+            ],
+            [
+                'BINARY',
+                'b',
+                'CREATE TABLE "a" (bb TEXT COLLATE RTRIM, b VARCHAR(42) NOT NULL COLLATE BINARY)',
+            ],
+            [
+                'BINARY',
+                'b',
+                'CREATE TABLE "a" (bbb TEXT COLLATE NOCASE, bb TEXT COLLATE RTRIM, '
+                    . 'b VARCHAR(42) NOT NULL COLLATE BINARY)',
+            ],
+            [
+                'BINARY',
+                'b',
+                'CREATE TABLE "a" (b VARCHAR(42) NOT NULL COLLATE BINARY, bb TEXT COLLATE RTRIM)',
+            ],
+            [
+                'utf-8',
+                'bar#',
+                'CREATE TABLE dummy_table (id INTEGER NOT NULL, foo VARCHAR(255) COLLATE "utf-8" NOT NULL, '
+                    . '"bar#" VARCHAR(255) COLLATE "utf-8" NOT NULL, baz VARCHAR(255) COLLATE "utf-8" NOT NULL, '
+                    . 'PRIMARY KEY(id))',
+            ],
+            [
+                null,
+                'bar#',
+                'CREATE TABLE dummy_table (id INTEGER NOT NULL, foo VARCHAR(255) NOT NULL,'
+                    . ' "bar#" VARCHAR(255) NOT NULL, baz VARCHAR(255) NOT NULL, PRIMARY KEY(id))',
+            ],
+            [
+                'utf-8',
+                'baz',
+                'CREATE TABLE dummy_table (id INTEGER NOT NULL, foo VARCHAR(255) COLLATE "utf-8" NOT NULL,'
+                    . ' "bar#" INTEGER NOT NULL, baz VARCHAR(255) COLLATE "utf-8" NOT NULL, PRIMARY KEY(id))',
+            ],
+            [
+                null,
+                'baz',
+                'CREATE TABLE dummy_table (id INTEGER NOT NULL, foo VARCHAR(255) NOT NULL, "bar#" INTEGER NOT NULL, '
+                    . 'baz VARCHAR(255) NOT NULL, PRIMARY KEY(id))',
+            ],
+            [
+                'utf-8',
+                'bar/',
+                'CREATE TABLE dummy_table (id INTEGER NOT NULL, foo VARCHAR(255) COLLATE "utf-8" NOT NULL, '
+                    . '"bar/" VARCHAR(255) COLLATE "utf-8" NOT NULL, baz VARCHAR(255) COLLATE "utf-8" NOT NULL,'
+                    . ' PRIMARY KEY(id))',
+            ],
+            [
+                null,
+                'bar/',
+                'CREATE TABLE dummy_table (id INTEGER NOT NULL, foo VARCHAR(255) NOT NULL, '
+                    . '"bar/" VARCHAR(255) NOT NULL, baz VARCHAR(255) NOT NULL, PRIMARY KEY(id))',
+            ],
+            [
+                'utf-8',
+                'baz',
+                'CREATE TABLE dummy_table (id INTEGER NOT NULL, foo VARCHAR(255) COLLATE "utf-8" NOT NULL, '
+                    . '"bar/" INTEGER NOT NULL, baz VARCHAR(255) COLLATE "utf-8" NOT NULL, PRIMARY KEY(id))',
+            ],
+            [
+                null,
+                'baz',
+                'CREATE TABLE dummy_table (id INTEGER NOT NULL, foo VARCHAR(255) NOT NULL,'
+                    . ' "bar/" INTEGER NOT NULL, baz VARCHAR(255) NOT NULL, PRIMARY KEY(id))',
+            ],
         ];
     }
 
@@ -83,13 +160,15 @@ class SqliteSchemaManagerTest extends TestCase
             'Multiple similar columns with type comment 1' => [
                 null,
                 'b',
-                'CREATE TABLE "a" (a TEXT COLLATE RTRIM, "b" TEXT DEFAULT "a" COLLATE RTRIM, "bb" CLOB DEFAULT NULL COLLATE BINARY --(DC2Type:x)
+                'CREATE TABLE "a" (a TEXT COLLATE RTRIM, "b" TEXT DEFAULT "a" COLLATE RTRIM, '
+                    . '"bb" CLOB DEFAULT NULL COLLATE BINARY --(DC2Type:x)
 )',
             ],
             'Multiple similar columns with type comment 2' => [
                 '(DC2Type:x)',
                 'b',
-                'CREATE TABLE "a" (a TEXT COLLATE RTRIM, "bb" TEXT DEFAULT "a" COLLATE RTRIM, "b" CLOB DEFAULT NULL COLLATE BINARY --(DC2Type:x)
+                'CREATE TABLE "a" (a TEXT COLLATE RTRIM, "bb" TEXT DEFAULT "a" COLLATE RTRIM, '
+                    . '"b" CLOB DEFAULT NULL COLLATE BINARY --(DC2Type:x)
 )',
             ],
             'Multiple similar columns on different lines, with type comment 1' => [

tests/Schema/TableTest.php

@@ -468,7 +468,9 @@ class TableTest extends TestCase
         $table->setPrimaryKey(['baz']);
 
         $indexes = $table->getIndexes();
-        self::assertCount(2, $indexes, 'Table should only contain both the primary key table index and the unique one, even though it was overruled.');
+
+        // Table should only contain both the primary key table index and the unique one, even though it was overruled
+        self::assertCount(2, $indexes);
 
         self::assertTrue($table->hasPrimaryKey());
         self::assertTrue($table->hasIndex('idx_unique'));
@@ -525,7 +527,7 @@ class TableTest extends TestCase
         self::assertTrue($localTable->hasIndex('explicit_idx'));
     }
 
-    public function testAddingFulfillingExplicitIndexOverridingImplicitForeignKeyConstraintIndexWithSameNameDoesNotThrowException(): void
+    public function testAddingFulfillingExplicitIndexOverridingImplicitForeignKeyConstraintIndexWithSameName(): void
     {
         $foreignTable = new Table('foreign');
         $foreignTable->addColumn('id', 'integer');

tests/Schema/Visitor/RemoveNamespacedAssetsTest.php

@@ -25,7 +25,7 @@ class RemoveNamespacedAssetsTest extends TestCase
         $schema->visit(new RemoveNamespacedAssets());
 
         $tables = $schema->getTables();
-        self::assertEquals(['test.test', 'test.baz'], array_keys($tables), "Only 2 tables should be present, both in 'test' namespace.");
+        self::assertEquals(['test.test', 'test.baz'], array_keys($tables));
     }
 
     public function testCleanupForeignKeys(): void

tests/Tools/DumperTest.php

@@ -7,6 +7,9 @@ use ArrayObject;
 use DateTime;
 use DateTimeImmutable;
 use DateTimeZone;
+use Doctrine\DBAL\Tests\Tools\TestAsset\ChildClass;
+use Doctrine\DBAL\Tests\Tools\TestAsset\ChildWithSameAttributesClass;
+use Doctrine\DBAL\Tests\Tools\TestAsset\ParentClass;
 use Doctrine\DBAL\Tools\Dumper;
 use PHPUnit\Framework\TestCase;
 use stdClass;
@@ -118,23 +121,23 @@ class DumperTest extends TestCase
     {
         return [
             'different-attributes' => [
-                new TestAsset\ChildClass(),
+                new ChildClass(),
                 [
                     'childPublicAttribute' => 4,
                     'childProtectedAttribute:protected' => 5,
-                    'childPrivateAttribute:Doctrine\DBAL\Tests\Tools\TestAsset\ChildClass:private' => 6,
+                    'childPrivateAttribute:' . ChildClass::class . ':private' => 6,
                     'parentPublicAttribute' => 1,
                     'parentProtectedAttribute:protected' => 2,
-                    'parentPrivateAttribute:Doctrine\DBAL\Tests\Tools\TestAsset\ParentClass:private' => 3,
+                    'parentPrivateAttribute:' . ParentClass::class . ':private' => 3,
                 ],
             ],
             'same-attributes' => [
-                new TestAsset\ChildWithSameAttributesClass(),
+                new ChildWithSameAttributesClass(),
                 [
                     'parentPublicAttribute' => 4,
                     'parentProtectedAttribute:protected' => 5,
-                    'parentPrivateAttribute:Doctrine\DBAL\Tests\Tools\TestAsset\ChildWithSameAttributesClass:private' => 6,
-                    'parentPrivateAttribute:Doctrine\DBAL\Tests\Tools\TestAsset\ParentClass:private' => 3,
+                    'parentPrivateAttribute:' . ChildWithSameAttributesClass::class . ':private' => 6,
+                    'parentPrivateAttribute:' . ParentClass::class . ':private' => 3,
                 ],
             ],
         ];

tests/Types/ArrayTest.php

@@ -38,7 +38,11 @@ class ArrayTest extends TestCase
     public function testConversionFailure(): void
     {
         $this->expectException(ConversionException::class);
-        $this->expectExceptionMessage("Could not convert database value to 'array' as an error was triggered by the unserialization: 'unserialize(): Error at offset 0 of 7 bytes'");
+        $this->expectExceptionMessage(
+            "Could not convert database value to 'array' as an error was triggered by the unserialization:"
+                . " 'unserialize(): Error at offset 0 of 7 bytes'"
+        );
+
         $this->type->convertToPHPValue('abcdefg', $this->platform);
     }
 

tests/Types/ObjectTest.php

@@ -39,7 +39,10 @@ class ObjectTest extends TestCase
     public function testConversionFailure(): void
     {
         $this->expectException(ConversionException::class);
-        $this->expectExceptionMessage("Could not convert database value to 'object' as an error was triggered by the unserialization: 'unserialize(): Error at offset 0 of 7 bytes'");
+        $this->expectExceptionMessage(
+            "Could not convert database value to 'object' as an error was triggered by the unserialization:"
+                . " 'unserialize(): Error at offset 0 of 7 bytes'"
+        );
         $this->type->convertToPHPValue('abcdefg', $this->platform);
     }
 

tests/Types/StringTest.php

@@ -53,7 +53,7 @@ class StringTest extends TestCase
 
     public function testSQLConversion(): void
     {
-        self::assertFalse($this->type->canRequireSQLConversion(), 'String type can never require SQL conversion to work.');
+        self::assertFalse($this->type->canRequireSQLConversion());
         self::assertEquals('t.foo', $this->type->convertToDatabaseValueSQL('t.foo', $this->platform));
         self::assertEquals('t.foo', $this->type->convertToPHPValueSQL('t.foo', $this->platform));
     }

tests/phpstan-polyfill.php

-<?php
-
-declare(strict_types=1);
-
-// PHPStan does not read global constants from the stubs yet, remove this when it does
-if (defined('OCI_NO_AUTO_COMMIT')) {
-    return;
-}
-
-define('OCI_NO_AUTO_COMMIT', 0);