minor #41948 [PhpUnitBridge] Avoid triggering the autoloader in Deprecation::isLegacy() (derrabus)

nicolas-grekas · nicolas-grekas · commit 3c72d518f5f6 · 2021-07-02T18:03:29.000+02:00
This PR was merged into the 4.4 branch. Discussion ---------- [PhpUnitBridge] Avoid triggering the autoloader in Deprecation::isLegacy() | Q | A | ------------- | --- | Branch? | 4.4 | Bug fix? | yes | New feature? | no | Deprecations? | no | Tickets | N/A | License | MIT | Doc PR | N/A The low-deps tests are currently failing because Deprecation::isLegacy() keeps triggering the autoloader and might even do so while `DebugClassLoader` considers a failed autoload attempt as an error. Commits ------- b95dfed Avoid triggering the autoloader in Deprecation::isLegacy()
diff --git a/src/Symfony/Bridge/PhpUnit/DeprecationErrorHandler/Deprecation.php b/src/Symfony/Bridge/PhpUnit/DeprecationErrorHandler/Deprecation.php
@@ -20,6 +20,8 @@
 use Symfony\Component\Debug\DebugClassLoader as LegacyDebugClassLoader;
 use Symfony\Component\ErrorHandler\DebugClassLoader;
 
+class_exists(Groups::class);
+
 /**
  * @internal
  */
@@ -194,7 +196,7 @@ public function isLegacy()
         }
 
         $method = $this->originatingMethod();
-        $groups = class_exists(Groups::class) ? [new Groups(), 'groups'] : [Test::class, 'getGroups'];
+        $groups = class_exists(Groups::class, false) ? [new Groups(), 'groups'] : [Test::class, 'getGroups'];
 
         return 0 === strpos($method, 'testLegacy')
             || 0 === strpos($method, 'provideLegacy')
