Shiloh/brotherton-erpnext
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix Employee tax declaration test case

Browse Source
This commit is contained in:
Shreya 2018-06-10 00:38:30 +05:30
parent 664f30a018
commit 83d5dc119b
2 changed files with 19 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

31
erpnext/hr/doctype/employee_tax_exemption_declaration/test_employee_tax_exemption_declaration.py
View File

@ -8,7 +8,7 @@ import unittest
from erpnext.hr.doctype.salary_structure.test_salary_structure import make_employee from erpnext.hr.doctype.salary_structure.test_salary_structure import make_employee
class TestEmployeeTaxExemptionDeclaration(unittest.TestCase): class TestEmployeeTaxExemptionDeclaration(unittest.TestCase):
def setup(self): def setUp(self):
make_employee("employee@taxexepmtion.com") make_employee("employee@taxexepmtion.com")
make_employee("employee1@taxexepmtion.com") make_employee("employee1@taxexepmtion.com")
create_payroll_period() create_payroll_period()
@ -19,7 +19,7 @@ class TestEmployeeTaxExemptionDeclaration(unittest.TestCase):
declaration = frappe.get_doc({ declaration = frappe.get_doc({
"doctype": "Employee Tax Exemption Declaration", "doctype": "Employee Tax Exemption Declaration",
"employee": frappe.get_value("Employee", {"user_id":"employee@taxexepmtion.com"}, "name"), "employee": frappe.get_value("Employee", {"user_id":"employee@taxexepmtion.com"}, "name"),
"payroll_period": "Test Payroll Period", "payroll_period": "_Test Payroll Period",
"declarations": [dict(exemption_sub_category = "_Test Sub Category", "declarations": [dict(exemption_sub_category = "_Test Sub Category",
exemption_category = "_Test Category", exemption_category = "_Test Category",
amount = 150000)] amount = 150000)]
@ -27,7 +27,7 @@ class TestEmployeeTaxExemptionDeclaration(unittest.TestCase):
self.assertRaises(frappe.ValidationError, declaration.save) self.assertRaises(frappe.ValidationError, declaration.save)
declaration = frappe.get_doc({ declaration = frappe.get_doc({
"doctype": "Employee Tax Exemption Declaration", "doctype": "Employee Tax Exemption Declaration",
"payroll_period": "Test Payroll Period", "payroll_period": "_Test Payroll Period",
"employee": frappe.get_value("Employee", {"user_id":"employee@taxexepmtion.com"}, "name"), "employee": frappe.get_value("Employee", {"user_id":"employee@taxexepmtion.com"}, "name"),
"declarations": [dict(exemption_sub_category = "_Test Sub Category", "declarations": [dict(exemption_sub_category = "_Test Sub Category",
exemption_category = "_Test Category", exemption_category = "_Test Category",
@ -39,7 +39,8 @@ class TestEmployeeTaxExemptionDeclaration(unittest.TestCase):
declaration = frappe.get_doc({ declaration = frappe.get_doc({
"doctype": "Employee Tax Exemption Declaration", "doctype": "Employee Tax Exemption Declaration",
"employee": frappe.get_value("Employee", {"user_id":"employee@taxexepmtion.com"}, "name"), "employee": frappe.get_value("Employee", {"user_id":"employee@taxexepmtion.com"}, "name"),
"payroll_period": "Test Payroll Period", "company": "_Test Company",
"payroll_period": "_Test Payroll Period",
"declarations": [dict(exemption_sub_category = "_Test Sub Category", "declarations": [dict(exemption_sub_category = "_Test Sub Category",
exemption_category = "_Test Category", exemption_category = "_Test Category",
amount = 100000), amount = 100000),
@ -54,7 +55,8 @@ class TestEmployeeTaxExemptionDeclaration(unittest.TestCase):
declaration = frappe.get_doc({ declaration = frappe.get_doc({
"doctype": "Employee Tax Exemption Declaration", "doctype": "Employee Tax Exemption Declaration",
"employee": frappe.get_value("Employee", {"user_id":"employee@taxexepmtion.com"}, "name"), "employee": frappe.get_value("Employee", {"user_id":"employee@taxexepmtion.com"}, "name"),
"payroll_period": "Test Payroll Period", "company": "_Test Company",
"payroll_period": "_Test Payroll Period",
"declarations": [dict(exemption_sub_category = "_Test Sub Category", "declarations": [dict(exemption_sub_category = "_Test Sub Category",
exemption_category = "_Test Category", exemption_category = "_Test Category",
amount = 100000), amount = 100000),
@ -62,17 +64,18 @@ class TestEmployeeTaxExemptionDeclaration(unittest.TestCase):
exemption_category = "_Test Category", exemption_category = "_Test Category",
amount = 50000), amount = 50000),
] ]
}) }).insert()
self.assertTrue(declaration.submit) self.assertTrue(declaration.submit)
duplicate_declaration = frappe.get_doc({ duplicate_declaration = frappe.get_doc({
"doctype": "Employee Tax Exemption Declaration", "doctype": "Employee Tax Exemption Declaration",
"employee": frappe.get_value("Employee", {"user_id":"employee@taxexepmtion.com"}, "name"), "employee": frappe.get_value("Employee", {"user_id":"employee@taxexepmtion.com"}, "name"),
"payroll_period": "Test Payroll Period", "company": "_Test Company",
"payroll_period": "_Test Payroll Period",
"declarations": [dict(exemption_sub_category = "_Test Sub Category", "declarations": [dict(exemption_sub_category = "_Test Sub Category",
exemption_category = "_Test Category", exemption_category = "_Test Category",
amount = 100000) amount = 100000)
] ]
}) }).insert()
self.assertRaises(frappe.DocstatusTransitionError, duplicate_declaration.submit) self.assertRaises(frappe.DocstatusTransitionError, duplicate_declaration.submit)
duplicate_declaration.employee = frappe.get_value("Employee", {"user_id":"employee1@taxexepmtion.com"}, "name") duplicate_declaration.employee = frappe.get_value("Employee", {"user_id":"employee1@taxexepmtion.com"}, "name")
self.assertTrue(duplicate_declaration.submit) self.assertTrue(duplicate_declaration.submit)
@ -93,20 +96,22 @@ def create_exemption_category():
category = frappe.get_doc({ category = frappe.get_doc({
"doctype": "Employee Tax Exemption Category", "doctype": "Employee Tax Exemption Category",
"name": "_Test Category", "name": "_Test Category",
"deduction_component": "_Test Tax", "deduction_component": "Income Tax",
"max_amount": 100000 "max_amount": 100000
}).insert() }).insert()
if not frappe.db.exists("Employee Tax Exemption Sub Category", "_Test Category"): if not frappe.db.exists("Employee Tax Exemption Sub Category", "_Test Sub Category"):
frappe.get_doc({ frappe.get_doc({
"doctype": "Employee Tax Exemption Sub Category", "doctype": "Employee Tax Exemption Sub Category",
"name": "_Test Sub Category", "name": "_Test Sub Category",
"exemption_category": "_Test Category", "exemption_category": "_Test Category",
"max_amount": 100000 "max_amount": 100000,
"is_active": 1
}).insert() }).insert()
if not frappe.db.exists("Employee Tax Exemption Sub Category", "_Test Category"): if not frappe.db.exists("Employee Tax Exemption Sub Category", "_Test1 Sub Category"):
frappe.get_doc({ frappe.get_doc({
"doctype": "Employee Tax Exemption Sub Category", "doctype": "Employee Tax Exemption Sub Category",
"name": "_Test1 Sub Category", "name": "_Test1 Sub Category",
"exemption_category": "_Test Category", "exemption_category": "_Test Category",
"max_amount": 50000 "max_amount": 50000,
"is_active": 1
}).insert() }).insert()

2
erpnext/hr/doctype/leave_application/test_leave_application.py
View File

@ -433,7 +433,7 @@ def get_leave_period():
to_date = "{0}-12-31".format(now_datetime().year), to_date = "{0}-12-31".format(now_datetime().year),
company = "_Test Company", company = "_Test Company",
is_active = 1 is_active = 1
)).insert() )).insert()
def allocate_leaves(employee, leave_period, leave_type, new_leaves_allocated, eligible_leaves=0): def allocate_leaves(employee, leave_period, leave_type, new_leaves_allocated, eligible_leaves=0):
allocate_leave = frappe.get_doc({ allocate_leave = frappe.get_doc({