mirror of
https://github.com/frappe/erpnext.git
synced 2026-09-09 23:39:28 +00:00
fix(stock): use correct field when reading previous stock closing balance (backport #55898) (#58854)
Co-authored-by: Nabin Hait <nabinhait@gmail.com> Fixes #54819
This commit is contained in:
committed by
GitHub
parent
596e086e1c
commit
70f391d5e7
@@ -331,7 +331,7 @@ class StockClosing:
|
||||
],
|
||||
filters={
|
||||
"company": self.company,
|
||||
"closing_stock_balance": self.last_closing_balance.name,
|
||||
"stock_closing_entry": self.last_closing_balance.name,
|
||||
},
|
||||
)
|
||||
|
||||
|
||||
@@ -5,9 +5,11 @@ from unittest.mock import patch
|
||||
|
||||
import frappe
|
||||
from frappe.core.doctype.user_permission.test_user_permission import create_user
|
||||
from frappe.utils import today
|
||||
from frappe.utils import add_days, today
|
||||
|
||||
from erpnext.stock.doctype.item.test_item import make_item
|
||||
from erpnext.stock.doctype.stock_closing_entry.stock_closing_entry import StockClosing
|
||||
from erpnext.stock.doctype.stock_entry.stock_entry_utils import make_stock_entry
|
||||
from erpnext.tests.utils import ERPNextTestSuite
|
||||
|
||||
# On ERPNextTestSuite, the doctype test records and all
|
||||
@@ -24,6 +26,27 @@ class TestStockClosingEntry(ERPNextTestSuite):
|
||||
Use this class for testing interactions between multiple components.
|
||||
"""
|
||||
|
||||
def test_closing_entry_reads_previous_closing_balance(self):
|
||||
item = make_item(properties={"is_stock_item": 1}).name
|
||||
first_date = add_days(today(), -10)
|
||||
self.make_stock_closing_entry(first_date, first_date)
|
||||
|
||||
second_from_date = add_days(first_date, 1)
|
||||
make_stock_entry(
|
||||
item_code=item,
|
||||
to_warehouse=WAREHOUSE,
|
||||
qty=10,
|
||||
rate=100,
|
||||
posting_date=second_from_date,
|
||||
company=COMPANY,
|
||||
)
|
||||
|
||||
closing = StockClosing(COMPANY, second_from_date, add_days(second_from_date, 1))
|
||||
entries = closing.get_sle_entries()
|
||||
|
||||
self.assertEqual(closing.last_closing_balance.name, self.last_closing_entry)
|
||||
self.assertIn(item, {row.item_code for row in entries})
|
||||
|
||||
def make_stock_closing_entry(self, from_date, to_date):
|
||||
entry = frappe.get_doc(
|
||||
doctype="Stock Closing Entry",
|
||||
|
||||
Reference in New Issue
Block a user