[FIX] purchase_duplicate_check: Code review fixing

Author: geomer198

purchase_duplicate_check/models/purchase_order.py

@@ -20,12 +20,12 @@ def _prepare_pending_orders_message(self, product_id):
             )
             type_ = order.state in ["draft", "sent"] and "RFQ" or "PO"
             message_parts.append(
-                f"{type_}: {order_href} date: {order.create_date.date()} Qty: {line.product_qty}<br/>"  # noqa
+                f"{type_}: {order_href}; date: {order.create_date.date()}; Qty: {line.product_qty};<br/>"  # noqa
             )
         return "".join(message_parts)
 
     def _is_activity_enabled(self) -> bool:
-        """Check if activity for repeating orders is enabled"""
+        """Check if activity for duplicated orders is enabled"""
         return (
             self.env["ir.config_parameter"]
             .sudo()

purchase_duplicate_check/tests/test_purchase_order.py

@@ -42,7 +42,7 @@ def test_prepare_pending_orders_message(self):
         """Test flow where prepare message for purchase order"""
         message = self.order1._prepare_pending_orders_message(self.product_2.id)
         self.assertFalse(message, "Message must be empty")
-        expected_message = f"RFQ: <a href='/web#id={self.order1.id}&model=purchase.order'>{self.order1.name}</a> date: {self.order1.create_date.date()} Qty: 10.0<br/>"  # noqa
+        expected_message = f"RFQ: <a href='/web#id={self.order1.id}&model=purchase.order'>{self.order1.name}</a>; date: {self.order1.create_date.date()}; Qty: 10.0;<br/>"  # noqa
         message = self.order1._prepare_pending_orders_message(self.product_1.id)
         self.assertEqual(message, expected_message, "Messages must be the same")