From 0170d873c6d5339ebec21231af439068ec201b4e Mon Sep 17 00:00:00 2001 From: monishdeb Date: Tue, 29 Dec 2015 19:48:39 +0530 Subject: [PATCH] unit test fixes 4.7beta4 --- CRM/Contribute/BAO/Contribution.php | 2 +- tests/phpunit/api/v3/CustomValueTest.php | 5 +++-- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/CRM/Contribute/BAO/Contribution.php b/CRM/Contribute/BAO/Contribution.php index 74c6aefcff..3582ef87a0 100644 --- a/CRM/Contribute/BAO/Contribution.php +++ b/CRM/Contribute/BAO/Contribution.php @@ -3062,7 +3062,7 @@ INNER JOIN civicrm_activity ON civicrm_activity_contact.activity_id = civicrm_ac $trxnParams['trxn_date'] = !empty($params['contribution']->cancel_date) ? $params['contribution']->cancel_date : date('YmdHis'); } //CRM-16259, set is_payment flag for non pending status - if (!in_array(CRM_Utils_Array::value('contribution_status_id', $params), $pendingStatus)) { + if (!in_array($contributionStatus, $pendingStatus)) { $trxnParams['is_payment'] = 1; } if (!empty($params['payment_processor'])) { diff --git a/tests/phpunit/api/v3/CustomValueTest.php b/tests/phpunit/api/v3/CustomValueTest.php index 05dde076c5..b1fa3cb66d 100644 --- a/tests/phpunit/api/v3/CustomValueTest.php +++ b/tests/phpunit/api/v3/CustomValueTest.php @@ -109,14 +109,15 @@ class api_v3_CustomValueTest extends CiviUnitTestCase { foreach ($customFieldDataType as $dataType => $label) { switch ($dataType) { + case 'Country': + case 'StateProvince': + $this->foreignKeyChecksOff(); case 'String': case 'Link': case 'Int': case 'Float': case 'Money': case 'Date': - case 'Country': - case 'StateProvince': case 'Boolean': //Based on the custom field data-type choose desired SQL operators(to test with) and basic $type -- 2.25.1