mirror of
https://github.com/frappe/erpnext.git
synced 2026-08-14 15:11:52 +00:00
test: add Payment Entry GL characterization snapshots
Extend the Phase-0 GL safety net with five representative Payment Entry scenarios (receive against SI, pay against PI, with deductions, with taxes, multi-currency) ahead of moving PE onto the composer.
This commit is contained in:
Nabin Hait
30
erpnext/accounts/gl_snapshots/pe_multi_currency.json
Normal file
30
erpnext/accounts/gl_snapshots/pe_multi_currency.json
Normal file
@@ -0,0 +1,30 @@
|
||||
[
|
||||
{
|
||||
"account": "_Test Bank - _TC",
|
||||
"account_currency": "INR",
|
||||
"against": "_Test Supplier USD",
|
||||
"cost_center": null,
|
||||
"credit": 1000.0,
|
||||
"credit_in_account_currency": 1000.0,
|
||||
"debit": 0.0,
|
||||
"debit_in_account_currency": 0.0,
|
||||
"is_opening": "No",
|
||||
"party": null,
|
||||
"party_type": null,
|
||||
"posting_date": "2024-01-15"
|
||||
},
|
||||
{
|
||||
"account": "_Test Payable USD - _TC",
|
||||
"account_currency": "USD",
|
||||
"against": "_Test Bank - _TC",
|
||||
"cost_center": null,
|
||||
"credit": 0.0,
|
||||
"credit_in_account_currency": 0.0,
|
||||
"debit": 1000.0,
|
||||
"debit_in_account_currency": 12.5,
|
||||
"is_opening": "No",
|
||||
"party": "_Test Supplier USD",
|
||||
"party_type": "Supplier",
|
||||
"posting_date": "2024-01-15"
|
||||
}
|
||||
]
|
||||
30
erpnext/accounts/gl_snapshots/pe_pay_against_pi.json
Normal file
30
erpnext/accounts/gl_snapshots/pe_pay_against_pi.json
Normal file
@@ -0,0 +1,30 @@
|
||||
[
|
||||
{
|
||||
"account": "Creditors - _TC",
|
||||
"account_currency": "INR",
|
||||
"against": "_Test Bank - _TC",
|
||||
"cost_center": null,
|
||||
"credit": 0.0,
|
||||
"credit_in_account_currency": 0.0,
|
||||
"debit": 250.0,
|
||||
"debit_in_account_currency": 250.0,
|
||||
"is_opening": "No",
|
||||
"party": "_Test Supplier",
|
||||
"party_type": "Supplier",
|
||||
"posting_date": "2024-01-15"
|
||||
},
|
||||
{
|
||||
"account": "_Test Bank - _TC",
|
||||
"account_currency": "INR",
|
||||
"against": "_Test Supplier",
|
||||
"cost_center": null,
|
||||
"credit": 250.0,
|
||||
"credit_in_account_currency": 250.0,
|
||||
"debit": 0.0,
|
||||
"debit_in_account_currency": 0.0,
|
||||
"is_opening": "No",
|
||||
"party": null,
|
||||
"party_type": null,
|
||||
"posting_date": "2024-01-15"
|
||||
}
|
||||
]
|
||||
30
erpnext/accounts/gl_snapshots/pe_receive_against_si.json
Normal file
30
erpnext/accounts/gl_snapshots/pe_receive_against_si.json
Normal file
@@ -0,0 +1,30 @@
|
||||
[
|
||||
{
|
||||
"account": "Debtors - _TC",
|
||||
"account_currency": "INR",
|
||||
"against": "_Test Cash - _TC",
|
||||
"cost_center": null,
|
||||
"credit": 1000.0,
|
||||
"credit_in_account_currency": 1000.0,
|
||||
"debit": 0.0,
|
||||
"debit_in_account_currency": 0.0,
|
||||
"is_opening": "No",
|
||||
"party": "_Test Customer",
|
||||
"party_type": "Customer",
|
||||
"posting_date": "2024-01-15"
|
||||
},
|
||||
{
|
||||
"account": "_Test Cash - _TC",
|
||||
"account_currency": "INR",
|
||||
"against": "_Test Customer",
|
||||
"cost_center": null,
|
||||
"credit": 0.0,
|
||||
"credit_in_account_currency": 0.0,
|
||||
"debit": 1000.0,
|
||||
"debit_in_account_currency": 1000.0,
|
||||
"is_opening": "No",
|
||||
"party": null,
|
||||
"party_type": null,
|
||||
"posting_date": "2024-01-15"
|
||||
}
|
||||
]
|
||||
58
erpnext/accounts/gl_snapshots/pe_with_deductions.json
Normal file
58
erpnext/accounts/gl_snapshots/pe_with_deductions.json
Normal file
@@ -0,0 +1,58 @@
|
||||
[
|
||||
{
|
||||
"account": "Debtors - _TC",
|
||||
"account_currency": "INR",
|
||||
"against": "_Test Cash - _TC",
|
||||
"cost_center": null,
|
||||
"credit": 1000.0,
|
||||
"credit_in_account_currency": 1000.0,
|
||||
"debit": 0.0,
|
||||
"debit_in_account_currency": 0.0,
|
||||
"is_opening": "No",
|
||||
"party": "_Test Customer",
|
||||
"party_type": "Customer",
|
||||
"posting_date": "2024-01-15"
|
||||
},
|
||||
{
|
||||
"account": "Debtors - _TC",
|
||||
"account_currency": "INR",
|
||||
"against": "_Test Cash - _TC",
|
||||
"cost_center": null,
|
||||
"credit": 50.0,
|
||||
"credit_in_account_currency": 50.0,
|
||||
"debit": 0.0,
|
||||
"debit_in_account_currency": 0.0,
|
||||
"is_opening": "No",
|
||||
"party": "_Test Customer",
|
||||
"party_type": "Customer",
|
||||
"posting_date": "2024-01-15"
|
||||
},
|
||||
{
|
||||
"account": "Write Off - _TC",
|
||||
"account_currency": "INR",
|
||||
"against": "_Test Customer",
|
||||
"cost_center": "_Test Cost Center - _TC",
|
||||
"credit": 0.0,
|
||||
"credit_in_account_currency": 0.0,
|
||||
"debit": 50.0,
|
||||
"debit_in_account_currency": 50.0,
|
||||
"is_opening": "No",
|
||||
"party": null,
|
||||
"party_type": null,
|
||||
"posting_date": "2024-01-15"
|
||||
},
|
||||
{
|
||||
"account": "_Test Cash - _TC",
|
||||
"account_currency": "INR",
|
||||
"against": "_Test Customer",
|
||||
"cost_center": null,
|
||||
"credit": 0.0,
|
||||
"credit_in_account_currency": 0.0,
|
||||
"debit": 1000.0,
|
||||
"debit_in_account_currency": 1000.0,
|
||||
"is_opening": "No",
|
||||
"party": null,
|
||||
"party_type": null,
|
||||
"posting_date": "2024-01-15"
|
||||
}
|
||||
]
|
||||
44
erpnext/accounts/gl_snapshots/pe_with_taxes.json
Normal file
44
erpnext/accounts/gl_snapshots/pe_with_taxes.json
Normal file
@@ -0,0 +1,44 @@
|
||||
[
|
||||
{
|
||||
"account": "Creditors - _TC",
|
||||
"account_currency": "INR",
|
||||
"against": "_Test Bank - _TC",
|
||||
"cost_center": null,
|
||||
"credit": 0.0,
|
||||
"credit_in_account_currency": 0.0,
|
||||
"debit": 1000.0,
|
||||
"debit_in_account_currency": 1000.0,
|
||||
"is_opening": "No",
|
||||
"party": "_Test Supplier",
|
||||
"party_type": "Supplier",
|
||||
"posting_date": "2024-01-15"
|
||||
},
|
||||
{
|
||||
"account": "_Test Account Service Tax - _TC",
|
||||
"account_currency": "INR",
|
||||
"against": "_Test Supplier",
|
||||
"cost_center": "_Test Cost Center - _TC",
|
||||
"credit": 0.0,
|
||||
"credit_in_account_currency": 0.0,
|
||||
"debit": 100.0,
|
||||
"debit_in_account_currency": 100.0,
|
||||
"is_opening": "No",
|
||||
"party": null,
|
||||
"party_type": null,
|
||||
"posting_date": "2024-01-15"
|
||||
},
|
||||
{
|
||||
"account": "_Test Bank - _TC",
|
||||
"account_currency": "INR",
|
||||
"against": "_Test Supplier",
|
||||
"cost_center": null,
|
||||
"credit": 1100.0,
|
||||
"credit_in_account_currency": 1100.0,
|
||||
"debit": 0.0,
|
||||
"debit_in_account_currency": 0.0,
|
||||
"is_opening": "No",
|
||||
"party": null,
|
||||
"party_type": null,
|
||||
"posting_date": "2024-01-15"
|
||||
}
|
||||
]
|
||||
@@ -20,6 +20,7 @@ from erpnext.accounts.doctype.account.test_account import create_account
|
||||
from erpnext.accounts.doctype.mode_of_payment.test_mode_of_payment import (
|
||||
set_default_account_for_mode_of_payment,
|
||||
)
|
||||
from erpnext.accounts.doctype.payment_entry.payment_entry import get_payment_entry
|
||||
from erpnext.accounts.doctype.purchase_invoice.purchase_invoice import make_debit_note
|
||||
from erpnext.accounts.doctype.purchase_invoice.test_purchase_invoice import make_purchase_invoice
|
||||
from erpnext.accounts.doctype.sales_invoice.sales_invoice import make_sales_return
|
||||
@@ -40,6 +41,30 @@ def make_dated_purchase_invoice(**args):
|
||||
return pi
|
||||
|
||||
|
||||
def make_dated_payment_entry(**args):
|
||||
"""Standalone Payment Entry (no invoice reference) on a fixed posting date.
|
||||
|
||||
Mirrors test_payment_entry.create_payment_entry without importing that test
|
||||
module, whose import drags in test-record dependencies that conflict during
|
||||
discovery."""
|
||||
pe = frappe.new_doc("Payment Entry")
|
||||
pe.company = COMPANY
|
||||
pe.payment_type = args.get("payment_type") or "Pay"
|
||||
pe.party_type = args.get("party_type") or "Supplier"
|
||||
pe.party = args.get("party") or "_Test Supplier"
|
||||
pe.paid_from = args.get("paid_from") or "_Test Bank - _TC"
|
||||
pe.paid_to = args.get("paid_to") or "Creditors - _TC"
|
||||
pe.paid_amount = args.get("paid_amount") or 1000
|
||||
pe.setup_party_account_field()
|
||||
pe.set_missing_values()
|
||||
pe.set_exchange_rate()
|
||||
pe.received_amount = pe.paid_amount / pe.target_exchange_rate
|
||||
pe.reference_no = "Test001"
|
||||
pe.posting_date = POSTING_DATE
|
||||
pe.reference_date = POSTING_DATE
|
||||
return pe
|
||||
|
||||
|
||||
class TestGLCharacterization(IntegrationTestCase):
|
||||
@classmethod
|
||||
def setUpClass(cls):
|
||||
@@ -187,3 +212,70 @@ class TestGLCharacterization(IntegrationTestCase):
|
||||
debit_note.insert()
|
||||
debit_note.submit()
|
||||
assert_gl_snapshot(self, "pi_return", "Purchase Invoice", debit_note.name)
|
||||
|
||||
def test_pe_receive_against_si(self):
|
||||
si = create_sales_invoice(posting_date=POSTING_DATE, qty=10, rate=100)
|
||||
pe = get_payment_entry("Sales Invoice", si.name, bank_account="_Test Cash - _TC")
|
||||
pe.posting_date = POSTING_DATE
|
||||
pe.reference_no = "PE-REC-1"
|
||||
pe.reference_date = POSTING_DATE
|
||||
pe.insert()
|
||||
pe.submit()
|
||||
assert_gl_snapshot(self, "pe_receive_against_si", "Payment Entry", pe.name)
|
||||
|
||||
def test_pe_pay_against_pi(self):
|
||||
pi = make_dated_purchase_invoice(qty=5, rate=50)
|
||||
pi.insert()
|
||||
pi.submit()
|
||||
pe = get_payment_entry("Purchase Invoice", pi.name, bank_account="_Test Bank - _TC")
|
||||
pe.posting_date = POSTING_DATE
|
||||
pe.reference_no = "PE-PAY-1"
|
||||
pe.reference_date = POSTING_DATE
|
||||
pe.insert()
|
||||
pe.submit()
|
||||
assert_gl_snapshot(self, "pe_pay_against_pi", "Payment Entry", pe.name)
|
||||
|
||||
def test_pe_with_deductions(self):
|
||||
si = create_sales_invoice(posting_date=POSTING_DATE, qty=10, rate=100)
|
||||
pe = get_payment_entry("Sales Invoice", si.name, bank_account="_Test Cash - _TC")
|
||||
pe.posting_date = POSTING_DATE
|
||||
pe.reference_no = "PE-DED-1"
|
||||
pe.reference_date = POSTING_DATE
|
||||
pe.received_amount = pe.received_amount - 50
|
||||
pe.append(
|
||||
"deductions",
|
||||
{
|
||||
"account": "Write Off - _TC",
|
||||
"cost_center": "_Test Cost Center - _TC",
|
||||
"amount": 50,
|
||||
},
|
||||
)
|
||||
pe.insert()
|
||||
pe.submit()
|
||||
assert_gl_snapshot(self, "pe_with_deductions", "Payment Entry", pe.name)
|
||||
|
||||
def test_pe_with_taxes(self):
|
||||
frappe.db.set_single_value("Accounts Settings", "merge_similar_account_heads", 1)
|
||||
pe = make_dated_payment_entry(party="_Test Supplier", paid_to="Creditors - _TC")
|
||||
pe.append(
|
||||
"taxes",
|
||||
{
|
||||
"account_head": "_Test Account Service Tax - _TC",
|
||||
"charge_type": "Actual",
|
||||
"tax_amount": 100,
|
||||
"add_deduct_tax": "Add",
|
||||
"description": "Service Tax",
|
||||
"cost_center": "_Test Cost Center - _TC",
|
||||
},
|
||||
)
|
||||
pe.save()
|
||||
pe.submit()
|
||||
assert_gl_snapshot(self, "pe_with_taxes", "Payment Entry", pe.name)
|
||||
|
||||
def test_pe_multi_currency(self):
|
||||
pe = make_dated_payment_entry(party="_Test Supplier USD", paid_to="_Test Payable USD - _TC")
|
||||
pe.target_exchange_rate = 80
|
||||
pe.received_amount = pe.paid_amount / pe.target_exchange_rate
|
||||
pe.save()
|
||||
pe.submit()
|
||||
assert_gl_snapshot(self, "pe_multi_currency", "Payment Entry", pe.name)
|
||||
|
||||
Reference in New Issue
Block a user