diff options
| author | David Beguin <dbe@odoo.com> | 2018-12-12 15:44:49 +0000 |
|---|---|---|
| committer | Thibault Delavallée <tde@odoo.com> | 2019-05-13 15:18:40 +0000 |
| commit | 3f707ba000a51011b5bc3f049632e051b5b7ef41 (patch) | |
| tree | 181e08b16ede9e3087318401af9035b9f9220be0 /addons/test_mass_mailing | |
| parent | 5723ee1f21c6182494621a128204f988813fa408 (diff) | |
[REF] mail, various : make message_receive_bounce from mail.thread private
Purpose: make some methods from mail.thread mixin private and update their
naming if necessary. Some methods prepare data for business flows and deal
with internal information. Public methods should either use of give some
access to those methods. Computation itself should be keps internal.
In this commit we put message_receive_bounce as private. Future commit(s)
will probably rewrite part of the bounce logic.
Some addons are updated accordingly to the method change.
Related to task ID 1911679
Linked to PR #29483
Diffstat (limited to 'addons/test_mass_mailing')
| -rw-r--r-- | addons/test_mass_mailing/tests/test_mail_auto_blacklist.py | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/addons/test_mass_mailing/tests/test_mail_auto_blacklist.py b/addons/test_mass_mailing/tests/test_mail_auto_blacklist.py index b4a679d907e..ac04a835847 100644 --- a/addons/test_mass_mailing/tests/test_mail_auto_blacklist.py +++ b/addons/test_mass_mailing/tests/test_mail_auto_blacklist.py @@ -22,28 +22,28 @@ class TestAutoBlacklist(common.TransactionCase): 'bounced': datetime.datetime.now() - datetime.timedelta(weeks=2), 'email': self.mailing_contact_1.email }) - self.mailing_contact_1.message_receive_bounce(self.mailing_contact_1.email, self.mailing_contact_1) + self.mailing_contact_1._message_receive_bounce(self.mailing_contact_1.email, self.mailing_contact_1) mail_statistics.create({ 'model': 'mail.mass_mailing.contact', 'res_id': self.mailing_contact_1.id, 'bounced': datetime.datetime.now() - datetime.timedelta(weeks=3), 'email': self.mailing_contact_1.email }) - self.mailing_contact_1.message_receive_bounce(self.mailing_contact_1.email, self.mailing_contact_1) + self.mailing_contact_1._message_receive_bounce(self.mailing_contact_1.email, self.mailing_contact_1) mail_statistics.create({ 'model': 'mail.mass_mailing.contact', 'res_id': self.mailing_contact_1.id, 'bounced': datetime.datetime.now() - datetime.timedelta(weeks=4), 'email': self.mailing_contact_1.email }) - self.mailing_contact_1.message_receive_bounce(self.mailing_contact_1.email, self.mailing_contact_1) + self.mailing_contact_1._message_receive_bounce(self.mailing_contact_1.email, self.mailing_contact_1) mail_statistics.create({ 'model': 'mail.mass_mailing.contact', 'res_id': self.mailing_contact_1.id, 'bounced': datetime.datetime.now() - datetime.timedelta(weeks=5), 'email': self.mailing_contact_1.email }) - self.mailing_contact_1.message_receive_bounce(self.mailing_contact_1.email, self.mailing_contact_1) + self.mailing_contact_1._message_receive_bounce(self.mailing_contact_1.email, self.mailing_contact_1) # create mass mailing record @@ -77,7 +77,7 @@ class TestAutoBlacklist(common.TransactionCase): 'email': self.mailing_contact_1.email }) # call bounced - self.mailing_contact_1.message_receive_bounce(self.mailing_contact_1.email, self.mailing_contact_1) + self.mailing_contact_1._message_receive_bounce(self.mailing_contact_1.email, self.mailing_contact_1) # check blacklist blacklist_record = mail_blacklist.search([('email', '=', self.mailing_contact_1.email)]) |
