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

[Test] Added test cases to evaluate customer credit limit

Browse Source
This commit is contained in:
shreyas 2016-10-20 15:56:18 +05:30
parent e05656963a
commit 17f43da2a6
2 changed files with 52 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
erpnext/selling/doctype/customer/customer.py
View File

@ -127,13 +127,13 @@ class Customer(TransactionBase):
frappe.throw(_("A Customer Group exists with same name please change the Customer name or rename the Customer Group"), frappe.NameError)
def validate_credit_limit_on_change(self):
customer = frappe.get_doc('Customer', self.name)
companies = [c.name for c in frappe.get_all("Company")]
if self.get("__islocal") or self.credit_limit == frappe.db.get_value("Customer", self.name, "credit_limit"):
return
for company in companies:
outstanding_amt = get_customer_outstanding(customer.name, company)
for company in frappe.get_all("Company"):
outstanding_amt = get_customer_outstanding(self.name, company.name)
if flt(self.credit_limit) < outstanding_amt:
frappe.throw(_("New credit limit is less than current outstanding amount for the customer. Credit limit has to be atleast {0}").format(outstanding_amt))
frappe.throw(_("""New credit limit is less than current outstanding amount for the customer. Credit limit has to be atleast {0}""").format(outstanding_amt))
def delete_customer_address(self):
addresses = frappe.db.sql("""select name, lead from `tabAddress`

48
erpnext/selling/doctype/customer/test_customer.py
View File

@ -8,6 +8,7 @@ import unittest
from frappe.test_runner import make_test_records
from erpnext.exceptions import PartyFrozen, PartyDisabled
from frappe.utils import flt
test_ignore = ["Price List"]
@ -111,6 +112,52 @@ class TestCustomer(unittest.TestCase):
self.assertEquals("_Test Customer 1 - 1", duplicate_customer.name)
self.assertEquals(test_customer_1.customer_name, duplicate_customer.customer_name)
def test_customer_credit_limit(self):
from erpnext.selling.doctype.customer.customer import get_credit_limit, get_customer_outstanding
from erpnext.stock.doctype.delivery_note.test_delivery_note import create_delivery_note
from erpnext.accounts.doctype.sales_invoice.test_sales_invoice import create_sales_invoice
from erpnext.selling.doctype.sales_order.test_sales_order import make_sales_order
from erpnext.selling.doctype.sales_order.sales_order import make_sales_invoice
outstanding_amt = get_customer_outstanding('_Test Customer', '_Test Company')
credit_limit = get_credit_limit('_Test Customer', '_Test Company')
if credit_limit == 0.0:
frappe.db.set_value("Customer", '_Test Customer', 'credit_limit', 50.0)
# Sales Order
so = make_sales_order(do_not_submit=True)
self.assertRaises(frappe.ValidationError, so.submit)
# Delivery Note
dn = create_delivery_note(do_not_submit=True)
self.assertRaises(frappe.ValidationError, dn.submit)
# Sales Invoice
si = create_sales_invoice(do_not_submit=True)
self.assertRaises(frappe.ValidationError, si.submit)
if credit_limit > outstanding_amt:
frappe.db.set_value("Customer", '_Test Customer', 'credit_limit', credit_limit)
# Makes Sales invoice from Sales Order
so.save(ignore_permissions=True)
si = make_sales_invoice(so.name)
si.save(ignore_permissions=True)
self.assertRaises(frappe.ValidationError, make_sales_order)
def test_customer_credit_limit_on_change(self):
from erpnext.selling.doctype.customer.customer import get_credit_limit, get_customer_outstanding
from erpnext.selling.doctype.sales_order.test_sales_order import make_sales_order
outstanding_amt = get_customer_outstanding('_Test Customer', '_Test Company')
credit_limit = get_credit_limit('_Test Customer', '_Test Company')
customer = frappe.get_doc("Customer", '_Test Customer')
customer.credit_limit = flt(outstanding_amt - 100)
self.assertRaises(frappe.ValidationError, customer.save)
def get_customer_dict(customer_name):
return {
"customer_group": "_Test Customer Group",
@ -119,4 +166,3 @@ def get_customer_dict(customer_name):
"doctype": "Customer",
"territory": "_Test Territory"
}