fix: codacy

frappe · Nov 2, 2020 · 72f98bc · 72f98bc
1 parent f7de9ec
commit 72f98bc
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 6 deletions.
diff --git a/erpnext/hr/doctype/leave_period/leave_period.py b/erpnext/hr/doctype/leave_period/leave_period.py
@@ -9,7 +9,6 @@
 from frappe.model.document import Document
 from erpnext.hr.utils import validate_overlap
 from frappe.utils.background_jobs import enqueue
-from six import iteritems
 
 class LeavePeriod(Document):
 

diff --git a/erpnext/hr/doctype/leave_period/test_leave_period.py b/erpnext/hr/doctype/leave_period/test_leave_period.py
@@ -5,9 +5,6 @@
 
 import frappe, erpnext
 import unittest
-from frappe.utils import today, add_months
-from erpnext.hr.doctype.employee.test_employee import make_employee
-from erpnext.hr.doctype.leave_application.leave_application import get_leave_balance_on
 
 test_dependencies = ["Employee", "Leave Type", "Leave Policy"]
 

diff --git a/erpnext/hr/doctype/leave_policy_assignment/test_leave_policy_assignment.py b/erpnext/hr/doctype/leave_policy_assignment/test_leave_policy_assignment.py
@@ -13,7 +13,7 @@ class TestLeavePolicyAssignment(unittest.TestCase):
 
 	def setUp(self):
 		for doctype in ["Leave Application", "Leave Allocation", "Leave Policy Assignment", "Leave Ledger Entry"]:
-			frappe.db.sql("delete from `tab{0}`".format(doctype))
+			frappe.db.sql("delete from `tab{0}`".format(doctype)) #nosec
 
 	def test_grant_leaves(self):
 		leave_period = get_leave_period()
@@ -98,6 +98,6 @@ def test_allow_to_grant_all_leave_after_cancellation_of_every_leave_allocation(s
 
 	def tearDown(self):
 		for doctype in ["Leave Application", "Leave Allocation", "Leave Policy Assignment", "Leave Ledger Entry"]:
-			frappe.db.sql("delete from `tab{0}`".format(doctype))
+			frappe.db.sql("delete from `tab{0}`".format(doctype)) #nosec