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

fix: Define specific exceptions and fix tests

Browse Source
This commit is contained in:
Deepesh Garg 2020-11-25 13:46:07 +05:30
parent df065f7044
commit 4e991fe668
3 changed files with 23 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

27
erpnext/accounts/doctype/accounting_dimension_filter/test_accounting_dimension_filter.py
View File

@ -6,7 +6,8 @@ from __future__ import unicode_literals
import frappe
import unittest
from erpnext.accounts.doctype.sales_invoice.test_sales_invoice import create_sales_invoice
from erpnext.accounts.doctype.accounting_dimension.test_accounting_dimension import create_dimension
from erpnext.accounts.doctype.accounting_dimension.test_accounting_dimension import create_dimension, disable_dimension
from erpnext.exceptions import InvalidAccountDimension, MandatoryDimension
class TestAccountingDimensionFilter(unittest.TestCase):
def setUp(self):
@ -16,19 +17,25 @@ class TestAccountingDimensionFilter(unittest.TestCase):
def test_allowed_dimension_validation(self):
si = create_sales_invoice(do_not_save=1)
si.items[0].cost_center = 'Main - _TC'
si.location = 'Block 1'
si.save()
self.assertRaises(frappe.ValidationError, si.submit)
self.assertRaises(InvalidAccountDimension, si.submit)
def test_mandatory_dimension_validation(self):
si = create_sales_invoice(do_not_save=1)
si.items[0].location = ''
si.location = 'Block 1'
# Test with no department for Sales Account
si.items[0].department = ''
si.items[0].cost_center = '_Test Cost Center 2 - _TC'
si.save()
self.assertRaises(frappe.ValidationError, si.submit)
self.assertRaises(MandatoryDimension, si.submit)
def tearDown(self):
disable_dimension_filter()
disable_dimension()
def create_accounting_dimension_filter():
if not frappe.db.get_value('Accounting Dimension Filter',
@ -52,10 +59,10 @@ def create_accounting_dimension_filter():
doc.save()
if not frappe.db.get_value('Accounting Dimension Filter',
{'accounting_dimension': 'Location'}):
{'accounting_dimension': 'Department'}):
frappe.get_doc({
'doctype': 'Accounting Dimension Filter',
'accounting_dimension': 'Location',
'accounting_dimension': 'Department',
'allow_or_restrict': 'Allow',
'company': '_Test Company',
'accounts': [{
@ -63,12 +70,12 @@ def create_accounting_dimension_filter():
'is_mandatory': 1
}],
'dimensions': [{
'accounting_dimension': 'Location',
'dimension_value': 'Block 1'
'accounting_dimension': 'Department',
'dimension_value': '_Test Department - _TC'
}]
}).insert()
else:
doc = frappe.get_doc('Accounting Dimension Filter', {'accounting_dimension': 'Location'})
doc = frappe.get_doc('Accounting Dimension Filter', {'accounting_dimension': 'Department'})
doc.disabled = 0
doc.save()
@ -77,6 +84,6 @@ def disable_dimension_filter():
doc.disabled = 1
doc.save()
doc = frappe.get_doc('Accounting Dimension Filter', {'accounting_dimension': 'Location'})
doc = frappe.get_doc('Accounting Dimension Filter', {'accounting_dimension': 'Department'})
doc.disabled = 1
doc.save()

8
erpnext/accounts/doctype/gl_entry/gl_entry.py
View File

@ -11,7 +11,7 @@ from frappe.model.meta import get_field_precision
from erpnext.accounts.party import validate_party_gle_currency, validate_party_frozen_disabled
from erpnext.accounts.utils import get_account_currency
from erpnext.accounts.utils import get_fiscal_year
from erpnext.exceptions import InvalidAccountCurrency
from erpnext.exceptions import InvalidAccountCurrency, InvalidAccountDimension, MandatoryDimension
from erpnext.accounts.doctype.accounting_dimension.accounting_dimension import get_checks_for_pl_and_bs_accounts
from erpnext.accounts.doctype.accounting_dimension_filter.accounting_dimension_filter import get_dimension_filter_map
from six import iteritems
@ -101,16 +101,16 @@ class GLEntry(Document):
if self.account == account:
if value['is_mandatory'] and not self.get(dimension):
frappe.throw(_("{0} is mandatory for account {1}").format(
frappe.bold(frappe.unscrub(dimension)), frappe.bold(self.account)))
frappe.bold(frappe.unscrub(dimension)), frappe.bold(self.account)), MandatoryDimension)
if value['allow_or_restrict'] == 'Allow':
if self.get(dimension) and self.get(dimension) not in value['allowed_dimensions']:
frappe.throw(_("Invalid value {0} for account {1}").format(
frappe.bold(self.get(dimension)), frappe.bold(self.account)))
frappe.bold(self.get(dimension)), frappe.bold(self.account)), InvalidAccountDimension)
else:
if self.get(dimension) and self.get(dimension) in value['allowed_dimensions']:
frappe.throw(_("Invalid value {0} for account {1}").format(
frappe.bold(self.get(dimension)), frappe.bold(self.account)))
frappe.bold(self.get(dimension)), frappe.bold(self.account)), InvalidAccountDimension)
def check_pl_account(self):
if self.is_opening=='Yes' and \

2
erpnext/exceptions.py
View File

@ -6,3 +6,5 @@ class PartyFrozen(frappe.ValidationError): pass
class InvalidAccountCurrency(frappe.ValidationError): pass
class InvalidCurrency(frappe.ValidationError): pass
class PartyDisabled(frappe.ValidationError):pass
class InvalidAccountDimension(frappe.ValidationError): pass
class MandatoryDimension(frappe.ValidationError): pass