bug #32902 [PhpUnitBridge] Allow sutFqcnResolver to return array (VincentLanglet)

nicolas-grekas · nicolas-grekas · commit 926246ba72f3 · 2019-08-04T17:24:16.000+02:00
This PR was merged into the 3.4 branch. Discussion ---------- [PhpUnitBridge] Allow sutFqcnResolver to return array | Q | A | ------------- | --- | Branch? | 3.4 | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | #... | License | MIT | Doc PR | Phpunit handle multiple covers annotation: ``` /** * @Covers ClassOne * @Covers ClassTwo */ ``` The coverage solver was only allowed to return `ClassOne` or `ClassTwo` since it was transformed to `[ClassName]`. Returning `[ClassOne, ClassTwo]` was transform to [[ClassOne, ClassTwo]]` which was creating an error with phpunit. This PR fixed this case. Commits ------- 1451c0b Allow sutFqcnResolver to return array
diff --git a/src/Symfony/Bridge/PhpUnit/Legacy/CoverageListenerTrait.php b/src/Symfony/Bridge/PhpUnit/Legacy/CoverageListenerTrait.php
@@ -76,7 +76,7 @@ public function startTest($test)
         $cache = $r->getValue();
         $cache = array_replace_recursive($cache, array(
             \get_class($test) => array(
-                'covers' => array($sutFqcn),
+                'covers' => \is_array($sutFqcn) ? $sutFqcn : array($sutFqcn),
             ),
         ));
         $r->setValue($testClass, $cache);
