Skip to content
This repository was archived by the owner on Feb 16, 2026. It is now read-only.

[ADD] extra test cases for picking validation #5

Closed
thomaspaulb wants to merge 1 commit into from
Closed

[ADD] extra test cases for picking validation #5

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
54 changes: 28 additions & 26 deletions purchase_sale_inter_company/tests/test_inter_company_purchase_sale.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -354,14 +354,16 @@ def test_sync_picking(self):
so_picking_id.move_lines.product_qty,
)

so_picking_id.state = "done"
# Validate sale order, create backorder
wizard_data = so_picking_id.with_user(self.user_company_b).button_validate()
wizard = (
self.env["stock.backorder.confirmation"]
.with_context(**wizard_data.get("context"))
.create({})
)
wizard.process()
wizard.with_user(self.user_company_b).process()
self.assertEqual(so_picking_id.state, "done")
self.assertNotEqual((sale.picking_ids - so_picking_id).state, "done")

# Quantities should have been synced
self.assertNotEqual(po_picking_id, so_picking_id)
Expand Down Expand Up @@ -455,22 +457,24 @@ def test_sync_picking_lot(self):
self.company_b.sync_picking = True

purchase = self._create_purchase_order(
self.partner_company_b, self.stockable_product_serial
self.partner_company_b,
self.stockable_product_serial + self.consumable_product,
)
sale = self._approve_po(purchase)

# validate the SO picking
po_picking_id = purchase.picking_ids
so_picking_id = sale.picking_ids

so_move = so_picking_id.move_lines
so_move.move_line_ids = [
so_moves = so_picking_id.move_lines
so_moves[1].quantity_done = 2
so_moves[0].move_line_ids = [
(
0,
0,
{
"location_id": so_move.location_id.id,
"location_dest_id": so_move.location_dest_id.id,
"location_id": so_moves[0].location_id.id,
"location_dest_id": so_moves[0].location_dest_id.id,
"product_id": self.stockable_product_serial.id,
"product_uom_id": self.stockable_product_serial.uom_id.id,
"qty_done": 1,
Expand All @@ -482,21 +486,8 @@ def test_sync_picking_lot(self):
0,
0,
{
"location_id": so_move.location_id.id,
"location_dest_id": so_move.location_dest_id.id,
"product_id": self.stockable_product_serial.id,
"product_uom_id": self.stockable_product_serial.uom_id.id,
"qty_done": 1,
"lot_id": self.serial_2.id,
"picking_id": so_picking_id.id,
},
),
(
0,
0,
{
"location_id": so_move.location_id.id,
"location_dest_id": so_move.location_dest_id.id,
"location_id": so_moves[0].location_id.id,
"location_dest_id": so_moves[0].location_dest_id.id,
"product_id": self.stockable_product_serial.id,
"product_uom_id": self.stockable_product_serial.uom_id.id,
"qty_done": 1,
Expand All @@ -505,9 +496,17 @@ def test_sync_picking_lot(self):
},
),
]
so_picking_id.button_validate()
wizard_data = so_picking_id.with_user(self.user_company_b).button_validate()
wizard = (
self.env["stock.backorder.confirmation"]
.with_context(**wizard_data.get("context"))
.create({})
)
wizard.with_user(self.user_company_b).process()
self.assertEqual(so_picking_id.state, "done")
self.assertNotEqual((sale.picking_ids - so_picking_id).state, "done")

so_lots = so_move.mapped("move_line_ids.lot_id")
so_lots = so_moves.mapped("move_line_ids.lot_id")
po_lots = po_picking_id.mapped("move_lines.move_line_ids.lot_id")
self.assertEqual(
len(so_lots),
Expand All @@ -518,15 +517,18 @@ def test_sync_picking_lot(self):
so_lots, po_lots, msg="The lots of the moves should be different objects"
)
self.assertEqual(
so_lots.mapped("name"),
po_lots.mapped("name"),
so_lots.sudo().mapped("name"),
po_lots.sudo().mapped("name"),
msg="The lots should have the same name in both moves",
)
self.assertIn(
serial_3_company_a,
po_lots,
msg="Serial 333 already existed, a new one shouldn't have been created",
)
# A backorder should have been made for both
self.assertTrue(len(sale.picking_ids) > 1)
self.assertEqual(len(purchase.picking_ids), len(sale.picking_ids))

def test_sync_picking_same_product_multiple_lines(self):
"""
Expand Down