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] minimal test for Mailing.update_email_resetdate #15832

Merged
merged 1 commit into from
Nov 12, 2019

Conversation

eileenmcnaughton
Copy link
Contributor

Overview

Adds test for no fatal in Mailing.update_email_resetdate that I wrote to step through @seamuslee001 PR

Before

No test

After

Minimal one

Technical Details

Also removes a couple of unused variables, adds a few exception comments

Comments

ALso removes a couple of unused variables, adds a few exception comments
@civibot
Copy link

civibot bot commented Nov 12, 2019

(Standard links)

@civibot civibot bot added the master label Nov 12, 2019
@seamuslee001 seamuslee001 merged commit aa301f1 into civicrm:master Nov 12, 2019
@seamuslee001 seamuslee001 deleted the mail_s branch November 12, 2019 06:06
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