test: test case to check removed attached file
(cherry picked from commit 76b57a433886c3a38b9c4883db8652aabe8d32d8)
This commit is contained in:
parent
b582e9c42c
commit
7833138c57
@ -319,7 +319,7 @@ def remove_attached_file(docname):
|
||||
if file_name := frappe.db.get_value(
|
||||
"File", {"attached_to_name": docname, "attached_to_doctype": "Repost Item Valuation"}, "name"
|
||||
):
|
||||
frappe.delete_doc("File", file_name)
|
||||
frappe.delete_doc("File", file_name, delete_permanently=True)
|
||||
|
||||
|
||||
def repost_sl_entries(doc):
|
||||
|
@ -424,3 +424,38 @@ class TestRepostItemValuation(FrappeTestCase, StockTestMixin):
|
||||
|
||||
self.assertRaises(frappe.ValidationError, riv.save)
|
||||
doc.cancel()
|
||||
|
||||
def test_remove_attached_file(self):
|
||||
item_code = make_item("_Test Remove Attached File Item", properties={"is_stock_item": 1})
|
||||
|
||||
make_purchase_receipt(
|
||||
item_code=item_code,
|
||||
qty=1,
|
||||
rate=100,
|
||||
)
|
||||
|
||||
pr1 = make_purchase_receipt(
|
||||
item_code=item_code,
|
||||
qty=1,
|
||||
rate=100,
|
||||
posting_date=add_days(today(), days=-1),
|
||||
)
|
||||
|
||||
if docname := frappe.db.exists("Repost Item Valuation", {"voucher_no": pr1.name}):
|
||||
self.assertFalse(
|
||||
frappe.db.get_value(
|
||||
"File",
|
||||
{"attached_to_doctype": "Repost Item Valuation", "attached_to_name": docname},
|
||||
"name",
|
||||
)
|
||||
)
|
||||
else:
|
||||
repost_entries = create_item_wise_repost_entries(pr1.doctype, pr1.name)
|
||||
for entry in repost_entries:
|
||||
self.assertFalse(
|
||||
frappe.db.get_value(
|
||||
"File",
|
||||
{"attached_to_doctype": "Repost Item Valuation", "attached_to_name": entry.name},
|
||||
"name",
|
||||
)
|
||||
)
|
||||
|
Loading…
x
Reference in New Issue
Block a user