[WIP] PHPUnit: set executionOrder as random and enable resolveDependencies

phpunit.xml.dist

@@ -7,6 +7,8 @@
     bootstrap="tests/App/bootstrap.php"
     colors="true"
     beStrictAboutOutputDuringTests="true"
+    executionOrder="random"
+    resolveDependencies="true"
 >
 
     <php>

src/Services/DatabaseTools/ORMDatabaseTool.php

@@ -165,7 +165,8 @@ public function loadFixtures(array $classNames = [], bool $append = false): Abst
 
     protected function disableForeignKeyChecksIfApplicable(): void
     {
-        if (!$this->isMysql()) {
+        // Don't disable FOREIGN_KEY_CHECKS is the database is not created yet
+        if (!$this->isMysql() || !$this->connection->isConnected()) {
             return;
         }
 

tests/Test/ConfigSqliteTest.php

@@ -26,8 +26,10 @@ class_alias('\Doctrine\Persistence\ObjectManager', '\Doctrine\Common\Persistence
 use Symfony\Bundle\FrameworkBundle\Test\KernelTestCase;
 
 /**
+ * @runTestsInSeparateProcesses
+ * @preserveGlobalState disabled
+ *
  * @IgnoreAnnotation("depends")
- * @IgnoreAnnotation("expectedException")
  */
 class ConfigSqliteTest extends KernelTestCase
 {
@@ -221,7 +223,7 @@ public function testLoadDependentFixturesWithDependencyInjected(): void
     /**
      * Use nelmio/alice.
      */
-    public function testLoadFixturesFiles(): void
+    public function testLoadFixturesFiles(): array
     {
         $fixtures = $this->loadFixtureFiles([
             '@AcmeBundle/DataFixtures/ORM/user.yml',
@@ -264,6 +266,8 @@ public function testLoadFixturesFiles(): void
         $this->assertTrue(
             $user->getEnabled()
         );
+
+        return $fixtures;
     }
 
     /**
@@ -285,6 +289,22 @@ public function testLoadNonexistentFixturesFiles(): void
      */
     public function testLoadFixturesFilesWithPurgeModeTruncate(): void
     {
+        // Load initial fixtures
+        $this->testLoadFixturesFiles();
+
+        $em = $this->getContainer()
+            ->get('doctrine.orm.entity_manager');
+
+        $users = $em->getRepository('LiipAcme:User')
+            ->findAll();
+
+        // There are 10 users in the database
+        $this->assertSame(
+            10,
+            count($users)
+        );
+
+        // Load fixtures with append = true
         $fixtures = $this->loadFixtureFiles([
             '@AcmeBundle/DataFixtures/ORM/user.yml',
         ], true, null, 'doctrine', ORMPurger::PURGE_MODE_TRUNCATE);
@@ -297,7 +317,17 @@ public function testLoadFixturesFilesWithPurgeModeTruncate(): void
             $fixtures
         );
 
-        $id = 1;
+        $users = $em->getRepository('LiipAcme:User')
+            ->findAll();
+
+        // There are only 10 users in the database
+        $this->assertSame(
+            10,
+            count($users)
+        );
+
+        // Auto-increment hasn't been altered, so ids start from 11
+        $id = 11;
         /** @var \Liip\Acme\Tests\App\Entity\User $user */
         foreach ($fixtures as $user) {
             $this->assertSame($id++, $user->getId());

tests/Test/ConfigTest.php

@@ -35,10 +35,6 @@ class_alias('\Doctrine\Persistence\ObjectManager', '\Doctrine\Common\Persistence
  * @runTestsInSeparateProcesses
  * @preserveGlobalState disabled
  *
- * Avoid conflict with PHPUnit annotation when reading QueryCount
- * annotation:
- *
- * @IgnoreAnnotation("expectedException")
  */
 class ConfigTest extends KernelTestCase
 {