fix: clear entity manager to avoid conflicts

when loading fixtures
alexislefebvre · Jul 2, 2024 · 80f51e5 · 80f51e5
1 parent ebe5cce
commit 80f51e5
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 1 deletion.
diff --git a/tests/Test/ConfigMysqlCacheDbTest.php b/tests/Test/ConfigMysqlCacheDbTest.php
@@ -13,6 +13,7 @@
 
 namespace Liip\Acme\Tests\Test;
 
+use Doctrine\ORM\EntityManagerInterface;
 use Liip\Acme\Tests\App\Entity\User;
 use Liip\Acme\Tests\AppConfigMysqlCacheDb\AppConfigMysqlKernelCacheDb;
 use PHPUnit\Framework\Attributes\PreserveGlobalState;
@@ -107,13 +108,16 @@ public function testLoadFixturesAndCheckBackup(): void
 
     public function testLoadFixturesCheckReferences(): void
     {
-        $this->markTestSkipped('This test is broken right now.');
         $referenceRepository = $this->databaseTool->loadFixtures([
             'Liip\Acme\Tests\App\DataFixtures\ORM\LoadUserData',
         ])->getReferenceRepository();
 
         $this->assertCount(1, $referenceRepository->getReferences());
 
+        /** @var EntityManagerInterface $entityManager */
+        $entityManager = $this->getTestContainer()->get(EntityManagerInterface::class);
+        $entityManager->clear();
+
         $referenceRepository = $this->databaseTool->loadFixtures([
             'Liip\Acme\Tests\App\DataFixtures\ORM\LoadUserData',
             'Liip\Acme\Tests\App\DataFixtures\ORM\LoadSecondUserData',

diff --git a/tests/Test/ConfigTest.php b/tests/Test/ConfigTest.php
@@ -13,6 +13,7 @@
 
 namespace Liip\Acme\Tests\Test;
 
+use Doctrine\ORM\EntityManagerInterface;
 use Doctrine\Persistence\ObjectRepository;
 use Liip\Acme\Tests\App\Entity\User;
 use Liip\Acme\Tests\AppConfig\AppConfigKernel;
@@ -44,6 +45,7 @@ class ConfigTest extends KernelTestCase
     private $userRepository;
     /** @var SqliteDatabaseBackup */
     private $sqliteDatabaseBackup;
+    private EntityManagerInterface $entityManager;
 
     protected function setUp(): void
     {
@@ -62,6 +64,10 @@ protected function setUp(): void
         $this->sqliteDatabaseBackup = $this->getTestContainer()->get(SqliteDatabaseBackup::class);
 
         $this->assertInstanceOf(SqliteDatabaseBackup::class, $this->sqliteDatabaseBackup);
+
+        $this->entityManager = $this->getTestContainer()->get(EntityManagerInterface::class);
+
+        $this->assertInstanceOf(EntityManagerInterface::class, $this->entityManager);
     }
 
     /**
@@ -135,6 +141,8 @@ public function testCacheCanBeDisabled(): void
 
         sleep(2);
 
+        $this->clearEntityManager();
+
         // Reload the fixtures.
         $this->databaseTool->loadFixtures($fixtures);
 
@@ -183,6 +191,8 @@ public function testBackupIsRefreshed(): void
 
         sleep(2);
 
+        $this->clearEntityManager();
+
         // Reload the fixtures.
         $this->databaseTool->loadFixtures($fixtures);
 
@@ -207,6 +217,8 @@ public function testBackupIsRefreshed(): void
 
         sleep(2);
 
+        $this->clearEntityManager();
+
         // Update the filemtime of the fixture file used as a dependency.
         touch($dependentFixtureFilePath);
 
@@ -237,6 +249,11 @@ protected static function getKernelClass(): string
         return AppConfigKernel::class;
     }
 
+    protected function clearEntityManager(): void
+    {
+        $this->entityManager->clear();
+    }
+
     protected function tearDown(): void
     {
         parent::tearDown();