# Copyright 2026 Criptomart # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl). from odoo.tests import tagged from odoo.tests.common import TransactionCase @tagged("-at_install", "post_install") class TestBatchSummary(TransactionCase): @classmethod def setUpClass(cls): super().setUpClass() cls.uom_unit = cls.env.ref("uom.product_uom_unit") cls.uom_dozen = cls.env.ref("uom.product_uom_dozen") cls.location_src = cls.env.ref("stock.stock_location_stock") cls.location_dest = cls.env.ref("stock.stock_location_customers") cls.picking_type = cls.env.ref("stock.picking_type_out") cls.product = cls.env["product.product"].create( { "name": "Test Product", "uom_id": cls.uom_unit.id, "uom_po_id": cls.uom_unit.id, } ) cls.batch = cls.env["stock.picking.batch"].create( {"name": "Batch Test", "picking_type_id": cls.picking_type.id} ) cls.picking1 = cls.env["stock.picking"].create( { "picking_type_id": cls.picking_type.id, "location_id": cls.location_src.id, "location_dest_id": cls.location_dest.id, "batch_id": cls.batch.id, } ) cls.picking2 = cls.env["stock.picking"].create( { "picking_type_id": cls.picking_type.id, "location_id": cls.location_src.id, "location_dest_id": cls.location_dest.id, "batch_id": cls.batch.id, } ) # Helpers def _add_move(self, picking, qty_demanded, qty_done, uom): move = self.env["stock.move"].create( { "name": self.product.name, "product_id": self.product.id, "product_uom_qty": qty_demanded, "product_uom": uom.id, "picking_id": picking.id, "location_id": self.location_src.id, "location_dest_id": self.location_dest.id, } ) if qty_done: self.env["stock.move.line"].create( { "move_id": move.id, "picking_id": picking.id, "product_id": self.product.id, "product_uom_id": self.product.uom_id.id, "location_id": self.location_src.id, "location_dest_id": self.location_dest.id, "quantity": qty_done, } ) return move def _recompute_batch(self): self.batch.invalidate_recordset() self.batch._compute_summary_line_ids() def _create_batch_with_pickings(self): batch = self.env["stock.picking.batch"].create( {"name": "Batch Flow", "picking_type_id": self.picking_type.id} ) picking = self.env["stock.picking"].create( { "picking_type_id": self.picking_type.id, "location_id": self.location_src.id, "location_dest_id": self.location_dest.id, } ) move = self.env["stock.move"].create( { "name": self.product.name, "product_id": self.product.id, "product_uom_qty": 2.0, "product_uom": self.uom_unit.id, "picking_id": picking.id, "location_id": self.location_src.id, "location_dest_id": self.location_dest.id, } ) self.env["stock.move.line"].create( { "move_id": move.id, "picking_id": picking.id, "product_id": self.product.id, "product_uom_id": self.uom_unit.id, "location_id": self.location_src.id, "location_dest_id": self.location_dest.id, "quantity": 2.0, } ) picking.batch_id = batch.id return batch # Tests def test_totals_and_pending_with_conversion(self): """Totals aggregate per product with UoM conversion and pending.""" # demand 12 units, done 5 units self._add_move(self.picking1, qty_demanded=12, qty_done=5, uom=self.uom_unit) # demand 2 dozens (24 units), done 6 units self._add_move(self.picking2, qty_demanded=2, qty_done=6, uom=self.uom_dozen) self._recompute_batch() self.assertEqual(len(self.batch.summary_line_ids), 1) line = self.batch.summary_line_ids self.assertEqual(line.product_id, self.product) self.assertEqual(line.product_uom_id, self.uom_unit) self.assertAlmostEqual(line.qty_demanded, 36.0) self.assertAlmostEqual(line.qty_done, 11.0) self.assertAlmostEqual(line.qty_pending, 25.0) def test_collected_flag_preserved_on_recompute(self): """Collected stays checked after totals change.""" self._add_move(self.picking1, qty_demanded=1, qty_done=1, uom=self.uom_unit) self._recompute_batch() line = self.batch.summary_line_ids line.is_collected = True # Add more demand/done to trigger an update self._add_move(self.picking1, qty_demanded=3, qty_done=2, uom=self.uom_unit) self._recompute_batch() self.assertTrue(self.batch.summary_line_ids.is_collected) def test_cancelled_moves_are_ignored(self): """Cancelled moves do not count in the summary and lines are removed.""" move1 = self._add_move( self.picking1, qty_demanded=4, qty_done=2, uom=self.uom_unit ) move2 = self._add_move( self.picking2, qty_demanded=6, qty_done=3, uom=self.uom_unit ) self._recompute_batch() self.assertEqual(len(self.batch.summary_line_ids), 1) move1.write({"state": "cancel"}) move2.write({"state": "cancel"}) self._recompute_batch() self.assertFalse(self.batch.summary_line_ids) def test_no_required_product_error_on_confirm(self): """Confirming batch with pickings must not create summary lines without product.""" batch = self._create_batch_with_pickings() # Trigger compute via confirm flow batch.action_confirm() self.assertTrue(batch.summary_line_ids) self.assertFalse( batch.summary_line_ids.filtered(lambda line: not line.product_id) ) def test_empty_batch_add_pickings_then_confirm(self): """Create empty draft batch, add multiple pickings, then confirm.""" # 1. Create empty batch in draft state batch = self.env["stock.picking.batch"].create( {"name": "Batch Empty Start", "picking_type_id": self.picking_type.id} ) self.assertEqual(batch.state, "draft") self.assertFalse(batch.picking_ids) self.assertFalse(batch.summary_line_ids) # 2. Create pickings with moves (without batch assignment) product2 = self.env["product.product"].create( { "name": "Test Product 2", "uom_id": self.uom_unit.id, "uom_po_id": self.uom_unit.id, } ) picking_a = self.env["stock.picking"].create( { "picking_type_id": self.picking_type.id, "location_id": self.location_src.id, "location_dest_id": self.location_dest.id, } ) self.env["stock.move"].create( { "name": self.product.name, "product_id": self.product.id, "product_uom_qty": 5.0, "product_uom": self.uom_unit.id, "picking_id": picking_a.id, "location_id": self.location_src.id, "location_dest_id": self.location_dest.id, } ) picking_b = self.env["stock.picking"].create( { "picking_type_id": self.picking_type.id, "location_id": self.location_src.id, "location_dest_id": self.location_dest.id, } ) self.env["stock.move"].create( { "name": product2.name, "product_id": product2.id, "product_uom_qty": 10.0, "product_uom": self.uom_unit.id, "picking_id": picking_b.id, "location_id": self.location_src.id, "location_dest_id": self.location_dest.id, } ) picking_c = self.env["stock.picking"].create( { "picking_type_id": self.picking_type.id, "location_id": self.location_src.id, "location_dest_id": self.location_dest.id, } ) self.env["stock.move"].create( { "name": self.product.name, "product_id": self.product.id, "product_uom_qty": 3.0, "product_uom": self.uom_unit.id, "picking_id": picking_c.id, "location_id": self.location_src.id, "location_dest_id": self.location_dest.id, } ) # 3. Add pickings to the batch picking_a.batch_id = batch picking_b.batch_id = batch picking_c.batch_id = batch self.assertEqual(len(batch.picking_ids), 3) # 4. Confirm the batch — this should not raise product_id required error batch.action_confirm() self.assertEqual(batch.state, "in_progress") # 5. Verify summary lines are correct self.assertTrue(batch.summary_line_ids) self.assertFalse( batch.summary_line_ids.filtered(lambda line: not line.product_id) ) # Two products expected products_in_summary = batch.summary_line_ids.mapped("product_id") self.assertIn(self.product, products_in_summary) self.assertIn(product2, products_in_summary) # Check aggregated quantities line_product1 = batch.summary_line_ids.filtered( lambda line: line.product_id == self.product ) self.assertAlmostEqual(line_product1.qty_demanded, 8.0) # 5 + 3 line_product2 = batch.summary_line_ids.filtered( lambda line: line.product_id == product2 ) self.assertAlmostEqual(line_product2.qty_demanded, 10.0)