diff --git a/erpnext/accounts/doctype/payment_ledger_entry/test_payment_ledger_entry.py b/erpnext/accounts/doctype/payment_ledger_entry/test_payment_ledger_entry.py index 6ea63fd6c24..c30d2a80488 100644 --- a/erpnext/accounts/doctype/payment_ledger_entry/test_payment_ledger_entry.py +++ b/erpnext/accounts/doctype/payment_ledger_entry/test_payment_ledger_entry.py @@ -10,75 +10,23 @@ from erpnext.accounts.doctype.payment_entry.payment_entry import get_payment_ent from erpnext.accounts.doctype.payment_entry.test_payment_entry import create_payment_entry 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.stock.doctype.item.test_item import create_item from erpnext.tests.utils import ERPNextTestSuite class TestPaymentLedgerEntry(ERPNextTestSuite): def setUp(self): self.ple = qb.DocType("Payment Ledger Entry") - self.create_company() - self.create_item() - self.create_customer() - - def create_company(self): - company_name = "_Test Payment Ledger" - company = None - if frappe.db.exists("Company", company_name): - company = frappe.get_doc("Company", company_name) - else: - company = frappe.get_doc( - { - "doctype": "Company", - "company_name": company_name, - "country": "India", - "default_currency": "INR", - "create_chart_of_accounts_based_on": "Standard Template", - "chart_of_accounts": "Standard", - } - ) - company = company.save() - - self.company = company.name - self.cost_center = company.cost_center - self.warehouse = "All Warehouses - _PL" - self.income_account = "Sales - _PL" - self.expense_account = "Cost of Goods Sold - _PL" - self.debit_to = "Debtors - _PL" - self.creditors = "Creditors - _PL" - - # create bank account - if frappe.db.exists("Account", "HDFC - _PL"): - self.bank = "HDFC - _PL" - else: - bank_acc = frappe.get_doc( - { - "doctype": "Account", - "account_name": "HDFC", - "parent_account": "Bank Accounts - _PL", - "company": self.company, - } - ) - bank_acc.save() - self.bank = bank_acc.name - - def create_item(self): - item_name = "_Test PL Item" - item = create_item( - item_code=item_name, is_stock_item=0, company=self.company, warehouse=self.warehouse - ) - self.item = item if isinstance(item, str) else item.item_code - - def create_customer(self): - name = "_Test PL Customer" - if frappe.db.exists("Customer", name): - self.customer = name - else: - customer = frappe.new_doc("Customer") - customer.customer_name = name - customer.type = "Individual" - customer.save() - self.customer = customer.name + self.company = "_Test Company" + self.cost_center = "Main - _TC" + self.warehouse = "Stores - _TC" + self.income_account = "Sales - _TC" + self.expense_account = "Cost of Goods Sold - _TC" + self.debit_to = "Debtors - _TC" + self.creditors = "Creditors - _TC" + self.bank = "Cash - _TC" + self.item = "_Test Item" + self.customer = "_Test Customer" + self.clear_old_entries() def create_sales_invoice( self, qty=1, rate=100, posting_date=None, do_not_save=False, do_not_submit=False diff --git a/erpnext/accounts/doctype/period_closing_voucher/test_period_closing_voucher.py b/erpnext/accounts/doctype/period_closing_voucher/test_period_closing_voucher.py index 95ab540d657..cf226d4e5c7 100644 --- a/erpnext/accounts/doctype/period_closing_voucher/test_period_closing_voucher.py +++ b/erpnext/accounts/doctype/period_closing_voucher/test_period_closing_voucher.py @@ -19,7 +19,6 @@ class TestPeriodClosingVoucher(ERPNextTestSuite): frappe.db.set_single_value("Accounts Settings", "use_legacy_controller_for_pcv", 1) def test_closing_entry(self): - company = create_company() cost_center = create_cost_center("Test Cost Center 1") jv1 = make_journal_entry( @@ -28,10 +27,10 @@ class TestPeriodClosingVoucher(ERPNextTestSuite): account1="Cash - TPC", account2="Sales - TPC", cost_center=cost_center, - company=company, + company="Test PCV Company", save=False, ) - jv1.company = company + jv1.company = "Test PCV Company" jv1.save() jv1.submit() @@ -41,10 +40,10 @@ class TestPeriodClosingVoucher(ERPNextTestSuite): account1="Cost of Goods Sold - TPC", account2="Cash - TPC", cost_center=cost_center, - company=company, + company="Test PCV Company", save=False, ) - jv2.company = company + jv2.company = "Test PCV Company" jv2.save() jv2.submit() @@ -68,14 +67,13 @@ class TestPeriodClosingVoucher(ERPNextTestSuite): self.assertEqual(pcv_gle, expected_gle) def test_cost_center_wise_posting(self): - company = create_company() surplus_account = create_account() cost_center1 = create_cost_center("Main") cost_center2 = create_cost_center("Western Branch") create_sales_invoice( - company=company, + company="Test PCV Company", cost_center=cost_center1, income_account="Sales - TPC", expense_account="Cost of Goods Sold - TPC", @@ -86,7 +84,7 @@ class TestPeriodClosingVoucher(ERPNextTestSuite): posting_date="2021-03-15", ) create_sales_invoice( - company=company, + company="Test PCV Company", cost_center=cost_center2, income_account="Sales - TPC", expense_account="Cost of Goods Sold - TPC", @@ -131,12 +129,11 @@ class TestPeriodClosingVoucher(ERPNextTestSuite): ) def test_period_closing_with_finance_book_entries(self): - company = create_company() surplus_account = create_account() cost_center = create_cost_center("Test Cost Center 1") create_sales_invoice( - company=company, + company="Test PCV Company", income_account="Sales - TPC", expense_account="Cost of Goods Sold - TPC", cost_center=cost_center, @@ -153,9 +150,9 @@ class TestPeriodClosingVoucher(ERPNextTestSuite): amount=400, cost_center=cost_center, posting_date="2021-03-15", - company=company, + company="Test PCV Company", ) - jv.company = company + jv.company = "Test PCV Company" jv.finance_book = create_finance_book().name jv.save() jv.submit() @@ -182,7 +179,6 @@ class TestPeriodClosingVoucher(ERPNextTestSuite): self.assertSequenceEqual(pcv_gle, expected_gle) def test_gl_entries_restrictions(self): - company = create_company() cost_center = create_cost_center("Test Cost Center 1") self.make_period_closing_voucher(posting_date="2021-03-31") @@ -193,16 +189,15 @@ class TestPeriodClosingVoucher(ERPNextTestSuite): account1="Cash - TPC", account2="Sales - TPC", cost_center=cost_center, - company=company, + company="Test PCV Company", save=False, ) - jv1.company = company + jv1.company = "Test PCV Company" jv1.save() self.assertRaises(frappe.ValidationError, jv1.submit) def test_closing_balance_with_dimensions_and_test_reposting_entry(self): - company = create_company() cost_center1 = create_cost_center("Test Cost Center 1") cost_center2 = create_cost_center("Test Cost Center 2") @@ -212,10 +207,10 @@ class TestPeriodClosingVoucher(ERPNextTestSuite): account1="Cash - TPC", account2="Sales - TPC", cost_center=cost_center1, - company=company, + company="Test PCV Company", save=False, ) - jv1.company = company + jv1.company = "Test PCV Company" jv1.save() jv1.submit() @@ -225,10 +220,10 @@ class TestPeriodClosingVoucher(ERPNextTestSuite): account1="Cash - TPC", account2="Sales - TPC", cost_center=cost_center2, - company=company, + company="Test PCV Company", save=False, ) - jv2.company = company + jv2.company = "Test PCV Company" jv2.save() jv2.submit() @@ -255,11 +250,11 @@ class TestPeriodClosingVoucher(ERPNextTestSuite): account1="Cash - TPC", account2="Sales - TPC", cost_center=cost_center2, - company=company, + company="Test PCV Company", save=False, ) - jv3.company = company + jv3.company = "Test PCV Company" jv3.save() jv3.submit() @@ -294,12 +289,12 @@ class TestPeriodClosingVoucher(ERPNextTestSuite): self.assertEqual(cc2_closing_balance.credit, 500) self.assertEqual(cc2_closing_balance.credit_in_account_currency, 500) - warehouse = frappe.db.get_value("Warehouse", {"company": company}, "name") + warehouse = frappe.db.get_value("Warehouse", {"company": "Test PCV Company"}, "name") repost_doc = frappe.get_doc( { "doctype": "Repost Item Valuation", - "company": company, + "company": "Test PCV Company", "posting_date": "2020-03-15", "based_on": "Item and Warehouse", "item_code": "Test Item 1", @@ -340,7 +335,6 @@ class TestPeriodClosingVoucher(ERPNextTestSuite): {"enable_immutable_ledger": 1}, ) def test_immutable_ledger_reverse_entry_uses_passed_posting_date_after_pcv(self): - company = create_company() cost_center = create_cost_center("Test Cost Center 1") jv = make_journal_entry( @@ -349,10 +343,10 @@ class TestPeriodClosingVoucher(ERPNextTestSuite): account1="Cash - TPC", account2="Sales - TPC", cost_center=cost_center, - company=company, + company="Test PCV Company", save=False, ) - jv.company = company + jv.company = "Test PCV Company" jv.save() jv.submit() @@ -378,19 +372,6 @@ class TestPeriodClosingVoucher(ERPNextTestSuite): self.assertEqual(totals_after_cancel.total_debit, totals_after_cancel.total_credit) -def create_company(): - company = frappe.get_doc( - { - "doctype": "Company", - "company_name": "Test PCV Company", - "country": "United States", - "default_currency": "USD", - } - ) - company.insert(ignore_if_duplicate=True) - return company.name - - def create_account(): account = frappe.get_doc( { diff --git a/erpnext/accounts/report/financial_ratios/test_financial_ratios.py b/erpnext/accounts/report/financial_ratios/test_financial_ratios.py new file mode 100644 index 00000000000..ef45fa0f1e6 --- /dev/null +++ b/erpnext/accounts/report/financial_ratios/test_financial_ratios.py @@ -0,0 +1,73 @@ +# Copyright (c) 2023, Frappe Technologies Pvt. Ltd. and Contributors +# MIT License. See license.txt + +import frappe +from frappe.utils import today + +from erpnext.accounts.report.financial_ratios.financial_ratios import execute +from erpnext.tests.utils import ERPNextTestSuite + + +class TestFinancialRatios(ERPNextTestSuite): + def setUp(self): + self.company = "_Test Company" + self.abbr = "_TC" + # The report matches the group accounts by their account_type, which the + # standard chart of accounts does not set on group accounts by default. + self.set_account_type("Fixed Assets", "Fixed Asset") + self.set_account_type("Direct Income", "Direct Income") + + def set_account_type(self, account_name, account_type): + frappe.db.set_value("Account", f"{account_name} - {self.abbr}", "account_type", account_type) + + def test_fixed_asset_turnover_uses_net_fixed_assets(self): + # Acquire a fixed asset worth 10,000 funded by equity. + self.make_journal_entry("Buildings", "Capital Stock", 10000) + # Book sales of 20,000 collected in cash. Total assets now = 30,000 + # (Buildings 10,000 + Cash 20,000), while net fixed assets stay at 10,000. + self.make_journal_entry("Cash", "Sales", 20000) + + columns, data = execute(self.get_report_filters()) + year_key = columns[1]["fieldname"] + ratio_row = next((row for row in data if row.get("ratio") == "Fixed Asset Turnover Ratio"), None) + self.assertIsNotNone(ratio_row, "Fixed Asset Turnover Ratio row not found in report output") + + # Net Sales / Net Fixed Assets = 20,000 / 10,000 = 2.0 + # (the old behaviour divided by total assets, giving 20,000 / 30,000 = 0.667) + self.assertEqual(ratio_row[year_key], 2.0) + + def get_report_filters(self): + active_fy = frappe.db.get_value( + "Fiscal Year", + {"disabled": 0, "year_start_date": ("<=", today()), "year_end_date": (">=", today())}, + ["name", "year_start_date", "year_end_date"], + as_dict=True, + ) + return frappe._dict( + company=self.company, + from_fiscal_year=active_fy.name, + to_fiscal_year=active_fy.name, + period_start_date=active_fy.year_start_date, + period_end_date=active_fy.year_end_date, + filter_based_on="Fiscal Year", + periodicity="Yearly", + ) + + def make_journal_entry(self, debit_account, credit_account, amount): + journal_entry = frappe.new_doc("Journal Entry") + journal_entry.posting_date = today() + journal_entry.company = self.company + for account, debit, credit in ( + (debit_account, amount, 0), + (credit_account, 0, amount), + ): + journal_entry.append( + "accounts", + { + "account": f"{account} - {self.abbr}", + "debit_in_account_currency": debit, + "credit_in_account_currency": credit, + }, + ) + journal_entry.insert() + journal_entry.submit() diff --git a/erpnext/accounts/report/gross_profit/test_gross_profit.py b/erpnext/accounts/report/gross_profit/test_gross_profit.py index 72fef842849..416f2982414 100644 --- a/erpnext/accounts/report/gross_profit/test_gross_profit.py +++ b/erpnext/accounts/report/gross_profit/test_gross_profit.py @@ -14,71 +14,17 @@ from erpnext.tests.utils import ERPNextTestSuite class TestGrossProfit(ERPNextTestSuite): def setUp(self): - self.create_company() - self.create_item() - self.create_bundle() - self.create_customer() - - def create_company(self): - company_name = "_Test Gross Profit" - abbr = "_GP" - if frappe.db.exists("Company", company_name): - company = frappe.get_doc("Company", company_name) - else: - company = frappe.get_doc( - { - "doctype": "Company", - "company_name": company_name, - "country": "India", - "default_currency": "INR", - "create_chart_of_accounts_based_on": "Standard Template", - "chart_of_accounts": "Standard", - } - ) - company = company.save() - - self.company = company.name - self.cost_center = company.cost_center - self.warehouse = "Stores - " + abbr - self.finished_warehouse = "Finished Goods - " + abbr - self.income_account = "Sales - " + abbr - self.expense_account = "Cost of Goods Sold - " + abbr - self.debit_to = "Debtors - " + abbr - self.creditors = "Creditors - " + abbr - - def create_item(self): - item = create_item( - item_code="_Test GP Item", is_stock_item=1, company=self.company, warehouse=self.warehouse - ) - self.item = item if isinstance(item, str) else item.item_code - - def create_bundle(self): - from erpnext.selling.doctype.product_bundle.test_product_bundle import make_product_bundle - - item2 = create_item( - item_code="_Test GP Item 2", is_stock_item=1, company=self.company, warehouse=self.warehouse - ) - self.item2 = item2 if isinstance(item2, str) else item2.item_code - - # This will be parent item - bundle = create_item( - item_code="_Test GP bundle", is_stock_item=0, company=self.company, warehouse=self.warehouse - ) - self.bundle = bundle if isinstance(bundle, str) else bundle.item_code - - # Create Product Bundle - self.product_bundle = make_product_bundle(parent=self.bundle, items=[self.item, self.item2]) - - def create_customer(self): - name = "_Test GP Customer" - if frappe.db.exists("Customer", name): - self.customer = name - else: - customer = frappe.new_doc("Customer") - customer.customer_name = name - customer.type = "Individual" - customer.save() - self.customer = customer.name + self.company = "_Test Company" + self.cost_center = "Main - _TC" + self.warehouse = "Stores - _TC" + self.finished_warehouse = "Finished Goods - _TC" + self.income_account = "Sales - _TC" + self.expense_account = "Cost of Goods Sold - _TC" + self.debit_to = "Debtors - _TC" + self.item = "_Test Item" + self.item2 = "_Test Item Home Desktop 100" + self.bundle = "_Test Product Bundle Item" + self.customer = "_Test Customer" def create_sales_invoice( self, qty=1, rate=100, posting_date=None, do_not_save=False, do_not_submit=False @@ -212,7 +158,7 @@ class TestGrossProfit(ERPNextTestSuite): "posting_date": frappe.utils.datetime.date.fromisoformat(nowdate()), "item_code": self.item, "item_name": self.item, - "warehouse": "Stores - _GP", + "warehouse": "Stores - _TC", "qty": 1.0, "avg._selling_rate": 100.0, "valuation_rate": 150.0, @@ -241,7 +187,7 @@ class TestGrossProfit(ERPNextTestSuite): "posting_date": frappe.utils.datetime.date.fromisoformat(nowdate()), "item_code": self.item, "item_name": self.item, - "warehouse": "Stores - _GP", + "warehouse": "Stores - _TC", "qty": 1.0, "avg._selling_rate": 100.0, "valuation_rate": 100.0, @@ -373,7 +319,7 @@ class TestGrossProfit(ERPNextTestSuite): "posting_date": frappe.utils.datetime.date.fromisoformat(nowdate()), "item_code": self.item, "item_name": self.item, - "warehouse": "Stores - _GP", + "warehouse": "Stores - _TC", "qty": 4.0, "avg._selling_rate": 100.0, "valuation_rate": 125.0, @@ -414,7 +360,7 @@ class TestGrossProfit(ERPNextTestSuite): "posting_date": frappe.utils.datetime.date.fromisoformat(nowdate()), "item_code": self.item, "item_name": self.item, - "warehouse": "Stores - _GP", + "warehouse": "Stores - _TC", "qty": 0.0, "avg._selling_rate": 100, "valuation_rate": 0.0, @@ -460,7 +406,7 @@ class TestGrossProfit(ERPNextTestSuite): "posting_date": frappe.utils.datetime.date.fromisoformat(nowdate()), "item_code": self.item, "item_name": self.item, - "warehouse": "Stores - _GP", + "warehouse": "Stores - _TC", "qty": -1.0, "avg._selling_rate": 100.0, "valuation_rate": 0.0, @@ -553,7 +499,7 @@ class TestGrossProfit(ERPNextTestSuite): "posting_date": frappe.utils.datetime.date.fromisoformat(nowdate()), "item_code": self.item, "item_name": self.item, - "warehouse": "Stores - _GP", + "warehouse": "Stores - _TC", "qty": 4.0, "avg._selling_rate": 800.0, "valuation_rate": 700.0, diff --git a/erpnext/accounts/report/payment_ledger/test_payment_ledger.py b/erpnext/accounts/report/payment_ledger/test_payment_ledger.py index 00206059672..9628e853a13 100644 --- a/erpnext/accounts/report/payment_ledger/test_payment_ledger.py +++ b/erpnext/accounts/report/payment_ledger/test_payment_ledger.py @@ -9,7 +9,12 @@ from erpnext.tests.utils import ERPNextTestSuite class TestPaymentLedger(ERPNextTestSuite): def setUp(self): - self.create_company() + self.company = "_Test Company" + self.cost_center = "Main - _TC" + self.warehouse = "Stores - _TC" + self.income_account = "Sales - _TC" + self.expense_account = "Cost of Goods Sold - _TC" + self.debit_to = "Debtors - _TC" self.cleanup() def cleanup(self): @@ -22,30 +27,6 @@ class TestPaymentLedger(ERPNextTestSuite): for doctype in doctypes: qb.from_(doctype).delete().where(doctype.company == self.company).run() - def create_company(self): - name = "Test Payment Ledger" - company = None - if frappe.db.exists("Company", name): - company = frappe.get_doc("Company", name) - else: - company = frappe.get_doc( - { - "doctype": "Company", - "company_name": name, - "country": "India", - "default_currency": "INR", - "create_chart_of_accounts_based_on": "Standard Template", - "chart_of_accounts": "Standard", - } - ) - company = company.save() - self.company = company.name - self.cost_center = company.cost_center - self.warehouse = "All Warehouses" + " - " + company.abbr - self.income_account = company.default_income_account - self.expense_account = company.default_expense_account - self.debit_to = company.default_receivable_account - def test_unpaid_invoice_outstanding(self): sinv = create_sales_invoice( company=self.company, diff --git a/erpnext/controllers/tests/test_accounts_controller.py b/erpnext/controllers/tests/test_accounts_controller.py index 4fabe83ebfb..10b6ca8edf3 100644 --- a/erpnext/controllers/tests/test_accounts_controller.py +++ b/erpnext/controllers/tests/test_accounts_controller.py @@ -18,39 +18,9 @@ from erpnext.buying.doctype.purchase_order.test_purchase_order import ( prepare_data_for_internal_transfer, ) from erpnext.projects.doctype.project.test_project import make_project -from erpnext.stock.doctype.item.test_item import create_item from erpnext.tests.utils import ERPNextTestSuite -def make_customer(customer_name, currency=None): - if not frappe.db.exists("Customer", customer_name): - customer = frappe.new_doc("Customer") - customer.customer_name = customer_name - customer.customer_type = "Individual" - - if currency: - customer.default_currency = currency - customer.save() - return customer.name - else: - return customer_name - - -def make_supplier(supplier_name, currency=None): - if not frappe.db.exists("Supplier", supplier_name): - supplier = frappe.new_doc("Supplier") - supplier.supplier_name = supplier_name - supplier.supplier_type = "Individual" - supplier.supplier_group = "All Supplier Groups" - - if currency: - supplier.default_currency = currency - supplier.save() - return supplier.name - else: - return supplier_name - - class TestAccountsController(ERPNextTestSuite): """ Test Exchange Gain/Loss booking on various scenarios. @@ -67,79 +37,29 @@ class TestAccountsController(ERPNextTestSuite): """ def setUp(self): - self.create_company() + self.company = "_Test Company" + self.company_abbr = "_TC" + self.cost_center = "Main - _TC" + self.warehouse = "Stores - _TC" + self.finished_warehouse = "Finished Goods - _TC" + self.income_account = "Sales - _TC" + self.expense_account = "Cost of Goods Sold - _TC" + self.debit_to = "Debtors - _TC" + self.debit_usd = "_Test Receivable USD - _TC" + self.debtors_usd = "_Test Receivable USD - _TC" + self.cash = "Cash - _TC" + self.creditors = "Creditors - _TC" + self.creditors_usd = "_Test Payable USD - _TC" + self.item = "_Test Item" + self.customer = "_Test Customer USD" + self.supplier = "_Test Supplier USD" self.create_account() - self.create_item() - self.create_parties() self.clear_old_entries() frappe.flags.is_reverse_depr_entry = False - def create_company(self): - company_name = "_Test Company" - self.company_abbr = abbr = "_TC" - if frappe.db.exists("Company", company_name): - company = frappe.get_doc("Company", company_name) - else: - company = frappe.get_doc( - { - "doctype": "Company", - "company_name": company_name, - "country": "India", - "default_currency": "INR", - "create_chart_of_accounts_based_on": "Standard Template", - "chart_of_accounts": "Standard", - } - ) - company = company.save() - - self.company = company.name - self.cost_center = company.cost_center - self.warehouse = "Stores - " + abbr - self.finished_warehouse = "Finished Goods - " + abbr - self.income_account = "Sales - " + abbr - self.expense_account = "Cost of Goods Sold - " + abbr - self.debit_to = "Debtors - " + abbr - self.debit_usd = "Debtors USD - " + abbr - self.cash = "Cash - " + abbr - self.creditors = "Creditors - " + abbr - - def create_item(self): - item = create_item( - item_code="_Test Notebook", is_stock_item=0, company=self.company, warehouse=self.warehouse - ) - self.item = item if isinstance(item, str) else item.item_code - - def create_parties(self): - self.create_customer() - self.create_supplier() - - def create_customer(self): - self.customer = make_customer("_Test MC Customer USD", "USD") - - def create_supplier(self): - self.supplier = make_supplier("_Test MC Supplier USD", "USD") - def create_account(self): + # Advance accounts are not in persistent test data — create them on demand. accounts = [ - frappe._dict( - { - "attribute_name": "debtors_usd", - "name": "Debtors USD", - "account_type": "Receivable", - "account_currency": "USD", - "parent_account": "Accounts Receivable - " + self.company_abbr, - } - ), - frappe._dict( - { - "attribute_name": "creditors_usd", - "name": "Creditors USD", - "account_type": "Payable", - "account_currency": "USD", - "parent_account": "Accounts Payable - " + self.company_abbr, - } - ), - # Advance accounts under Asset and Liability header frappe._dict( { "attribute_name": "advance_received_usd", @@ -185,6 +105,7 @@ class TestAccountsController(ERPNextTestSuite): company.save() customer = frappe.get_doc("Customer", self.customer) + customer.accounts = [] customer.append( "accounts", { @@ -196,6 +117,7 @@ class TestAccountsController(ERPNextTestSuite): customer.save() supplier = frappe.get_doc("Supplier", self.supplier) + supplier.accounts = [] supplier.append( "accounts", { diff --git a/erpnext/regional/report/vat_audit_report/test_vat_audit_report.py b/erpnext/regional/report/vat_audit_report/test_vat_audit_report.py index ded1bde3133..897692df59b 100644 --- a/erpnext/regional/report/vat_audit_report/test_vat_audit_report.py +++ b/erpnext/regional/report/vat_audit_report/test_vat_audit_report.py @@ -16,7 +16,7 @@ from erpnext.tests.utils import ERPNextTestSuite class TestVATAuditReport(ERPNextTestSuite): def setUp(self): - make_company("_Test Company SA VAT", "_TCSV") + self.company = "_Test Company SA VAT" create_account( account_name="VAT - 0%", diff --git a/erpnext/setup/doctype/company/test_records.json b/erpnext/setup/doctype/company/test_records.json index 74615e60162..d3faeec4672 100644 --- a/erpnext/setup/doctype/company/test_records.json +++ b/erpnext/setup/doctype/company/test_records.json @@ -214,5 +214,14 @@ "doctype": "Company", "chart_of_accounts": "Standard", "create_chart_of_accounts_based_on": "Standard Template" + }, + { + "abbr": "TPC", + "company_name": "Test PCV Company", + "country": "United States", + "default_currency": "USD", + "doctype": "Company", + "chart_of_accounts": "Standard", + "create_chart_of_accounts_based_on": "Standard Template" } ]