From b09889643fa04edbfc79012cb0a8c7e74afaef27 Mon Sep 17 00:00:00 2001 From: Nabin Hait Date: Thu, 2 Jul 2026 00:04:13 +0530 Subject: [PATCH] test: don't override tearDown; rely on ERPNextTestSuite rollback --- .../bom_variance_report/test_bom_variance_report.py | 12 ++---------- 1 file changed, 2 insertions(+), 10 deletions(-) diff --git a/erpnext/manufacturing/report/bom_variance_report/test_bom_variance_report.py b/erpnext/manufacturing/report/bom_variance_report/test_bom_variance_report.py index e70bad995b4..718dba7d29c 100644 --- a/erpnext/manufacturing/report/bom_variance_report/test_bom_variance_report.py +++ b/erpnext/manufacturing/report/bom_variance_report/test_bom_variance_report.py @@ -19,18 +19,10 @@ class TestBOMVarianceReport(ERPNextTestSuite): ) self.raw_materials = self.get_bom_raw_materials() - self.overproduction_percentage = frappe.db.get_single_value( - "Manufacturing Settings", "overproduction_percentage_for_work_order" - ) + # allow over-production so a Work Order can produce more than planned; ERPNextTestSuite + # rolls this back at tearDown, so no manual restore is needed frappe.db.set_single_value("Manufacturing Settings", "overproduction_percentage_for_work_order", 100) - def tearDown(self): - frappe.db.set_single_value( - "Manufacturing Settings", - "overproduction_percentage_for_work_order", - self.overproduction_percentage, - ) - def get_bom_raw_materials(self): return { row.item_code: row.qty