Merge pull request #6819 from morozov/fix-test-switch-primary-key-order

Fix SchemaManagerFunctionalTestCase::testSwitchPrimaryKeyOrder()
doctrine · Mar 4, 2025 · be9c119 · be9c119
2 parents 98cf3a2 + 4093dd9
commit be9c119
Showing 1 changed file with 16 additions and 1 deletion.
diff --git a/tests/Functional/Schema/SchemaManagerFunctionalTestCase.php b/tests/Functional/Schema/SchemaManagerFunctionalTestCase.php
@@ -1462,8 +1462,21 @@ public function testChangeIndexWithForeignKeys(): void
         self::assertTrue($child->hasIndex('idx_2'));
     }
 
+    /** @throws Exception */
     public function testSwitchPrimaryKeyOrder(): void
     {
+        $platform = $this->connection->getDatabasePlatform();
+
+        if (
+            $platform instanceof DB2Platform
+            || $platform instanceof OraclePlatform
+            || $platform instanceof SQLServerPlatform
+        ) {
+            self::markTestIncomplete(
+                'Dropping primary key constraint on the currently used database platform is not implemented.',
+            );
+        }
+
         $prototype = new Table('test_switch_pk_order');
         $prototype->addColumn('foo_id', Types::INTEGER);
         $prototype->addColumn('bar_id', Types::INTEGER);
@@ -1481,11 +1494,13 @@ public function testSwitchPrimaryKeyOrder(): void
             $schemaManager->introspectTable('test_switch_pk_order'),
             $table,
         );
+        self::assertFalse($diff->isEmpty());
+        $schemaManager->alterTable($diff);
 
         $table      = $schemaManager->introspectTable('test_switch_pk_order');
         $primaryKey = $table->getPrimaryKey();
         self::assertNotNull($primaryKey);
-        self::assertSame(['foo_id', 'bar_id'], array_map('strtolower', $primaryKey->getColumns()));
+        self::assertSame(['bar_id', 'foo_id'], array_map('strtolower', $primaryKey->getColumns()));
     }
 
     public function testDropColumnWithDefault(): void