summaryrefslogtreecommitdiff
path: root/addons/sms/tests
diff options
context:
space:
mode:
authorThibault Delavallée <tde@odoo.com>2020-05-26 08:13:07 +0000
committerThibault Delavallée <tde@odoo.com>2020-05-26 10:35:58 +0000
commitc13c3318b2eb32366fe342edb934461c4efbffcd (patch)
tree1843fde1258ec023e877b784ca054270053c9040 /addons/sms/tests
parent4a2ac044f8bd4a5a1f96f0e45c008473dc15eec3 (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.py5
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