chore: Tests for Purchase Receipt
This commit is contained in:
parent
0411026ad6
commit
e35fd5e305
@ -1098,7 +1098,7 @@
|
|||||||
"no_copy": 1,
|
"no_copy": 1,
|
||||||
"oldfieldname": "status",
|
"oldfieldname": "status",
|
||||||
"oldfieldtype": "Select",
|
"oldfieldtype": "Select",
|
||||||
"options": "\nDraft\nTo Bill\nCompleted\nCancelled\nClosed",
|
"options": "\nDraft\nTo Bill\nCompleted\nReturn Issued\nCancelled\nClosed",
|
||||||
"print_hide": 1,
|
"print_hide": 1,
|
||||||
"print_width": "150px",
|
"print_width": "150px",
|
||||||
"read_only": 1,
|
"read_only": 1,
|
||||||
@ -1266,7 +1266,7 @@
|
|||||||
"idx": 146,
|
"idx": 146,
|
||||||
"is_submittable": 1,
|
"is_submittable": 1,
|
||||||
"links": [],
|
"links": [],
|
||||||
"modified": "2020-08-03 23:18:47.739997",
|
"modified": "2020-09-08 11:22:09.056684",
|
||||||
"modified_by": "Administrator",
|
"modified_by": "Administrator",
|
||||||
"module": "Stock",
|
"module": "Stock",
|
||||||
"name": "Delivery Note",
|
"name": "Delivery Note",
|
||||||
|
@ -895,7 +895,7 @@
|
|||||||
"no_copy": 1,
|
"no_copy": 1,
|
||||||
"oldfieldname": "status",
|
"oldfieldname": "status",
|
||||||
"oldfieldtype": "Select",
|
"oldfieldtype": "Select",
|
||||||
"options": "\nDraft\nTo Bill\nCompleted\nCancelled\nClosed",
|
"options": "\nDraft\nTo Bill\nCompleted\nReturn Issued\nCancelled\nClosed",
|
||||||
"print_hide": 1,
|
"print_hide": 1,
|
||||||
"print_width": "150px",
|
"print_width": "150px",
|
||||||
"read_only": 1,
|
"read_only": 1,
|
||||||
@ -1120,7 +1120,7 @@
|
|||||||
"idx": 261,
|
"idx": 261,
|
||||||
"is_submittable": 1,
|
"is_submittable": 1,
|
||||||
"links": [],
|
"links": [],
|
||||||
"modified": "2020-08-03 23:20:26.381024",
|
"modified": "2020-09-08 11:21:25.465966",
|
||||||
"modified_by": "Administrator",
|
"modified_by": "Administrator",
|
||||||
"module": "Stock",
|
"module": "Stock",
|
||||||
"name": "Purchase Receipt",
|
"name": "Purchase Receipt",
|
||||||
|
@ -255,11 +255,13 @@ class TestPurchaseReceipt(unittest.TestCase):
|
|||||||
self.assertEqual(frappe.db.get_value("Serial No", serial_no, "warehouse"),
|
self.assertEqual(frappe.db.get_value("Serial No", serial_no, "warehouse"),
|
||||||
pr.get("items")[0].rejected_warehouse)
|
pr.get("items")[0].rejected_warehouse)
|
||||||
|
|
||||||
def test_purchase_return(self):
|
def test_purchase_return_partial(self):
|
||||||
|
|
||||||
pr = make_purchase_receipt(company="_Test Company with perpetual inventory", warehouse = "Stores - TCP1", supplier_warehouse = "Work in Progress - TCP1")
|
pr = make_purchase_receipt(company="_Test Company with perpetual inventory", warehouse = "Stores - TCP1", supplier_warehouse = "Work in Progress - TCP1")
|
||||||
|
|
||||||
return_pr = make_purchase_receipt(company="_Test Company with perpetual inventory", warehouse = "Stores - TCP1", supplier_warehouse = "Work in Progress - TCP1", is_return=1, return_against=pr.name, qty=-2)
|
return_pr = make_purchase_receipt(company="_Test Company with perpetual inventory", warehouse = "Stores - TCP1", supplier_warehouse = "Work in Progress - TCP1", is_return=1, return_against=pr.name, qty=-2, do_not_submit=1)
|
||||||
|
return_pr.items[0].purchase_receipt_item = pr.items[0].name
|
||||||
|
return_pr.submit()
|
||||||
|
|
||||||
# check sle
|
# check sle
|
||||||
outgoing_rate = frappe.db.get_value("Stock Ledger Entry", {"voucher_type": "Purchase Receipt",
|
outgoing_rate = frappe.db.get_value("Stock Ledger Entry", {"voucher_type": "Purchase Receipt",
|
||||||
@ -283,6 +285,31 @@ class TestPurchaseReceipt(unittest.TestCase):
|
|||||||
self.assertEqual(expected_values[gle.account][0], gle.debit)
|
self.assertEqual(expected_values[gle.account][0], gle.debit)
|
||||||
self.assertEqual(expected_values[gle.account][1], gle.credit)
|
self.assertEqual(expected_values[gle.account][1], gle.credit)
|
||||||
|
|
||||||
|
# hack because new_doc isn't considering is_return portion of status_updater
|
||||||
|
returned = frappe.get_doc("Purchase Receipt", return_pr.name)
|
||||||
|
returned.update_prevdoc_status()
|
||||||
|
pr.load_from_db()
|
||||||
|
|
||||||
|
# Check if Original PR updated
|
||||||
|
self.assertEqual(pr.items[0].returned_qty, 2)
|
||||||
|
self.assertEqual(pr.per_returned, 40)
|
||||||
|
|
||||||
|
def test_purchase_return_full(self):
|
||||||
|
pr = make_purchase_receipt(company="_Test Company with perpetual inventory", warehouse = "Stores - TCP1", supplier_warehouse = "Work in Progress - TCP1")
|
||||||
|
|
||||||
|
return_pr = make_purchase_receipt(company="_Test Company with perpetual inventory", warehouse = "Stores - TCP1", supplier_warehouse = "Work in Progress - TCP1", is_return=1, return_against=pr.name, qty=-5, do_not_submit=1)
|
||||||
|
return_pr.items[0].purchase_receipt_item = pr.items[0].name
|
||||||
|
return_pr.submit()
|
||||||
|
|
||||||
|
# hack because new_doc isn't considering is_return portion of status_updater
|
||||||
|
returned = frappe.get_doc("Purchase Receipt", return_pr.name)
|
||||||
|
returned.update_prevdoc_status()
|
||||||
|
pr.load_from_db()
|
||||||
|
|
||||||
|
# Check if Original PR updated
|
||||||
|
self.assertEqual(pr.items[0].returned_qty, 5)
|
||||||
|
self.assertEqual(pr.per_returned, 100)
|
||||||
|
self.assertEqual(pr.status, 'Return Issued')
|
||||||
|
|
||||||
def test_purchase_return_for_rejected_qty(self):
|
def test_purchase_return_for_rejected_qty(self):
|
||||||
from erpnext.stock.doctype.warehouse.test_warehouse import get_warehouse
|
from erpnext.stock.doctype.warehouse.test_warehouse import get_warehouse
|
||||||
|
Loading…
x
Reference in New Issue
Block a user