bug #60645 [PhpUnitBridge] Skip bootstrap for PHPUnit >=10 (HypeMC)

nicolas-grekas · nicolas-grekas · commit 47f3667bff71 · 2025-06-04T10:32:26.000+02:00
This PR was merged into the 7.2 branch. Discussion ---------- [PhpUnitBridge] Skip bootstrap for PHPUnit >=10 | Q | A | ------------- | --- | Branch? | 7.2 | Bug fix? | yes | New feature? | no | Deprecations? | no | Issues | Fix symfony/recipes#1425 (comment) | License | MIT As suggested by `@wouterj`, this will skip the bootstrap file that is automatically [required by the bridge's `composer.json`](https://github.com/symfony/symfony/blob/158dff8106151b943fad16f85bcf8b70b7bb1178/src/Symfony/Bridge/PhpUnit/composer.json#L32). I'm not sure whether to skip the entire file or only the part that registers the `DeprecationErrorHandler`. Commits ------- 36974c3 [PhpUnitBridge] Skip bootstrap for PHPUnit >=10
diff --git a/src/Symfony/Bridge/PhpUnit/bootstrap.php b/src/Symfony/Bridge/PhpUnit/bootstrap.php
@@ -14,7 +14,11 @@
 use Symfony\Bridge\PhpUnit\DeprecationErrorHandler;
 
 // Detect if we need to serialize deprecations to a file.
-if (in_array(\PHP_SAPI, ['cli', 'phpdbg'], true) && $file = getenv('SYMFONY_DEPRECATIONS_SERIALIZE')) {
+if (
+    // Skip if we're using PHPUnit >=10
+    !class_exists(PHPUnit\Metadata\Metadata::class)
+    && in_array(\PHP_SAPI, ['cli', 'phpdbg'], true) && $file = getenv('SYMFONY_DEPRECATIONS_SERIALIZE')
+) {
     DeprecationErrorHandler::collectDeprecations($file);
 
     return;
@@ -46,6 +50,10 @@
     }
 }
 
-if ('disabled' !== getenv('SYMFONY_DEPRECATIONS_HELPER')) {
+if (
+    // Skip if we're using PHPUnit >=10
+    !class_exists(PHPUnit\Metadata\Metadata::class, false)
+    && 'disabled' !== getenv('SYMFONY_DEPRECATIONS_HELPER')
+) {
     DeprecationErrorHandler::register(getenv('SYMFONY_DEPRECATIONS_HELPER'));
 }
