From ece8d0041572fc6ffdbe4acb9f2a6b64b8f4a7b5 Mon Sep 17 00:00:00 2001
From: Nishka Gosalia <nishkagosalia@Nishkas-MacBook-Air.local>
Date: Fri, 6 Feb 2026 14:34:08 +0530
Subject: [PATCH] fix: test cases

---
 erpnext/manufacturing/doctype/bom/bom.py      |  2 +-
 .../doctype/job_card/test_job_card.py         | 36 ++++++++++++++-----
 2 files changed, 28 insertions(+), 10 deletions(-)

diff --git a/erpnext/manufacturing/doctype/bom/bom.py b/erpnext/manufacturing/doctype/bom/bom.py
index 943091487d2..64688c4621e 100644
--- a/erpnext/manufacturing/doctype/bom/bom.py
+++ b/erpnext/manufacturing/doctype/bom/bom.py
@@ -333,7 +333,7 @@ class BOM(WebsiteGenerator):
 			if row.bom_no:
 				continue
 
-			operation_idx_with_no_rm[row.idx] = row.operation
+			operation_idx_with_no_rm[row.idx] = row
 
 		for row in self.items:
 			if row.operation_row_id and row.operation_row_id in operation_idx_with_no_rm:
diff --git a/erpnext/manufacturing/doctype/job_card/test_job_card.py b/erpnext/manufacturing/doctype/job_card/test_job_card.py
index 028a05d5c9d..bcda8913238 100644
--- a/erpnext/manufacturing/doctype/job_card/test_job_card.py
+++ b/erpnext/manufacturing/doctype/job_card/test_job_card.py
@@ -90,21 +90,39 @@ class TestJobCard(ERPNextTestSuite):
 
 		cut_bom = create_semi_fg_bom(cut_fg.name, raw.name, inspection_required=1)
 		stitch_bom = create_semi_fg_bom(stitch_fg.name, cut_fg.name, inspection_required=0)
-		final_bom = frappe.new_doc("BOM")
-		final_bom.item = final.name
-		final_bom.quantity = 1
-		final_bom.with_operations = 1
-		final_bom.track_semi_finished_goods = 1
+		final_bom = frappe.new_doc(
+			"BOM", item=final.name, quantity=1, with_operations=1, track_semi_finished_goods=1
+		)
 		final_bom.append("items", {"item_code": raw.name, "qty": 1})
 		final_bom.append(
-			"operations", {"operation": cutting.name, "workstation": "_Test Workstation 1", "bom_no": cut_bom}
+			"operations",
+			{
+				"operation": cutting.name,
+				"workstation": "_Test Workstation 1",
+				"bom_no": cut_bom,
+				"skip_material_transfer": 1,
+			},
 		)
 		final_bom.append(
 			"operations",
-			{"operation": stitching.name, "workstation": "_Test Workstation 1", "bom_no": stitch_bom},
+			{
+				"operation": stitching.name,
+				"workstation": "_Test Workstation 1",
+				"bom_no": stitch_bom,
+				"skip_material_transfer": 1,
+			},
 		)
-		final_bom.append("operations", {"operation": ironing.name, "workstation": "_Test Workstation 1"})
-		final_bom.insert()
+		final_bom.append(
+			"operations",
+			{
+				"operation": ironing.name,
+				"workstation": "_Test Workstation 1",
+				"is_final_finished_good": 1,
+				"bom_no": final_bom.name,
+				"skip_material_transfer": 1,
+			},
+		)
+		final_bom.append("items", {"item_code": stitch_fg.name, "qty": 1, "operation_row_id": 3})
 		final_bom.submit()
 		work_order = make_work_order(final_bom.name, final.name, 1, variant_items=[], use_multi_level_bom=0)
 		work_order.wip_warehouse = "Work In Progress - WP"