Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add unit test for core#19628 #19972

Merged

Conversation

monishdeb
Copy link
Member

Overview

Added unit test for #19628 fix

@civibot
Copy link

civibot bot commented Apr 6, 2021

(Standard links)

@eileenmcnaughton
Copy link
Contributor

CRM_Core_BAO_ActionScheduleTest::testReminderWithCustomDateField
Incorrect recipients: Array
(
[expected] => Array
(
[0] => Array
(
[0] => [email protected]
)

    )

[actual] => Array
    (
    )

)

Failed asserting that two arrays are equal.
--- Expected
+++ Actual
@@ @@
Array (

  • 0 => Array (...)
    )

/home/jenkins/bknix-dfl/build/core-19972-6834/web/sites/all/modules/civicrm/tests/phpunit/CiviTest/CiviMailUtils.php:299
/home/jenkins/bknix-dfl/build/core-19972-6834/web/sites/all/modules/civicrm/tests/phpunit/CRM/Core/BAO/ActionScheduleTest.php:2197
/home/jenkins/bknix-dfl/build/core-19972-6834/web/sites/all/modules/civicrm/tests/phpunit/CRM/Core/BAO/ActionScheduleTest.php:996
/home/jenkins/bknix-dfl/build/core-19972-6834/web/sites/all/modules/civicrm/tests/phpunit/CiviTest/CiviUnitTestCase.php:226
/home/jenkins/bknix-dfl/extern/phpunit7/phpunit7.phar:615

@monishdeb monishdeb force-pushed the issue_2383_unit_test branch from db85c7a to d1e4008 Compare April 9, 2021 06:30
@eileenmcnaughton
Copy link
Contributor

woohoo

@eileenmcnaughton eileenmcnaughton merged commit 6c1f813 into civicrm:master Apr 9, 2021
@monishdeb monishdeb deleted the issue_2383_unit_test branch April 9, 2021 08:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants