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

[NFC] Add in Unit test to test domain token replacement in Reply process #24453

Merged
merged 1 commit into from
Sep 5, 2022

Conversation

seamuslee001
Copy link
Contributor

Overview

Adds in unit test to test the refactoring Eileen is doing in #24446

Before

No unit test on reply autoresponder

After

We have a unit test (may not be perfect but works at least locally)

ping @eileenmcnaughton

@civibot
Copy link

civibot bot commented Sep 5, 2022

(Standard links)

@eileenmcnaughton
Copy link
Contributor

looks like it needs some teardown

CRM_Mailing_MailingSystemTest::testMailingReplyAutoRespond
Failure in api call for Group create: DB Error: already exists
INSERT INTO civicrm_group (name , title , description , is_active , visibility , group_type ) VALUES ('Test Group 1' , 'New Test Group Created' , 'New Test Group Created' , 1 , 'Public Pages' , '12' ) [nativecode=1062 ** Duplicate entry 'Test Group 1' for key 'UI_name']
#0 /home/jenkins/bknix-dfl/build/core-24453-88bhx/web/sites/all/modules/civicrm/vendor/pear/pear-core-minimal/src/PEAR.php(944): CRM_Core_Error::exceptionHandler(Object(DB_Error))

Check for do not reply and also from email and to email
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