[FIX] delivery_auto_refresh: dynamic carrier line

Author: renda-dev

delivery_auto_refresh/models/sale_order.py

@@ -67,10 +67,12 @@ def _auto_refresh_delivery(self):
             if self.state in {"draft", "sent"}:
                 price_unit = self.carrier_id.rate_shipment(self)["price"]
                 if not self.is_all_service:
-                    self._create_delivery_line(self.carrier_id, price_unit)
-                self.with_context(auto_refresh_delivery=True).write(
-                    {"recompute_delivery_price": False}
-                )
+                    self.with_context(skip_validation_check=True)._create_delivery_line(
+                        self.carrier_id, price_unit
+                    )
+                self.with_context(
+                    auto_refresh_delivery=True, skip_validation_check=True
+                ).write({"recompute_delivery_price": False})
 
     @api.model
     def create(self, vals):

delivery_auto_refresh/tests/test_delivery_auto_refresh.py

@@ -303,8 +303,7 @@ def test_auto_refresh_so_and_unlink_line(self):
         self.assertFalse(delivery_line.exists())
 
     def test_auto_add_delivery_line_add_service(self):
-        """Delivery line should not be created because
-        there are only service products in SO"""
+        self.env["ir.config_parameter"].sudo().set_param(self.auto_add_delivery_line, 1)
         service = self.env["product.product"].create(
             {"name": "Service Test", "type": "service"}
         )