fix(test): use non-group customer group in test setup

This commit is contained in:
khushi8112
2026-04-06 12:38:22 +05:30
parent 97684d3dae
commit ea3fcc214b
14 changed files with 18 additions and 6 deletions

View File

@@ -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

View File

@@ -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

View File

@@ -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

View File

@@ -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()

View File

@@ -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()

View File

@@ -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

View File

@@ -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

View File

@@ -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

View File

@@ -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

View File

@@ -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",

View File

@@ -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()

View File

@@ -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()

View File

@@ -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()

View File

@@ -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: