mirror of
https://github.com/frappe/erpnext.git
synced 2026-04-12 11:25:09 +00:00
fix(test): use non-group customer group in test setup
This commit is contained in:
@@ -2043,6 +2043,7 @@ def create_customer(name="_Test Customer 2 USD", currency="USD"):
|
|||||||
customer.customer_name = name
|
customer.customer_name = name
|
||||||
customer.default_currency = currency
|
customer.default_currency = currency
|
||||||
customer.type = "Individual"
|
customer.type = "Individual"
|
||||||
|
customer.customer_group = "Individual"
|
||||||
customer.save()
|
customer.save()
|
||||||
customer = customer.name
|
customer = customer.name
|
||||||
return customer
|
return customer
|
||||||
|
|||||||
@@ -80,6 +80,7 @@ class TestPaymentLedgerEntry(FrappeTestCase):
|
|||||||
customer = frappe.new_doc("Customer")
|
customer = frappe.new_doc("Customer")
|
||||||
customer.customer_name = name
|
customer.customer_name = name
|
||||||
customer.type = "Individual"
|
customer.type = "Individual"
|
||||||
|
customer.customer_group = "Individual"
|
||||||
customer.save()
|
customer.save()
|
||||||
self.customer = customer.name
|
self.customer = customer.name
|
||||||
|
|
||||||
|
|||||||
@@ -2546,6 +2546,7 @@ def make_customer(customer_name, currency=None):
|
|||||||
customer = frappe.new_doc("Customer")
|
customer = frappe.new_doc("Customer")
|
||||||
customer.customer_name = customer_name
|
customer.customer_name = customer_name
|
||||||
customer.type = "Individual"
|
customer.type = "Individual"
|
||||||
|
customer.customer_group = "Individual"
|
||||||
|
|
||||||
if currency:
|
if currency:
|
||||||
customer.default_currency = currency
|
customer.default_currency = currency
|
||||||
|
|||||||
@@ -629,18 +629,21 @@ def create_parties():
|
|||||||
customer.customer_name = "_Test Subscription Customer"
|
customer.customer_name = "_Test Subscription Customer"
|
||||||
customer.default_currency = "USD"
|
customer.default_currency = "USD"
|
||||||
customer.append("accounts", {"company": "_Test Company", "account": "_Test Receivable USD - _TC"})
|
customer.append("accounts", {"company": "_Test Company", "account": "_Test Receivable USD - _TC"})
|
||||||
|
customer.customer_group = "Individual"
|
||||||
customer.insert()
|
customer.insert()
|
||||||
|
|
||||||
if not frappe.db.exists("Customer", "_Test Subscription Customer Multi Currency"):
|
if not frappe.db.exists("Customer", "_Test Subscription Customer Multi Currency"):
|
||||||
customer = frappe.new_doc("Customer")
|
customer = frappe.new_doc("Customer")
|
||||||
customer.customer_name = "Test Subscription Customer Multi Currency"
|
customer.customer_name = "Test Subscription Customer Multi Currency"
|
||||||
customer.default_currency = "USD"
|
customer.default_currency = "USD"
|
||||||
|
customer.customer_group = "Individual"
|
||||||
customer.insert()
|
customer.insert()
|
||||||
|
|
||||||
if not frappe.db.exists("Customer", "_Test Subscription Customer John Doe"):
|
if not frappe.db.exists("Customer", "_Test Subscription Customer John Doe"):
|
||||||
customer = frappe.new_doc("Customer")
|
customer = frappe.new_doc("Customer")
|
||||||
customer.customer_name = "_Test Subscription Customer John Doe"
|
customer.customer_name = "_Test Subscription Customer John Doe"
|
||||||
customer.append("accounts", {"company": "_Test Company", "account": "_Test Receivable - _TC"})
|
customer.append("accounts", {"company": "_Test Company", "account": "_Test Receivable - _TC"})
|
||||||
|
customer.customer_group = "Individual"
|
||||||
customer.insert()
|
customer.insert()
|
||||||
|
|
||||||
|
|
||||||
|
|||||||
@@ -779,6 +779,7 @@ class TestAccountsReceivable(AccountsTestMixin, FrappeTestCase):
|
|||||||
"customer_name": "Jane Doe",
|
"customer_name": "Jane Doe",
|
||||||
"type": "Individual",
|
"type": "Individual",
|
||||||
"default_currency": "USD",
|
"default_currency": "USD",
|
||||||
|
"customer_group": "Individual",
|
||||||
}
|
}
|
||||||
)
|
)
|
||||||
.insert()
|
.insert()
|
||||||
@@ -1002,6 +1003,7 @@ class TestAccountsReceivable(AccountsTestMixin, FrappeTestCase):
|
|||||||
"customer_name": "Jane Doe",
|
"customer_name": "Jane Doe",
|
||||||
"type": "Individual",
|
"type": "Individual",
|
||||||
"default_currency": "USD",
|
"default_currency": "USD",
|
||||||
|
"customer_group": "Individual",
|
||||||
}
|
}
|
||||||
)
|
)
|
||||||
.insert()
|
.insert()
|
||||||
|
|||||||
@@ -82,6 +82,7 @@ class TestGrossProfit(FrappeTestCase):
|
|||||||
customer = frappe.new_doc("Customer")
|
customer = frappe.new_doc("Customer")
|
||||||
customer.customer_name = name
|
customer.customer_name = name
|
||||||
customer.type = "Individual"
|
customer.type = "Individual"
|
||||||
|
customer.customer_group = "Individual"
|
||||||
customer.save()
|
customer.save()
|
||||||
self.customer = customer.name
|
self.customer = customer.name
|
||||||
|
|
||||||
|
|||||||
@@ -12,6 +12,7 @@ class AccountsTestMixin:
|
|||||||
customer = frappe.new_doc("Customer")
|
customer = frappe.new_doc("Customer")
|
||||||
customer.customer_name = customer_name
|
customer.customer_name = customer_name
|
||||||
customer.type = "Individual"
|
customer.type = "Individual"
|
||||||
|
customer.customer_group = "Individual"
|
||||||
|
|
||||||
if currency:
|
if currency:
|
||||||
customer.default_currency = currency
|
customer.default_currency = currency
|
||||||
@@ -36,6 +37,7 @@ class AccountsTestMixin:
|
|||||||
"account": default_account,
|
"account": default_account,
|
||||||
},
|
},
|
||||||
)
|
)
|
||||||
|
customer.customer_group = "Individual"
|
||||||
customer.save()
|
customer.save()
|
||||||
self.customer = customer_name
|
self.customer = customer_name
|
||||||
|
|
||||||
|
|||||||
@@ -7,12 +7,8 @@ from erpnext.accounts.party import get_default_price_list
|
|||||||
class PartyTestCase(FrappeTestCase):
|
class PartyTestCase(FrappeTestCase):
|
||||||
def test_get_default_price_list_should_return_none_for_invalid_group(self):
|
def test_get_default_price_list_should_return_none_for_invalid_group(self):
|
||||||
customer = frappe.get_doc(
|
customer = frappe.get_doc(
|
||||||
{
|
{"doctype": "Customer", "customer_name": "test customer", "customer_group": "Individual"}
|
||||||
"doctype": "Customer",
|
|
||||||
"customer_name": "test customer",
|
|
||||||
}
|
|
||||||
).insert(ignore_permissions=True, ignore_mandatory=True)
|
).insert(ignore_permissions=True, ignore_mandatory=True)
|
||||||
customer.customer_group = None
|
|
||||||
customer.save()
|
customer.save()
|
||||||
price_list = get_default_price_list(customer)
|
price_list = get_default_price_list(customer)
|
||||||
assert price_list is None
|
assert price_list is None
|
||||||
|
|||||||
@@ -29,6 +29,7 @@ def make_customer(customer_name, currency=None):
|
|||||||
customer = frappe.new_doc("Customer")
|
customer = frappe.new_doc("Customer")
|
||||||
customer.customer_name = customer_name
|
customer.customer_name = customer_name
|
||||||
customer.customer_type = "Individual"
|
customer.customer_type = "Individual"
|
||||||
|
customer.customer_group = "Individual"
|
||||||
|
|
||||||
if currency:
|
if currency:
|
||||||
customer.default_currency = currency
|
customer.default_currency = currency
|
||||||
|
|||||||
@@ -324,7 +324,7 @@ def _make_customer(source_name, target_doc=None, ignore_permissions=False):
|
|||||||
target.customer_name = source.lead_name
|
target.customer_name = source.lead_name
|
||||||
|
|
||||||
if not target.customer_group:
|
if not target.customer_group:
|
||||||
target.customer_group = frappe.db.get_default("Customer Group")
|
target.customer_group = "Individual"
|
||||||
|
|
||||||
doclist = get_mapped_doc(
|
doclist = get_mapped_doc(
|
||||||
"Lead",
|
"Lead",
|
||||||
|
|||||||
@@ -196,6 +196,7 @@ def create_customer():
|
|||||||
if not doc:
|
if not doc:
|
||||||
doc = frappe.new_doc("Customer")
|
doc = frappe.new_doc("Customer")
|
||||||
doc.customer_name = "_Test NC"
|
doc.customer_name = "_Test NC"
|
||||||
|
doc.customer_group = "Individual"
|
||||||
doc.insert()
|
doc.insert()
|
||||||
|
|
||||||
|
|
||||||
|
|||||||
@@ -148,6 +148,7 @@ def make_customer():
|
|||||||
"doctype": "Customer",
|
"doctype": "Customer",
|
||||||
"customer_name": "_Test UAE Customer",
|
"customer_name": "_Test UAE Customer",
|
||||||
"customer_type": "Company",
|
"customer_type": "Company",
|
||||||
|
"customer_group": "Individual",
|
||||||
}
|
}
|
||||||
)
|
)
|
||||||
customer.insert()
|
customer.insert()
|
||||||
|
|||||||
@@ -115,6 +115,7 @@ def make_customer():
|
|||||||
"doctype": "Customer",
|
"doctype": "Customer",
|
||||||
"customer_name": "_Test SA Customer",
|
"customer_name": "_Test SA Customer",
|
||||||
"customer_type": "Company",
|
"customer_type": "Company",
|
||||||
|
"customer_group": "Individual",
|
||||||
}
|
}
|
||||||
).insert()
|
).insert()
|
||||||
|
|
||||||
|
|||||||
@@ -450,6 +450,7 @@ def make_customer(customer_name):
|
|||||||
customer = frappe.new_doc("Customer")
|
customer = frappe.new_doc("Customer")
|
||||||
customer.customer_name = customer_name
|
customer.customer_name = customer_name
|
||||||
customer.customer_type = "Individual"
|
customer.customer_type = "Individual"
|
||||||
|
customer.customer_group = "Individual"
|
||||||
customer.insert()
|
customer.insert()
|
||||||
return customer.name
|
return customer.name
|
||||||
else:
|
else:
|
||||||
|
|||||||
Reference in New Issue
Block a user