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/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/sms/tests')
| -rw-r--r-- | addons/sms/tests/common.py | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/addons/sms/tests/common.py b/addons/sms/tests/common.py index 901939d15c2..a3d1e3b292a 100644 --- a/addons/sms/tests/common.py +++ b/addons/sms/tests/common.py @@ -66,7 +66,7 @@ class SMSCase(MockSMS): """ Main test class to use when testing SMS integrations. Contains helpers and tools related to notification sent by SMS. """ - def assertSMSSent(self, numbers, content): + def assertSMSSent(self, numbers, content=None): """ Check sent SMS. Order is not checked. Each number should have received the same content. Useful to check batch sending. @@ -76,7 +76,8 @@ class SMSCase(MockSMS): for number in numbers: sent_sms = next((sms for sms in self._sms if sms['number'] == number), None) self.assertTrue(bool(sent_sms), 'Number %s not found in %s' % (number, repr([s['number'] for s in self._sms]))) - self.assertEqual(sent_sms['body'], content) + if content is not None: + self.assertEqual(sent_sms['body'], content) def assertSMSCanceled(self, partner, number, error_code, content=None): """ Check canceled SMS. Search is done for a pair partner / number where |
