Merge ~cjwatson/launchpad:fix-dmarc-for-merges-and-code into launchpad:master

Proposed by Colin Watson
Status: Merged
Approved by: Colin Watson
Approved revision: ebde447fb7229596a547801857d7c5163387c23a
Merge reported by: Otto Co-Pilot
Merged at revision: not available
Proposed branch: ~cjwatson/launchpad:fix-dmarc-for-merges-and-code
Merge into: launchpad:master
Diff against target: 14 lines (+1/-2)
1 file modified
lib/lp/codehosting/scanner/tests/test_mergedetection.py (+1/-2)
Reviewer Review Type Date Requested Status
Tom Wardill (community) Approve
Review via email: mp+406577@code.launchpad.net

Commit message

Fix TestBranchMergeDetectionHandler for DMARC-compliant addresses

To post a comment you must log in.
Revision history for this message
Tom Wardill (twom) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/lib/lp/codehosting/scanner/tests/test_mergedetection.py b/lib/lp/codehosting/scanner/tests/test_mergedetection.py
index 2268ca8..fa5ecfc 100644
--- a/lib/lp/codehosting/scanner/tests/test_mergedetection.py
+++ b/lib/lp/codehosting/scanner/tests/test_mergedetection.py
@@ -302,8 +302,7 @@ class TestBranchMergeDetectionHandler(TestCaseWithFactory):
302 self.assertIn(302 self.assertIn(
303 'Work in progress => Merged',303 'Work in progress => Merged',
304 six.ensure_text(notifications[0].get_payload(decode=True)))304 six.ensure_text(notifications[0].get_payload(decode=True)))
305 self.assertEqual(305 self.assertEqual(proposal.address, notifications[0]['From'])
306 config.canonical.noreply_from_address, notifications[0]['From'])
307 recipients = set(msg['x-envelope-to'] for msg in notifications)306 recipients = set(msg['x-envelope-to'] for msg in notifications)
308 expected = set(307 expected = set(
309 [proposal.source_branch.registrant.preferredemail.email,308 [proposal.source_branch.registrant.preferredemail.email,

Subscribers

People subscribed via source and target branches

to status/vote changes: