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

odoo/addons/base_report_to_printer/models/printing_report_xml_action.py

@@ -22,7 +22,7 @@ class PrintingReportXmlAction(models.Model):
         comodel_name="res.users", string="User", required=True, ondelete="cascade"
     )
     action = fields.Selection(
-        selection=lambda s: s.env["printing.action"]._available_action_types(),
+        selection=lambda s: s.env["printing.action"]._available_action_types,
         required=True,
     )
     printer_id = fields.Many2one(comodel_name="printing.printer", string="Printer")

odoo/addons/base_report_to_printer/models/res_users.py

@@ -11,19 +11,27 @@ from odoo import api, fields, models
 class ResUsers(models.Model):
     _inherit = "res.users"
 
-    @api.model
+    @property
     def _user_available_action_types(self):
         return [
             (code, string)
-            for code, string in self.env["printing.action"]._available_action_types()
+            for code, string in self.env["printing.action"]._available_action_types
             if code != "user_default"
         ]
 
-    printing_action = fields.Selection(selection=_user_available_action_types)
+    printing_action = fields.Selection(
+        selection=lambda self: self._user_available_action_types
+    )
     printing_printer_id = fields.Many2one(
         comodel_name="printing.printer", string="Default Printer"
     )
 
+    @api.constrains("printing_action")
+    def _check_printing_action(self):
+        for rec in self:
+            if rec.printing_action == "user_default":
+                raise ValueError("user_default should not be available")
+
     @property
     def SELF_READABLE_FIELDS(self):
         return super().SELF_READABLE_FIELDS + ["printing_action", "printing_printer_id"]

odoo/addons/base_report_to_printer/readme/CONTRIBUTORS.md

@@ -15,3 +15,4 @@
 - Hughes Damry \<<hughes.damry@acsone.eu>\>
 - Akim Juillerat \<<akim.juillerat@camptocamp.com>\>
 - Jacques-Etienne Baudoux (BCIM) \<<je@bcim.be>\>
+- Tris Doan \<<tridm@trobz.com>\>

odoo/addons/base_report_to_printer/readme/CREDITS.md

@@ -0,0 +1 @@
+The migration of this module from 17.0 to 18.0 was financially supported by Camptocamp.

odoo/addons/base_report_to_printer/static/description/index.html

@@ -367,9 +367,9 @@ ul.auto-toc {
 !! This file is generated by oca-gen-addon-readme !!
 !! changes will be overwritten.                   !!
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
-!! source digest: sha256:617c8c00cdb15a695dde5b4ab62953885ca35720cf15abfdad4ad886bb5244c8
+!! source digest: sha256:7761acf7bc8cae2f3c56913a0da0ebc3d646ee29311fb4682bde14452b3a69d8
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! -->
-<p><a class="reference external image-reference" href="https://odoo-community.org/page/development-status"><img alt="Beta" src="https://img.shields.io/badge/maturity-Beta-yellow.png" /></a> <a class="reference external image-reference" href="http://www.gnu.org/licenses/agpl-3.0-standalone.html"><img alt="License: AGPL-3" src="https://img.shields.io/badge/licence-AGPL--3-blue.png" /></a> <a class="reference external image-reference" href="https://github.com/OCA/report-print-send/tree/17.0/base_report_to_printer"><img alt="OCA/report-print-send" src="https://img.shields.io/badge/github-OCA%2Freport--print--send-lightgray.png?logo=github" /></a> <a class="reference external image-reference" href="https://translation.odoo-community.org/projects/report-print-send-17-0/report-print-send-17-0-base_report_to_printer"><img alt="Translate me on Weblate" src="https://img.shields.io/badge/weblate-Translate%20me-F47D42.png" /></a> <a class="reference external image-reference" href="https://runboat.odoo-community.org/builds?repo=OCA/report-print-send&amp;target_branch=17.0"><img alt="Try me on Runboat" src="https://img.shields.io/badge/runboat-Try%20me-875A7B.png" /></a></p>
+<p><a class="reference external image-reference" href="https://odoo-community.org/page/development-status"><img alt="Beta" src="https://img.shields.io/badge/maturity-Beta-yellow.png" /></a> <a class="reference external image-reference" href="http://www.gnu.org/licenses/agpl-3.0-standalone.html"><img alt="License: AGPL-3" src="https://img.shields.io/badge/licence-AGPL--3-blue.png" /></a> <a class="reference external image-reference" href="https://github.com/OCA/report-print-send/tree/18.0/base_report_to_printer"><img alt="OCA/report-print-send" src="https://img.shields.io/badge/github-OCA%2Freport--print--send-lightgray.png?logo=github" /></a> <a class="reference external image-reference" href="https://translation.odoo-community.org/projects/report-print-send-18-0/report-print-send-18-0-base_report_to_printer"><img alt="Translate me on Weblate" src="https://img.shields.io/badge/weblate-Translate%20me-F47D42.png" /></a> <a class="reference external image-reference" href="https://runboat.odoo-community.org/builds?repo=OCA/report-print-send&amp;target_branch=18.0"><img alt="Try me on Runboat" src="https://img.shields.io/badge/runboat-Try%20me-875A7B.png" /></a></p>
 <p>This module allows users to send reports to a printer attached to the
 server.</p>
 <p>It adds an optional behaviour on reports to send it directly to a
@@ -409,7 +409,8 @@ preprinted paper such as payment slip.</p>
 <li><a class="reference internal" href="#credits" id="toc-entry-9">Credits</a><ul>
 <li><a class="reference internal" href="#authors" id="toc-entry-10">Authors</a></li>
 <li><a class="reference internal" href="#contributors" id="toc-entry-11">Contributors</a></li>
-<li><a class="reference internal" href="#maintainers" id="toc-entry-12">Maintainers</a></li>
+<li><a class="reference internal" href="#other-credits" id="toc-entry-12">Other credits</a></li>
+<li><a class="reference internal" href="#maintainers" id="toc-entry-13">Maintainers</a></li>
 </ul>
 </li>
 </ul>
@@ -454,9 +455,9 @@ Printers from CUPS</em></li>
 <li>To print a report on a specific printer or tray, you can change
 these in <em>Settings &gt; Printing &gt; Reports</em> to define default
 behaviour.</li>
-<li>To print a report on a specific printer and/or tray for a user,
-you can change these in <em>Settings &gt; Printing &gt; Reports</em> in
-<em>Specific actions per user</em></li>
+<li>To print a report on a specific printer and/or tray for a user, you
+can change these in <em>Settings &gt; Printing &gt; Reports</em> in <em>Specific
+actions per user</em></li>
 <li>Users may also select a default action, printer or tray in their
 preferences.</li>
 </ul>
@@ -492,7 +493,7 @@ need to notify the bus or use web_notify for it.</li>
 <p>Bugs are tracked on <a class="reference external" href="https://github.com/OCA/report-print-send/issues">GitHub Issues</a>.
 In case of trouble, please check there if your issue has already been reported.
 If you spotted it first, help us to smash it by providing a detailed and welcomed
-<a class="reference external" href="https://github.com/OCA/report-print-send/issues/new?body=module:%20base_report_to_printer%0Aversion:%2017.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**">feedback</a>.</p>
+<a class="reference external" href="https://github.com/OCA/report-print-send/issues/new?body=module:%20base_report_to_printer%0Aversion:%2018.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**">feedback</a>.</p>
 <p>Do not contact contributors directly about support or help with technical issues.</p>
 </div>
 <div class="section" id="credits">
@@ -528,10 +529,16 @@ If you spotted it first, help us to smash it by providing a detailed and welcome
 <li>Hughes Damry &lt;<a class="reference external" href="mailto:hughes.damry&#64;acsone.eu">hughes.damry&#64;acsone.eu</a>&gt;</li>
 <li>Akim Juillerat &lt;<a class="reference external" href="mailto:akim.juillerat&#64;camptocamp.com">akim.juillerat&#64;camptocamp.com</a>&gt;</li>
 <li>Jacques-Etienne Baudoux (BCIM) &lt;<a class="reference external" href="mailto:je&#64;bcim.be">je&#64;bcim.be</a>&gt;</li>
+<li>Tris Doan &lt;<a class="reference external" href="mailto:tridm&#64;trobz.com">tridm&#64;trobz.com</a>&gt;</li>
 </ul>
 </div>
+<div class="section" id="other-credits">
+<h2><a class="toc-backref" href="#toc-entry-12">Other credits</a></h2>
+<p>The migration of this module from 17.0 to 18.0 was financially supported
+by Camptocamp.</p>
+</div>
 <div class="section" id="maintainers">
-<h2><a class="toc-backref" href="#toc-entry-12">Maintainers</a></h2>
+<h2><a class="toc-backref" href="#toc-entry-13">Maintainers</a></h2>
 <p>This module is maintained by the OCA.</p>
 <a class="reference external image-reference" href="https://odoo-community.org">
 <img alt="Odoo Community Association" src="https://odoo-community.org/logo.png" />
@@ -539,7 +546,7 @@ If you spotted it first, help us to smash it by providing a detailed and welcome
 <p>OCA, or the Odoo Community Association, is a nonprofit organization whose
 mission is to support the collaborative development of Odoo features and
 promote its widespread use.</p>
-<p>This module is part of the <a class="reference external" href="https://github.com/OCA/report-print-send/tree/17.0/base_report_to_printer">OCA/report-print-send</a> project on GitHub.</p>
+<p>This module is part of the <a class="reference external" href="https://github.com/OCA/report-print-send/tree/18.0/base_report_to_printer">OCA/report-print-send</a> project on GitHub.</p>
 <p>You are welcome to contribute. To learn how please visit <a class="reference external" href="https://odoo-community.org/page/Contribute">https://odoo-community.org/page/Contribute</a>.</p>
 </div>
 </div>

odoo/addons/base_report_to_printer/static/src/js/qweb_action_manager.esm.js

@@ -1,6 +1,5 @@
-/** @odoo-module */
-import {_t} from "@web/core/l10n/translation";
 import {markup} from "@odoo/owl";
+import {_t} from "@web/core/l10n/translation";
 import {registry} from "@web/core/registry";
 
 async function cupsReportActionHandler(action, options, env) {
@@ -52,6 +51,7 @@ async function cupsReportActionHandler(action, options, env) {
                     title: `${terms.issue_on} ${print_action.printer_name}`,
                     type: "warning",
                     sticky: true,
+                    messageIsHtml: true,
                     buttons: [
                         {
                             name: _t("Print"),

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()