From 520a5cd1be311c26d4d174fdcc27fa7d2877fa0c Mon Sep 17 00:00:00 2001 From: Antonio Espinosa Date: Mon, 29 Aug 2016 19:24:46 +0200 Subject: [PATCH] Fix country search and add test to check it --- mail_tracking_mailgun/models/mail_tracking_email.py | 2 +- mail_tracking_mailgun/tests/test_mailgun.py | 1 + 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/mail_tracking_mailgun/models/mail_tracking_email.py b/mail_tracking_mailgun/models/mail_tracking_email.py index adf8d565..ebd7d658 100644 --- a/mail_tracking_mailgun/models/mail_tracking_email.py +++ b/mail_tracking_mailgun/models/mail_tracking_email.py @@ -18,7 +18,7 @@ class MailTrackingEmail(models.Model): country = False if country_code: country = self.env['res.country'].search([ - ('code', '=', country_code.capitalize()), + ('code', '=', country_code.upper()), ]) if country: return country.id diff --git a/mail_tracking_mailgun/tests/test_mailgun.py b/mail_tracking_mailgun/tests/test_mailgun.py index 91396f78..f4ecf16c 100644 --- a/mail_tracking_mailgun/tests/test_mailgun.py +++ b/mail_tracking_mailgun/tests/test_mailgun.py @@ -154,6 +154,7 @@ class TestMailgun(TransactionCase): self.assertEqual(event.ua_family, ua_family) self.assertEqual(event.ua_type, ua_type) self.assertEqual(event.mobile, False) + self.assertEqual(event.user_country_id.code, 'US') # https://documentation.mailgun.com/user_manual.html#tracking-clicks def test_event_clicked(self):