diogo/symfony
Archived
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

do not register the test listener twice

Browse Source 
If the listener is already configured through the PHPUnit config, there
is no need to also enable it explicitly in the test runner.
This commit is contained in:
Christian Flothmann 2017-02-27 14:57:41 +01:00
parent 57a81eb992
commit cee435fefd
1 changed files with 10 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
src/Symfony/Bridge/PhpUnit/TextUI/TestRunner.php
View File

@ -27,9 +27,16 @@ class TestRunner extends \PHPUnit_TextUI_TestRunner
*/
protected function handleConfiguration(array &$arguments)
{
$arguments['listeners'] = isset($arguments['listeners']) ? $arguments['listeners'] : array();
$arguments['listeners'][] = new SymfonyTestsListener();
$listener = new SymfonyTestsListener();
return parent::handleConfiguration($arguments);
$result = parent::handleConfiguration($arguments);
$arguments['listeners'] = isset($arguments['listeners']) ? $arguments['listeners'] : array();
if (!array_filter($arguments['listeners'], function ($listener) { return $listener instanceof SymfonyTestsListener; })) {
$arguments['listeners'][] = $listener;
}
return $result;
}
}