nsinnovations/erpnext
2
0
Fork 0
mirror of https://github.com/frappe/erpnext.git synced 2026-08-26 13:25:20 +00:00
Code Issues Packages Projects Releases Wiki Activity

test: add positive anchor and robust row pairing in Work Order Consumed Materials

Browse Source
This commit is contained in:
Nabin Hait
2026-07-02 14:02:45 +05:30
parent 47ee1d126d
commit f1e91b6be6
1 changed files with 9 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
erpnext/manufacturing/report/work_order_consumed_materials/test_work_order_consumed_materials.py
View File

@@ -62,9 +62,12 @@ class TestWorkOrderConsumedMaterials(ERPNextTestSuite):
self.assertEqual(len(rows), len(wo.required_items)) self.assertEqual(len(rows), len(wo.required_items))
reported = {row["raw_material_item_code"]: row for row in rows} # pair rows to required items by sorting rather than a dict keyed on item code, so
for item in wo.required_items: # a BOM with two lines for the same component wouldn't silently collapse to one row
row = reported[item.item_code] rows_sorted = sorted(rows, key=lambda r: (r["raw_material_item_code"], r["required_qty"]))
items_sorted = sorted(wo.required_items, key=lambda i: (i.item_code, i.required_qty))
for row, item in zip(rows_sorted, items_sorted, strict=True):
self.assertEqual(row["raw_material_item_code"], item.item_code)
self.assertEqual(row["required_qty"], item.required_qty) self.assertEqual(row["required_qty"], item.required_qty)
self.assertEqual(row["transferred_qty"], item.required_qty) self.assertEqual(row["transferred_qty"], item.required_qty)
self.assertEqual(row["consumed_qty"], item.required_qty) self.assertEqual(row["consumed_qty"], item.required_qty)
@@ -100,6 +103,9 @@ class TestWorkOrderConsumedMaterials(ERPNextTestSuite):
def test_date_range_filter_excludes_work_order(self): def test_date_range_filter_excludes_work_order(self):
wo = self.make_manufactured_work_order(qty=1) wo = self.make_manufactured_work_order(qty=1)
# positive anchor: the WO shows up within the default (current) window
self.assertIn(wo.name, {row.get("parent") for row in self.run_report()})
# a window that ends before the WO was created must not include it # a window that ends before the WO was created must not include it
data = self.run_report(from_date=add_days(nowdate(), -10), to_date=add_days(nowdate(), -5)) data = self.run_report(from_date=add_days(nowdate(), -10), to_date=add_days(nowdate(), -5))
self.assertNotIn(wo.name, {row.get("parent") for row in data}) self.assertNotIn(wo.name, {row.get("parent") for row in data})