pengyuthon/odoo_source
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[IMP] base: add test cases for 'email_split'

Browse Source 
Add test cases related to issues found in various leads management. All those
email inputs lead to an email found being '@gmail.com' (or equivalent) which
is not a valid email.

A consequence of that behavior is that 'email_normalized' for several leads
is the same ('@gmail.com') and they are considered as being the same email
identity. They could be included in a pack of leads to merge (see 'crm').

Task-3572208

X-original-commit: odoo/odoo@7498b9a0a1
Part-of: odoo/odoo#141856
This commit is contained in:
Thibault Delavallée
2023-11-17 16:11:51 +00:00
parent e08ddc21a6
commit d5b39338ff
1 changed files with 29 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files
odoo/addons/base/tests/test_mail.py
+29 -9
View File
@@ -593,9 +593,29 @@ class TestEmailTools(BaseCase):
("'(ss)' <123@gmail.com>, 'foo' <foo@bar>", ['123@gmail.com', 'foo@bar']), # comma + single-quoting
('"john@gmail.com"<johnny@gmail.com>', ['johnny@gmail.com']), # double-quoting
('"<jg>" <johnny@gmail.com>', ['johnny@gmail.com']), # double-quoting with brackets
('@gmail.com', ['@gmail.com']), # no left-part
# TFR cases
# - multi @
('fr@ncois.th@notgmail.com', ['@notgmail.com']),
('f@r@nc.gz,ois@notgmail.com', ['@nc.gz', 'ois@notgmail.com']),
('@notgmail.com esteban_gnole@coldmail.com@notgmail.com', ['@notgmail.com']),
# - multi emails (with invalid)
('Ivan@dezotos.com Cc iv.an@notgmail.com', ['@notgmail.com']),
('ivan-dredi@coldmail.com ivan.dredi@notgmail.com', ['@notgmail.com']),
('@notgmail.com ivan@coincoin.com.ar jeanine@coincoin.com.ar', ['@coincoin.com.ar']),
('@notgmail.com whoareyou@youhou.com. ivan.dezotos@notgmail.com', ['@notgmail.com']),
('francois@nc.gz CC: ois@notgmail.com ivan@dezotos.com', ['francois@nc.gzCC', '@dezotos.com']),
('francois@nc.gz CC: ois@notgmail.com,ivan@dezotos.com', ['francois@nc.gzCC', 'ois@notgmail.com', 'ivan@dezotos.com']),
# - separated with '/''
('ivan.plein@dezotos.com / ivan.plu@notgmail.com', ['@notgmail.com']),
('@notgmail.com ivan.parfois@notgmail.com/ ivan.souvent@notgmail.com', ['@notgmail.com']),
# - separated with '-''
('ivan@dezotos.com - ivan.dezotos@notgmail.com', ['@notgmail.com']),
('car.pool@notgmail.com - co (TAMBO) Registration car.warsh@notgmail.com', ['@notgmail.com']),
]
for text, expected in cases:
self.assertEqual(email_split(text), expected, 'email_split is broken')
for source, expected in cases:
with self.subTest(source=source):
self.assertEqual(email_split(source), expected)
def test_email_split_and_format(self):
""" Test 'email_split_and_format', notably in case of multi encapsulation
@@ -676,16 +696,16 @@ class TestEmailTools(BaseCase):
def test_email_formataddr(self):
""" Test custom 'formataddr', notably with IDNA support """
email = 'joe@example.com'
email_base = 'joe@example.com'
email_idna = 'joe@examplé.com'
cases = [
# (name, address), charsets expected
(('', email), ['ascii', 'utf-8'], 'joe@example.com'),
(('joe', email), ['ascii', 'utf-8'], '"joe" <joe@example.com>'),
(('joe doe', email), ['ascii', 'utf-8'], '"joe doe" <joe@example.com>'),
(('joe"doe', email), ['ascii', 'utf-8'], '"joe\\"doe" <joe@example.com>'),
(('joé', email), ['ascii'], '=?utf-8?b?am/DqQ==?= <joe@example.com>'),
(('joé', email), ['utf-8'], '"joé" <joe@example.com>'),
(('', email_base), ['ascii', 'utf-8'], 'joe@example.com'),
(('joe', email_base), ['ascii', 'utf-8'], '"joe" <joe@example.com>'),
(('joe doe', email_base), ['ascii', 'utf-8'], '"joe doe" <joe@example.com>'),
(('joe"doe', email_base), ['ascii', 'utf-8'], '"joe\\"doe" <joe@example.com>'),
(('joé', email_base), ['ascii'], '=?utf-8?b?am/DqQ==?= <joe@example.com>'),
(('joé', email_base), ['utf-8'], '"joé" <joe@example.com>'),
(('', email_idna), ['ascii'], 'joe@xn--exampl-gva.com'),
(('', email_idna), ['utf-8'], 'joe@examplé.com'),
(('joé', email_idna), ['ascii'], '=?utf-8?b?am/DqQ==?= <joe@xn--exampl-gva.com>'),