Fix createTable

Author: tatevikg1

src/Domain/Subscription/Service/Manager/DynamicListAttrTablesManager.php

@@ -5,6 +5,7 @@
 namespace PhpList\Core\Domain\Subscription\Service\Manager;
 
 use Doctrine\DBAL\Connection;
+use Doctrine\DBAL\Exception\TableExistsException;
 use Doctrine\DBAL\Schema\Table;
 use InvalidArgumentException;
 use PhpList\Core\Domain\Subscription\Model\AttributeTypeEnum;
@@ -77,6 +78,12 @@ public function createOptionsTableIfNotExists(string $listTable): void
         $table->setPrimaryKey(['id']);
         $table->addUniqueIndex(['name'], 'uniq_' . $fullTableName . '_name');
 
-        $schemaManager->createTable($table);
+        try {
+            $schemaManager->createTable($table);
+        } catch (TableExistsException $e) {
+            // Table was created by a concurrent process or a previous test run.
+            // Since this method is idempotent by contract, swallow the exception.
+            return;
+        }
     }
 }

tests/Integration/Domain/Subscription/Service/DynamicListAttrManagerFunctionalTest.php

@@ -11,6 +11,7 @@
 use PhpList\Core\TestingSupport\Traits\DatabaseTestTrait;
 use PHPUnit\Framework\Assert;
 use Symfony\Bundle\FrameworkBundle\Test\KernelTestCase;
+use Throwable;
 
 /**
  * Functional test for DynamicListAttrManager and DynamicListAttrRepository working together
@@ -90,4 +91,24 @@ public function testCreateInsertAndFetchOptionsEndToEnd(): void
         Assert::assertNotNull($oneName);
         Assert::assertTrue(in_array($oneName, ['Blue', 'Red'], true));
     }
+
+    protected function tearDown(): void
+    {
+        try {
+            if ($this->entityManager !== null) {
+                $connection = $this->entityManager->getConnection();
+                $fullTable = 'phplist_listattr_colours';
+                // Use raw SQL for cleanup to avoid relying on SchemaManager in tests
+                $connection->executeStatement('DROP TABLE IF EXISTS ' . $fullTable);
+            }
+            // phpcs:ignore Generic.CodeAnalysis.EmptyStatement.DetectedCatch
+        } catch (Throwable $e) {
+            // Ignore cleanup failures to not mask test results
+        } finally {
+            $this->repo = null;
+            $this->manager = null;
+            $this->tablesManager = null;
+            parent::tearDown();
+        }
+    }
 }