diff options
| author | Thibault Delavallée <tde@odoo.com> | 2020-05-26 08:13:07 +0000 |
|---|---|---|
| committer | Thibault Delavallée <tde@odoo.com> | 2020-05-26 10:35:58 +0000 |
| commit | c13c3318b2eb32366fe342edb934461c4efbffcd (patch) | |
| tree | 1843fde1258ec023e877b784ca054270053c9040 /addons/mass_mailing_sms/tests | |
| parent | 4a2ac044f8bd4a5a1f96f0e45c008473dc15eec3 (diff) | |
[FIX] sms: fix test helpers calling a removed method
Fix a crash when calling assertSMSStatistics without a number but taking the
number from a partner.
Also fix "sent" sms helper, content should be optional like other helpers.
Task ID 2247037
Community PR #50384
Diffstat (limited to 'addons/mass_mailing_sms/tests')
| -rw-r--r-- | addons/mass_mailing_sms/tests/common.py | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/addons/mass_mailing_sms/tests/common.py b/addons/mass_mailing_sms/tests/common.py index 6049fc610b7..4a17519b3d6 100644 --- a/addons/mass_mailing_sms/tests/common.py +++ b/addons/mass_mailing_sms/tests/common.py @@ -1,7 +1,6 @@ # -*- coding: utf-8 -*- from odoo.addons.mass_mailing.tests.common import MassMailCommon -from odoo.addons.phone_validation.tools import phone_validation from odoo.addons.sms.tests.common import SMSCase, SMSCommon @@ -50,7 +49,7 @@ class MassSMSCase(SMSCase): state = recipient_info.get('state', 'outgoing') content = recipient_info.get('content', None) if number is None and partner: - number = phone_validation.phone_get_sanitized_record_number(partner) + number = partner._sms_get_recipients_info()[partner.id]['sanitized'] notif = traces.filtered(lambda s: s.sms_number == number and s.state == state) self.assertTrue(notif, 'SMS: not found notification for number %s, (state: %s)' % (number, state)) |