added test cases
This commit is contained in:
parent
560ba391f9
commit
cc1c7ad2df
@ -125,7 +125,7 @@ class TestPurchaseReceipt(unittest.TestCase):
|
|||||||
pr = make_purchase_receipt()
|
pr = make_purchase_receipt()
|
||||||
|
|
||||||
return_pr = make_purchase_receipt(is_return=1, return_against=pr.name, qty=-2)
|
return_pr = make_purchase_receipt(is_return=1, return_against=pr.name, qty=-2)
|
||||||
|
|
||||||
# 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",
|
||||||
"voucher_no": return_pr.name}, "outgoing_rate")
|
"voucher_no": return_pr.name}, "outgoing_rate")
|
||||||
@ -148,7 +148,21 @@ class TestPurchaseReceipt(unittest.TestCase):
|
|||||||
self.assertEquals(expected_values[gle.account][1], gle.credit)
|
self.assertEquals(expected_values[gle.account][1], gle.credit)
|
||||||
|
|
||||||
set_perpetual_inventory(0)
|
set_perpetual_inventory(0)
|
||||||
|
|
||||||
|
def test_purchase_return_for_rejected_qty(self):
|
||||||
|
set_perpetual_inventory()
|
||||||
|
|
||||||
|
pr = make_purchase_receipt(received_qty=4, qty=2)
|
||||||
|
|
||||||
|
return_pr = make_purchase_receipt(is_return=1, return_against=pr.name, received_qty = -4, qty=-2)
|
||||||
|
|
||||||
|
actual_qty = frappe.db.get_value("Stock Ledger Entry", {"voucher_type": "Purchase Receipt",
|
||||||
|
"voucher_no": return_pr.name, 'warehouse': return_pr.items[0].rejected_warehouse}, "actual_qty")
|
||||||
|
|
||||||
|
self.assertEqual(actual_qty, -2)
|
||||||
|
|
||||||
|
set_perpetual_inventory(0)
|
||||||
|
|
||||||
def test_purchase_return_for_serialized_items(self):
|
def test_purchase_return_for_serialized_items(self):
|
||||||
def _check_serial_no_values(serial_no, field_values):
|
def _check_serial_no_values(serial_no, field_values):
|
||||||
serial_no = frappe.get_doc("Serial No", serial_no)
|
serial_no = frappe.get_doc("Serial No", serial_no)
|
||||||
@ -248,17 +262,23 @@ def make_purchase_receipt(**args):
|
|||||||
pr.currency = args.currency or "INR"
|
pr.currency = args.currency or "INR"
|
||||||
pr.is_return = args.is_return
|
pr.is_return = args.is_return
|
||||||
pr.return_against = args.return_against
|
pr.return_against = args.return_against
|
||||||
|
qty = args.qty or 5
|
||||||
|
received_qty = args.received_qty or qty
|
||||||
|
rejected_qty = args.rejected_qty or flt(received_qty) - flt(qty)
|
||||||
|
|
||||||
pr.append("items", {
|
pr.append("items", {
|
||||||
"item_code": args.item or args.item_code or "_Test Item",
|
"item_code": args.item or args.item_code or "_Test Item",
|
||||||
"warehouse": args.warehouse or "_Test Warehouse - _TC",
|
"warehouse": args.warehouse or "_Test Warehouse - _TC",
|
||||||
"qty": args.qty or 5,
|
"qty": qty,
|
||||||
"received_qty": args.qty or 5,
|
"received_qty": received_qty,
|
||||||
|
"rejected_qty": rejected_qty,
|
||||||
|
"rejected_warehouse": args.rejected_warehouse or "_Test Rejected Warehouse - _TC" if rejected_qty != 0 else "",
|
||||||
"rate": args.rate or 50,
|
"rate": args.rate or 50,
|
||||||
"conversion_factor": 1.0,
|
"conversion_factor": 1.0,
|
||||||
"serial_no": args.serial_no,
|
"serial_no": args.serial_no,
|
||||||
"stock_uom": "_Test UOM"
|
"stock_uom": "_Test UOM"
|
||||||
})
|
})
|
||||||
|
|
||||||
if not args.do_not_save:
|
if not args.do_not_save:
|
||||||
pr.insert()
|
pr.insert()
|
||||||
if not args.do_not_submit:
|
if not args.do_not_submit:
|
||||||
|
Loading…
x
Reference in New Issue
Block a user