From fee2fee83a9580a4365c67aa38fc89232ce8dec4 Mon Sep 17 00:00:00 2001 From: Himanshu Warekar Date: Tue, 28 May 2019 11:47:51 +0530 Subject: [PATCH] fix: test cases --- erpnext/support/doctype/service_level/test_service_level.py | 4 ++-- .../service_level_agreement/test_service_level_agreement.py | 5 +++-- 2 files changed, 5 insertions(+), 4 deletions(-) diff --git a/erpnext/support/doctype/service_level/test_service_level.py b/erpnext/support/doctype/service_level/test_service_level.py index 79f064358d..9131d807f6 100644 --- a/erpnext/support/doctype/service_level/test_service_level.py +++ b/erpnext/support/doctype/service_level/test_service_level.py @@ -166,13 +166,13 @@ def make_service_level(): } ] }) - service_level_exist = frappe.get_doc("Service Level", "_Test Service Level") + service_level_exist = frappe.db.exists("Service Level", {"service_level": "_Test Service Level"}) if not service_level_exist: service_level.insert() return service_level else: - return service_level_exist + return frappe.get_doc("Service Level", "_Test Service Level") def get_service_level(): return frappe.get_doc("Service Level", "_Test Service Level") diff --git a/erpnext/support/doctype/service_level_agreement/test_service_level_agreement.py b/erpnext/support/doctype/service_level_agreement/test_service_level_agreement.py index 2411d2163a..2679f97f55 100644 --- a/erpnext/support/doctype/service_level_agreement/test_service_level_agreement.py +++ b/erpnext/support/doctype/service_level_agreement/test_service_level_agreement.py @@ -94,6 +94,7 @@ def make_service_level_agreement(): }) default_service_level_agreement_exists = frappe.db.exists("Service Level Agreement", "SLA-Default Service Level Agreement") + if not default_service_level_agreement_exists: default_service_level_agreement.insert(ignore_permissions=True) @@ -180,13 +181,13 @@ def make_service_level_agreement(): ] }) - service_level_agreement_exists = frappe.get_doc("Service Level Agreement", "SLA-_Test Service Level Agreement") + service_level_agreement_exists = frappe.db.exists("Service Level Agreement", {"service_level_agreement_name": "_Test Service Level Agreement"}) if not service_level_agreement_exists: service_level_agreement.insert(ignore_permissions=True) return service_level_agreement else: - return service_level_agreement_exists + return frappe.get_doc("Service Level Agreement", "SLA-_Test Service Level Agreement") def get_service_level_agreement(): service_level_agreement = frappe.get_doc("Service Level Agreement", "SLA-_Test Service Level Agreement")