Merge pull request #36983 from deepeshgarg007/employee_utils

chore: Update employee for tests
frappe · Sep 6, 2023 · c2f8f1d · c2f8f1d
2 parents 0f065fe + ae01d70
commit c2f8f1d
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/erpnext/setup/doctype/employee/test_employee.py b/erpnext/setup/doctype/employee/test_employee.py
@@ -66,5 +66,8 @@ def make_employee(user, company=None, **kwargs):
 		employee.insert()
 		return employee.name
 	else:
-		frappe.db.set_value("Employee", {"employee_name": user}, "status", "Active")
-		return frappe.get_value("Employee", {"employee_name": user}, "name")
+		employee = frappe.get_doc("Employee", {"employee_name": user})
+		employee.update(kwargs)
+		employee.status = "Active"
+		employee.save()
+		return employee.name