delivery_carrier_deposit/models/deposit_slip.py

@@ -18,7 +18,7 @@ class DepositSlip(models.Model):
     def _compute_weight(self):
         weight = 0.0
         for picking in self.picking_ids:
-            weight += picking.weight
+            weight += picking.shipping_weight
         self.weight = weight
 
     name = fields.Char(

delivery_carrier_deposit/tests/__init__.py

@@ -0,0 +1 @@
+from . import test_deposit_slip

delivery_carrier_deposit/tests/models.py

@@ -0,0 +1,16 @@
+# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
+from odoo import fields, models
+
+
+class FakeDeliveryCarrier(models.Model):
+    _inherit = "delivery.carrier"
+
+    delivery_type = fields.Selection(
+        selection_add=[("test", "Test Carrier")], ondelete={"test": "set default"}
+    )
+
+    def test_send_shipping(self, pickings):
+        res = []
+        for _p in pickings:
+            res = res + [{"exact_price": 0.0, "tracking_number": False}]
+        return res

delivery_carrier_deposit/tests/test_deposit_slip.py

@@ -0,0 +1,44 @@
+# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).
+
+from odoo_test_helper import FakeModelLoader
+
+from odoo.tests.common import SavepointCase
+
+
+class TestDepositSlip(SavepointCase):
+    @classmethod
+    def setUpClass(cls):
+        super().setUpClass()
+        cls.loader = FakeModelLoader(cls.env, cls.__module__)
+        cls.loader.backup_registry()
+
+        # The fake class is imported here !! After the backup_registry
+        from .models import FakeDeliveryCarrier
+
+        cls.loader.update_registry((FakeDeliveryCarrier,))
+
+        delivery_free_product = cls.env.ref("delivery.product_product_delivery")
+        cls.carrier = cls.env["delivery.carrier"].create(
+            {
+                "name": "Test Carrier",
+                "delivery_type": "test",
+                "product_id": delivery_free_product.id,
+            }
+        )
+        cls.delivery_order = cls.env.ref("stock.outgoing_shipment_main_warehouse4")
+        cls.delivery_order.write({"carrier_id": cls.carrier.id})
+
+    def test_delivery_slip_creation(self):
+        self.delivery_order._action_done()
+        wizard = self.env["delivery.deposit.wizard"].create(
+            {
+                "delivery_type": "test",
+            }
+        )
+        wizard.create_deposit_slip()
+        deposit = self.env["deposit.slip"].search([("state", "=", "draft")])
+        self.assertEqual(len(deposit), 1)
+        self.assertEqual(len(deposit.picking_ids), 1)
+        self.assertEqual(deposit.weight, self.delivery_order.shipping_weight)
+        deposit.validate_deposit()
+        self.assertEqual(deposit.state, "done")

delivery_carrier_deposit/views/deposit_slip_view.xml

@@ -95,7 +95,7 @@
                             name="picking_ids"
                             nolabel="1"
                             widget="many2many"
-                            options="{'not_delete': True}"
+                            options="{'not_delete': True, 'no_create': True}"
                         >
                         <tree>
                             <field name="name" />

delivery_carrier_deposit/views/stock_picking_view.xml

@@ -8,7 +8,7 @@
         <field name="weight" position="after">
             <field
                     name="deposit_slip_id"
-                    attrs="{'readonly': True,'invisible': [('deposit_slip_id','=','')]}"
+                    attrs="{'readonly': True,'invisible': [('deposit_slip_id', '=', False)]}"
                 />
         </field>
     </field>

test-requirements.txt

@@ -0,0 +1 @@
+odoo-test-helper >= 1.1.0