From 6298b166bb533b33e511da6ca4e437475f9d47c4 Mon Sep 17 00:00:00 2001 From: "deb.monish" Date: Mon, 1 Jan 2018 23:27:53 +0530 Subject: [PATCH] test failure fixes --- CRM/Contribute/Form/Contribution.php | 3 +- api/v3/examples/ContributionPage/Submit.php | 4 +- .../CRM/Contribute/BAO/ContributionTest.php | 2 - .../CRM/Contribute/Form/ContributionTest.php | 66 +------------------ .../CRM/Financial/BAO/FinancialItemTest.php | 2 - .../CRM/Financial/Form/PaymentEditTest.php | 5 +- tests/phpunit/api/v3/ContributionPageTest.php | 1 + tests/phpunit/api/v3/ContributionTest.php | 2 - tests/phpunit/api/v3/MembershipTest.php | 2 - 9 files changed, 8 insertions(+), 79 deletions(-) diff --git a/CRM/Contribute/Form/Contribution.php b/CRM/Contribute/Form/Contribution.php index fab5b05a94..a9c18b80c4 100644 --- a/CRM/Contribute/Form/Contribution.php +++ b/CRM/Contribute/Form/Contribution.php @@ -1288,10 +1288,9 @@ class CRM_Contribute_Form_Contribution extends CRM_Contribute_Form_AbstractEditP public function testSubmit($params, $action, $creditCardMode = NULL) { $defaults = array( 'soft_credit_contact_id' => array(), + 'receive_date' => date('Y-m-d H:i:s'), 'receipt_date' => '', - 'receipt_date_time' => '', 'cancel_date' => '', - 'cancel_date_time' => '', 'hidden_Premium' => 1, ); $this->_bltID = 5; diff --git a/api/v3/examples/ContributionPage/Submit.php b/api/v3/examples/ContributionPage/Submit.php index c24c4dbcb3..a1d8932776 100644 --- a/api/v3/examples/ContributionPage/Submit.php +++ b/api/v3/examples/ContributionPage/Submit.php @@ -26,8 +26,8 @@ function contribution_page_submit_example() { ), 'cvv2' => 123, 'pledge_id' => '1', - 'cid' => '77', - 'contact_id' => '77', + 'cid' => '83', + 'contact_id' => '83', 'amount' => '100', 'is_pledge' => TRUE, 'pledge_block_id' => 2, diff --git a/tests/phpunit/CRM/Contribute/BAO/ContributionTest.php b/tests/phpunit/CRM/Contribute/BAO/ContributionTest.php index 847e710196..8ff2a2021e 100644 --- a/tests/phpunit/CRM/Contribute/BAO/ContributionTest.php +++ b/tests/phpunit/CRM/Contribute/BAO/ContributionTest.php @@ -1203,8 +1203,6 @@ WHERE eft.entity_id = %1 AND ft.to_financial_account_id <> %2"; $form->testSubmit(array( 'total_amount' => 100, 'financial_type_id' => $financialType['id'], - 'receive_date' => '04/21/2015', - 'receive_date_time' => '11:27PM', 'contact_id' => $contactId, 'contribution_status_id' => 1, 'price_set_id' => 0, diff --git a/tests/phpunit/CRM/Contribute/Form/ContributionTest.php b/tests/phpunit/CRM/Contribute/Form/ContributionTest.php index 68d96949e5..005ca08bf4 100644 --- a/tests/phpunit/CRM/Contribute/Form/ContributionTest.php +++ b/tests/phpunit/CRM/Contribute/Form/ContributionTest.php @@ -149,8 +149,6 @@ class CRM_Contribute_Form_ContributionTest extends CiviUnitTestCase { $form->testSubmit(array( 'total_amount' => 50, 'financial_type_id' => 1, - 'receive_date' => '04/21/2015', - 'receive_date_time' => '11:27PM', 'contact_id' => $this->_individualId, 'payment_instrument_id' => array_search('Check', $this->paymentInstruments), 'contribution_status_id' => 1, @@ -168,8 +166,6 @@ class CRM_Contribute_Form_ContributionTest extends CiviUnitTestCase { $form->testSubmit(array( 'total_amount' => 50, 'financial_type_id' => 1, - 'receive_date' => '04/21/2015', - 'receive_date_time' => '11:27PM', 'contact_id' => $this->_individualId, 'payment_instrument_id' => array_search('Credit Card', $this->paymentInstruments), 'contribution_status_id' => 1, @@ -195,8 +191,6 @@ class CRM_Contribute_Form_ContributionTest extends CiviUnitTestCase { $form->testSubmit(array( 'total_amount' => 50, 'financial_type_id' => 1, - 'receive_date' => '04/21/2015', - 'receive_date_time' => '11:27PM', 'contact_id' => $this->_individualId, 'contribution_status_id' => 1, 'credit_card_number' => 4444333322221111, @@ -267,8 +261,6 @@ class CRM_Contribute_Form_ContributionTest extends CiviUnitTestCase { $form->testSubmit(array( 'total_amount' => 50, 'financial_type_id' => 1, - 'receive_date' => '04/21/2015', - 'receive_date_time' => '11:27PM', 'contact_id' => $this->_individualId, 'contribution_status_id' => 1, 'credit_card_number' => 4444333322221111, @@ -332,8 +324,6 @@ class CRM_Contribute_Form_ContributionTest extends CiviUnitTestCase { $form->testSubmit(array( 'total_amount' => 60, 'financial_type_id' => 1, - 'receive_date' => '04/21/2015', - 'receive_date_time' => '11:27PM', 'contact_id' => $this->_individualId, 'contribution_status_id' => 1, 'credit_card_number' => 4444333322221111, @@ -393,8 +383,6 @@ class CRM_Contribute_Form_ContributionTest extends CiviUnitTestCase { $form->testSubmit(array( 'total_amount' => 50, 'financial_type_id' => 1, - 'receive_date' => '04/21/2015', - 'receive_date_time' => '11:27PM', 'contact_id' => $this->_individualId, 'payment_instrument_id' => array_search('Credit Card', $this->paymentInstruments), 'contribution_status_id' => 1, @@ -446,8 +434,6 @@ class CRM_Contribute_Form_ContributionTest extends CiviUnitTestCase { $form->testSubmit(array( 'total_amount' => 50, 'financial_type_id' => 1, - 'receive_date' => '04/21/2015', - 'receive_date_time' => '11:27PM', 'contact_id' => $this->_individualId, 'payment_instrument_id' => array_search('Credit Card', $this->paymentInstruments), 'contribution_status_id' => 1, @@ -506,8 +492,6 @@ class CRM_Contribute_Form_ContributionTest extends CiviUnitTestCase { $form->testSubmit(array( 'total_amount' => 50, 'financial_type_id' => 1, - 'receive_date' => '04/21/2015', - 'receive_date_time' => '11:27PM', 'contact_id' => $this->_individualId, 'payment_instrument_id' => array_search('Credit Card', $this->paymentInstruments), 'payment_processor_id' => $this->paymentProcessorID, @@ -546,8 +530,6 @@ class CRM_Contribute_Form_ContributionTest extends CiviUnitTestCase { $form->testSubmit(array( 'total_amount' => 50, 'financial_type_id' => 1, - 'receive_date' => '04/21/2015', - 'receive_date_time' => '11:27PM', 'contact_id' => $this->_individualId, 'payment_instrument_id' => array_search('Credit Card', $this->paymentInstruments), 'payment_processor_id' => $this->paymentProcessorID, @@ -572,7 +554,7 @@ class CRM_Contribute_Form_ContributionTest extends CiviUnitTestCase { */ public function testSubmitCreditCardWithRecur() { $form = new CRM_Contribute_Form_Contribution(); - $receiveDate = date('m/d/Y', strtotime('+1 month')); + $receiveDate = date('Y-m-d H:i:s', strtotime('+1 month')); $form->testSubmit(array( 'total_amount' => 50, 'financial_type_id' => 1, @@ -581,7 +563,6 @@ class CRM_Contribute_Form_ContributionTest extends CiviUnitTestCase { 'frequency_unit' => 'month', 'installments' => 2, 'receive_date' => $receiveDate, - 'receive_date_time' => '11:27PM', 'contact_id' => $this->_individualId, 'payment_instrument_id' => array_search('Credit Card', $this->paymentInstruments), 'payment_processor_id' => $this->paymentProcessorID, @@ -592,7 +573,7 @@ class CRM_Contribute_Form_ContributionTest extends CiviUnitTestCase { 'live' ); $contribution = $this->callAPISuccessGetSingle('Contribution', array('return' => 'receive_date')); - $this->assertEquals(date("m/d/Y", strtotime($contribution['receive_date'])), $receiveDate); + $this->assertEquals($contribution['receive_date'], $receiveDate); } /** @@ -603,8 +584,6 @@ class CRM_Contribute_Form_ContributionTest extends CiviUnitTestCase { $form->testSubmit(array( 'total_amount' => 50, 'financial_type_id' => 1, - 'receive_date' => '04/21/2015', - 'receive_date_time' => '11:27PM', 'contact_id' => $this->_individualId, 'payment_instrument_id' => array_search('Credit Card', $this->paymentInstruments), 'payment_processor_id' => $this->paymentProcessorID, @@ -630,8 +609,6 @@ class CRM_Contribute_Form_ContributionTest extends CiviUnitTestCase { $form->testSubmit(array( 'total_amount' => 50, 'financial_type_id' => 1, - 'receive_date' => '04/21/2015', - 'receive_date_time' => '11:27PM', 'contact_id' => $this->_individualId, 'is_email_receipt' => TRUE, 'from_email_address' => 'test@test.com', @@ -688,8 +665,6 @@ class CRM_Contribute_Form_ContributionTest extends CiviUnitTestCase { $params = array( 'total_amount' => 100, 'financial_type_id' => $donationFT, - 'receive_date' => '04/21/2015', - 'receive_date_time' => '11:27PM', 'contact_id' => $this->_individualId, 'is_email_receipt' => TRUE, 'from_email_address' => 'test@test.com', @@ -746,8 +721,6 @@ Price Field - Price Field 1 1 $ 100.00 $ 100.00 $form->testSubmit(array( 'total_amount' => 50, 'financial_type_id' => 1, - 'receive_date' => '04/21/2015', - 'receive_date_time' => '11:27PM', 'contact_id' => $this->_individualId, 'payment_instrument_id' => array_search('Check', $this->paymentInstruments), 'pledge_payment_id' => $pledgePaymentID, @@ -768,8 +741,6 @@ Price Field - Price Field 1 1 $ 100.00 $ 100.00 $form->testSubmit(array( 'total_amount' => 50, 'financial_type_id' => 1, - 'receive_date' => '04/21/2015', - 'receive_date_time' => '11:27PM', 'contact_id' => $this->_individualId, 'payment_instrument_id' => array_search('Check', $this->paymentInstruments), 'contribution_status_id' => 1, @@ -797,8 +768,6 @@ Price Field - Price Field 1 1 $ 100.00 $ 100.00 $form->testSubmit(array( 'total_amount' => 50, 'financial_type_id' => 1, - 'receive_date' => '04/21/2015', - 'receive_date_time' => '11:27PM', 'contact_id' => $this->_individualId, 'payment_instrument_id' => array_search('Check', $this->paymentInstruments), 'contribution_status_id' => 1, @@ -829,8 +798,6 @@ Price Field - Price Field 1 1 $ 100.00 $ 100.00 $form->testSubmit(array( 'total_amount' => 50, 'financial_type_id' => 1, - 'receive_date' => '04/21/2015', - 'receive_date_time' => '11:27PM', 'contact_id' => $this->_individualId, 'payment_instrument_id' => array_search('Check', $this->paymentInstruments), 'contribution_status_id' => 1, @@ -851,8 +818,6 @@ Price Field - Price Field 1 1 $ 100.00 $ 100.00 $form->testSubmit(array( 'total_amount' => 50, 'financial_type_id' => 1, - 'receive_date' => '04/21/2015', - 'receive_date_time' => '11:27PM', 'contact_id' => $this->_individualId, 'payment_instrument_id' => array_search('Check', $this->paymentInstruments), 'contribution_status_id' => 1, @@ -876,8 +841,6 @@ Price Field - Price Field 1 1 $ 100.00 $ 100.00 $form->testSubmit(array( 'total_amount' => -5, 'financial_type_id' => 1, - 'receive_date' => '04/24/2016', - 'receive_date_time' => '11:27PM', 'contact_id' => $this->_individualId, 'payment_instrument_id' => array_search('Check', $this->paymentInstruments), 'contribution_status_id' => 1, @@ -900,8 +863,6 @@ Price Field - Price Field 1 1 $ 100.00 $ 100.00 $form->testSubmit(array( 'total_amount' => 50, 'financial_type_id' => 1, - 'receive_date' => '04/21/2015', - 'receive_date_time' => '11:27PM', 'contact_id' => $this->_individualId, 'payment_instrument_id' => array_search('Check', $this->paymentInstruments), 'contribution_status_id' => 1, @@ -913,8 +874,6 @@ Price Field - Price Field 1 1 $ 100.00 $ 100.00 'total_amount' => 45, 'net_amount' => 45, 'financial_type_id' => 1, - 'receive_date' => '04/21/2015', - 'receive_date_time' => '11:27PM', 'contact_id' => $this->_individualId, 'payment_instrument_id' => array_search('Check', $this->paymentInstruments), 'contribution_status_id' => 1, @@ -943,8 +902,6 @@ Price Field - Price Field 1 1 $ 100.00 $ 100.00 $form->testSubmit(array( 'total_amount' => 50, 'financial_type_id' => 1, - 'receive_date' => '04/21/2015', - 'receive_date_time' => '11:27PM', 'contact_id' => $this->_individualId, 'payment_instrument_id' => array_search('Check', $this->paymentInstruments), 'check_number' => '123AX', @@ -957,8 +914,6 @@ Price Field - Price Field 1 1 $ 100.00 $ 100.00 'total_amount' => 50, 'net_amount' => 50, 'financial_type_id' => 1, - 'receive_date' => '04/21/2015', - 'receive_date_time' => '11:27PM', 'contact_id' => $this->_individualId, 'payment_instrument_id' => array_search('Credit Card', $this->paymentInstruments), 'card_type_id' => CRM_Core_PseudoConstant::getKey('CRM_Financial_DAO_FinancialTrxn', 'card_type_id', 'Visa'), @@ -1014,8 +969,6 @@ Price Field - Price Field 1 1 $ 100.00 $ 100.00 array( 'total_amount' => 50, 'financial_type_id' => 1, - 'receive_date' => '04/21/2015', - 'receive_date_time' => '11:27PM', 'contact_id' => $this->_individualId, 'payment_instrument_id' => array_search('Check', $this->paymentInstruments), 'check_number' => substr(sha1(rand()), 0, 7), @@ -1035,8 +988,6 @@ Price Field - Price Field 1 1 $ 100.00 $ 100.00 'total_amount' => 50, 'currency' => 'USD', 'financial_type_id' => 1, - 'receive_date' => '04/21/2015', - 'receive_date_time' => '11:27PM', 'contact_id' => $this->_individualId, 'payment_instrument_id' => array_search('Credit card', $this->paymentInstruments), 'payment_processor_id' => $this->paymentProcessorID, @@ -1066,8 +1017,6 @@ Price Field - Price Field 1 1 $ 100.00 $ 100.00 $form->testSubmit(array( 'total_amount' => 100, 'financial_type_id' => $this->_financialTypeId, - 'receive_date' => '04/21/2015', - 'receive_date_time' => '11:27PM', 'contact_id' => $this->_individualId, 'payment_instrument_id' => array_search('Check', $this->paymentInstruments), 'contribution_status_id' => 1, @@ -1093,8 +1042,6 @@ Price Field - Price Field 1 1 $ 100.00 $ 100.00 $form->testSubmit(array( 'id' => $contribution['id'], 'financial_type_id' => 3, - 'receive_date' => '04/21/2015', - 'receive_date_time' => '11:27PM', 'contact_id' => $this->_individualId, 'payment_instrument_id' => array_search('Check', $this->paymentInstruments), 'contribution_status_id' => 1, @@ -1118,8 +1065,6 @@ Price Field - Price Field 1 1 $ 100.00 $ 100.00 $form->testSubmit(array( 'total_amount' => 100, 'financial_type_id' => 3, - 'receive_date' => '04/21/2015', - 'receive_date_time' => '11:27PM', 'contact_id' => $this->_individualId, 'payment_instrument_id' => array_search('Check', $this->paymentInstruments), 'contribution_status_id' => 1, @@ -1279,8 +1224,7 @@ Price Field - Price Field 1 1 $ 100.00 $ 100.00 $form->testSubmit(array( 'total_amount' => 100, 'financial_type_id' => $this->_financialTypeId, - 'receive_date' => '04/21/2015', - 'receive_date_time' => '11:27PM', + 'receive_date' => '2015-04-21 00:00:00', 'contact_id' => $this->_individualId, 'payment_instrument_id' => array_search('Check', $this->paymentInstruments), 'contribution_status_id' => 1, @@ -1316,8 +1260,6 @@ Price Field - Price Field 1 1 $ 100.00 $ 100.00 array( 'total_amount' => 100, 'financial_type_id' => 3, - 'receive_date' => '04/21/2015', - 'receive_date_time' => '11:27PM', 'contact_id' => $this->_individualId, 'payment_instrument_id' => array_search('Credit Card', $this->paymentInstruments), 'contribution_status_id' => 1, @@ -1405,8 +1347,6 @@ Price Field - Price Field 1 1 $ 100.00 $ 100.00 array( 'total_amount' => 50, 'financial_type_id' => 1, - 'receive_date' => '04/21/2015', - 'receive_date_time' => '11:27PM', 'contact_id' => $this->_individualId, 'credit_card_number' => 4444333322221111, 'payment_instrument_id' => array_search('Credit Card', $this->paymentInstruments), diff --git a/tests/phpunit/CRM/Financial/BAO/FinancialItemTest.php b/tests/phpunit/CRM/Financial/BAO/FinancialItemTest.php index a9b56cf00f..cece2bb61d 100644 --- a/tests/phpunit/CRM/Financial/BAO/FinancialItemTest.php +++ b/tests/phpunit/CRM/Financial/BAO/FinancialItemTest.php @@ -326,8 +326,6 @@ class CRM_Financial_BAO_FinancialItemTest extends CiviUnitTestCase { $form->testSubmit(array( 'total_amount' => 100, 'financial_type_id' => 1, - 'receive_date' => '04/21/2015', - 'receive_date_time' => '11:27PM', 'contact_id' => $contactId, 'contribution_status_id' => 1, 'price_set_id' => 0, diff --git a/tests/phpunit/CRM/Financial/Form/PaymentEditTest.php b/tests/phpunit/CRM/Financial/Form/PaymentEditTest.php index e68771d13f..bbc8cb47b2 100644 --- a/tests/phpunit/CRM/Financial/Form/PaymentEditTest.php +++ b/tests/phpunit/CRM/Financial/Form/PaymentEditTest.php @@ -58,9 +58,8 @@ class CRM_Financial_Form_PaymentEditTest extends CiviUnitTestCase { $form = new CRM_Contribute_Form_Contribution(); $form->testSubmit(array( 'total_amount' => 50, + 'receive_date' => '2015-04-21 23:27:00', 'financial_type_id' => CRM_Core_PseudoConstant::getKey('CRM_Contribute_BAO_Contribution', 'financial_type_id', 'Donation'), - 'receive_date' => '04/21/2015', - 'receive_date_time' => '11:27PM', 'contact_id' => $this->_individualID, 'payment_instrument_id' => CRM_Core_PseudoConstant::getKey('CRM_Contribute_BAO_Contribution', 'payment_instrument_id', 'Check'), 'check_number' => '123XA', @@ -131,8 +130,6 @@ class CRM_Financial_Form_PaymentEditTest extends CiviUnitTestCase { $form->testSubmit(array( 'total_amount' => 50, 'financial_type_id' => CRM_Core_PseudoConstant::getKey('CRM_Contribute_BAO_Contribution', 'financial_type_id', 'Donation'), - 'receive_date' => '04/21/2015', - 'receive_date_time' => '11:27PM', 'contact_id' => $this->_individualID, 'payment_instrument_id' => $checkPaymentInstrumentID, 'check_number' => $checkNumber1, diff --git a/tests/phpunit/api/v3/ContributionPageTest.php b/tests/phpunit/api/v3/ContributionPageTest.php index ecaca1a1dd..6d462ab2ca 100644 --- a/tests/phpunit/api/v3/ContributionPageTest.php +++ b/tests/phpunit/api/v3/ContributionPageTest.php @@ -1687,6 +1687,7 @@ class api_v3_ContributionPageTest extends CiviUnitTestCase { 'last_name' => 'T', 'email' => 'JT@ohcanada.ca', 'is_pay_later' => TRUE, + 'receive_date' => date('Y-m-d H:i:s'), ); // Create PriceSet/PriceField diff --git a/tests/phpunit/api/v3/ContributionTest.php b/tests/phpunit/api/v3/ContributionTest.php index a145592304..2102975ec3 100644 --- a/tests/phpunit/api/v3/ContributionTest.php +++ b/tests/phpunit/api/v3/ContributionTest.php @@ -2902,8 +2902,6 @@ class api_v3_ContributionTest extends CiviUnitTestCase { 'net_amount' => 20, 'fee_amount' => 0, 'financial_type_id' => 1, - 'receive_date' => '04/21/2015', - 'receive_date_time' => '11:27PM', 'contact_id' => $this->_individualId, 'contribution_status_id' => 1, 'billing_middle_name' => '', diff --git a/tests/phpunit/api/v3/MembershipTest.php b/tests/phpunit/api/v3/MembershipTest.php index 3738afe928..1cc7def0b1 100644 --- a/tests/phpunit/api/v3/MembershipTest.php +++ b/tests/phpunit/api/v3/MembershipTest.php @@ -190,8 +190,6 @@ class api_v3_MembershipTest extends CiviUnitTestCase { $form->testSubmit(array( 'total_amount' => 100, 'financial_type_id' => 1, - 'receive_date' => '04/21/2015', - 'receive_date_time' => '11:27PM', 'contact_id' => $contactId, 'payment_instrument_id' => array_search('Check', $this->paymentInstruments), 'contribution_status_id' => 3, -- 2.25.1