From d233dd419fabd4b42d5b83b920ec7ee54fee57da Mon Sep 17 00:00:00 2001
From: Mark Sapiro
Date: Sun, 1 Jan 2017 19:22:07 -0800
Subject: Test coverage improvements. Replaced unreachable statement with
 assertion.

---
 src/mailman/rules/dmarc.py | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

(limited to 'src/mailman/rules/dmarc.py')

diff --git a/src/mailman/rules/dmarc.py b/src/mailman/rules/dmarc.py
index 43bbfc609..92db64ccd 100644
--- a/src/mailman/rules/dmarc.py
+++ b/src/mailman/rules/dmarc.py
@@ -149,10 +149,8 @@ def _DMARCProhibited(mlist, email, dmarc_domain, org=False):
             seen.add(cnames[item])
             want_names.add(cnames[item])
             want_names.discard(item)
-    if len(want_names) != 1:
-        elog.error(
-            """multiple DMARC entries in results for %s,
-            processing each to be strict""",
+    assert len(want_names) == 1, """\
+        Error in CNAME processing for {}; want_names != 1.""".format(
             dmarc_domain)
     for name in want_names:
         if name not in results_by_name:
-- 
cgit v1.2.3-70-g09d2