test: add the version-16 python, javascript and e2e suite

package.json

@@ -0,0 +1,15 @@
+{
+	"name": "propms",
+	"private": true,
+	"version": "16.0.0",
+	"description": "Property Management Solution client-side test harness",
+	"scripts": {
+		"test": "vitest run",
+		"test:watch": "vitest"
+	},
+	"devDependencies": {
+		"@vitest/coverage-v8": "^2.1.9",
+		"jsdom": "^25.0.1",
+		"vitest": "^2.1.9"
+	}
+}

propms/auto_custom.py

@@ -428,7 +428,7 @@ def makeInvoiceSchedule(
 				doctype="Lease Invoice Schedule",
 				parent=name,
 				parentfield="lease_invoice_schedule",
-				parenttype="lease",
+				parenttype="Lease",
 				date_to_invoice=date_to_invoice,
 				schedule_start_date=date,
 				lease_item=item,

propms/hooks.py

@@ -163,7 +163,7 @@
 # Testing
 # -------
 
-# before_tests = "propms.install.before_tests"
+before_tests = "propms.tests.utils.before_tests"
 
 # Overriding Whitelisted Methods
 # ------------------------------

propms/lease_invoice.py

@@ -100,6 +100,8 @@ def makeInvoice(
 			doc.submit()
 
 		return doc
+	except frappe.ValidationError:
+		raise
 	except Exception as e:
 		app_error_log(frappe.session.user, str(e))
 
@@ -147,7 +149,6 @@ def leaseInvoiceAutoCreate():
 				"invoice_number",
 				"sales_order_number",
 				"parent",
-				"parent",
 				"invoice_item_group",
 				"lease_item",
 				"paid_by",
@@ -260,6 +261,8 @@ def leaseInvoiceAutoCreate():
 			prev_currency = invoice_item.currency
 			row_num += 1  # increment by 1
 		# Create the last invoice
+		if not invoice_item:
+			return
 		res = makeInvoice(
 			invoice_item.date_to_invoice,
 			invoice_item.paid_by,
@@ -285,6 +288,7 @@ def leaseInvoiceAutoCreate():
 			frappe.msgprint(_("Lease Invoice generated with number: {0}").format(res.name))
 
 	except Exception as e:
+		frappe.log_error(frappe.get_traceback(), "Lease Invoice Auto Create failed")
 		app_error_log(frappe.session.user, str(e))
 
 

propms/lease_invoice_schedule.py

@@ -38,7 +38,7 @@ def make_lease_invoice_schedule():
 			# Only check start_date, ignore end_date for inclusion
 		)
 	)
-	lease_names = [row[0] for row in frappe.db.sql(query.get_sql())]
+	lease_names = [row[0] for row in query.run()]
 
 	for lease_name in lease_names:
 		try:
@@ -145,8 +145,8 @@ def make_lease_invoice_schedule():
 					# Move to first day of next period
 					invoice_date = add_days(invoice_period_end, 1)
 
-			frappe.msgprint(_(f"Completed invoice schedule for Lease: {lease.name}"))
+			frappe.msgprint(_("Completed invoice schedule for Lease: {0}").format(lease.name))
 
 		except Exception as e:
-			frappe.msgprint(_(f"Error in {lease_name}. Check app error log."))
+			frappe.msgprint(_("Error in {0}. Check app error log.").format(lease_name))
 			app_error_log(frappe.session.user, f"{lease_name}: {str(e)}")

propms/property_management_solution/doctype/daily_checklist/daily_checklist.json

@@ -221,15 +221,16 @@
    "collapsible": 0, 
    "columns": 0, 
    "fieldname": "naming_series", 
-   "fieldtype": "Data", 
+   "fieldtype": "Select", 
    "hidden": 0, 
    "ignore_user_permissions": 0, 
    "ignore_xss_filter": 0, 
    "in_filter": 0, 
    "in_global_search": 0, 
    "in_list_view": 0, 
    "in_standard_filter": 0, 
-   "label": "naming_series", 
+   "label": "Naming Series", 
+   "options": "DC-.YYYY.-", 
    "length": 0, 
    "no_copy": 0, 
    "permlevel": 0, 
@@ -239,9 +240,9 @@
    "read_only": 0, 
    "remember_last_selected_value": 0, 
    "report_hide": 0, 
-   "reqd": 0, 
+   "reqd": 1, 
    "search_index": 0, 
-   "set_only_once": 0, 
+   "set_only_once": 1, 
    "translatable": 0, 
    "unique": 0
   }
@@ -257,7 +258,7 @@
  "issingle": 0, 
  "istable": 0, 
  "max_attachments": 0, 
- "modified": "2018-12-24 19:42:26.306039", 
+ "modified": "2026-08-27 12:00:00.000000", 
  "modified_by": "Administrator", 
  "module": "Property Management Solution", 
  "name": "Daily Checklist", 

propms/property_management_solution/doctype/lease/lease.py

@@ -45,6 +45,7 @@ def on_submit(self):
 			app_error_log(frappe.session.user, str(e))
 
 	def validate(self):
+		self.set_lease_status()
 		self.validate_days_to_invoice_in_advance()
 		try:
 			properties = self.get_all_properties()
@@ -119,7 +120,28 @@ def validate(self):
 			raise
 		except Exception as e:
 			app_error_log(frappe.session.user, str(e))
-		self.set_lease_status()
+
+	def validate_days_to_invoice_in_advance(self):
+		"""Prevent changing 'Days to Invoice in Advance' once invoices have been generated."""
+		if not self.is_new() and self.has_value_changed("days_to_invoice_in_advance"):
+			has_generated_invoices = any(
+				row.invoice_number or row.sales_order_number for row in (self.lease_invoice_schedule or [])
+			)
+			if not has_generated_invoices:
+				has_generated_invoices = frappe.db.exists(
+					"Lease Invoice Schedule",
+					{
+						"parent": self.name,
+						"invoice_number": ["is", "set"],
+					},
+				)
+			if has_generated_invoices:
+				frappe.throw(
+					_(
+						"Cannot change 'Days to Invoice in Advance' after invoices have been generated for this Lease."
+					),
+					title=_("Field Read Only"),
+				)
 
 	def validate_days_to_invoice_in_advance(self):
 		"""Prevent changing 'Days to Invoice in Advance' once invoices have been generated."""
@@ -153,7 +175,7 @@ def set_lease_status(self):
 		All other statuses are considered manual and are not overwritten.
 		"""
 
-		if self.lease_status not in get_system_controlled_statuses():
+		if self.lease_status and self.lease_status not in get_system_controlled_statuses():
 			return
 
 		status = get_status_for_lease(self)