# -*- coding: utf-8 -*- # Part of Odoo. See LICENSE file for full copyright and licensing details. from odoo import Command, fields from odoo.exceptions import UserError from odoo.tests import Form from odoo.tests.common import TransactionCase class TestBatchPicking(TransactionCase): @classmethod def setUpClass(cls): """ Create 3 standard pickings and reserve them to have some move lines. The setup data looks like this: Picking1 Picking2 Picking3 ProductA ProductA ProductB Lot1: 5 units Lot4: 5 units SN6 : 1 unit Lot2: 5 units SN7 : 1 unit Lot3: 5 units SN8 : 1 unit ProductB SN9 : 1 unit SN1 : 1 unit SN10: 1 unit SN2 : 1 unit SN3 : 1 unit SN4 : 1 unit SN5 : 1 unit The picking_internal is the same as Picking1 move-wise, only using a different picking_type so it doesn't get auto-batched with the other pickings. """ super().setUpClass() cls.stock_location = cls.env.ref('stock.stock_location_stock') cls.customer_location = cls.env.ref('stock.stock_location_customers') cls.picking_type_out = cls.env['ir.model.data']._xmlid_to_res_id('stock.picking_type_out') cls.picking_type_in = cls.env['ir.model.data']._xmlid_to_res_id('stock.picking_type_in') cls.picking_type_internal = cls.env['ir.model.data']._xmlid_to_res_id('stock.picking_type_internal') cls.user_demo = cls.env['res.users'].search([('login', '=', 'demo')]) cls.productA = cls.env['product.product'].create({ 'name': 'Product A', 'is_storable': True, 'tracking': 'lot', 'categ_id': cls.env.ref('product.product_category_all').id, }) cls.lots_p_a = cls.env['stock.lot'].create([{ 'name': 'lot_product_a_' + str(i + 1), 'product_id': cls.productA.id, } for i in range(4)]) cls.productB = cls.env['product.product'].create({ 'name': 'Product B', 'is_storable': True, 'tracking': 'serial', 'categ_id': cls.env.ref('product.product_category_all').id, }) cls.lots_p_b = cls.env['stock.lot'].create([{ 'name': 'lot_product_a_' + str(i + 1), 'product_id': cls.productB.id, } for i in range(10)]) Quant = cls.env['stock.quant'] for lot in cls.lots_p_a: Quant._update_available_quantity(cls.productA, cls.stock_location, 5.0, lot_id=lot) for lot in cls.lots_p_b: Quant._update_available_quantity(cls.productB, cls.stock_location, 1.0, lot_id=lot) cls.picking_client_1 = cls.env['stock.picking'].create({ 'location_id': cls.stock_location.id, 'location_dest_id': cls.customer_location.id, 'picking_type_id': cls.picking_type_out, 'company_id': cls.env.company.id, 'state': 'draft', }) cls.env['stock.move'].create({ 'name': cls.productA.name, 'product_id': cls.productA.id, 'product_uom_qty': 15, 'product_uom': cls.productA.uom_id.id, 'picking_id': cls.picking_client_1.id, 'location_id': cls.stock_location.id, 'location_dest_id': cls.customer_location.id, }) cls.env['stock.move'].create({ 'name': cls.productB.name, 'product_id': cls.productB.id, 'product_uom_qty': 5, 'product_uom': cls.productB.uom_id.id, 'picking_id': cls.picking_client_1.id, 'location_id': cls.stock_location.id, 'location_dest_id': cls.customer_location.id, }) cls.picking_client_2 = cls.env['stock.picking'].create({ 'location_id': cls.stock_location.id, 'location_dest_id': cls.customer_location.id, 'picking_type_id': cls.picking_type_out, 'company_id': cls.env.company.id, 'state': 'draft', }) cls.env['stock.move'].create({ 'name': cls.productA.name, 'product_id': cls.productA.id, 'product_uom_qty': 5, 'product_uom': cls.productA.uom_id.id, 'picking_id': cls.picking_client_2.id, 'location_id': cls.stock_location.id, 'location_dest_id': cls.customer_location.id, }) cls.picking_client_3 = cls.env['stock.picking'].create({ 'location_id': cls.stock_location.id, 'location_dest_id': cls.customer_location.id, 'picking_type_id': cls.picking_type_out, 'company_id': cls.env.company.id, 'state': 'draft', }) cls.env['stock.move'].create({ 'name': cls.productB.name, 'product_id': cls.productB.id, 'product_uom_qty': 5, 'product_uom': cls.productB.uom_id.id, 'picking_id': cls.picking_client_3.id, 'location_id': cls.stock_location.id, 'location_dest_id': cls.customer_location.id, }) cls.picking_internal = cls.env['stock.picking'].create({ 'location_id': cls.stock_location.id, 'location_dest_id': cls.customer_location.id, 'picking_type_id': cls.picking_type_internal, 'company_id': cls.env.company.id, 'state': 'draft', }) cls.env['stock.move'].create({ 'name': cls.productA.name, 'product_id': cls.productA.id, 'product_uom_qty': 15, 'product_uom': cls.productA.uom_id.id, 'picking_id': cls.picking_internal.id, 'location_id': cls.customer_location.id, 'location_dest_id': cls.stock_location.id, }) cls.env['stock.move'].create({ 'name': cls.productB.name, 'product_id': cls.productB.id, 'product_uom_qty': 5, 'product_uom': cls.productB.uom_id.id, 'picking_id': cls.picking_internal.id, 'location_id': cls.customer_location.id, 'location_dest_id': cls.stock_location.id, }) cls.all_pickings = cls.picking_client_1 | cls.picking_client_2 | cls.picking_client_3 cls.all_pickings.action_confirm() cls.picking_internal.action_confirm() def test_creation_from_lines(self): """ Select all the move_lines and create a wave from them """ all_lines = self.all_pickings.move_line_ids res_dict = all_lines.action_open_add_to_wave() res_dict['context'] = {'active_model': 'stock.move.line', 'active_ids': all_lines.ids} self.assertEqual(res_dict.get('res_model'), 'stock.add.to.wave') wizard_form = Form.from_action(self.env, res_dict) wizard_form.mode = 'new' wizard_form.user_id = self.user_demo wizard_form.save().attach_pickings() wave = self.env['stock.picking.batch'].search([ ('is_wave', '=', True) ]) self.assertTrue(wave) self.assertEqual(wave.picking_ids, self.all_pickings) self.assertEqual(wave.move_line_ids, all_lines) self.assertEqual(wave.user_id, self.user_demo) def test_creation_from_pickings(self): """ Select all the picking_ids and create a wave from them """ action = self.env['ir.actions.actions']._for_xml_id('stock_picking_batch.stock_add_to_wave_action_stock_picking') action['context'] = {'active_model': 'stock.picking', 'active_ids': self.all_pickings.ids} self.assertEqual(action.get('res_model'), 'stock.add.to.wave') wizard_form = Form.from_action(self.env, action) wizard_form.mode = 'new' wizard = wizard_form.save() res = wizard.attach_pickings() self.assertEqual(set(res['context']['picking_to_wave']), set(self.all_pickings.ids)) def test_add_to_existing_wave_from_lines(self): res_dict = self.picking_client_1.move_line_ids.action_open_add_to_wave() res_dict['context'] = {'active_model': 'stock.move.line', 'active_ids': self.picking_client_1.move_line_ids.ids} wizard_form = Form.from_action(self.env, res_dict) wizard_form.mode = 'new' wizard_form.user_id = self.user_demo wizard_form.save().attach_pickings() wave = self.env['stock.picking.batch'].search([ ('is_wave', '=', True) ]) res_dict = self.picking_client_2.move_line_ids.action_open_add_to_wave() res_dict['context'] = {'active_model': 'stock.move.line', 'active_ids': self.picking_client_2.move_line_ids.ids} wizard_form = Form.from_action(self.env, res_dict) wizard_form.mode = 'existing' wizard_form.wave_id = wave wizard_form.save().attach_pickings() wave = self.env['stock.picking.batch'].search([ ('is_wave', '=', True) ]) self.assertEqual(len(wave), 1) self.assertEqual(wave.picking_ids, self.picking_client_1 | self.picking_client_2) def test_add_to_existing_wave_from_pickings(self): res_dict = self.picking_client_1.move_line_ids.action_open_add_to_wave() res_dict['context'] = {'active_model': 'stock.move.line', 'active_ids': self.picking_client_1.move_line_ids.ids} wizard_form = Form.from_action(self.env, res_dict) wizard_form.mode = 'new' wizard_form.user_id = self.user_demo action = wizard_form.save().attach_pickings() wave = self.env['stock.picking.batch'].search([ ('is_wave', '=', True) ]) action = self.env['ir.actions.actions']._for_xml_id('stock_picking_batch.stock_add_to_wave_action_stock_picking') action['context'] = {'active_model': 'stock.picking', 'active_ids': self.all_pickings.ids} self.assertEqual(action.get('res_model'), 'stock.add.to.wave') wizard_form = Form.from_action(self.env, action) wizard_form.mode = 'existing' wizard_form.wave_id = wave wizard = wizard_form.save() res = wizard.attach_pickings() self.assertEqual(set(res['context']['picking_to_wave']), set(self.all_pickings.ids)) self.assertEqual(res['context']['active_wave_id'], wave.id) def test_wave_split_picking(self): lines = self.picking_client_1.move_ids.filtered(lambda m: m.product_id == self.productB).move_line_ids move = lines.move_id self.assertEqual(len(move), 1) all_db_pickings = self.env['stock.picking'].search([]) all_db_pickings.write({'scheduled_date': '2025-01-01 00:00:00'}) res_dict = lines.action_open_add_to_wave() res_dict['context'] = {'active_model': 'stock.move.line', 'active_ids': lines.ids} self.assertEqual(res_dict.get('res_model'), 'stock.add.to.wave') wizard_form = Form.from_action(self.env, res_dict) wizard_form.mode = 'new' wizard_form.save().attach_pickings() wave = self.env['stock.picking.batch'].search([ ('is_wave', '=', True) ]) self.assertTrue(wave) # Original picking lost a stock move self.assertTrue(move.picking_id) self.assertFalse(move.picking_id == self.picking_client_1) self.assertTrue(self.picking_client_1.move_ids) self.assertTrue(move.picking_id.batch_id == wave) self.assertTrue(lines.batch_id == wave) new_all_db_picking = self.env['stock.picking'].search([]) self.assertEqual(len(all_db_pickings) + 1, len(new_all_db_picking)) new_picking = new_all_db_picking - all_db_pickings self.assertEqual(new_picking.scheduled_date, fields.Datetime.to_datetime('2025-01-01 00:00:00')) def test_wave_split_move(self): lines = self.picking_internal.move_ids.filtered(lambda m: m.product_id == self.productB).move_line_ids[0:2] move = lines.move_id all_db_pickings = self.env['stock.picking'].search([]) res_dict = lines.action_open_add_to_wave() res_dict['context'] = {'active_model': 'stock.move.line', 'active_ids': lines.ids} self.assertEqual(res_dict.get('res_model'), 'stock.add.to.wave') wizard_form = Form.from_action(self.env, res_dict) wizard_form.mode = 'new' wizard_form.save().attach_pickings() wave = self.env['stock.picking.batch'].search([ ('is_wave', '=', True) ]) self.assertTrue(wave) self.assertTrue(wave.picking_type_id) # Original picking lost a stock move self.assertTrue(move.picking_id) self.assertTrue(move.picking_id == self.picking_internal) self.assertFalse(lines.move_id == move) self.assertFalse(move.picking_id.batch_id) new_move = lines.move_id self.assertTrue(new_move.picking_id.batch_id == wave) self.assertTrue(lines.batch_id == wave) new_all_db_picking = self.env['stock.picking'].search([]) self.assertEqual(len(all_db_pickings) + 1, len(new_all_db_picking)) def test_wave_split_move_uom(self): self.uom_dozen = self.env.ref('uom.product_uom_dozen') sns = self.env['stock.lot'].create([{ 'name': 'sn-' + str(i), 'product_id': self.productB.id, } for i in range(12)]) for i in range(12): self.env['stock.quant']._update_available_quantity(self.productB, self.stock_location, 1.0, lot_id=sns[i]) dozen_move = self.env['stock.move'].create({ 'name': self.productB.name, 'product_id': self.productB.id, 'product_uom_qty': 1, 'product_uom': self.uom_dozen.id, 'picking_id': self.picking_client_1.id, 'location_id': self.stock_location.id, 'location_dest_id': self.customer_location.id, }) dozen_move._action_confirm() dozen_move._action_assign() self.assertEqual(len(dozen_move.move_line_ids), 12) self.assertEqual(dozen_move.move_line_ids.product_uom_id, self.env.ref('uom.product_uom_unit')) lines = dozen_move.move_line_ids[0:5] res_dict = lines.action_open_add_to_wave() res_dict['context'] = {'active_model': 'stock.move.line', 'active_ids': lines.ids} self.assertEqual(res_dict.get('res_model'), 'stock.add.to.wave') wizard_form = Form.from_action(self.env, res_dict) wizard_form.mode = 'new' wizard_form.save().attach_pickings() wave = self.env['stock.picking.batch'].search([ ('is_wave', '=', True) ]) self.assertFalse(lines.move_id == dozen_move) self.assertEqual(lines.batch_id, wave) self.assertEqual(dozen_move.product_uom_qty, 0.58) def test_wave_mutliple_move_lines(self): self.productA = self.env['product.product'].create({ 'name': 'Product Test A', 'is_storable': True, 'categ_id': self.env.ref('product.product_category_all').id, }) picking = self.env['stock.picking'].create({ 'location_id': self.stock_location.id, 'location_dest_id': self.customer_location.id, 'picking_type_id': self.picking_type_out, 'company_id': self.env.company.id, }) ml1 = self.env['stock.move.line'].create({ 'product_id': self.productA.id, 'quantity': 5, 'product_uom_id': self.productA.uom_id.id, 'picking_id': picking.id, 'location_id': self.stock_location.id, 'location_dest_id': self.customer_location.id, }) self.env['stock.move.line'].create({ 'product_id': self.productA.id, 'quantity': 5, 'product_uom_id': self.productA.uom_id.id, 'picking_id': picking.id, 'location_id': self.stock_location.id, 'location_dest_id': self.customer_location.id, }) ml2 = self.env['stock.move.line'].create({ 'product_id': self.productA.id, 'quantity': 5, 'product_uom_id': self.productA.uom_id.id, 'picking_id': picking.id, 'location_id': self.stock_location.id, 'location_dest_id': self.customer_location.id, }) ml1._add_to_wave() wave = self.env['stock.picking.batch'].search([ ('is_wave', '=', True) ]) self.assertFalse(picking.batch_id) self.assertEqual(ml1.picking_id.batch_id.id, wave.id) self.assertEqual(ml1.picking_id.move_ids.quantity, 5) self.assertEqual(ml1.picking_id.move_ids.product_uom_qty, 5) self.assertEqual(ml2.picking_id.id, picking.id) self.assertEqual(ml2.picking_id.move_ids.quantity, 10) self.assertEqual(ml2.picking_id.move_ids.product_uom_qty, 0) def test_wave_trigger_errors(self): with self.assertRaises(UserError): lines = self.picking_client_1.move_line_ids res_dict = lines.action_open_add_to_wave() wizard_form = Form.from_action(self.env, res_dict) wizard_form.mode = 'new' wizard = wizard_form.save() wizard.attach_pickings() with self.assertRaises(UserError): self.picking_client_1.company_id = self.env.company self.picking_client_2.company_id = self.env['res.company'].create({'name': 'Company 2'}) lines = (self.picking_client_1 | self.picking_client_2).move_line_ids res_dict = lines.action_open_add_to_wave() res_dict['context'] = {'active_model': 'stock.move.line', 'active_ids': lines.ids} wizard_form = Form.from_action(self.env, res_dict) wizard_form.mode = 'new' wizard = wizard_form.save() wizard.attach_pickings() def test_not_assign_to_wave(self): """ Picking - Move line A 5 from Container to Cust -> Going to a wave picking - Move line A 5 from Container to Cust -> Validate --------------------------------------------- Create - Move A 5 from Container to Cust Check it creates a new picking and it's not assign to the wave """ location = self.env['stock.location'].create({ 'name': 'Container', 'location_id': self.stock_location.id }) self.env['stock.quant']._update_available_quantity(self.productA, location, 5.0, lot_id=self.lots_p_a[0]) self.env['stock.quant']._update_available_quantity(self.productA, location, 5.0, lot_id=self.lots_p_a[1]) picking_1 = self.env['stock.picking'].create({ 'location_id': location.id, 'location_dest_id': self.customer_location.id, 'picking_type_id': self.picking_type_out, 'company_id': self.env.company.id, 'state': 'draft', }) self.env['stock.move'].create({ 'name': 'Test Wave', 'product_id': self.productA.id, 'product_uom_qty': 10, 'product_uom': self.productA.uom_id.id, 'picking_id': picking_1.id, 'picking_type_id': self.picking_type_out, 'location_id': location.id, 'location_dest_id': self.customer_location.id, }) picking_1.action_confirm() picking_1.action_assign() self.assertEqual(len(picking_1.move_line_ids), 2) move_line_to_wave = picking_1.move_line_ids[0] move_line_to_wave._add_to_wave() picking_1.move_line_ids.quantity = 5 picking_1.move_ids.picked = True picking_1._action_done() new_move = self.env['stock.move'].create({ 'name': 'Test Wave', 'product_id': self.productA.id, 'product_uom_qty': 5, 'product_uom': self.productA.uom_id.id, 'picking_type_id': self.picking_type_out, 'location_id': location.id, 'location_dest_id': self.customer_location.id, }) new_move._action_confirm() self.assertTrue(new_move.picking_id) self.assertTrue(new_move.picking_id.id not in [picking_1.id, move_line_to_wave.picking_id.id]) def test_operation_type_in_wave(self): """ Check that the operation type of the picking is set correclty in the wave. """ warehouse = self.env['stock.warehouse'].search([], limit=1) warehouse.reception_steps = 'three_steps' self.productA = self.env['product.product'].create({ 'name': 'Product Test A', 'is_storable': True, 'categ_id': self.env.ref('product.product_category_all').id, }) self.productB = self.env['product.product'].create({ 'name': 'Product Test B', 'is_storable': True, 'categ_id': self.env.ref('product.product_category_all').id, }) picking = self.env['stock.picking'].create({ 'location_id': self.customer_location.id, 'location_dest_id': self.stock_location.id, 'picking_type_id': self.picking_type_in, 'company_id': self.env.company.id, 'state': 'draft', }) self.env['stock.move'].create({ 'name': self.productA.name, 'product_id': self.productA.id, 'product_uom_qty': 1, 'product_uom': self.productA.uom_id.id, 'picking_id': picking.id, 'location_id': self.customer_location.id, 'location_dest_id': warehouse.wh_input_stock_loc_id.id, }) self.env['stock.move'].create({ 'name': self.productB.name, 'product_id': self.productB.id, 'product_uom_qty': 5, 'product_uom': self.productB.uom_id.id, 'picking_id': picking.id, 'location_id': self.customer_location.id, 'location_dest_id': warehouse.wh_input_stock_loc_id.id, }) picking.action_confirm() picking.move_ids.move_line_ids.write({'quantity': 1}) picking.move_ids.picked = True res_dict = picking.button_validate() self.env[res_dict['res_model']].with_context(res_dict['context']).process() move_line = self.env["stock.move.line"].search([('product_id', '=', self.productA.id), ('location_id', '=', warehouse.wh_input_stock_loc_id.id)]) move_line._add_to_wave() wave = self.env['stock.picking.batch'].search([ ('is_wave', '=', True) ]) self.assertEqual(wave.picking_type_id, move_line.picking_type_id) def test_validatation_of_partially_empty_picking(self): """ Check that you can validate a wave transfer containing an empty picking, that the picking stays unchanged (except for the 'picked' state of the move) and is removed from the transfer """ self.productA.tracking = 'none' self.productB.tracking = 'none' (picking_1, picking_2) = self.env['stock.picking'].create([ { 'picking_type_id': self.picking_type_in, }, { 'picking_type_id': self.picking_type_in, }, ]) self.env['stock.move'].create([ { 'name': self.productA.name, 'product_id': self.productA.id, 'product_uom_qty': 2, 'product_uom': self.productA.uom_id.id, 'picking_id': picking_1.id, 'location_id': picking_1.location_id.id, 'location_dest_id': picking_1.location_dest_id.id, }, { 'name': self.productB.name, 'product_id': self.productB.id, 'product_uom_qty': 3, 'product_uom': self.productB.uom_id.id, 'picking_id': picking_2.id, 'location_id': picking_2.location_id.id, 'location_dest_id': picking_2.location_dest_id.id, }, ]) (picking_1 | picking_2).action_confirm() wave = self.env['stock.picking.batch'].create({ 'name': 'Wave transfer', 'picking_ids': [Command.link(picking_1.id), Command.link(picking_2.id)], 'is_wave': True, }) wave.move_ids.filtered(lambda m: m.product_id == self.productB).quantity = 0.0 wave.move_ids.picked = True wave.action_done() self.assertEqual(wave.state, 'done') self.assertEqual(wave.picking_ids, picking_1) self.assertEqual([picking_1.state, picking_1.move_ids.quantity, picking_1.move_ids.picked], ['done', 2.0, True]) self.assertEqual([picking_2.state, picking_2.move_ids.quantity, picking_2.move_ids.picked], ['assigned', 0.0, False]) def test_add_partially_assigned_move_to_batch(self): """ Checks that a picking is linked to the wave transfer in case all of its moves are to be linked with the wave transfer. """ picking = self.picking_internal # update a move for the moved qty to be less than the initial demand picking.move_ids[0].quantity = 10.0 self.assertRecordValues(picking.move_ids, [{'product_uom_qty': 15.0, 'quantity': 10.0}, {'product_uom_qty': 5.0, 'quantity': 5.0}]) lines = picking.move_ids.move_line_ids res_dict = lines.action_open_add_to_wave() res_dict['context'] = {'active_model': 'stock.move.line', 'active_ids': lines.ids} wizard_form = Form.from_action(self.env, res_dict) wizard_form.mode = 'new' wizard_form.save().attach_pickings() # check that the picking was added to the wave transfer wave = picking.batch_id.filtered(lambda b: b.is_wave) self.assertTrue(wave) # check that the lines are still linked to the original picking self.assertEqual(lines.move_id.picking_id, picking) # check that no other picking was added to the wave transfer self.assertEqual(wave.move_ids, picking.move_ids) def test_dont_add_empty_move_to_batch(self): """ Checks that a picking is not linked to the wave transfer in case one of its move is not to be linked with the wave transfer. """ picking = self.picking_internal move_1, move_2 = picking.move_ids # update a move for the moved qty to 0 move_1.quantity = 0 self.assertRecordValues(picking.move_ids, [{'product_uom_qty': 15.0, 'quantity': 0.0}, {'product_uom_qty': 5.0, 'quantity': 5.0}]) lines = picking.move_ids.move_line_ids res_dict = lines.action_open_add_to_wave() res_dict['context'] = {'active_model': 'stock.move.line', 'active_ids': lines.ids} wizard_form = Form.from_action(self.env, res_dict) wizard_form.mode = 'new' wizard_form.save().attach_pickings() # check that a new picking was added to the wave transfer new_picking = move_2.picking_id self.assertIsNot(picking, new_picking) wave = new_picking.batch_id.filtered(lambda b: b.is_wave) self.assertTrue(wave) self.assertEqual(wave.move_ids, move_2) # check that the original picking was not added to a wave transfer self.assertFalse(picking.batch_id.filtered(lambda b: b.is_wave)) self.assertEqual(picking.move_ids, move_1) def test_validate_wave_with_zero_qty_picking(self): """ Check that we can validate a wave transfer containing a picking without quantity. In that case, the picking remains unchanged and is removed from the wave """ (self.productA | self.productB).tracking = 'none' self.env['stock.quant']._update_available_quantity(self.productA, self.stock_location, 10.0) self.env['stock.quant']._update_available_quantity(self.productB, self.stock_location, 10.0) picking_1, picking_2 = self.env['stock.picking'].create([ {'picking_type_id': self.picking_type_out}, {'picking_type_id': self.picking_type_out}, ]) self.env['stock.move'].create([ { 'name': self.productA.name, 'product_id': self.productA.id, 'product_uom_qty': 2, 'product_uom': self.productA.uom_id.id, 'picking_id': picking_1.id, 'location_id': picking_1.location_id.id, 'location_dest_id': picking_1.location_dest_id.id, }, { 'name': self.productB.name, 'product_id': self.productB.id, 'product_uom_qty': 3, 'product_uom': self.productB.uom_id.id, 'picking_id': picking_2.id, 'location_id': picking_2.location_id.id, 'location_dest_id': picking_2.location_dest_id.id, }, ]) (picking_1 | picking_2).action_confirm() wave = self.env['stock.picking.batch'].create({ 'name': 'Wave transfer', 'picking_ids': [Command.link(picking_1.id), Command.link(picking_2.id)], }) self.assertEqual((picking_1 | picking_2).mapped('state'), ['assigned', 'assigned']) picking_1.move_ids.quantity = 0.0 wave.action_done() self.assertEqual(wave.state, 'done') self.assertRecordValues(picking_1.move_ids, [{'state': 'confirmed', 'quantity': 0.0, 'picked': False}]) self.assertRecordValues(picking_2.move_ids, [{'state': 'done', 'quantity': 3.0, 'picked': True}])