ssh-ed25519 AAAAC3NzaC1lZDI1NTE5AAAAIM+MPHfW8vX4CgPA9taZb1mBMAs/IJYfo41AHEi6N46i m.francke@secure42.de

[IMP] increase test coverage

[IMP] increase test coverage

[IMP] increase test coverage

fghf

fgghf

dfdfg

Author: mathiasfrancke

mail_quoted_reply/tests/test_reply.py

@@ -71,6 +71,9 @@ def test_unquoted_reply(self):
         wizard = (
             self.env[action["res_model"]].with_context(**action["context"]).create({})
         )
+        wizard = wizard.with_context(
+            default_subject="Some subject for testing purposes"
+        )
         self.assertTrue(wizard.partner_ids)
         self.assertEqual(message.email_from, wizard.partner_ids.email_formatted)
         # the onchange in the composer isn't triggered in tests, so we check for the

mail_quoted_reply/wizard/mail_compose_message.py

@@ -1,6 +1,6 @@
 from markupsafe import Markup
 
-from odoo import api, models, tools
+from odoo import api, models
 
 
 class MailComposeMessage(models.TransientModel):
@@ -31,7 +31,8 @@ def _compute_body(self):
     def _compute_subject(self):
         res = super()._compute_subject()
         for composer in self:
-            subj = composer.env.context.get("default_subject", False)
+            context = composer.env.context
+            subj = context.get("default_subject", False)
             if subj:
-                composer.subject = tools.ustr(subj)
+                composer.subject = str(subj)
         return res