nsinnovations/erpnext
2
0
Fork 0
mirror of https://github.com/frappe/erpnext.git synced 2026-05-23 06:59:20 +00:00
Code Issues Packages Projects Releases Wiki Activity

test: improve logic for test_employee_query_with_user_permissions

Browse Source
This commit is contained in:
Sagar Vora
2025-06-09 22:12:36 +05:30
parent ea68e94665
commit 2a85c401b4
1 changed files with 20 additions and 25 deletions
Download Patch File Download Diff File Expand all files Collapse all files

45
erpnext/controllers/tests/test_queries.py
View File

@@ -106,9 +106,9 @@ class TestQueries(ERPNextTestSuite):
value=1, value=1,
property_type="Check", property_type="Check",
) )
ps.save() frappe.clear_cache(doctype="Payment Entry")
user = create_user("test_employee_query@example.com", ("Accounts User", "HR User")) user = create_user("test_employee_query@example.com", "Accounts User", "HR User")
add_user_permissions( add_user_permissions(
{ {
"user": user.name, "user": user.name,
@@ -121,29 +121,24 @@ class TestQueries(ERPNextTestSuite):
} }
) )
frappe.reload_doc("accounts", "doctype", "payment entry") with IntegrationTestCase.set_user(user.name):
params = {
"doctype": "Employee",
"txt": "",
"searchfield": "name",
"start": 0,
"page_len": 20,
"filters": None,
"reference_doctype": "Payment Entry",
"ignore_user_permissions": 1,
}
frappe.set_user(user.name) result = queries.employee_query(**params)
params = { self.assertGreater(len(result), 1)
"doctype": "Employee",
"txt": "",
"searchfield": "name",
"start": 0,
"page_len": 20,
"filters": None,
"reference_doctype": "Payment Entry",
"ignore_user_permissions": 1,
}
result = queries.employee_query(**params) ps.delete(ignore_permissions=1, force=1, delete_permanently=1)
self.assertGreater(len(result), 1) frappe.clear_cache(doctype="Payment Entry")
ps.delete(ignore_permissions=1, force=1, delete_permanently=1) # only one employee should be returned even though ignore_user_permissions is passed as 1
frappe.reload_doc("accounts", "doctype", "payment entry") result = queries.employee_query(**params)
frappe.clear_cache() self.assertEqual(len(result), 1)
# only one employee should be returned even though ignore_user_permissions is passed as 1
result = queries.employee_query(**params)
self.assertEqual(len(result), 1)
frappe.set_user("Administrator")