From 59ddd051801cf70c7841686ec2b45af523d0050e Mon Sep 17 00:00:00 2001 From: Jeremy Kerr Date: Tue, 1 May 2012 15:51:38 +0800 Subject: tests/notifications: Add test for escaped patch names .. to test for the fix introduced in 18986b7e. Signed-off-by: Jeremy Kerr --- apps/patchwork/tests/notifications.py | 14 ++++++++++++++ 1 file changed, 14 insertions(+) diff --git a/apps/patchwork/tests/notifications.py b/apps/patchwork/tests/notifications.py index f14b30b..df7e3f9 100644 --- a/apps/patchwork/tests/notifications.py +++ b/apps/patchwork/tests/notifications.py @@ -172,6 +172,20 @@ class PatchNotificationEmailTest(TestCase): self.assertEquals(msg.to, [self.submitter.email]) self.assertTrue(self.patch.get_absolute_url() in msg.body) + def testNotificationEscaping(self): + self.patch.name = 'Patch name with " character' + self.patch.save() + PatchChangeNotification(patch = self.patch, + orig_state = self.patch.state).save() + self._expireNotifications() + + errors = send_notifications() + self.assertEquals(errors, []) + self.assertEquals(len(mail.outbox), 1) + msg = mail.outbox[0] + self.assertEquals(msg.to, [self.submitter.email]) + self.assertFalse('"' in msg.body) + def testNotificationOptout(self): """ensure opt-out addresses don't get notifications""" PatchChangeNotification(patch = self.patch, -- cgit v1.2.3