disable global test listener when not registered

xabbuh · xabbuh · commit e068661240ce · 2017-03-01T20:11:57.000+01:00
The global test listener is always initialized to register the clock
mock and DNS mock as soon as possible. However, when the listener is
registered locally through the PHPUnit config, it will never be
registered as a listener. In thise case, the state of the local
listener must be reset to correctly report expected deprecation test
results.
diff --git a/src/Symfony/Bridge/PhpUnit/SymfonyTestsListener.php b/src/Symfony/Bridge/PhpUnit/SymfonyTestsListener.php
@@ -68,6 +68,12 @@ public function __destruct()
         }
     }
 
+    public function globalListenerDisabled()
+    {
+        self::$globallyEnabled = false;
+        $this->state = -1;
+    }
+
     public function startTestSuite(\PHPUnit_Framework_TestSuite $suite)
     {
         $suiteName = $suite->getName();
diff --git a/src/Symfony/Bridge/PhpUnit/TextUI/TestRunner.php b/src/Symfony/Bridge/PhpUnit/TextUI/TestRunner.php
@@ -33,7 +33,17 @@ protected function handleConfiguration(array &$arguments)
 
         $arguments['listeners'] = isset($arguments['listeners']) ? $arguments['listeners'] : array();
 
-        if (!array_filter($arguments['listeners'], function ($listener) { return $listener instanceof SymfonyTestsListener; })) {
+        $registeredLocally = false;
+
+        foreach ($arguments['listeners'] as $registeredListener) {
+            if ($registeredListener instanceof SymfonyTestsListener) {
+                $registeredListener->globalListenerDisabled();
+                $registeredLocally = true;
+                break;
+            }
+        }
+
+        if (!$registeredLocally) {
             $arguments['listeners'][] = $listener;
         }
 

Original file line number	Diff line number	Diff line change
`@@ -68,6 +68,12 @@ public function __destruct()`
`68`	`68`	`}`
`69`	`69`	`}`
`70`	`70`
	`71`	`+ public function globalListenerDisabled()`
	`72`	`+ {`
	`73`	`+ self::$globallyEnabled = false;`
	`74`	`+ $this->state = -1;`
	`75`	`+ }`
	`76`	`+`
`71`	`77`	`public function startTestSuite(\PHPUnit_Framework_TestSuite $suite)`
`72`	`78`	`{`
`73`	`79`	`$suiteName = $suite->getName();`