[FIX] web_font_size_report_layout: tests

Author: syci

web_font_size_report_layout/tests/test_company_font_size.py

@@ -36,3 +36,12 @@ def test_multi_company_independence(self):
         co2.write({"report_font_size": "14"})
         self.assertEqual(co1.report_font_size, "9")
         self.assertEqual(co2.report_font_size, "14")
+
+    def test_multiple_changes_persist_in_sequence(self):
+        co = self.Company.create({"name": "Co G"})
+        co.write({"report_font_size": "14"})
+        self.assertEqual(co.report_font_size, "14")
+        co.write({"report_font_size": "10"})
+        self.assertEqual(co.report_font_size, "10")
+        co.write({"report_font_size": "12"})
+        self.assertEqual(co.report_font_size, "12")

web_font_size_report_layout/tests/test_document_layout_onchange.py

@@ -1,3 +1,5 @@
+from unittest.mock import patch
+
 from odoo.tests.common import TransactionCase, tagged
 
 
@@ -11,30 +13,26 @@ def setUpClass(cls):
         cls.Wizard = cls.env["base.document.layout"]
 
     def test_wizard_default_inherits_company_value(self):
-        wiz = cls = self.__class__
-        wiz = cls.Wizard.with_company(cls.company.id).create(
-            {"company_id": cls.company.id}
+        wiz = self.Wizard.with_company(self.company.id).create(
+            {"company_id": self.company.id}
         )
         self.assertEqual(wiz.report_font_size, "11")
 
     def test_onchange_updates_wizard_field_without_crash(self):
-        wiz = cls = self.__class__
-        wiz = cls.Wizard.with_company(cls.company.id).create(
-            {"company_id": cls.company.id}
+        wiz = self.Wizard.with_company(self.company.id).create(
+            {"company_id": self.company.id}
         )
         wiz.report_font_size = "14"
         if hasattr(wiz, "_onchange_report_font_size"):
             wiz._onchange_report_font_size()
         self.assertEqual(wiz.report_font_size, "14")
 
     def test_apply_wizard_persists_on_company(self):
-        wiz = cls = self.__class__
-        wiz = cls.Wizard.with_company(cls.company.id).create(
-            {"company_id": cls.company.id}
+        wiz = self.Wizard.with_company(self.company.id).create(
+            {"company_id": self.company.id}
         )
         wiz.report_font_size = "12"
 
-        done = False
         for meth in (
             "execute",
             "action_confirm",
@@ -43,9 +41,56 @@ def test_apply_wizard_persists_on_company(self):
         ):
             if hasattr(wiz, meth):
                 getattr(wiz, meth)()
-                done = True
                 break
-        if not done:
-            cls.company.write({"report_font_size": wiz.report_font_size})
+        else:
+            self.company.write({"report_font_size": wiz.report_font_size})
+
+        self.assertEqual(self.company.report_font_size, "12")
+
+    def test_wizard_reflects_company_change_on_new_instance(self):
+        self.company.write({"report_font_size": "14"})
+        wiz = self.Wizard.with_company(self.company.id).create(
+            {"company_id": self.company.id}
+        )
+        self.assertEqual(wiz.report_font_size, "14")
+
+    def test_onchange_handles_compute_exception_sets_preview_false(self):
+        wiz = self.Wizard.with_company(self.company.id).create(
+            {"company_id": self.company.id}
+        )
+        if "preview" not in wiz._fields:
+            self.skipTest(
+                "This Odoo version has no 'preview' field on base.document.layout"
+            )
+
+        wiz.report_font_size = "14"
+        with patch.object(
+            type(wiz), "_compute_preview", side_effect=Exception("boom"), create=True
+        ):
+            wiz._onchange_report_font_size()
+        self.assertFalse(bool(wiz.preview))
+
+    def test_apply_uses_action_apply_when_present(self):
+        wiz = self.Wizard.with_company(self.company.id).create(
+            {"company_id": self.company.id}
+        )
+        wiz.report_font_size = "13"
+
+        def _fake_action_apply(self):
+            self.company_id.report_font_size = self.report_font_size
+
+        with patch.object(type(wiz), "action_apply", _fake_action_apply, create=True):
+            done = False
+            for meth in (
+                "execute",
+                "action_confirm",
+                "action_apply",
+                "action_configure_document_layout",
+            ):
+                if hasattr(wiz, meth):
+                    getattr(wiz, meth)()
+                    done = True
+                    break
 
-        self.assertEqual(cls.company.report_font_size, "12")
+        self.assertTrue(done, "Expected loop to call injected action_apply")
+        self.assertEqual(self.company.report_font_size, "13")