From: eileen <emcnaughton@wikimedia.org>
Date: Mon, 22 Mar 2021 10:45:44 +0000 (+1300)
Subject: Remove never-gonna-happen todos
X-Git-Url: https://vcs.fsf.org/?a=commitdiff_plain;h=202b2c69aebdf55a3dfab1f0b0617a14164eecda;p=civicrm-core.git

Remove never-gonna-happen todos
---

diff --git a/tests/phpunit/CRM/Core/BAO/ActionScheduleTest.php b/tests/phpunit/CRM/Core/BAO/ActionScheduleTest.php
index 6efa0146fc..21d96074e5 100644
--- a/tests/phpunit/CRM/Core/BAO/ActionScheduleTest.php
+++ b/tests/phpunit/CRM/Core/BAO/ActionScheduleTest.php
@@ -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;