[MIG] report_py3o: Migration to 19.0

Author: kopeyev

report_py3o/__manifest__.py

@@ -4,7 +4,7 @@
     "name": "Py3o Report Engine",
     "summary": "Reporting engine based on Libreoffice (ODT -> ODT, "
     "ODT -> PDF, ODT -> DOC, ODT -> DOCX, ODS -> ODS, etc.)",
-    "version": "18.0.1.0.2",
+    "version": "19.0.1.0.0",
     "category": "Reporting",
     "license": "AGPL-3",
     "author": "XCG Consulting, ACSONE SA/NV, Odoo Community Association (OCA)",

report_py3o/tests/test_report_py3o.py

@@ -24,11 +24,11 @@
 from odoo.exceptions import ValidationError
 from odoo.tests.common import TransactionCase
 
-from odoo.addons.base.tests.test_mimetypes import PNG
+from odoo.addons.base.tests.test_mimetypes import PNG as ODOO_PNG
 
-from ..models._py3o_parser_context import format_multiline_value
 from ..models.ir_actions_report import PY3O_CONVERSION_COMMAND_PARAMETER
-from ..models.py3o_report import TemplateNotFound
+from ..wizard._py3o_parser_context import format_multiline_value
+from ..wizard.py3o_report import Py3oReport, TemplateNotFound
 
 logger = logging.getLogger(__name__)
 
@@ -52,7 +52,7 @@ def temporary_copy(path):
 class TestReportPy3o(TransactionCase):
     def setUp(self):
         super().setUp()
-        self.env.user.image_1920 = PNG
+        self.env.user.image_1920 = ODOO_PNG
         self.report = self.env.ref("report_py3o.res_users_report_py3o")
         self.py3o_report = self.env["py3o.report"].create(
             {"ir_actions_report_id": self.report.id}
@@ -91,13 +91,19 @@ def test_reports(self):
 
     def test_reports_merge_zip(self):
         self.report.py3o_filetype = "odt"
-        users = self.env["res.users"].search([])
-        self.assertTrue(len(users) > 0)
-        py3o_report = self.env["py3o.report"]
+        users = self.env["res.users"].search([], limit=2)
+        if len(users) < 2:
+            users |= self.env["res.users"].create(
+                {
+                    "name": "Test User 2",
+                    "login": f"test_user_2_{self.env.cr.dbname}",
+                    "email": f"test_user_2_{self.env.cr.dbname}@example.com",
+                }
+            )
+        users.write({"image_1920": ODOO_PNG})
+        self.assertGreaterEqual(len(users), 2)
         _zip_results = self.py3o_report._zip_results
-        with mock.patch.object(
-            py3o_report.__class__, "_zip_results"
-        ) as patched_zip_results:
+        with mock.patch.object(Py3oReport, "_zip_results") as patched_zip_results:
             patched_zip_results.side_effect = _zip_results
             content, filetype = self.report._render(self.report.id, users.ids)
             self.assertEqual(1, patched_zip_results.call_count)
@@ -147,7 +153,7 @@ def test_report_post_process(self):
         self.assertEqual(self.env.user.id, attachements.res_id)
         self.assertEqual(b"test result", b64decode(attachements.datas))
 
-    @tools.misc.mute_logger("odoo.addons.report_py3o.models.py3o_report")
+    @tools.misc.mute_logger("odoo.addons.report_py3o.wizard.py3o_report")
     def test_report_template_configs(self):
         # the demo template is specified with a relative path in in the module
         # path
@@ -169,9 +175,9 @@ def test_report_template_configs(self):
             self.report._render(self.report.id, self.env.user.ids)
         with temporary_copy(flbk_filename) as tmp_filename:
             self.report.py3o_template_fallback = tmp_filename
-            tools.config.misc["report_py3o"] = {
-                "root_tmpl_path": os.path.realpath(os.path.dirname(tmp_filename))
-            }
+            tools.config["root_tmpl_path"] = os.path.realpath(
+                os.path.dirname(tmp_filename)
+            )
             res = self.report._render(self.report.id, self.env.user.ids)
             self.assertTrue(res)
 
@@ -192,7 +198,7 @@ def test_report_template_configs(self):
         res = self.report._render(self.report.id, self.env.user.ids)
         self.assertTrue(res)
 
-    @tools.misc.mute_logger("odoo.addons.report_py3o.models.py3o_report")
+    @tools.misc.mute_logger("odoo.addons.report_py3o.wizard.py3o_report")
     def test_report_template_fallback_validity(self):
         tmpl_name = self.report.py3o_template_fallback
         with as_file(files(f"odoo.addons.{self.report.module}")) as _asf:

report_py3o/views/py3o_template.xml

@@ -7,13 +7,11 @@
             <search>
                 <field name="name" />
                 <field name="filetype" />
-                <group string="Group By" name="groupby">
-                    <filter
-                        name="filetype_groupby"
-                        string="File Type"
-                        context="{'group_by': 'filetype'}"
-                    />
-                </group>
+                <filter
+                    name="filetype_groupby"
+                    string="File Type"
+                    context="{'group_by': 'filetype'}"
+                />
             </search>
         </field>
     </record>

report_py3o/wizard/py3o_report.py

@@ -86,7 +86,7 @@ class Py3oReport(models.TransientModel):
     def _is_valid_template_path(self, path):
         """Check if the path is a trusted path for py3o templates."""
         real_path = os.path.realpath(path)
-        root_path = tools.config.get_misc("report_py3o", "root_tmpl_path")
+        root_path = tools.config.get("root_tmpl_path")
         if not root_path:
             logger.warning(
                 "You must provide a root template path into odoo.cfg to be "