From e67165d6cecf82caabef09ed20b411c2cb5f81df Mon Sep 17 00:00:00 2001 From: ruthra kumar Date: Sun, 1 Mar 2026 21:33:45 +0530 Subject: [PATCH] refactor(test): hardcoded names over dynamic ones Much faster bootstrap without those get_doc calls --- .../test_process_deferred_accounting.py | 4 ++-- .../purchase_invoice/test_purchase_invoice.py | 4 ++-- erpnext/controllers/tests/test_mapper.py | 2 +- .../tests/test_subcontracting_controller.py | 2 +- erpnext/crm/doctype/lead/test_lead.py | 2 +- .../doctype/opportunity/test_opportunity.py | 2 +- erpnext/crm/doctype/prospect/test_prospect.py | 4 ++-- .../plaid_settings/test_plaid_settings.py | 2 +- erpnext/manufacturing/doctype/bom/test_bom.py | 10 +++++----- .../doctype/job_card/test_job_card.py | 4 ++-- .../doctype/work_order/test_work_order.py | 4 ++-- .../projects/doctype/project/test_project.py | 2 +- erpnext/projects/doctype/task/test_task.py | 2 +- .../doctype/timesheet/test_timesheet.py | 18 +++++++++--------- .../doctype/department/test_department.py | 2 +- .../setup/doctype/employee/test_employee.py | 12 ++++++------ .../delivery_note/test_delivery_note.py | 6 +++--- .../delivery_trip/test_delivery_trip.py | 4 ++-- .../test_landed_cost_voucher.py | 8 ++++---- .../purchase_receipt/test_purchase_receipt.py | 6 +++--- .../test_serial_and_batch_bundle.py | 6 +++--- .../stock/doctype/serial_no/test_serial_no.py | 4 ++-- .../doctype/stock_entry/test_stock_entry.py | 2 +- .../test_stock_reconciliation.py | 2 +- erpnext/stock/tests/test_utils.py | 2 +- erpnext/tests/utils.py | 9 +++++++++ 26 files changed, 67 insertions(+), 58 deletions(-) diff --git a/erpnext/accounts/doctype/process_deferred_accounting/test_process_deferred_accounting.py b/erpnext/accounts/doctype/process_deferred_accounting/test_process_deferred_accounting.py index 5751bcaabd5..95be179b66d 100644 --- a/erpnext/accounts/doctype/process_deferred_accounting/test_process_deferred_accounting.py +++ b/erpnext/accounts/doctype/process_deferred_accounting/test_process_deferred_accounting.py @@ -52,7 +52,7 @@ class TestProcessDeferredAccounting(ERPNextTestSuite): start_date="2023-05-01", end_date="2023-06-30", type="Income", - company=self.companies[0].name, + company="_Test Company", ) process_deferred_accounting.insert() @@ -83,7 +83,7 @@ class TestProcessDeferredAccounting(ERPNextTestSuite): start_date="2019-01-01", end_date="2019-01-31", type="Income", - company=self.companies[0].name, + company="_Test Company", ) pda.submit() pda.cancel() diff --git a/erpnext/accounts/doctype/purchase_invoice/test_purchase_invoice.py b/erpnext/accounts/doctype/purchase_invoice/test_purchase_invoice.py index 1c51daccd60..33117c639dc 100644 --- a/erpnext/accounts/doctype/purchase_invoice/test_purchase_invoice.py +++ b/erpnext/accounts/doctype/purchase_invoice/test_purchase_invoice.py @@ -2272,7 +2272,7 @@ class TestPurchaseInvoice(ERPNextTestSuite, StockTestMixin): def test_create_purchase_invoice_without_mandatory(self): pi = frappe.new_doc("Purchase Invoice") - pi.company = self.companies[0].name + pi.company = "_Test Company" pi.flags.ignore_mandatory = True pi.insert(ignore_permissions=True) @@ -2401,7 +2401,7 @@ class TestPurchaseInvoice(ERPNextTestSuite, StockTestMixin): "doctype": "Serial No", "item_code": serial_item, "serial_no": serial_no, - "company": self.companies[0].name, + "company": "_Test Company", } ).insert() diff --git a/erpnext/controllers/tests/test_mapper.py b/erpnext/controllers/tests/test_mapper.py index 3abb0ddd827..3a804d78b8f 100644 --- a/erpnext/controllers/tests/test_mapper.py +++ b/erpnext/controllers/tests/test_mapper.py @@ -39,7 +39,7 @@ class TestMapper(ERPNextTestSuite): "order_type": "Sales", "transaction_date": nowdate(), "valid_till": add_months(nowdate(), 1), - "company": self.companies[0].name, + "company": "_Test Company", } ) for item in item_list: diff --git a/erpnext/controllers/tests/test_subcontracting_controller.py b/erpnext/controllers/tests/test_subcontracting_controller.py index c78b1110ed6..0dbacb3c22d 100644 --- a/erpnext/controllers/tests/test_subcontracting_controller.py +++ b/erpnext/controllers/tests/test_subcontracting_controller.py @@ -742,7 +742,7 @@ class TestSubcontractingController(ERPNextTestSuite): "doctype": "Serial No", "item_code": "Subcontracted SRM Item 2", "serial_no": serial_no, - "company": self.companies[0].name, + "company": "_Test Company", } ).insert() diff --git a/erpnext/crm/doctype/lead/test_lead.py b/erpnext/crm/doctype/lead/test_lead.py index f4320faf79d..fdf35c705ad 100644 --- a/erpnext/crm/doctype/lead/test_lead.py +++ b/erpnext/crm/doctype/lead/test_lead.py @@ -120,7 +120,7 @@ class TestLead(ERPNextTestSuite): create_todo("followup", "Lead", lead.name) opportunity = make_opportunity(lead.name) - opportunity.company = self.companies[0].name + opportunity.company = "_Test Company" opportunity.save() self.assertEqual(opportunity.get("party_name"), lead.name) diff --git a/erpnext/crm/doctype/opportunity/test_opportunity.py b/erpnext/crm/doctype/opportunity/test_opportunity.py index da8530115a7..91c0f29d063 100644 --- a/erpnext/crm/doctype/opportunity/test_opportunity.py +++ b/erpnext/crm/doctype/opportunity/test_opportunity.py @@ -54,7 +54,7 @@ class TestOpportunity(ERPNextTestSuite): self.assertEqual(doc.status, "Quotation") def test_make_new_lead_if_required(self): - opp_doc = make_opportunity_from_lead(self.companies[0].name) + opp_doc = make_opportunity_from_lead("_Test Company") self.assertTrue(opp_doc.party_name) self.assertEqual(opp_doc.opportunity_from, "Lead") diff --git a/erpnext/crm/doctype/prospect/test_prospect.py b/erpnext/crm/doctype/prospect/test_prospect.py index 856176e9121..92158a30dbb 100644 --- a/erpnext/crm/doctype/prospect/test_prospect.py +++ b/erpnext/crm/doctype/prospect/test_prospect.py @@ -11,7 +11,7 @@ from erpnext.tests.utils import ERPNextTestSuite class TestProspect(ERPNextTestSuite): def test_add_lead_to_prospect_and_address_linking(self): - company = self.companies[0].name + company = "_Test Company" lead_doc = make_lead() address_doc = make_address(address_title=lead_doc.name) address_doc.append("links", {"link_doctype": lead_doc.doctype, "link_name": lead_doc.name}) @@ -37,7 +37,7 @@ class TestProspect(ERPNextTestSuite): "doctype": "Prospect", "company_name": "_Test Prospect", "customer_group": "_Test Customer Group", - "company": self.companies[0].name, + "company": "_Test Company", } ) prospect.insert() diff --git a/erpnext/erpnext_integrations/doctype/plaid_settings/test_plaid_settings.py b/erpnext/erpnext_integrations/doctype/plaid_settings/test_plaid_settings.py index 318a690cfc1..12105703772 100644 --- a/erpnext/erpnext_integrations/doctype/plaid_settings/test_plaid_settings.py +++ b/erpnext/erpnext_integrations/doctype/plaid_settings/test_plaid_settings.py @@ -55,7 +55,7 @@ class TestPlaidSettings(ERPNextTestSuite): } bank = json.dumps(frappe.get_doc("Bank", "Citi").as_dict(), default=json_handler) - company = self.companies[0].name + company = "_Test Company" add_bank_accounts(bank_accounts, bank, company) diff --git a/erpnext/manufacturing/doctype/bom/test_bom.py b/erpnext/manufacturing/doctype/bom/test_bom.py index 7a5bdba4f4e..6df3a0036fe 100644 --- a/erpnext/manufacturing/doctype/bom/test_bom.py +++ b/erpnext/manufacturing/doctype/bom/test_bom.py @@ -392,7 +392,7 @@ class TestBOM(ERPNextTestSuite): item_code = make_item(properties={"is_stock_item": 1}).name bom = frappe.new_doc("BOM") - bom.company = self.companies[0].name + bom.company = "_Test Company" bom.item = item_code bom.append("items", frappe._dict(item_code=item_code)) bom.save() @@ -406,13 +406,13 @@ class TestBOM(ERPNextTestSuite): item2 = make_item(properties={"is_stock_item": 1}).name bom1 = frappe.new_doc("BOM") - bom1.company = self.companies[0].name + bom1.company = "_Test Company" bom1.item = item1 bom1.append("items", frappe._dict(item_code=item2)) bom1.save() bom2 = frappe.new_doc("BOM") - bom2.company = self.companies[0].name + bom2.company = "_Test Company" bom2.item = item2 bom2.append("items", frappe._dict(item_code=item1)) bom2.save() @@ -570,7 +570,7 @@ class TestBOM(ERPNextTestSuite): @timeout def test_clear_inpection_quality(self): bom = frappe.copy_doc(self.globalTestRecords["BOM"][2], ignore_no_copy=True) - bom.company = self.companies[0].name + bom.company = "_Test Company" bom.docstatus = 0 bom.is_default = 0 bom.quality_inspection_template = "_Test Quality Inspection Template" @@ -616,7 +616,7 @@ class TestBOM(ERPNextTestSuite): # Step 1: Create BOM bom = frappe.new_doc("BOM") - bom.company = self.companies[0].name + bom.company = "_Test Company" bom.item = fg_item.item_code bom.quantity = 1 bom.append( diff --git a/erpnext/manufacturing/doctype/job_card/test_job_card.py b/erpnext/manufacturing/doctype/job_card/test_job_card.py index 0d36f106999..d0f9bfc920d 100644 --- a/erpnext/manufacturing/doctype/job_card/test_job_card.py +++ b/erpnext/manufacturing/doctype/job_card/test_job_card.py @@ -87,7 +87,7 @@ class TestJobCard(ERPNextTestSuite): quantity=1, with_operations=1, track_semi_finished_goods=1, - company=self.companies[9].name, + company="_Test Company", ) final_bom.append("items", {"item_code": raw.name, "qty": 1}) final_bom.append( @@ -122,7 +122,7 @@ class TestJobCard(ERPNextTestSuite): final_bom.insert() final_bom.submit() work_order = make_work_order(final_bom.name, final.name, 1, variant_items=[], use_multi_level_bom=0) - work_order.company = self.companies[9].name + work_order.company = "_Test Company" work_order.wip_warehouse = "Work In Progress - WP" work_order.fg_warehouse = "Finished Goods - WP" work_order.scrap_warehouse = "All Warehouses - WP" diff --git a/erpnext/manufacturing/doctype/work_order/test_work_order.py b/erpnext/manufacturing/doctype/work_order/test_work_order.py index 88ea156665c..0a92064c973 100644 --- a/erpnext/manufacturing/doctype/work_order/test_work_order.py +++ b/erpnext/manufacturing/doctype/work_order/test_work_order.py @@ -960,7 +960,7 @@ class TestWorkOrder(ERPNextTestSuite): bom_item, fg_qty=1, process_loss_percentage=10, - company=self.companies[0].name, + company="_Test Company", ) bom_doc.submit() @@ -1147,7 +1147,7 @@ class TestWorkOrder(ERPNextTestSuite): "is_active": 1, "is_default": 1, "quantity": 1.0, - "company": self.companies[0].name, + "company": "_Test Company", "with_operations": 1, "operations": [ { diff --git a/erpnext/projects/doctype/project/test_project.py b/erpnext/projects/doctype/project/test_project.py index a29860e0b6b..aa37c34ef89 100644 --- a/erpnext/projects/doctype/project/test_project.py +++ b/erpnext/projects/doctype/project/test_project.py @@ -17,7 +17,7 @@ class TestProject(ERPNextTestSuite): from erpnext.setup.doctype.employee.test_employee import make_employee project_name = "Test Project Costing" - employee = make_employee("employee@frappe.io", company=self.companies[0].name) + employee = make_employee("employee@frappe.io", company="_Test Company") project = make_project({"project_name": project_name}) timesheet = make_timesheet( employee=employee, diff --git a/erpnext/projects/doctype/task/test_task.py b/erpnext/projects/doctype/task/test_task.py index ccd1f789ca2..0d02fa9c9fb 100644 --- a/erpnext/projects/doctype/task/test_task.py +++ b/erpnext/projects/doctype/task/test_task.py @@ -15,7 +15,7 @@ class TestTask(ERPNextTestSuite): from erpnext.setup.doctype.employee.test_employee import make_employee project_name = "Test Project Costing" - employee = make_employee("employee@frappe.io", company=self.companies[0].name) + employee = make_employee("employee@frappe.io", company="_Test Company") project = make_project({"project_name": project_name}) task = create_task("_Test Task 1") task.project = project.name diff --git a/erpnext/projects/doctype/timesheet/test_timesheet.py b/erpnext/projects/doctype/timesheet/test_timesheet.py index 01da979b31d..cf3d9a17b2d 100644 --- a/erpnext/projects/doctype/timesheet/test_timesheet.py +++ b/erpnext/projects/doctype/timesheet/test_timesheet.py @@ -64,7 +64,7 @@ class TestTimesheet(ERPNextTestSuite): ) def test_timesheet_base_amount(self): - emp = make_employee("test_employee_6@salary.com", company=self.companies[0].name) + emp = make_employee("test_employee_6@salary.com", company="_Test Company") timesheet = make_timesheet(emp, simulate=True, is_billable=1) self.assertEqual(timesheet.time_logs[0].base_billing_rate, 50) @@ -73,7 +73,7 @@ class TestTimesheet(ERPNextTestSuite): self.assertEqual(timesheet.time_logs[0].base_costing_amount, 40) def test_timesheet_billing_amount(self): - emp = make_employee("test_employee_6@salary.com", company=self.companies[0].name) + emp = make_employee("test_employee_6@salary.com", company="_Test Company") timesheet = make_timesheet(emp, simulate=True, is_billable=1) self.assertEqual(timesheet.total_hours, 2) @@ -83,7 +83,7 @@ class TestTimesheet(ERPNextTestSuite): self.assertEqual(timesheet.total_billable_amount, 100) def test_timesheet_billing_amount_not_billable(self): - emp = make_employee("test_employee_6@salary.com", company=self.companies[0].name) + emp = make_employee("test_employee_6@salary.com", company="_Test Company") timesheet = make_timesheet(emp, simulate=True, is_billable=0) self.assertEqual(timesheet.total_hours, 2) @@ -93,7 +93,7 @@ class TestTimesheet(ERPNextTestSuite): self.assertEqual(timesheet.total_billable_amount, 0) def test_sales_invoice_from_timesheet(self): - emp = make_employee("test_employee_6@salary.com", company=self.companies[0].name) + emp = make_employee("test_employee_6@salary.com", company="_Test Company") timesheet = make_timesheet(emp, simulate=True, is_billable=1) sales_invoice = make_sales_invoice(timesheet.name, "_Test Item", "_Test Customer", currency="INR") @@ -111,7 +111,7 @@ class TestTimesheet(ERPNextTestSuite): @ERPNextTestSuite.change_settings("Projects Settings", {"fetch_timesheet_in_sales_invoice": 1}) def test_timesheet_billing_based_on_project(self): - emp = make_employee("test_employee_6@salary.com", company=self.companies[0].name) + emp = make_employee("test_employee_6@salary.com", company="_Test Company") project = frappe.get_value("Project", {"project_name": "_Test Project"}) timesheet = make_timesheet( @@ -127,7 +127,7 @@ class TestTimesheet(ERPNextTestSuite): self.assertEqual(ts.time_logs[0].sales_invoice, sales_invoice.name) def test_timesheet_time_overlap(self): - emp = make_employee("test_employee_6@salary.com", company=self.companies[0].name) + emp = make_employee("test_employee_6@salary.com", company="_Test Company") settings = frappe.get_single("Projects Settings") initial_setting = settings.ignore_employee_time_overlap @@ -183,7 +183,7 @@ class TestTimesheet(ERPNextTestSuite): settings.save() def test_timesheet_not_overlapping_with_continuous_timelogs(self): - emp = make_employee("test_employee_6@salary.com", company=self.companies[0].name) + emp = make_employee("test_employee_6@salary.com", company="_Test Company") update_activity_type("_Test Activity Type") timesheet = frappe.new_doc("Timesheet") @@ -212,7 +212,7 @@ class TestTimesheet(ERPNextTestSuite): timesheet.save() # should not throw an error def test_to_time(self): - emp = make_employee("test_employee_6@salary.com", company=self.companies[0].name) + emp = make_employee("test_employee_6@salary.com", company="_Test Company") from_time = now_datetime() timesheet = frappe.new_doc("Timesheet") @@ -278,7 +278,7 @@ class TestTimesheet(ERPNextTestSuite): This test ensures Timesheet status is recalculated correctly across billing and return lifecycle events. """ - emp = make_employee("test_employee_6@salary.com", company=self.companies[0].name) + emp = make_employee("test_employee_6@salary.com", company="_Test Company") timesheet = make_timesheet(emp, simulate=True, is_billable=1, do_not_submit=True) timesheet_detail = timesheet.append("time_logs", {}) diff --git a/erpnext/setup/doctype/department/test_department.py b/erpnext/setup/doctype/department/test_department.py index 01ba6b13913..6ad31d2443b 100644 --- a/erpnext/setup/doctype/department/test_department.py +++ b/erpnext/setup/doctype/department/test_department.py @@ -8,7 +8,7 @@ from erpnext.tests.utils import ERPNextTestSuite class TestDepartment(ERPNextTestSuite): def test_remove_department_data(self): - doc = create_department("Test Department", company=self.companies[0].name) + doc = create_department("Test Department", company="_Test Company") frappe.delete_doc("Department", doc.name) diff --git a/erpnext/setup/doctype/employee/test_employee.py b/erpnext/setup/doctype/employee/test_employee.py index ec5ef46d416..57d08d6c3ae 100644 --- a/erpnext/setup/doctype/employee/test_employee.py +++ b/erpnext/setup/doctype/employee/test_employee.py @@ -13,8 +13,8 @@ from erpnext.tests.utils import ERPNextTestSuite class TestEmployee(ERPNextTestSuite): def test_employee_status_left(self): - employee1 = make_employee("test_employee_1@company.com", company=self.companies[0].name) - employee2 = make_employee("test_employee_2@company.com", company=self.companies[0].name) + employee1 = make_employee("test_employee_1@company.com", company="_Test Company") + employee2 = make_employee("test_employee_2@company.com", company="_Test Company") employee1_doc = frappe.get_doc("Employee", employee1) employee2_doc = frappe.get_doc("Employee", employee2) employee2_doc.reload() @@ -25,7 +25,7 @@ class TestEmployee(ERPNextTestSuite): self.assertRaises(InactiveEmployeeStatusError, employee1_doc.save) def test_user_has_employee(self): - employee = make_employee("test_emp_user_creation@company.com", company=self.companies[0].name) + employee = make_employee("test_emp_user_creation@company.com", company="_Test Company") employee_doc = frappe.get_doc("Employee", employee) user = employee_doc.user_id self.assertTrue("Employee" in frappe.get_roles(user)) @@ -35,12 +35,12 @@ class TestEmployee(ERPNextTestSuite): def test_employee_user_permission(self): employee1 = make_employee( - "employee_1_test@company.com", create_user_permission=1, company=self.companies[0].name + "employee_1_test@company.com", create_user_permission=1, company="_Test Company" ) employee2 = make_employee( - "employee_2_test@company.com", create_user_permission=1, company=self.companies[0].name + "employee_2_test@company.com", create_user_permission=1, company="_Test Company" ) - make_employee("employee_3_test@company.com", create_user_permission=1, company=self.companies[0].name) + make_employee("employee_3_test@company.com", create_user_permission=1, company="_Test Company") employee1_doc = frappe.get_doc("Employee", employee1) employee2_doc = frappe.get_doc("Employee", employee2) diff --git a/erpnext/stock/doctype/delivery_note/test_delivery_note.py b/erpnext/stock/doctype/delivery_note/test_delivery_note.py index 480d54ac505..5d9c55cd671 100644 --- a/erpnext/stock/doctype/delivery_note/test_delivery_note.py +++ b/erpnext/stock/doctype/delivery_note/test_delivery_note.py @@ -161,7 +161,7 @@ class TestDeliveryNote(ERPNextTestSuite): "doctype": "Serial No", "item_code": "_Test Serialized Item With Series", "serial_no": make_autoname("SRDD", "Serial No"), - "company": self.companies[0].name, + "company": "_Test Company", } ) serial_no.save() @@ -226,7 +226,7 @@ class TestDeliveryNote(ERPNextTestSuite): "doctype": "Serial No", "item_code": sn_item, "serial_no": sn, - "company": self.companies[0].name, + "company": "_Test Company", } ) sn_doc.insert() @@ -1836,7 +1836,7 @@ class TestDeliveryNote(ERPNextTestSuite): "doctype": "Serial No", "item_code": sn_item, "serial_no": sn, - "company": self.companies[0].name, + "company": "_Test Company", } ) sn_doc.insert() diff --git a/erpnext/stock/doctype/delivery_trip/test_delivery_trip.py b/erpnext/stock/doctype/delivery_trip/test_delivery_trip.py index 98c0adfc532..783a28a47d8 100644 --- a/erpnext/stock/doctype/delivery_trip/test_delivery_trip.py +++ b/erpnext/stock/doctype/delivery_trip/test_delivery_trip.py @@ -22,14 +22,14 @@ class TestDeliveryTrip(ERPNextTestSuite): create_test_contact_and_address() address = create_address(driver) - self.delivery_trip = create_delivery_trip(driver, address, company=self.companies[0].name) + self.delivery_trip = create_delivery_trip(driver, address, company="_Test Company") def test_delivery_trip_notify_customers(self): # set default outgoing outgoing = frappe.get_doc( { "doctype": "Email Account", - "company": self.companies[0].name, + "company": "_Test Company", "enable_outgoing": 1, "default_outgoing": 1, "awaiting_password": 1, diff --git a/erpnext/stock/doctype/landed_cost_voucher/test_landed_cost_voucher.py b/erpnext/stock/doctype/landed_cost_voucher/test_landed_cost_voucher.py index 8bb1d327891..bfbf9bc4de2 100644 --- a/erpnext/stock/doctype/landed_cost_voucher/test_landed_cost_voucher.py +++ b/erpnext/stock/doctype/landed_cost_voucher/test_landed_cost_voucher.py @@ -401,7 +401,7 @@ class TestLandedCostVoucher(ERPNextTestSuite): "doctype": "Serial No", "item_code": item_code, "serial_no": serial_no, - "company": self.companies[0].name, + "company": "_Test Company", } ).insert() @@ -670,7 +670,7 @@ class TestLandedCostVoucher(ERPNextTestSuite): "doctype": "Serial No", "item_code": sn_item, "serial_no": sn, - "company": self.companies[0].name, + "company": "_Test Company", } ) sn_doc.insert() @@ -821,7 +821,7 @@ class TestLandedCostVoucher(ERPNextTestSuite): "doctype": "Serial No", "item_code": sn_item, "serial_no": sn, - "company": self.companies[0].name, + "company": "_Test Company", } ) sn_doc.insert() @@ -1011,7 +1011,7 @@ class TestLandedCostVoucher(ERPNextTestSuite): "doctype": "Serial No", "item_code": sn_item, "serial_no": sn, - "company": self.companies[0].name, + "company": "_Test Company", } ) sn_doc.insert() diff --git a/erpnext/stock/doctype/purchase_receipt/test_purchase_receipt.py b/erpnext/stock/doctype/purchase_receipt/test_purchase_receipt.py index 20ea5459e1f..d7f7abfbdf2 100644 --- a/erpnext/stock/doctype/purchase_receipt/test_purchase_receipt.py +++ b/erpnext/stock/doctype/purchase_receipt/test_purchase_receipt.py @@ -739,7 +739,7 @@ class TestPurchaseReceipt(ERPNextTestSuite): "doctype": "Serial No", "item_code": item_code, "serial_no": serial_no[0], - "company": self.companies[0].name, + "company": "_Test Company", } ).insert() @@ -2698,7 +2698,7 @@ class TestPurchaseReceipt(ERPNextTestSuite): "doctype": "Serial No", "item_code": serial_item, "serial_no": serial_no, - "company": self.companies[0].name, + "company": "_Test Company", } ).insert() @@ -3990,7 +3990,7 @@ class TestPurchaseReceipt(ERPNextTestSuite): "doctype": "Serial No", "item_code": serial_item, "serial_no": serial_no, - "company": self.companies[0].name, + "company": "_Test Company", } ).insert() diff --git a/erpnext/stock/doctype/serial_and_batch_bundle/test_serial_and_batch_bundle.py b/erpnext/stock/doctype/serial_and_batch_bundle/test_serial_and_batch_bundle.py index 79bcb63e531..c6929fe4cdb 100644 --- a/erpnext/stock/doctype/serial_and_batch_bundle/test_serial_and_batch_bundle.py +++ b/erpnext/stock/doctype/serial_and_batch_bundle/test_serial_and_batch_bundle.py @@ -40,7 +40,7 @@ class TestSerialandBatchBundle(ERPNextTestSuite): "doctype": "Serial No", "serial_no": sn, "item_code": serial_item_code, - "company": self.companies[0].name, + "company": "_Test Company", } ).insert(ignore_permissions=True) @@ -707,7 +707,7 @@ class TestSerialandBatchBundle(ERPNextTestSuite): "doctype": "Serial No", "serial_no": serial_no, "item_code": item, - "company": self.companies[0].name, + "company": "_Test Company", } ).insert(ignore_permissions=True) @@ -805,7 +805,7 @@ class TestSerialandBatchBundle(ERPNextTestSuite): "doctype": "Serial No", "serial_no": serial_no, "item_code": sn_item, - "company": self.companies[0].name, + "company": "_Test Company", } ).insert(ignore_permissions=True) serial_nos.append(serial_no) diff --git a/erpnext/stock/doctype/serial_no/test_serial_no.py b/erpnext/stock/doctype/serial_no/test_serial_no.py index a82cbe5d306..3ad635eb65b 100644 --- a/erpnext/stock/doctype/serial_no/test_serial_no.py +++ b/erpnext/stock/doctype/serial_no/test_serial_no.py @@ -34,7 +34,7 @@ class TestSerialNo(ERPNextTestSuite): sr.warehouse = "_Test Warehouse - _TC" sr.serial_no = "_TCSER0001" sr.purchase_rate = 10 - sr.company = self.companies[0].name + sr.company = "_Test Company" self.assertRaises(SerialNoCannotCreateDirectError, sr.insert) sr.warehouse = None @@ -198,7 +198,7 @@ class TestSerialNo(ERPNextTestSuite): "doctype": "Serial No", "item_code": item_code, "serial_no": serial_no, - "company": self.companies[0].name, + "company": "_Test Company", } ).insert() diff --git a/erpnext/stock/doctype/stock_entry/test_stock_entry.py b/erpnext/stock/doctype/stock_entry/test_stock_entry.py index e666b5743c6..9a3ef3f2655 100644 --- a/erpnext/stock/doctype/stock_entry/test_stock_entry.py +++ b/erpnext/stock/doctype/stock_entry/test_stock_entry.py @@ -648,7 +648,7 @@ class TestStockEntry(ERPNextTestSuite): doc = frappe.new_doc("Serial No") doc.serial_no = serial_no doc.item_code = "_Test Serialized Item" - doc.company = self.companies[0].name + doc.company = "_Test Company" doc.insert(ignore_permissions=True) se = frappe.copy_doc(self.globalTestRecords["Stock Entry"][0]) diff --git a/erpnext/stock/doctype/stock_reconciliation/test_stock_reconciliation.py b/erpnext/stock/doctype/stock_reconciliation/test_stock_reconciliation.py index 99231878f06..bc823e95fcb 100644 --- a/erpnext/stock/doctype/stock_reconciliation/test_stock_reconciliation.py +++ b/erpnext/stock/doctype/stock_reconciliation/test_stock_reconciliation.py @@ -612,7 +612,7 @@ class TestStockReconciliation(ERPNextTestSuite, StockTestMixin): "doctype": "Serial No", "item_code": item_code, "serial_no": "SR-CREATED-SR-NO", - "company": self.companies[0].name, + "company": "_Test Company", } ).insert() diff --git a/erpnext/stock/tests/test_utils.py b/erpnext/stock/tests/test_utils.py index cdbc2b4b41b..dfbe6d24806 100644 --- a/erpnext/stock/tests/test_utils.py +++ b/erpnext/stock/tests/test_utils.py @@ -89,7 +89,7 @@ class TestStockUtilities(ERPNextTestSuite, StockTestMixin): doctype="Serial No", item_code=serial_item.name, serial_no=frappe.generate_hash(), - company=self.companies[0].name, + company="_Test Company", ).insert() serial_scan = scan_barcode(serial.name) diff --git a/erpnext/tests/utils.py b/erpnext/tests/utils.py index b03171c662a..9bed50d4afb 100644 --- a/erpnext/tests/utils.py +++ b/erpnext/tests/utils.py @@ -2992,9 +2992,18 @@ class ERPNextTestSuite(unittest.TestCase): return decorator + @classmethod + def setUpClass(cls): + cls.globalTestRecords = {} + def tearDown(self): frappe.db.rollback() + def load_test_records(self, doctype): + if doctype not in self.globalTestRecords: + records = load_test_records_for(doctype) + self.globalTestRecords[doctype] = records[doctype] + @contextmanager def set_user(self, user: str): try: