refactor: removed unrequired code and test for standalone delivery no…

…te serial return (#30276) (#30278)

(cherry picked from commit 2a0ca7c)

Co-authored-by: Noah Jacob <noahjacobkurian@gmail.com>

erpnext/stock/doctype/delivery_note/delivery_note.py

@@ -13,10 +13,7 @@
 from erpnext.controllers.accounts_controller import get_taxes_and_charges
 from erpnext.controllers.selling_controller import SellingController
 from erpnext.stock.doctype.batch.batch import set_batch_nos
-from erpnext.stock.doctype.serial_no.serial_no import (
-	get_delivery_note_serial_no,
-	update_serial_nos_after_submit,
-)
+from erpnext.stock.doctype.serial_no.serial_no import get_delivery_note_serial_no
 
 form_grid_templates = {
 	"items": "templates/form_grid/item_grid.html"
@@ -218,9 +215,6 @@ def on_submit(self):
 		# Updating stock ledger should always be called after updating prevdoc status,
 		# because updating reserved qty in bin depends upon updated delivered qty in SO
 		self.update_stock_ledger()
-		if self.is_return:
-			update_serial_nos_after_submit(self, "items")
-
 		self.make_gl_entries()
 		self.repost_future_sle_and_gle()
 

erpnext/stock/doctype/delivery_note/test_delivery_note.py

@@ -823,11 +823,6 @@ def test_payment_terms_are_fetched_when_creating_sales_invoice(self):
 
 		automatically_fetch_payment_terms(enable=0)
 
-	def test_standalone_serial_no_return(self):
-		dn = create_delivery_note(item_code="_Test Serialized Item With Series", is_return=True, qty=-1)
-		dn.reload()
-		self.assertTrue(dn.items[0].serial_no)
-
 def create_delivery_note(**args):
 	dn = frappe.new_doc("Delivery Note")
 	args = frappe._dict(args)