Differential D19007

Add unit tests for mail failover behaviors when multiple mailers are configured
ClosedPublic
Actions

Authored by epriestley on Feb 6 2018, 4:43 PM.

Details

Reviewers

amckinley

Maniphest Tasks

T12677: Support multiple mail delivery services for automatic failover
T13053: Plans: Mail Tags and Failover

Commits

rP1f53aa27e459: Add unit tests for mail failover behaviors when multiple mailers are configured

Summary

Depends on D19006. Ref T13053. Ref T12677. When multiple mailers are configured but one or more fail, test that we recover (or don't) appropriately.

Test Plan

Ran unit tests.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Feb 6 2018, 4:43 PM

Harbormaster completed remote builds in B19365: Diff 45572.Feb 6 2018, 4:44 PM

epriestley requested review of this revision.Feb 6 2018, 4:44 PM

epriestley added a child revision: D19009: Add a Postmark mail adapter so it can be configured as an outbound mailer.Feb 6 2018, 5:44 PM

amckinley accepted this revision.Feb 7 2018, 10:40 PM

This revision is now accepted and ready to land.Feb 7 2018, 10:40 PM

Closed by commit rP1f53aa27e459: Add unit tests for mail failover behaviors when multiple mailers are configured (authored by epriestley). · Explain WhyFeb 8 2018, 2:17 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

metamta/

storage/

PhabricatorMetaMTAMail.php

16 lines

__tests__/

PhabricatorMetaMTAMailTestCase.php

78 lines

Diff 45606

View Options

src/applications/metamta/storage/PhabricatorMetaMTAMail.php

View Options

src/applications/metamta/storage/tests/PhabricatorMetaMTAMailTestCase.php

Add unit tests for mail failover behaviors when multiple mailers are configuredClosedPublicActions