DAOTest - Use docblock for type
authorTim Otten <totten@civicrm.org>
Thu, 25 Jul 2019 16:53:42 +0000 (09:53 -0700)
committerTim Otten <totten@civicrm.org>
Thu, 25 Jul 2019 16:53:42 +0000 (09:53 -0700)
This patch is going in as part of some cleanup/refactoring in a mid-cycle
change. Using the docblock allows the type to be communicated without
breaking on mid-cycle contract change.

tests/phpunit/CRM/Core/DAOTest.php

index 667a7e1943e52e8b4a514fb5e75f5ead2d214b28..32e8e38babb3c084a07ff4b7e4385702e4d5b629 100644 (file)
@@ -483,7 +483,10 @@ class CRM_Core_DAOTest extends CiviUnitTestCase {
    * @throws \Exception
    */
   public function testModifyQuery() {
-    $listener = function(\Symfony\Component\EventDispatcher\Event $e) {
+    /**
+     * @param \Civi\Core\Event\QueryEvent $e
+     */
+    $listener = function($e) {
       $e->query = '/* User :  hooked */' . $e->query;
     };
     Civi::dispatcher()->addListener('civi.db.query', $listener);
@@ -498,6 +501,9 @@ class CRM_Core_DAOTest extends CiviUnitTestCase {
    * Demonstrate it is modified showing the query now breaks.
    */
   public function testModifyAndBreakQuery() {
+    /**
+     * @param \Civi\Core\Event\QueryEvent $e
+     */
     $listener = function($e) {
       $e->query = '/* Forgot trailing comment marker' . $e->query;
     };