From a8aa73baae601f2caab8b8c78ac14b09a3801a6b Mon Sep 17 00:00:00 2001 From: ruthra kumar Date: Wed, 17 Jun 2026 08:23:31 +0530 Subject: [PATCH] refactor(test): remove even more dead code (cherry picked from commit 3d109571ee98de7768c3e94317d5ce23c250dad0) --- .../test_payment_ledger_entry.py | 13 ------------- .../report/payment_ledger/test_payment_ledger.py | 11 ----------- .../controllers/tests/test_accounts_controller.py | 13 ------------- 3 files changed, 37 deletions(-) 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 c30d2a80488..1262a354b46 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 @@ -26,7 +26,6 @@ class TestPaymentLedgerEntry(ERPNextTestSuite): 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 @@ -99,18 +98,6 @@ class TestPaymentLedgerEntry(ERPNextTestSuite): ) return so - def clear_old_entries(self): - doctype_list = [ - "GL Entry", - "Payment Ledger Entry", - "Sales Invoice", - "Purchase Invoice", - "Payment Entry", - "Journal Entry", - ] - for doctype in doctype_list: - qb.from_(qb.DocType(doctype)).delete().where(qb.DocType(doctype).company == self.company).run() - def create_journal_entry(self, acc1=None, acc2=None, amount=0, posting_date=None, cost_center=None): je = frappe.new_doc("Journal Entry") je.posting_date = posting_date or nowdate() diff --git a/erpnext/accounts/report/payment_ledger/test_payment_ledger.py b/erpnext/accounts/report/payment_ledger/test_payment_ledger.py index 9628e853a13..79a36c95f47 100644 --- a/erpnext/accounts/report/payment_ledger/test_payment_ledger.py +++ b/erpnext/accounts/report/payment_ledger/test_payment_ledger.py @@ -15,17 +15,6 @@ class TestPaymentLedger(ERPNextTestSuite): self.income_account = "Sales - _TC" self.expense_account = "Cost of Goods Sold - _TC" self.debit_to = "Debtors - _TC" - self.cleanup() - - def cleanup(self): - doctypes = [] - doctypes.append(qb.DocType("GL Entry")) - doctypes.append(qb.DocType("Payment Ledger Entry")) - doctypes.append(qb.DocType("Sales Invoice")) - doctypes.append(qb.DocType("Payment Entry")) - - for doctype in doctypes: - qb.from_(doctype).delete().where(doctype.company == self.company).run() def test_unpaid_invoice_outstanding(self): sinv = create_sales_invoice( diff --git a/erpnext/controllers/tests/test_accounts_controller.py b/erpnext/controllers/tests/test_accounts_controller.py index 10b6ca8edf3..3400578d307 100644 --- a/erpnext/controllers/tests/test_accounts_controller.py +++ b/erpnext/controllers/tests/test_accounts_controller.py @@ -54,7 +54,6 @@ class TestAccountsController(ERPNextTestSuite): self.customer = "_Test Customer USD" self.supplier = "_Test Supplier USD" self.create_account() - self.clear_old_entries() frappe.flags.is_reverse_depr_entry = False def create_account(self): @@ -243,18 +242,6 @@ class TestAccountsController(ERPNextTestSuite): pinv.submit() return pinv - def clear_old_entries(self): - doctype_list = [ - "GL Entry", - "Payment Ledger Entry", - "Sales Invoice", - "Purchase Invoice", - "Payment Entry", - "Journal Entry", - ] - for doctype in doctype_list: - qb.from_(qb.DocType(doctype)).delete().where(qb.DocType(doctype).company == self.company).run() - def create_payment_reconciliation(self): pr = frappe.new_doc("Payment Reconciliation") pr.company = self.company