From 66c25e4ce24fbd23750847b8ec6f7411e42dcdc7 Mon Sep 17 00:00:00 2001 From: eileen Date: Tue, 19 Feb 2019 12:20:59 +1300 Subject: [PATCH] [NFC, test class] formatting, remove unused variables --- .../CRM/Contribute/Form/SearchTest.php | 174 +++++++++--------- 1 file changed, 86 insertions(+), 88 deletions(-) diff --git a/tests/phpunit/CRM/Contribute/Form/SearchTest.php b/tests/phpunit/CRM/Contribute/Form/SearchTest.php index 9bbf4dda1a..5d2237e158 100644 --- a/tests/phpunit/CRM/Contribute/Form/SearchTest.php +++ b/tests/phpunit/CRM/Contribute/Form/SearchTest.php @@ -37,7 +37,7 @@ class CRM_Contribute_Form_SearchTest extends CiviUnitTestCase { protected $_individual; - protected $_tablesToTruncate = array('civicrm_contribution', 'civicrm_line_item'); + protected $_tablesToTruncate = ['civicrm_contribution', 'civicrm_line_item']; public function setUp() { parent::setUp(); @@ -52,12 +52,12 @@ class CRM_Contribute_Form_SearchTest extends CiviUnitTestCase { */ public function testBatchFilter() { $this->quickCleanup($this->_tablesToTruncate); - $contactID1 = $this->individualCreate(array(), 1); - $contactID2 = $this->individualCreate(array(), 2); + $contactID1 = $this->individualCreate([], 1); + $contactID2 = $this->individualCreate([], 2); $batchTitle = CRM_Batch_BAO_Batch::generateBatchName(); // create batch - $batch = $this->callAPISuccess('Batch', 'create', array( + $batch = $this->callAPISuccess('Batch', 'create', [ 'created_id' => $this->_individual, 'created_date' => CRM_Utils_Date::processDate(date("Y-m-d"), date("H:i:s")), 'status_id' => CRM_Core_Pseudoconstant::getKey('CRM_Batch_BAO_Batch', 'status_id', 'Data Entry'), @@ -65,17 +65,17 @@ class CRM_Contribute_Form_SearchTest extends CiviUnitTestCase { 'item_count' => 2, 'total' => 100, 'type_id' => array_search('Contribution', CRM_Batch_BAO_Batch::buildOptions('type_id')), - )); + ]); $batchID = $batch['id']; - $batchEntry = array( - 'primary_profiles' => array(1 => NULL, 2 => NULL, 3 => NULL), - 'primary_contact_id' => array( + $batchEntry = [ + 'primary_profiles' => [1 => NULL, 2 => NULL, 3 => NULL], + 'primary_contact_id' => [ 1 => $contactID1, 2 => $contactID2, - ), - 'field' => array( - 1 => array( + ], + 'field' => [ + 1 => [ 'financial_type' => 1, 'total_amount' => 70, 'receive_date' => '2013-07-24', @@ -83,8 +83,8 @@ class CRM_Contribute_Form_SearchTest extends CiviUnitTestCase { 'payment_instrument' => 1, 'check_number' => NULL, 'contribution_status_id' => 1, - ), - 2 => array( + ], + 2 => [ 'financial_type' => 1, 'total_amount' => 30, 'receive_date' => '2014-07-24', @@ -92,13 +92,13 @@ class CRM_Contribute_Form_SearchTest extends CiviUnitTestCase { 'payment_instrument' => 1, 'check_number' => NULL, 'contribution_status_id' => 1, - ), - ), + ], + ], 'actualBatchTotal' => 100, - ); + ]; // create random contribution to check IS NULL filter more precisely - $nonBatchContri = $this->callAPISuccess('Contribution', 'create', array( + $nonBatchContri = $this->callAPISuccess('Contribution', 'create', [ 'financial_type_id' => 1, 'total_amount' => 123, 'receive_date' => '2014-07-24', @@ -107,7 +107,7 @@ class CRM_Contribute_Form_SearchTest extends CiviUnitTestCase { 'check_number' => NULL, 'contribution_status_id' => 1, 'contact_id' => $this->_individual, - )); + ]); $nonBatchContriID = $nonBatchContri['id']; // process batch entries @@ -117,40 +117,40 @@ class CRM_Contribute_Form_SearchTest extends CiviUnitTestCase { // fetch created contributions $entities = $this->callAPISuccess('EntityBatch', 'get', array('batch_id' => $batchID)); - $ids = array(); + $ids = []; foreach ($entities['values'] as $value) { $ids[] = $value['entity_id']; } list($batchContriID1, $batchContriID2) = $ids; - $useCases = array( + $useCases = [ // Case 1: Search for ONLY those contributions which are created from batch - array( + [ 'form_value' => array('contribution_batch_id' => 'IS NOT NULL'), 'expected_count' => 2, 'expected_contribution' => array($batchContriID1, $batchContriID2), 'expected_qill' => 'Batch Name Not Null', - ), + ], // Case 2: Search for ONLY those contributions which are NOT created from batch - array( + [ 'form_value' => array('contribution_batch_id' => 'IS NULL'), 'expected_count' => 1, 'expected_contribution' => array($nonBatchContriID), 'expected_qill' => 'Batch Name Is Null', - ), + ], // Case 3: Search for ONLY those contributions which are created from batch ID - $batchID - array( + [ 'form_value' => array('contribution_batch_id' => $batchID), 'expected_count' => 2, 'expected_contribution' => array($batchContriID1, $batchContriID2), 'expected_qill' => 'Batch Name = ' . $batchTitle, - ), - ); + ], + ]; foreach ($useCases as $case) { $fv = $case['form_value']; CRM_Contact_BAO_Query::processSpecialFormValue($fv, array('contribution_batch_id')); $query = new CRM_Contact_BAO_Query(CRM_Contact_BAO_Query::convertFormValues($fv)); - list($select, $from, $where, $having) = $query->query(); + list($select, $from, $where) = $query->query(); // get and assert contribution count $contributions = CRM_Core_DAO::executeQuery(sprintf('SELECT DISTINCT civicrm_contribution.id %s %s AND civicrm_contribution.id IS NOT NULL', $from, $where))->fetchAll(); @@ -192,7 +192,7 @@ class CRM_Contribute_Form_SearchTest extends CiviUnitTestCase { 'total_amount' => 100, ); CRM_Core_BAO_FinancialTrxn::create($params); - $Contribution2 = $this->callAPISuccess('Contribution', 'create', array( + $this->callAPISuccess('Contribution', 'create', array( 'financial_type_id' => 1, 'total_amount' => 150, 'receive_date' => date('Ymd'), @@ -255,7 +255,7 @@ class CRM_Contribute_Form_SearchTest extends CiviUnitTestCase { $fv = $case['form_value']; CRM_Contact_BAO_Query::processSpecialFormValue($fv, array('financial_trxn_card_type_id')); $query = new CRM_Contact_BAO_Query(CRM_Contact_BAO_Query::convertFormValues($fv)); - list($select, $from, $where, $having) = $query->query(); + list($select, $from, $where) = $query->query(); // get and assert contribution count $contributions = CRM_Core_DAO::executeQuery(sprintf('SELECT DISTINCT civicrm_contribution.id %s %s AND civicrm_contribution.id IS NOT NULL', $from, $where))->fetchAll(); @@ -298,7 +298,7 @@ class CRM_Contribute_Form_SearchTest extends CiviUnitTestCase { 'pan_truncation' => 1234, ); CRM_Core_BAO_FinancialTrxn::create($params); - $Contribution2 = $this->callAPISuccess('Contribution', 'create', array( + $this->callAPISuccess('Contribution', 'create', array( 'financial_type_id' => 1, 'total_amount' => 150, 'receive_date' => date('Ymd'), @@ -355,7 +355,7 @@ class CRM_Contribute_Form_SearchTest extends CiviUnitTestCase { $fv = $case['form_value']; CRM_Contact_BAO_Query::processSpecialFormValue($fv, array('financial_trxn_pan_truncation')); $query = new CRM_Contact_BAO_Query(CRM_Contact_BAO_Query::convertFormValues($fv)); - list($select, $from, $where, $having) = $query->query(); + list($select, $from, $where) = $query->query(); // get and assert contribution count $contributions = CRM_Core_DAO::executeQuery(sprintf('SELECT DISTINCT civicrm_contribution.id %s %s AND civicrm_contribution.id IS NOT NULL', $from, $where))->fetchAll(); @@ -377,10 +377,10 @@ class CRM_Contribute_Form_SearchTest extends CiviUnitTestCase { */ public function testContributionRecurStatusFilter() { $this->quickCleanup($this->_tablesToTruncate); - $contactID1 = $this->individualCreate(array(), 1); - $contactID2 = $this->individualCreate(array(), 2); + $contactID1 = $this->individualCreate([], 1); + $contactID2 = $this->individualCreate([], 2); // "In Progress" recurring contribution for contactID1 - $ContributionRecur1 = $this->callAPISuccess('ContributionRecur', 'create', array( + $ContributionRecur1 = $this->callAPISuccess('ContributionRecur', 'create', [ 'sequential' => 1, 'contact_id' => $contactID1, 'frequency_interval' => 1, @@ -390,9 +390,9 @@ class CRM_Contribute_Form_SearchTest extends CiviUnitTestCase { 'payment_instrument_id' => 1, 'contribution_status_id' => 5, 'financial_type_id' => "Donation", - )); - $Contribution1 = $this->callAPISuccess('Contribution', 'create', array( - 'financial_type_id' => 1, + ]); + $Contribution1 = $this->callAPISuccess('Contribution', 'create', [ + 'financial_type_id' => 'Donation', 'total_amount' => 11, 'receive_date' => date('Ymd'), 'receive_date_time' => NULL, @@ -400,19 +400,18 @@ class CRM_Contribute_Form_SearchTest extends CiviUnitTestCase { 'contribution_status_id' => 1, 'contact_id' => $contactID1, 'contribution_recur_id' => $ContributionRecur1['id'], - 'financial_type_id' => "Donation", - )); - $params = array( + ]); + $params = [ 'to_financial_account_id' => 1, 'status_id' => 1, 'contribution_id' => $Contribution1['id'], 'payment_instrument_id' => 1, 'card_type_id' => 1, 'total_amount' => 11, - ); + ]; CRM_Core_BAO_FinancialTrxn::create($params); // "Completed" recurring contribution for contactID2 - $ContributionRecur2 = $this->callAPISuccess('ContributionRecur', 'create', array( + $ContributionRecur2 = $this->callAPISuccess('ContributionRecur', 'create', [ 'sequential' => 1, 'contact_id' => $contactID2, 'frequency_interval' => 1, @@ -422,9 +421,9 @@ class CRM_Contribute_Form_SearchTest extends CiviUnitTestCase { 'payment_instrument_id' => 1, 'contribution_status_id' => 1, 'financial_type_id' => "Donation", - )); - $Contribution2 = $this->callAPISuccess('Contribution', 'create', array( - 'financial_type_id' => 1, + ]); + $Contribution2 = $this->callAPISuccess('Contribution', 'create', [ + 'financial_type_id' => 'Donation', 'total_amount' => 22, 'receive_date' => date('Ymd'), 'receive_date_time' => NULL, @@ -432,41 +431,40 @@ class CRM_Contribute_Form_SearchTest extends CiviUnitTestCase { 'contribution_status_id' => 1, 'contact_id' => $contactID2, 'contribution_recur_id' => $ContributionRecur2['id'], - 'financial_type_id' => "Donation", - )); - $params = array( + ]); + $params = [ 'to_financial_account_id' => 1, 'status_id' => 1, 'contribution_id' => $Contribution2['id'], 'payment_instrument_id' => 1, 'card_type_id' => 1, 'total_amount' => 22, - ); + ]; CRM_Core_BAO_FinancialTrxn::create($params); - $useCases = array( + $useCases = [ // Case 1: Search for ONLY those recurring contributions with status "In Progress" - array( - 'form_value' => array('contribution_recur_contribution_status_id' => 5), + [ + 'form_value' => ['contribution_recur_contribution_status_id' => 5], 'expected_count' => 1, - 'expected_contact' => array($contactID1), + 'expected_contact' => [$contactID1], 'expected_qill' => "Recurring Contribution Status = 'In Progress'", - ), + ], // Case 2: Search for ONLY those recurring contributions with status "Completed" - array( - 'form_value' => array('contribution_recur_contribution_status_id' => 1), + [ + 'form_value' => ['contribution_recur_contribution_status_id' => 1], 'expected_count' => 1, - 'expected_contact' => array($contactID2), + 'expected_contact' => [$contactID2], 'expected_qill' => "Recurring Contribution Status = 'Completed'", - ), + ], // Case 3: Search for ONLY those recurring contributions with status "Cancelled" - array( - 'form_value' => array('contribution_recur_contribution_status_id' => 3), + [ + 'form_value' => ['contribution_recur_contribution_status_id' => 3], 'expected_count' => 0, - 'expected_contact' => array(), + 'expected_contact' => [], 'expected_qill' => "Recurring Contribution Status = 'Cancelled'", - ), - ); + ], + ]; foreach ($useCases as $case) { $fv = $case['form_value']; @@ -493,9 +491,9 @@ class CRM_Contribute_Form_SearchTest extends CiviUnitTestCase { */ public function testCancelledFilter() { $this->quickCleanup($this->_tablesToTruncate); - $contactID1 = $this->individualCreate(array(), 1); - $contactID2 = $this->individualCreate(array(), 2); - $Contribution1 = $this->callAPISuccess('Contribution', 'create', array( + $contactID1 = $this->individualCreate([], 1); + $contactID2 = $this->individualCreate([], 2); + $Contribution1 = $this->callAPISuccess('Contribution', 'create', [ 'financial_type_id' => 1, 'total_amount' => 100, 'receive_date' => date('Ymd'), @@ -505,8 +503,8 @@ class CRM_Contribute_Form_SearchTest extends CiviUnitTestCase { 'cancel_date' => date('Ymd'), 'cancel_reason' => 'Insufficient funds', 'contact_id' => $contactID1, - )); - $Contribution2 = $this->callAPISuccess('Contribution', 'create', array( + ]); + $this->callAPISuccess('Contribution', 'create', [ 'financial_type_id' => 1, 'total_amount' => 150, 'receive_date' => date('Ymd', strtotime(date('Y-m-d') . ' - 1 days')), @@ -516,8 +514,8 @@ class CRM_Contribute_Form_SearchTest extends CiviUnitTestCase { 'cancel_date' => date('Ymd', strtotime(date('Y-m-d') . ' - 1 days')), 'cancel_reason' => 'Insufficient funds', 'contact_id' => $contactID2, - )); - $Contribution3 = $this->callAPISuccess('Contribution', 'create', array( + ]); + $Contribution3 = $this->callAPISuccess('Contribution', 'create', [ 'financial_type_id' => 1, 'total_amount' => 200, 'receive_date' => date('Ymd'), @@ -527,37 +525,37 @@ class CRM_Contribute_Form_SearchTest extends CiviUnitTestCase { 'cancel_date' => date('Ymd'), 'cancel_reason' => 'Invalid Credit Card Number', 'contact_id' => $contactID1, - )); + ]); - $useCases = array( + $useCases = [ // Case 1: Search for Cancelled Date - array( - 'form_value' => array('cancel_date' => date('Y-m-d')), + [ + 'form_value' => ['cancel_date' => date('Y-m-d')], 'expected_count' => 2, - 'expected_contribution' => array($Contribution1['id'], $Contribution3['id']), + 'expected_contribution' => [$Contribution1['id'], $Contribution3['id']], 'expected_qill' => "Cancel Date Like '%" . date('Y-m-d') . "%'", - ), + ], // Case 2: Search for Cancelled Reason - array( - 'form_value' => array('cancel_reason' => 'Invalid Credit Card Number'), + [ + 'form_value' => ['cancel_reason' => 'Invalid Credit Card Number'], 'expected_count' => 1, - 'expected_contribution' => array($Contribution3['id']), + 'expected_contribution' => [$Contribution3['id']], 'expected_qill' => "Cancellation / Refund Reason Like '%Invalid Credit Card Number%'", - ), + ], // Case 3: Search for Cancelled Date and Cancelled Reason - array( - 'form_value' => array('cancel_date' => date('Y-m-d'), 'cancel_reason' => 'Insufficient funds'), + [ + 'form_value' => ['cancel_date' => date('Y-m-d'), 'cancel_reason' => 'Insufficient funds'], 'expected_count' => 1, - 'expected_contribution' => array($Contribution1['id']), + 'expected_contribution' => [$Contribution1['id']], 'expected_qill' => "Cancel Date Like '%" . date('Y-m-d') . "%'ANDCancellation / Refund Reason Like '%Insufficient funds%'", - ), - ); + ], + ]; foreach ($useCases as $case) { $fv = $case['form_value']; - CRM_Contact_BAO_Query::processSpecialFormValue($fv, array('cancel_date', 'cancel_reason')); + CRM_Contact_BAO_Query::processSpecialFormValue($fv, ['cancel_date', 'cancel_reason']); $query = new CRM_Contact_BAO_Query(CRM_Contact_BAO_Query::convertFormValues($fv)); - list($select, $from, $where, $having) = $query->query(); + list($select, $from, $where) = $query->query(); // get and assert contribution count $contributions = CRM_Core_DAO::executeQuery(sprintf('SELECT DISTINCT civicrm_contribution.id %s %s AND civicrm_contribution.id IS NOT NULL AND civicrm_contribution.contribution_status_id = 3', $from, $where))->fetchAll(); -- 2.25.1