PHPUnit: set “processIsolation="true"” in conf instead of each test

liip · Oct 4, 2022 · 5bf63d1 · 5bf63d1
1 parent 5631c43
commit 5bf63d1
Show file tree

Hide file tree

Showing 11 changed files with 4 additions and 12 deletions.
diff --git a/phpunit.xml.dist b/phpunit.xml.dist
@@ -9,6 +9,7 @@
     beStrictAboutOutputDuringTests="true"
     executionOrder="random"
     resolveDependencies="true"
+    processIsolation="true"
 >
 
     <php>

diff --git a/tests/Test/ConfigEventsTest.php b/tests/Test/ConfigEventsTest.php
@@ -30,7 +30,6 @@
  * Tests/App/AppKernel.php.
  * So it must be loaded in a separate process.
  *
- * @runTestsInSeparateProcesses
  * @preserveGlobalState disabled
  * @IgnoreAnnotation("dataProvider")
  *

diff --git a/tests/Test/ConfigMysqlCacheDbTest.php b/tests/Test/ConfigMysqlCacheDbTest.php
@@ -31,7 +31,6 @@
  * Tests/App/AppKernel.php.
  * So it must be loaded in a separate process.
  *
- * @runTestsInSeparateProcesses
  * @preserveGlobalState disabled
  * @IgnoreAnnotation("group")
  *

diff --git a/tests/Test/ConfigMysqlKeepDatabaseAndSchemaTest.php b/tests/Test/ConfigMysqlKeepDatabaseAndSchemaTest.php
@@ -30,7 +30,6 @@
  * Tests/App/AppKernel.php.
  * So it must be loaded in a separate process.
  *
- * @runTestsInSeparateProcesses
  * @preserveGlobalState disabled
  * @IgnoreAnnotation("group")
  *

diff --git a/tests/Test/ConfigMysqlTest.php b/tests/Test/ConfigMysqlTest.php
@@ -44,7 +44,6 @@ class_alias('\Doctrine\Persistence\ObjectManager', '\Doctrine\Common\Persistence
  * Tests/App/AppKernel.php.
  * So it must be loaded in a separate process.
  *
- * @runTestsInSeparateProcesses
  * @preserveGlobalState disabled
  * @IgnoreAnnotation("group")
  *

diff --git a/tests/Test/ConfigMysqlUrlTest.php b/tests/Test/ConfigMysqlUrlTest.php
@@ -29,7 +29,6 @@
  * Tests/App/AppKernel.php.
  * So it must be loaded in a separate process.
  *
- * @runTestsInSeparateProcesses
  * @preserveGlobalState disabled
  *
  * @internal

diff --git a/tests/Test/ConfigPgsqlTest.php b/tests/Test/ConfigPgsqlTest.php
@@ -30,7 +30,6 @@
  * Tests/App/AppKernel.php.
  * So it must be loaded in a separate process.
  *
- * @runTestsInSeparateProcesses
  * @preserveGlobalState disabled
  *
  * @internal

diff --git a/tests/Test/ConfigPhpcrTest.php b/tests/Test/ConfigPhpcrTest.php
@@ -32,7 +32,6 @@
  * Tests/App/AppKernel.php.
  * So it must be loaded in a separate process.
  *
- * @runTestsInSeparateProcesses
  * @preserveGlobalState disabled
  *
  * @internal

diff --git a/tests/Test/ConfigSqliteTest.php b/tests/Test/ConfigSqliteTest.php
@@ -30,9 +30,9 @@ class_alias('\Doctrine\Persistence\ObjectManager', '\Doctrine\Common\Persistence
 use Liip\TestFixturesBundle\Services\DatabaseTools\AbstractDatabaseTool;
 use Liip\TestFixturesBundle\Services\DatabaseTools\ORMSqliteDatabaseTool;
 use Symfony\Bundle\FrameworkBundle\Test\KernelTestCase;
+use function count;
 
 /**
- * @runTestsInSeparateProcesses
  * @preserveGlobalState disabled
  *
  * @IgnoreAnnotation("depends")
@@ -370,7 +370,7 @@ public function testLoadFixturesFilesWithPurgeModeTruncate(): void
         // There are 10 users in the database
         $this->assertSame(
             10,
-            \count($users)
+            count($users)
         );
 
         $this->databaseTool->setPurgeMode(ORMPurger::PURGE_MODE_TRUNCATE);
@@ -393,7 +393,7 @@ public function testLoadFixturesFilesWithPurgeModeTruncate(): void
         // There are only 10 users in the database
         $this->assertSame(
             10,
-            \count($users)
+            count($users)
         );
 
         // Auto-increment hasn't been altered, so ids start from 11

diff --git a/tests/Test/ConfigSqliteUrlTest.php b/tests/Test/ConfigSqliteUrlTest.php
@@ -25,7 +25,6 @@ class_alias('\Doctrine\Persistence\ObjectManager', '\Doctrine\Common\Persistence
 /**
  * Run SQLite tests by using an URL for Doctrine.
  *
- * @runTestsInSeparateProcesses
  * @preserveGlobalState disabled
  *
  * @IgnoreAnnotation("depends")

diff --git a/tests/Test/ConfigTest.php b/tests/Test/ConfigTest.php
@@ -37,7 +37,6 @@ class_alias('\Doctrine\Persistence\ObjectManager', '\Doctrine\Common\Persistence
  * Tests/App/AppKernel.php.
  * So it must be loaded in a separate process.
  *
- * @runTestsInSeparateProcesses
  * @preserveGlobalState disabled
  *
  * Avoid conflict with PHPUnit annotation when reading QueryCount