diff --git a/erpnext/controllers/tests/test_queries.py b/erpnext/controllers/tests/test_queries.py index 9e92d7ce638..1e0c670c1c0 100644 --- a/erpnext/controllers/tests/test_queries.py +++ b/erpnext/controllers/tests/test_queries.py @@ -106,9 +106,9 @@ class TestQueries(ERPNextTestSuite): value=1, 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( { "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) - params = { - "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) + self.assertGreater(len(result), 1) - result = queries.employee_query(**params) - self.assertGreater(len(result), 1) + ps.delete(ignore_permissions=1, force=1, delete_permanently=1) + frappe.clear_cache(doctype="Payment Entry") - ps.delete(ignore_permissions=1, force=1, delete_permanently=1) - frappe.reload_doc("accounts", "doctype", "payment entry") - frappe.clear_cache() - - # 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") + # 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) diff --git a/erpnext/manufacturing/doctype/job_card/test_job_card.py b/erpnext/manufacturing/doctype/job_card/test_job_card.py index 251235e772f..9e34d9e65ec 100644 --- a/erpnext/manufacturing/doctype/job_card/test_job_card.py +++ b/erpnext/manufacturing/doctype/job_card/test_job_card.py @@ -23,6 +23,7 @@ from erpnext.manufacturing.doctype.work_order.test_work_order import make_wo_ord from erpnext.manufacturing.doctype.work_order.work_order import WorkOrder from erpnext.manufacturing.doctype.workstation.test_workstation import make_workstation from erpnext.stock.doctype.stock_entry.stock_entry_utils import make_stock_entry +from erpnext.tests.utils import ERPNextTestSuite EXTRA_TEST_RECORD_DEPENDENCIES = ["UOM"] @@ -36,7 +37,13 @@ class UnitTestJobCard(UnitTestCase): pass -class TestJobCard(IntegrationTestCase): +class TestJobCard(ERPNextTestSuite): + @classmethod + def setUpClass(cls): + super().setUpClass() + # used in job card time log + cls.make_employees() + def setUp(self): self.make_bom_for_jc_tests() self.transfer_material_against: Literal["Work Order", "Job Card"] = "Work Order"