Fix bug in matching In-Reply-To header to existing Message-ID and add unit test.

Author: Erik Forsberg

roundup/mailgw.py

@@ -72,7 +72,7 @@ class node. Any parts of other types are each stored in separate files
 an exception, the original message is bounced back to the sender with the
 explanatory message given in the exception.
 
-$Id: mailgw.py,v 1.180 2006-12-19 01:13:31 richard Exp $
+$Id: mailgw.py,v 1.181 2007-01-21 15:18:26 forsberg Exp $
 """
 __docformat__ = 'restructuredtext'
 
@@ -718,7 +718,7 @@ def handle_message(self, message):
         # try in-reply-to to match the message if there's no nodeid
         inreplyto = message.getheader('in-reply-to') or ''
         if nodeid is None and inreplyto:
-            l = self.db.getclass('msg').stringFind(inreplyto=inreplyto)
+            l = self.db.getclass('msg').stringFind(messageid=inreplyto)
             if l:
                 nodeid = l[0]
 

test/test_mailgw.py

@@ -8,7 +8,7 @@
 # but WITHOUT ANY WARRANTY; without even the implied warranty of
 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
 #
-# $Id: test_mailgw.py,v 1.79 2006-10-05 23:08:21 richard Exp $
+# $Id: test_mailgw.py,v 1.80 2007-01-21 15:18:25 forsberg Exp $
 
 # TODO: test bcc
 
@@ -1285,6 +1285,22 @@ def testCommandDelimitersIgnore(self):
             'testing [assignedto=mary]')
         self.assertEqual(self.db.issue.get(nodeid, 'assignedto'), None)
 
+    def testReplytoMatch(self):
+        self.instance.config.MAILGW_SUBJECT_PREFIX_PARSING = 'loose'        
+        nodeid = self.doNewIssue()
+        nodeid2 = self._handle_mail('''Content-Type: text/plain;
+  charset="iso-8859-1"
+From: Chef <[email protected]>
+To: [email protected]
+Message-Id: <dummy_test_message_id2>
+In-Reply-To: <dummy_test_message_id>
+Subject: Testing...
+
+Followup message.
+''')
+        self.assertEqual(nodeid, nodeid2)
+
+
 def test_suite():
     suite = unittest.TestSuite()
     suite.addTest(unittest.makeSuite(MailgwTestCase))