refactor(test): hardcoded names over dynamic ones

ruthra-kumar · ruthra-kumar · commit d65aba39483a · 2026-03-02T07:08:44.000+05:30
Much faster bootstrap without those get_doc calls
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
@@ -52,7 +52,7 @@ def test_creation_of_ledger_entry_on_submit(self):
 			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 @@ def test_pda_submission_and_cancellation(self):
 			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
@@ -2272,7 +2272,7 @@ def test_repost_accounting_entries(self):
 
 	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 @@ def test_purchase_invoice_with_use_serial_batch_field_for_rejected_qty(self):
 						"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
@@ -39,7 +39,7 @@ def make_quotation(self, item_list, customer):
 				"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
@@ -742,7 +742,7 @@ def test_incorrect_serial_no_components_based_on_material_transfer(self):
 					"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
@@ -120,7 +120,7 @@ def test_opportunity_from_lead(self):
 		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
@@ -54,7 +54,7 @@ def test_opportunity_status(self):
 		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
@@ -11,7 +11,7 @@
 
 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 @@ def test_make_customer_from_prospect(self):
 				"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
@@ -55,7 +55,7 @@ def test_new_transaction(self):
 		}
 
 		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
@@ -392,7 +392,7 @@ def test_bom_recursion_1st_level(self):
 		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 @@ def test_bom_recursion_transitive(self):
 		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 @@ def test_bom_versioning(self):
 	@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 @@ def test_set_default_bom_for_item_having_single_bom(self):
 
 		# 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
@@ -87,7 +87,7 @@ def test_quality_inspection_mandatory_check(self):
 			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 @@ def test_quality_inspection_mandatory_check(self):
 		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
@@ -960,7 +960,7 @@ def test_wo_completion_with_pl_bom(self):
 				bom_item,
 				fg_qty=1,
 				process_loss_percentage=10,
-				company=self.companies[0].name,
+				company="_Test Company",
 			)
 			bom_doc.submit()
 
@@ -1147,7 +1147,7 @@ def test_fix_time_operations(self):
 				"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
@@ -17,7 +17,7 @@ def test_project_total_costing_and_billing_amount(self):
 		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
@@ -15,7 +15,7 @@ def test_task_total_costing_and_billing_amount(self):
 		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
@@ -64,7 +64,7 @@ def test_timesheet_post_update(self):
 		)
 
 	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 @@ def test_timesheet_base_amount(self):
 		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 @@ def test_timesheet_billing_amount(self):
 		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 @@ def test_timesheet_billing_amount_not_billable(self):
 		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 @@ def test_sales_invoice_from_timesheet(self):
 
 	@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 @@ def test_timesheet_billing_based_on_project(self):
 		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 @@ def test_timesheet_time_overlap(self):
 		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 @@ def test_timesheet_not_overlapping_with_continuous_timelogs(self):
 		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 @@ def test_partial_billing_and_return(self):
 		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
@@ -8,7 +8,7 @@
 
 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
@@ -13,8 +13,8 @@
 
 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 @@ def test_employee_status_left(self):
 		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 @@ def test_user_has_employee(self):
 
 	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
@@ -161,7 +161,7 @@ def test_serialize_status(self):
 				"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 @@ def test_delivery_note_return_against_denormalized_serial_no(self):
 						"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 @@ def test_delivery_note_legacy_serial_no_valuation(self):
 						"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
@@ -22,14 +22,14 @@ def setUp(self):
 		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
diff --git a/erpnext/stock/doctype/purchase_receipt/test_purchase_receipt.py b/erpnext/stock/doctype/purchase_receipt/test_purchase_receipt.py
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
diff --git a/erpnext/stock/doctype/serial_no/test_serial_no.py b/erpnext/stock/doctype/serial_no/test_serial_no.py
diff --git a/erpnext/stock/doctype/stock_entry/test_stock_entry.py b/erpnext/stock/doctype/stock_entry/test_stock_entry.py
diff --git a/erpnext/stock/doctype/stock_reconciliation/test_stock_reconciliation.py b/erpnext/stock/doctype/stock_reconciliation/test_stock_reconciliation.py
diff --git a/erpnext/stock/tests/test_utils.py b/erpnext/stock/tests/test_utils.py
diff --git a/erpnext/tests/utils.py b/erpnext/tests/utils.py

Original file line number	Diff line number	Diff line change
`@@ -39,7 +39,7 @@ def make_quotation(self, item_list, customer):`
`39`	`39`	`"order_type": "Sales",`
`40`	`40`	`"transaction_date": nowdate(),`
`41`	`41`	`"valid_till": add_months(nowdate(), 1),`
`42`		`- "company": self.companies[0].name,`
	`42`	`+ "company": "_Test Company",`
`43`	`43`	`}`
`44`	`44`	`)`
`45`	`45`	`for item in item_list:`
Original file line number	Diff line number	Diff line change
`@@ -742,7 +742,7 @@ def test_incorrect_serial_no_components_based_on_material_transfer(self):`
`742`	`742`	`"doctype": "Serial No",`
`743`	`743`	`"item_code": "Subcontracted SRM Item 2",`
`744`	`744`	`"serial_no": serial_no,`
`745`		`- "company": self.companies[0].name,`
	`745`	`+ "company": "_Test Company",`
`746`	`746`	`}`
`747`	`747`	`).insert()`
`748`	`748`
Original file line number	Diff line number	Diff line change
`@@ -55,7 +55,7 @@ def test_new_transaction(self):`
`55`	`55`	`}`
`56`	`56`
`57`	`57`	`bank = json.dumps(frappe.get_doc("Bank", "Citi").as_dict(), default=json_handler)`
`58`		`- company = self.companies[0].name`
	`58`	`+ company = "_Test Company"`
`59`	`59`
`60`	`60`	`add_bank_accounts(bank_accounts, bank, company)`
`61`	`61`