nsinnovations/erpnext
2
0
Fork 0
mirror of https://github.com/frappe/erpnext.git synced 2026-08-10 21:21:46 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #56767 from frappe/chore/incorrect-balance-qty-negative-case

Browse Source 
test: cover inconsistent balance detection in Incorrect Balance Qty report
This commit is contained in:
Nabin Hait
2026-07-02 14:57:08 +05:30
committed by GitHub
parent b4d3a879d2 27f5235e67
commit cabdb7417d
1 changed files with 25 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

25
erpnext/stock/report/incorrect_balance_qty_after_transaction/test_incorrect_balance_qty_after_transaction.py
View File

@@ -3,6 +3,7 @@
import frappe
from erpnext.stock.doctype.item.test_item import make_item
from erpnext.stock.doctype.stock_entry.stock_entry_utils import make_stock_entry
from erpnext.stock.report.incorrect_balance_qty_after_transaction.incorrect_balance_qty_after_transaction import (
execute,
@@ -28,6 +29,30 @@ class TestIncorrectBalanceQtyAfterTransaction(ERPNextTestSuite):
flagged = [row for row in data if row.get("item_code") == item]
self.assertEqual(flagged, [])
def test_inconsistent_balance_qty_is_flagged(self):
# a unique item keeps this SLE the only ledger entry for the item/warehouse
item = make_item(properties={"is_stock_item": 1}).name
entry = make_stock_entry(
item_code=item, to_warehouse=WAREHOUSE, qty=10, rate=100, posting_date="2026-06-01"
)
# Corrupt the running balance so it no longer matches the cumulative actual_qty --
# exactly the inconsistency this report exists to detect. set_value bypasses the
# ledger recompute that would otherwise keep the two in sync.
sle_name = frappe.db.get_value(
"Stock Ledger Entry",
{"voucher_no": entry.name, "item_code": item, "warehouse": WAREHOUSE},
"name",
)
frappe.db.set_value("Stock Ledger Entry", sle_name, "qty_after_transaction", 5)
flagged = [row for row in self.run_report(item_code=item) if row.get("name") == sle_name]
self.assertEqual(len(flagged), 1, "The tampered stock ledger entry should be flagged")
row = flagged[0]
self.assertEqual(row["expected_balance_qty"], 10) # cumulative actual_qty
self.assertEqual(row["qty_after_transaction"], 5) # tampered balance
self.assertEqual(row["differnce"], 5)
def test_sequence_of_movements_not_flagged(self):
item = "_Test Item 2"
make_stock_entry(item_code=item, to_warehouse=WAREHOUSE, qty=20, rate=50, posting_date="2026-06-01")