dev/core#720 add unit test, remove legacy code style.
authoreileen <emcnaughton@wikimedia.org>
Thu, 14 Feb 2019 22:50:56 +0000 (11:50 +1300)
committereileen <emcnaughton@wikimedia.org>
Fri, 15 Feb 2019 03:48:17 +0000 (16:48 +1300)
commitfaba82fcdd5b99364b6b838f0b418a2151237a06
tree4ebfde0b8a7b97900af80f4b9594db906313b199
parent7716b56a545a96689426ecd0eddf48213371b96f
dev/core#720 add unit test, remove legacy code style.

This PR adds tests to the summaryQuery code in preparation for changes.

The one non-test change is to remove the & from summaryContribution fn.

It is only called from one place in the code & that place does not return as reference.

```
  /**
   * @return mixed
   */
  public function getSummary() {
    return $this->_query->summaryContribution($this->_context);
  }
```
CRM/Contact/BAO/Query.php
tests/phpunit/CRM/Contact/BAO/QueryTest.php
tests/phpunit/CRMTraits/Financial/FinancialACLTrait.php
tests/phpunit/CRMTraits/Financial/PriceSetTrait.php
tests/phpunit/CiviTest/CiviUnitTestCase.php