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

Branch

mailer8

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 19365
Build 26184: Run Core Tests
Build 26183: arc lint + arc unit

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 45572

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