[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

2 files changed, 4 insertions, 4 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))
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