SyntaxConformance::testSqlOperators cleanup fix - ensure entities are deleted

[eileenmcnaughton]

Overview

Fixes SyntaxConformance::testSqlOperators to clean up after itself

Before

The tearDown works for other tests in the class but not this one

$ids var is unused but appears to have been for delete tracking

After

Entities deleted

Technical Details

Tear Down iterates $this->deletableTestObjects

  public function tearDown() {
    foreach ($this->deletableTestObjects as $entityName => $entities) {
      foreach ($entities as $entityID) {
        CRM_Core_DAO::deleteTestObjects($entityName, ['id' => $entityID]);
      }
    }
  }

Comments

[civibot]

(Standard links)

• If this is your first pull-request for CiviCRM, please browse CONTRIBUTING.md for information about the development and testing processes.
• If you are reviewing this pull-request, you may wish to consult the test sites and the Review Standards (long template, short template).