nsinnovations/erpnext
2
0
Fork 0
mirror of https://github.com/frappe/erpnext.git synced 2026-06-07 07:02:54 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: update cost center tests to use dynamic expense account

Browse Source 
Existing tests hardcoded "Cost of Goods Sold" as expected GL account,
but SDBNB overrides it on DN submission. Use dn.items[0].expense_account
to work with both SDBNB-enabled and legacy companies.
This commit is contained in:
kavin-114
2026-03-27 15:19:50 +05:30
parent 6ee7dc0b49
commit 78993c1ebe
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
erpnext/stock/doctype/delivery_note/test_delivery_note.py
View File

@@ -1224,7 +1224,7 @@ class TestDeliveryNote(ERPNextTestSuite):
self.assertTrue(gl_entries) self.assertTrue(gl_entries)
expected_values = { expected_values = {
"Cost of Goods Sold - TCP1": {"cost_center": cost_center}, dn.items[0].expense_account: {"cost_center": cost_center},
stock_in_hand_account: {"cost_center": cost_center}, stock_in_hand_account: {"cost_center": cost_center},
} }
for _i, gle in enumerate(gl_entries): for _i, gle in enumerate(gl_entries):
@@ -1253,7 +1253,7 @@ class TestDeliveryNote(ERPNextTestSuite):
self.assertTrue(gl_entries) self.assertTrue(gl_entries)
expected_values = { expected_values = {
"Cost of Goods Sold - TCP1": {"cost_center": cost_center}, dn.items[0].expense_account: {"cost_center": cost_center},
stock_in_hand_account: {"cost_center": cost_center}, stock_in_hand_account: {"cost_center": cost_center},
} }
for _i, gle in enumerate(gl_entries): for _i, gle in enumerate(gl_entries):