Remove never-gonna-happen todos
authoreileen <emcnaughton@wikimedia.org>
Mon, 22 Mar 2021 10:45:44 +0000 (23:45 +1300)
committereileen <emcnaughton@wikimedia.org>
Mon, 22 Mar 2021 10:45:44 +0000 (23:45 +1300)
tests/phpunit/CRM/Core/BAO/ActionScheduleTest.php

index 6efa0146fc596cb8b4551c5cbacd7559c2a41cb6..21d96074e50ac734c53364487a3ac09962320c97 100644 (file)
@@ -1111,12 +1111,6 @@ class CRM_Core_BAO_ActionScheduleTest extends CiviUnitTestCase {
     ]);
   }
 
-  /**
-   * For contacts/activities which don't match the schedule filter,
-   * an email should *not* be sent.
-   */
-  // TODO // function testActivityDateTime_NonMatch() { }
-
   /**
    * For contacts/members which match schedule based on join/start date,
    * an email should be sent.
@@ -1752,7 +1746,6 @@ class CRM_Core_BAO_ActionScheduleTest extends CiviUnitTestCase {
       [
         // Before the 2-week mark, no email.
         'time' => '2012-05-31 01:00:00',
-        // 'time' => '2012-06-01 01:00:00', // FIXME: Is this the right boundary?
         'recipients' => [],
       ],
     ]);
@@ -2126,12 +2119,6 @@ class CRM_Core_BAO_ActionScheduleTest extends CiviUnitTestCase {
     ]);
   }
 
-  // TODO // function testMembershipEndDate_NonMatch() { }
-  // TODO // function testEventTypeStartDate_Match() { }
-  // TODO // function testEventTypeEndDate_Match() { }
-  // TODO // function testEventNameStartDate_Match() { }
-  // TODO // function testEventNameEndDate_Match() { }
-
   /**
    * Run a series of cron jobs and make an assertion about email deliveries.
    *
@@ -2157,6 +2144,7 @@ class CRM_Core_BAO_ActionScheduleTest extends CiviUnitTestCase {
 
   /**
    * @var array
+   *
    * (DAO_Name => array(int)) List of items to garbage-collect during tearDown
    */
   private $_testObjects;