issued2550877 fixed, the tests adapted to the more precise header comparison. Added a hint to the code how headers are handled.

Author: bernhardreiter

test/test_mailgw.py

@@ -57,7 +57,15 @@ def open(self, journaltag):
 
 class DiffHelper:
     def compareMessages(self, new, old):
-        """Compare messages for semantic equivalence."""
+        """Compare messages for semantic equivalence.
+
+        Will raise an AssertionError with a diff for inequality.
+
+        Note that header fieldnames are case-insensitive.
+        So if a header fieldname appears more than once in different casing
+        and the values are not equal, there will be more than one entry
+        in the diff. Typical examples are "From:"/ "FROM:" and "TO:"/"To:".
+        """
         new = email.message_from_string(new.strip())
         old = email.message_from_string(old.strip())
 
@@ -458,7 +466,7 @@ def testNewIssueNoAuthorInfo(self):
 ''')
         self.compareMessages(self._get_mail(),
 '''FROM: [email protected]
-TO: [email protected], [email protected], [email protected]
+TO: [email protected], [email protected]
 Content-Type: text/plain; charset="utf-8"
 Subject: [issue1] Testing...
 To: [email protected], [email protected]
@@ -501,7 +509,7 @@ def testNewIssueNoAuthorEmail(self):
 ''')
         self.compareMessages(self._get_mail(),
 '''FROM: [email protected]
-TO: [email protected], [email protected], [email protected]
+TO: [email protected], [email protected]
 Content-Type: text/plain; charset="utf-8"
 Subject: [issue1] Testing...
 To: [email protected], [email protected]
@@ -2998,7 +3006,7 @@ def testSecurityMessagePermissionContent(self):
         assert os.path.exists(SENDMAILDEBUG)
         self.compareMessages(self._get_mail(),
 '''FROM: [email protected]
-TO: [email protected], [email protected]
+TO: [email protected]
 Content-Type: text/plain; charset="utf-8"
 Subject: [issue1] Testing...
 To: [email protected]