Skip to content

Commit

Permalink
Merge pull request #36986 from frappe/mergify/bp/version-14-hotfix/pr…
Browse files Browse the repository at this point in the history
…-36983

chore: Update employee for tests (backport #36983)
  • Loading branch information
deepeshgarg007 authored Sep 6, 2023
2 parents e210b28 + 24a4815 commit 62569b1
Showing 1 changed file with 5 additions and 2 deletions.
7 changes: 5 additions & 2 deletions erpnext/setup/doctype/employee/test_employee.py
Original file line number Diff line number Diff line change
Expand Up @@ -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

0 comments on commit 62569b1

Please sign in to comment.