Test column diff on all platforms

src/Types/EnumType.php

@@ -13,7 +13,6 @@
 use Doctrine\DBAL\Types\Exception\InvalidType;
 use Doctrine\DBAL\Types\Exception\ValueNotConvertible;
 use ReflectionClass;
-use StringBackedEnum;
 use Throwable;
 use UnitEnum;
 

tests/Functional/Schema/ComparatorTestUtils.php

@@ -38,6 +38,15 @@ public static function diffFromDesiredToActualTable(
         );
     }
 
+    public static function assertNoDiffDetected(Connection $connection, Comparator $comparator, Table $table): void
+    {
+        $schemaManager = $connection->createSchemaManager();
+
+        $diff = self::diffFromActualToDesiredTable($schemaManager, $comparator, $table);
+
+        TestCase::assertTrue($diff->isEmpty());
+    }
+
     public static function assertDiffNotEmpty(Connection $connection, Comparator $comparator, Table $table): void
     {
         $schemaManager = $connection->createSchemaManager();

tests/Functional/Schema/MySQL/ComparatorTest.php

@@ -16,11 +16,6 @@
 use Doctrine\DBAL\Schema\Table;
 use Doctrine\DBAL\Tests\Functional\Schema\ComparatorTestUtils;
 use Doctrine\DBAL\Tests\FunctionalTestCase;
-use Doctrine\DBAL\Tests\Types\EnumObject;
-use Doctrine\DBAL\Tests\Types\EnumNative;
-use Doctrine\DBAL\Tests\Types\EnumNativeBacked;
-use Doctrine\DBAL\Types\EnumType;
-use Doctrine\DBAL\Types\Type;
 use Doctrine\DBAL\Types\Types;
 use PHPUnit\Framework\Attributes\DataProvider;
 
@@ -220,36 +215,16 @@ public function testMariaDb1043NativeJsonUpgradeDetected(): void
         ComparatorTestUtils::assertDiffNotEmpty($this->connection, $this->comparator, $table);
     }
 
-    public function testEnumDiffEmpty(): void
-    {
-        if (! $this->platform instanceof MySQLPlatform) {
-            self::markTestSkipped();
-        }
-
-        $table = new Table('enum_test_table');
-
-        $table->addColumn('enum_col', Types::ENUM, ['members' => ['a', 'b']]);
-        $this->dropAndCreateTable($table);
-
-        // Revert column to previous ENUM declaration
-        $sql = 'ALTER TABLE enum_test_table CHANGE enum_col enum_col ENUM(\'a\', \'b\') NOT NULL';
-        $this->connection->executeStatement($sql);
-
-        ComparatorTestUtils::assertDiffNotEmpty($this->connection, $this->comparator, $table);
-    }
-
     public function testEnumDiffDetected(): void
     {
-        if (! $this->platform instanceof MySQLPlatform) {
-            self::markTestSkipped();
-        }
-
         $table = new Table('enum_test_table');
 
         $table->addColumn('enum_col', Types::ENUM, ['members' => ['a', 'b']]);
         $this->dropAndCreateTable($table);
 
-        // Revert column to previous ENUM declaration
+        ComparatorTestUtils::assertNoDiffDetected($this->connection, $this->comparator, $table);
+
+        // Alter column to previous state and check diff
         $sql = 'ALTER TABLE enum_test_table CHANGE enum_col enum_col ENUM(\'NOT_A_MEMBER_ANYMORE\') NOT NULL';
         $this->connection->executeStatement($sql);
 

tests/Functional/Schema/Oracle/ComparatorTest.php

@@ -60,4 +60,20 @@ public function testChangeBinaryColumnFixed(): void
             $table,
         )->isEmpty());
     }
+
+    public function testEnumDiffDetected(): void
+    {
+        $table = new Table('enum_test_table');
+
+        $table->addColumn('enum_col', Types::ENUM, ['members' => ['a', 'b']]);
+        $this->dropAndCreateTable($table);
+
+        ComparatorTestUtils::assertNoDiffDetected($this->connection, $this->comparator, $table);
+
+        // Alter column to previous state and check diff
+        $sql = 'ALTER TABLE enum_test_table CHANGE enum_col enum_col ENUM(\'NOT_A_MEMBER_ANYMORE\') NOT NULL';
+        $this->connection->executeStatement($sql);
+
+        ComparatorTestUtils::assertDiffNotEmpty($this->connection, $this->comparator, $table);
+    }
 }

tests/Functional/Schema/PostgreSQL/ComparatorTest.php

@@ -92,4 +92,20 @@ private function testColumnModification(callable $addColumn, callable $modifyCol
             $table,
         )->isEmpty());
     }
+
+    public function testEnumDiffDetected(): void
+    {
+        $table = new Table('enum_test_table');
+
+        $table->addColumn('enum_col', Types::ENUM, ['members' => ['a', 'b']]);
+        $this->dropAndCreateTable($table);
+
+        ComparatorTestUtils::assertNoDiffDetected($this->connection, $this->comparator, $table);
+
+        // Alter column to previous state and check diff
+        $sql = 'ALTER TABLE enum_test_table CHANGE enum_col enum_col ENUM(\'NOT_A_MEMBER_ANYMORE\') NOT NULL';
+        $this->connection->executeStatement($sql);
+
+        ComparatorTestUtils::assertDiffNotEmpty($this->connection, $this->comparator, $table);
+    }
 }

tests/Functional/Schema/SQLite/ComparatorTest.php

@@ -42,4 +42,20 @@ public function testChangeTableCollation(): void
         $column->setPlatformOption('collation', 'NOCASE');
         ComparatorTestUtils::assertDiffNotEmpty($this->connection, $this->comparator, $table);
     }
+
+    public function testEnumDiffDetected(): void
+    {
+        $table = new Table('enum_test_table');
+
+        $table->addColumn('enum_col', Types::ENUM, ['members' => ['a', 'b']]);
+        $this->dropAndCreateTable($table);
+
+        ComparatorTestUtils::assertNoDiffDetected($this->connection, $this->comparator, $table);
+
+        // Alter column to previous state and check diff
+        $sql = 'ALTER TABLE enum_test_table ALTER COLUMN enum_col enum_col ENUM(\'NOT_A_MEMBER_ANYMORE\') NOT NULL';
+        $this->connection->executeStatement($sql);
+
+        ComparatorTestUtils::assertDiffNotEmpty($this->connection, $this->comparator, $table);
+    }
 }