odoo-addon-base-report-to-printer 17.0.1.1.1__py3-none-any.whl → 18.0.1.0.0.4__py3-none-any.whl

odoo/addons/base_report_to_printer/tests/test_ir_actions_report.py

@@ -2,6 +2,7 @@
 # Copyright 2016 SYLEAM
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
 
+import logging
 from unittest import mock
 
 from odoo.tests.common import TransactionCase
@@ -57,14 +58,14 @@ class TestIrActionsReportXml(TransactionCase):
 
     def test_print_action_for_report_name_gets_report(self):
         """It should get report by name"""
-        with mock.patch("%s._get_report_from_name" % model) as mk:
+        with mock.patch(f"{model}._get_report_from_name") as mk:
             expect = "test"
             self.Model.print_action_for_report_name(expect)
             mk.assert_called_once_with(expect)
 
     def test_print_action_for_report_name_returns_if_no_report(self):
         """It should return empty dict when no matching report"""
-        with mock.patch("%s._get_report_from_name" % model) as mk:
+        with mock.patch(f"{model}._get_report_from_name") as mk:
             expect = "test"
             mk.return_value = False
             res = self.Model.print_action_for_report_name(expect)
@@ -72,7 +73,7 @@ class TestIrActionsReportXml(TransactionCase):
 
     def test_print_action_for_report_name_returns_if_report(self):
         """It should return correct serializable result for behaviour"""
-        with mock.patch("%s._get_report_from_name" % model) as mk:
+        with mock.patch(f"{model}._get_report_from_name") as mk:
             res = self.Model.print_action_for_report_name("test")
             behaviour = mk().behaviour()
             expect = {
@@ -102,14 +103,19 @@ class TestIrActionsReportXml(TransactionCase):
         report = self.Model.search([], limit=1)
         self.env.user.printing_action = "client"
         self.env.user.printing_printer_id = self.new_printer()
-        self.assertEqual(
-            report.behaviour(),
-            {
-                "action": "client",
-                "printer": self.env.user.printing_printer_id,
-                "tray": False,
-            },
-        )
+        with (
+            self.assertLogs(level=logging.WARNING) as logs,
+        ):
+            self.assertEqual(
+                report.behaviour(),
+                {
+                    "action": "client",
+                    "printer": self.env.user.printing_printer_id,
+                    "tray": False,
+                },
+            )
+            self.assertEqual(len(logs.records), 1)
+            self.assertEqual(logs.records[0].levelno, logging.WARNING)
 
     def test_behaviour_report_values(self):
         """It should return the action and printer from report"""
@@ -117,14 +123,19 @@ class TestIrActionsReportXml(TransactionCase):
         self.env.user.printing_action = "client"
         report.property_printing_action_id = self.new_action()
         report.printing_printer_id = self.new_printer()
-        self.assertEqual(
-            report.behaviour(),
-            {
-                "action": report.property_printing_action_id.action_type,
-                "printer": report.printing_printer_id,
-                "tray": False,
-            },
-        )
+        with (
+            self.assertLogs(level=logging.WARNING) as logs,
+        ):
+            self.assertEqual(
+                report.behaviour(),
+                {
+                    "action": report.property_printing_action_id.action_type,
+                    "printer": report.printing_printer_id,
+                    "tray": False,
+                },
+            )
+            self.assertEqual(len(logs.records), 1)
+            self.assertEqual(logs.records[0].levelno, logging.WARNING)
 
     def test_behaviour_user_action(self):
         """It should return the action and printer from user action"""
@@ -188,14 +199,19 @@ class TestIrActionsReportXml(TransactionCase):
         printing_action = self.new_printing_action()
         printing_action.user_id = self.env.user
         printing_action.printer_id = self.new_printer()
-        self.assertEqual(
-            report.behaviour(),
-            {
-                "action": printing_action.action,
-                "printer": printing_action.printer_id,
-                "tray": False,
-            },
-        )
+        with (
+            self.assertLogs(level=logging.WARNING) as logs,
+        ):
+            self.assertEqual(
+                report.behaviour(),
+                {
+                    "action": printing_action.action,
+                    "printer": printing_action.printer_id,
+                    "tray": False,
+                },
+            )
+            self.assertEqual(len(logs.records), 1)
+            self.assertEqual(logs.records[0].levelno, logging.WARNING)
 
     def test_behaviour_printing_action_user_defaults(self):
         """It should return the action and printer from user with printing
@@ -215,57 +231,71 @@ class TestIrActionsReportXml(TransactionCase):
         """
         It should return the correct tray
         """
-        report = self.Model.search([], limit=1)
-        action = self.env["printing.report.xml.action"].create(
-            {"user_id": self.env.user.id, "report_id": report.id, "action": "server"}
-        )
-        printer = self.new_printer()
-        tray_vals = {"name": "Tray", "system_name": "Tray", "printer_id": printer.id}
-        user_tray = self.new_tray({"system_name": "User tray"}, tray_vals)
-        report_tray = self.new_tray({"system_name": "Report tray"}, tray_vals)
-        action_tray = self.new_tray({"system_name": "Action tray"}, tray_vals)
-
-        # No report passed
-        self.env.user.printer_tray_id = False
-        options = printer.print_options()
-        self.assertFalse("InputSlot" in options)
-
-        # No tray defined
-        self.env.user.printer_tray_id = False
-        report.printer_tray_id = False
-        action.printer_tray_id = False
-        options = report.behaviour()
-        self.assertTrue("tray" in options)
-
-        # Only user tray is defined
-        self.env.user.printer_tray_id = user_tray
-        report.printer_tray_id = False
-        action.printer_tray_id = False
-        self.assertEqual("User tray", report.behaviour()["tray"])
-
-        # Only report tray is defined
-        self.env.user.printer_tray_id = False
-        report.printer_tray_id = report_tray
-        action.printer_tray_id = False
-        self.assertEqual("Report tray", report.behaviour()["tray"])
-
-        # Only action tray is defined
-        self.env.user.printer_tray_id = False
-        report.printer_tray_id = False
-        action.printer_tray_id = action_tray
-        self.assertEqual("Action tray", report.behaviour()["tray"])
-
-        # User and report tray defined
-        self.env.user.printer_tray_id = user_tray
-        report.printer_tray_id = report_tray
-        action.printer_tray_id = False
-        self.assertEqual("Report tray", report.behaviour()["tray"])
-
-        # All trays are defined
-        self.env.user.printer_tray_id = user_tray
-        report.printer_tray_id = report_tray
-        action.printer_tray_id = action_tray
-        self.assertEqual("Action tray", report.behaviour()["tray"])
+        with (
+            self.assertLogs(level=logging.WARNING) as logs,
+        ):
+            report = self.Model.search([], limit=1)
+            action = self.env["printing.report.xml.action"].create(
+                {
+                    "user_id": self.env.user.id,
+                    "report_id": report.id,
+                    "action": "server",
+                }
+            )
+            printer = self.new_printer()
+            tray_vals = {
+                "name": "Tray",
+                "system_name": "Tray",
+                "printer_id": printer.id,
+            }
+            user_tray = self.new_tray({"system_name": "User tray"}, tray_vals)
+            report_tray = self.new_tray({"system_name": "Report tray"}, tray_vals)
+            action_tray = self.new_tray({"system_name": "Action tray"}, tray_vals)
+
+            # No report passed
+            self.env.user.printer_tray_id = False
+            options = printer.print_options()
+            self.assertFalse("InputSlot" in options)
+
+            # No tray defined
+            self.env.user.printer_tray_id = False
+            report.printer_tray_id = False
+            action.printer_tray_id = False
+            options = report.behaviour()
+            self.assertTrue("tray" in options)
+
+            # Only user tray is defined
+            self.env.user.printer_tray_id = user_tray
+            report.printer_tray_id = False
+            action.printer_tray_id = False
+            self.assertEqual("User tray", report.behaviour()["tray"])
+
+            # Only report tray is defined
+            self.env.user.printer_tray_id = False
+            report.printer_tray_id = report_tray
+            action.printer_tray_id = False
+            self.assertEqual("Report tray", report.behaviour()["tray"])
+
+            # Only action tray is defined
+            self.env.user.printer_tray_id = False
+            report.printer_tray_id = False
+            action.printer_tray_id = action_tray
+            self.assertEqual("Action tray", report.behaviour()["tray"])
+
+            # User and report tray defined
+            self.env.user.printer_tray_id = user_tray
+            report.printer_tray_id = report_tray
+            action.printer_tray_id = False
+            self.assertEqual("Report tray", report.behaviour()["tray"])
+
+            # All trays are defined
+            self.env.user.printer_tray_id = user_tray
+            report.printer_tray_id = report_tray
+            action.printer_tray_id = action_tray
+            self.assertEqual("Action tray", report.behaviour()["tray"])
+            self.assertEqual(len(logs.records), 6)
+            for record in logs.records:
+                self.assertEqual(record.levelno, logging.WARNING)
 
     def test_onchange_printer_tray_id_empty(self):
         action = self.Model.new({"printer_tray_id": False})
@@ -305,11 +335,16 @@ class TestIrActionsReportXml(TransactionCase):
         printing_action.user_id = self.env.user
         printing_action.printer_id = self.new_printer()
         printing_action.printer_id.multi_thread = True
-        self.assertEqual(
-            report.behaviour(),
-            {
-                "action": printing_action.action,
-                "printer": printing_action.printer_id,
-                "tray": False,
-            },
-        )
+        with (
+            self.assertLogs(level=logging.WARNING) as logs,
+        ):
+            self.assertEqual(
+                report.behaviour(),
+                {
+                    "action": printing_action.action,
+                    "printer": printing_action.printer_id,
+                    "tray": False,
+                },
+            )
+            self.assertEqual(len(logs.records), 1)
+            self.assertEqual(logs.records[0].levelno, logging.WARNING)

odoo/addons/base_report_to_printer/tests/test_printing_job.py

@@ -1,6 +1,7 @@
 # Copyright 2016 LasLabs Inc.
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
 
+import logging
 from unittest import mock
 
 from odoo import fields
@@ -42,17 +43,23 @@ class TestPrintingJob(TransactionCase):
         values["printer_id"] = printer.id
         return self.env["printing.job"].create(values)
 
-    @mock.patch("%s.cups" % model)
-    def test_cancel_job_error(self, cups):
+    def test_cancel_job_error(self):
         """It should catch any exception from CUPS and update status"""
-        cups.Connection.side_effect = Exception
-        printer = self.new_printer()
-        job = self.new_job(printer, {"job_id_cups": 2})
-        job.action_cancel()
-        cups.Connection.side_effect = None
-        self.assertEqual(cups.Connection().cancelJob.call_count, 0)
+        with (
+            mock.patch(f"{model}.cups") as cups,
+            self.assertLogs(level=logging.WARNING) as logs,
+        ):
+            cups.Connection.side_effect = Exception
+            printer = self.new_printer()
+            job = self.new_job(printer, {"job_id_cups": 2})
+            job.action_cancel()
+            cups.Connection.side_effect = None
+            self.assertEqual(cups.Connection().cancelJob.call_count, 0)
+
+            self.assertEqual(len(logs.records), 3)
+            self.assertEqual(logs.records[0].levelno, logging.WARNING)
 
-    @mock.patch("%s.cups" % model)
+    @mock.patch(f"{model}.cups")
     def test_cancel_job(self, cups):
         """It should catch any exception from CUPS and update status"""
         printer = self.new_printer()

odoo/addons/base_report_to_printer/tests/test_printing_printer.py

@@ -1,6 +1,7 @@
 # Copyright 2016 LasLabs Inc.
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
 
+import logging
 import tempfile
 from unittest import mock
 
@@ -79,11 +80,11 @@ class TestPrintingPrinter(TransactionCase):
         )
         self.assertTrue("InputSlot" in self.Model.print_options(report, tray="Test"))
 
-    @mock.patch("%s.cups" % server_model)
+    @mock.patch(f"{server_model}.cups")
     def test_print_report(self, cups):
         """It should print a report through CUPS"""
         fd, file_name = tempfile.mkstemp()
-        with mock.patch("%s.mkstemp" % model) as mkstemp:
+        with mock.patch(f"{model}.mkstemp") as mkstemp:
             mkstemp.return_value = fd, file_name
             printer = self.new_record()
             printer.print_document(self.report, b"content to print", doc_format="pdf")
@@ -91,37 +92,52 @@ class TestPrintingPrinter(TransactionCase):
                 printer.system_name, file_name, file_name, options={}
             )
 
-    @mock.patch("%s.cups" % server_model)
-    def test_print_report_error(self, cups):
+    def test_print_report_error(self):
         """It should print a report through CUPS"""
-        cups.Connection.side_effect = Exception
-        fd, file_name = tempfile.mkstemp()
-        with mock.patch("%s.mkstemp" % model) as mkstemp:
-            mkstemp.return_value = fd, file_name
-            printer = self.new_record()
-            with self.assertRaises(UserError):
-                printer.print_document(
-                    self.report, b"content to print", doc_format="pdf"
-                )
-
-    @mock.patch("%s.cups" % server_model)
-    def test_print_file(self, cups):
+        with (
+            mock.patch(f"{model}.cups") as cups,
+            self.assertLogs(level=logging.WARNING) as logs,
+        ):
+            cups.Connection.side_effect = Exception
+            fd, file_name = tempfile.mkstemp()
+            with mock.patch(f"{model}.mkstemp") as mkstemp:
+                mkstemp.return_value = fd, file_name
+                printer = self.new_record()
+                with self.assertRaises(UserError):
+                    printer.print_document(
+                        self.report, b"content to print", doc_format="pdf"
+                    )
+            self.assertEqual(len(logs.records), 1)
+            self.assertEqual(logs.records[0].levelno, logging.WARNING)
+
+    def test_print_file(self):
         """It should print a file through CUPS"""
-        file_name = "file_name"
-        printer = self.new_record()
-        printer.print_file(file_name, "pdf")
-        cups.Connection().printFile.assert_called_once_with(
-            printer.system_name, file_name, file_name, options={}
-        )
+        with (
+            mock.patch(f"{server_model}.cups") as cups,
+            self.assertLogs(level=logging.WARNING) as logs,
+        ):
+            file_name = "file_name"
+            printer = self.new_record()
+            printer.print_file(file_name, "pdf")
+            cups.Connection().printFile.assert_called_once_with(
+                printer.system_name, file_name, file_name, options={}
+            )
+            self.assertEqual(len(logs.records), 1)
+            self.assertEqual(logs.records[0].levelno, logging.WARNING)
 
-    @mock.patch("%s.cups" % server_model)
-    def test_print_file_error(self, cups):
+    def test_print_file_error(self):
         """It should print a file through CUPS"""
-        cups.Connection.side_effect = Exception
-        file_name = "file_name"
-        printer = self.new_record()
-        with self.assertRaises(UserError):
-            printer.print_file(file_name)
+        with (
+            mock.patch(f"{server_model}.cups") as cups,
+            self.assertLogs(level=logging.WARNING) as logs,
+        ):
+            cups.Connection.side_effect = Exception
+            file_name = "file_name"
+            printer = self.new_record()
+            with self.assertRaises(UserError):
+                printer.print_file(file_name)
+            self.assertEqual(len(logs.records), 1)
+            self.assertEqual(logs.records[0].levelno, logging.WARNING)
 
     def test_set_default(self):
         """It should set a single record as default"""
@@ -142,7 +158,7 @@ class TestPrintingPrinter(TransactionCase):
         printer.unset_default()
         self.assertFalse(printer.default)
 
-    @mock.patch("%s.cups" % server_model)
+    @mock.patch(f"{server_model}.cups")
     def test_cancel_all_jobs(self, cups):
         """It should cancel all jobs"""
         printer = self.new_record()
@@ -151,7 +167,7 @@ class TestPrintingPrinter(TransactionCase):
             name=printer.system_name, purge_jobs=False
         )
 
-    @mock.patch("%s.cups" % server_model)
+    @mock.patch(f"{server_model}.cups")
     def test_cancel_and_purge_all_jobs(self, cups):
         """It should cancel all jobs"""
         printer = self.new_record()
@@ -160,21 +176,21 @@ class TestPrintingPrinter(TransactionCase):
             name=printer.system_name, purge_jobs=True
         )
 
-    @mock.patch("%s.cups" % server_model)
+    @mock.patch(f"{server_model}.cups")
     def test_enable_printer(self, cups):
         """It should enable the printer"""
         printer = self.new_record()
         printer.enable()
         cups.Connection().enablePrinter.assert_called_once_with(printer.system_name)
 
-    @mock.patch("%s.cups" % server_model)
+    @mock.patch(f"{server_model}.cups")
     def test_disable_printer(self, cups):
         """It should disable the printer"""
         printer = self.new_record()
         printer.disable()
         cups.Connection().disablePrinter.assert_called_once_with(printer.system_name)
 
-    @mock.patch("%s.cups" % server_model)
+    @mock.patch(f"{server_model}.cups")
     def test_print_test_page(self, cups):
         """It should print a test page"""
         printer = self.new_record()

odoo/addons/base_report_to_printer/tests/test_printing_printer_tray.py

@@ -99,7 +99,7 @@ class TestPrintingPrinter(TransactionCase):
             }
         }
 
-    @mock.patch("%s.cups" % server_model)
+    @mock.patch(f"{server_model}.cups")
     def test_update_printers(self, cups):
         """
         Check that the update_printers method calls _prepare_update_from_cups
@@ -111,7 +111,7 @@ class TestPrintingPrinter(TransactionCase):
         self.ServerModel.update_printers()
         self.assertEqual(self.printer.name, "My custom name")
 
-    @mock.patch("%s.cups" % server_model)
+    @mock.patch(f"{server_model}.cups")
     def test_prepare_update_from_cups_no_ppd(self, cups):
         """
         Check that the tray_ids field has no value when no PPD is available
@@ -124,7 +124,7 @@ class TestPrintingPrinter(TransactionCase):
         vals = self.printer._prepare_update_from_cups(connection, cups_printer)
         self.assertFalse("tray_ids" in vals)
 
-    @mock.patch("%s.cups" % server_model)
+    @mock.patch(f"{server_model}.cups")
     def test_prepare_update_from_cups_empty_ppd(self, cups):
         """
         Check that the tray_ids field has no value when the PPD file has
@@ -142,7 +142,7 @@ class TestPrintingPrinter(TransactionCase):
         vals = self.printer._prepare_update_from_cups(connection, cups_printer)
         self.assertFalse("tray_ids" in vals)
 
-    @mock.patch("%s.cups" % server_model)
+    @mock.patch(f"{server_model}.cups")
     @mock.patch("os.unlink")
     def test_prepare_update_from_cups_unlink_error(self, os_unlink, cups):
         """
@@ -159,7 +159,7 @@ class TestPrintingPrinter(TransactionCase):
         with self.assertRaises(OSError):
             self.printer._prepare_update_from_cups(connection, cups_printer)
 
-    @mock.patch("%s.cups" % server_model)
+    @mock.patch(f"{server_model}.cups")
     @mock.patch("os.unlink")
     def test_prepare_update_from_cups_unlink_error_enoent(self, os_unlink, cups):
         """
@@ -181,7 +181,7 @@ class TestPrintingPrinter(TransactionCase):
             [(0, 0, {"name": "Auto (Default)", "system_name": "Auto"})],
         )
 
-    @mock.patch("%s.cups" % server_model)
+    @mock.patch(f"{server_model}.cups")
     def test_prepare_update_from_cups(self, cups):
         """
         Check the return value when adding a single tray
@@ -197,7 +197,7 @@ class TestPrintingPrinter(TransactionCase):
             [(0, 0, {"name": "Auto (Default)", "system_name": "Auto"})],
         )
 
-    @mock.patch("%s.cups" % server_model)
+    @mock.patch(f"{server_model}.cups")
     def test_prepare_update_from_cups_with_multiple_trays(self, cups):
         """
         Check the return value when adding multiple trays at once
@@ -216,7 +216,7 @@ class TestPrintingPrinter(TransactionCase):
             ],
         )
 
-    @mock.patch("%s.cups" % server_model)
+    @mock.patch(f"{server_model}.cups")
     def test_prepare_update_from_cups_already_known_trays(self, cups):
         """
         Check that calling the method twice doesn't create the trays multiple
@@ -236,7 +236,7 @@ class TestPrintingPrinter(TransactionCase):
             [(0, 0, {"name": "Auto (Default)", "system_name": "Auto"})],
         )
 
-    @mock.patch("%s.cups" % server_model)
+    @mock.patch(f"{server_model}.cups")
     def test_prepare_update_from_cups_unknown_trays(self, cups):
         """
         Check that trays which are not in the PPD file are removed from Odoo

odoo/addons/base_report_to_printer/tests/test_printing_printer_wizard.py

@@ -1,6 +1,7 @@
 # Copyright 2016 LasLabs Inc.
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
 
+import logging
 from unittest import mock
 
 from odoo.exceptions import UserError
@@ -36,7 +37,7 @@ class TestPrintingPrinterWizard(TransactionCase):
             "uri": self.printer_vals["device-uri"],
         }
 
-    @mock.patch("%s.cups" % model)
+    @mock.patch(f"{model}.cups")
     def test_action_ok_inits_connection(self, cups):
         """It should initialize CUPS connection"""
         self.Model.action_ok()
@@ -44,7 +45,7 @@ class TestPrintingPrinterWizard(TransactionCase):
             host=self.server.address, port=self.server.port
         )
 
-    @mock.patch("%s.cups" % model)
+    @mock.patch(f"{model}.cups")
     def test_action_ok_gets_printers(self, cups):
         """It should get printers from CUPS"""
         cups.Connection().getPrinters.return_value = {"sys_name": self.printer_vals}
@@ -52,14 +53,19 @@ class TestPrintingPrinterWizard(TransactionCase):
         self.Model.action_ok()
         cups.Connection().getPrinters.assert_called_once_with()
 
-    @mock.patch("%s.cups" % model)
-    def test_action_ok_raises_warning_on_error(self, cups):
+    def test_action_ok_raises_warning_on_error(self):
         """It should raise Warning on any error"""
-        cups.Connection.side_effect = StopTest
-        with self.assertRaises(UserError):
-            self.Model.action_ok()
+        with (
+            mock.patch(f"{model}.cups") as cups,
+            self.assertLogs(level=logging.WARNING) as logs,
+        ):
+            cups.Connection.side_effect = StopTest
+            with self.assertRaises(UserError):
+                self.Model.action_ok()
+            self.assertEqual(len(logs.records), 1)
+            self.assertEqual(logs.records[0].levelno, logging.WARNING)
 
-    @mock.patch("%s.cups" % model)
+    @mock.patch(f"{model}.cups")
     def test_action_ok_creates_new_printer(self, cups):
         """It should create new printer w/ proper vals"""
         cups.Connection().getPrinters.return_value = {"sys_name": self.printer_vals}
@@ -75,7 +81,7 @@ class TestPrintingPrinterWizard(TransactionCase):
 
             self.assertEqual(val, rec_id[key])
 
-    @mock.patch("%s.cups" % model)
+    @mock.patch(f"{model}.cups")
     def test_action_ok_skips_existing_printer(self, cups):
         """It should not recreate existing printers"""
         cups.Connection().getPrinters.return_value = {"sys_name": self.printer_vals}