CRM-16799 Contribution.get api - support 'is_test'
authoreileenmcnaughton <eileen@fuzion.co.nz>
Sun, 5 Jul 2015 08:16:48 +0000 (08:16 +0000)
committereileenmcnaughton <eileen@fuzion.co.nz>
Sun, 5 Jul 2015 08:16:48 +0000 (08:16 +0000)
It would probably be more correct to accept is_test & make contribution_test an alias of that for backwards compatibility, but this
is the quick version which at least supports the 'right' param

api/v3/Contribution.php
tests/phpunit/api/v3/ContributionTest.php

index b19302837e08137739c1af25341a03fa272f1b6f..9d4f3d2825a8ea99c2b35ad268c28d112bd4d727 100644 (file)
@@ -256,6 +256,7 @@ function _civicrm_api3_contribution_get_spec(&$params) {
     'api.default' => 0,
     'type' => CRM_Utils_Type::T_BOOLEAN,
     'title' => 'Get Test Contributions?',
+    'api.aliases' => array('is_test'),
   );
   $params['financial_type_id']['api.aliases'] = array('contribution_type_id');
   $params['payment_instrument_id']['api.aliases'] = array('contribution_payment_instrument', 'payment_instrument');
index e34a99e15e1066e8bb74e4161fd8730233ba57ed..3aa0e3592147b222117cc5a026fae53583575d4c 100644 (file)
@@ -197,6 +197,14 @@ class api_v3_ContributionTest extends CiviUnitTestCase {
     ));
   }
 
+  /**
+   * Test that test contributions can be retrieved.
+   */
+  public function testGetTestContribution() {
+    $this->callAPISuccess('Contribution', 'create', array_merge($this->_params, array('is_test' => 1)));
+    $this->callAPISuccessGetSingle('Contribution', array('is_test' => 1));
+  }
+
   /**
    * We need to ensure previous tested behaviour still works as part of the api contract.
    */
@@ -214,7 +222,7 @@ class api_v3_ContributionTest extends CiviUnitTestCase {
       'source' => 'SSF',
       'contribution_status_id' => 1,
     );
-    $this->_contribution = $this->callAPISuccess('contribution', 'create', $p);
+    $this->_contribution = $this->callAPISuccess('Contribution', 'create', $p);
 
     $params = array(
       'contribution_id' => $this->_contribution['id'],