From f2f1b2597dd323203d2ad99bddf65599ed614744 Mon Sep 17 00:00:00 2001
From: Nabin Hait <nabinhait@gmail.com>
Date: Sun, 5 Jul 2026 12:13:48 +0530
Subject: [PATCH] test: add coverage for Italy e-invoice utility helpers

---
 erpnext/regional/italy/test_utils.py | 86 ++++++++++++++++++++++++++++
 1 file changed, 86 insertions(+)
 create mode 100644 erpnext/regional/italy/test_utils.py

diff --git a/erpnext/regional/italy/test_utils.py b/erpnext/regional/italy/test_utils.py
new file mode 100644
index 00000000000..f716bba8719
--- /dev/null
+++ b/erpnext/regional/italy/test_utils.py
@@ -0,0 +1,86 @@
+# Copyright (c) 2026, Frappe Technologies Pvt. Ltd. and Contributors
+# See license.txt
+
+import types
+
+import frappe
+
+from erpnext.regional.italy.utils import (
+	append_row_as_charges,
+	get_conditions,
+	get_unamended_name,
+	update_summary_details,
+)
+from erpnext.tests.utils import ERPNextTestSuite
+
+
+class TestItalyUtils(ERPNextTestSuite):
+	"""Pure helpers behind the Italian e-invoice export."""
+
+	def test_get_conditions_builds_filter_map(self):
+		base = get_conditions({})
+		self.assertEqual(base["docstatus"], 1)
+		self.assertEqual(base["company_tax_id"], ("!=", ""))
+		self.assertNotIn("company", base)
+
+		scoped = get_conditions({"company": "_Test Company", "customer": "_Test Customer"})
+		self.assertEqual(scoped["company"], "_Test Company")
+		self.assertEqual(scoped["customer"], "_Test Customer")
+
+		# a single bound uses >=/<=, both bounds use a between range
+		self.assertEqual(get_conditions({"from_date": "2026-01-01"})["posting_date"], (">=", "2026-01-01"))
+		self.assertEqual(get_conditions({"to_date": "2026-06-30"})["posting_date"], ("<=", "2026-06-30"))
+		self.assertEqual(
+			get_conditions({"from_date": "2026-01-01", "to_date": "2026-06-30"})["posting_date"],
+			("between", ["2026-01-01", "2026-06-30"]),
+		)
+
+	def test_update_summary_details_accumulates_and_flags_exemption(self):
+		summary = {}
+		tax = frappe._dict(tax_exemption_reason="N4", tax_exemption_law="Art. 10")
+
+		update_summary_details(summary, tax, 22.0, 44.0, 200.0)
+		update_summary_details(summary, tax, 22.0, 22.0, 100.0)
+		self.assertEqual(summary["22.0"]["tax_amount"], 66.0)
+		self.assertEqual(summary["22.0"]["taxable_amount"], 300.0)
+		# exemption fields are only populated for the zero-rate bucket
+		self.assertEqual(summary["22.0"]["tax_exemption_reason"], "")
+
+		update_summary_details(summary, tax, 0.0, 0.0, 500.0)
+		self.assertEqual(summary["0.0"]["tax_exemption_reason"], "N4")
+		self.assertEqual(summary["0.0"]["tax_exemption_law"], "Art. 10")
+
+	def test_append_row_as_charges_computes_amount(self):
+		items, summary = [], {}
+		tax = frappe._dict(rate=22.0, account_head="VAT - IT", tax_exemption_reason="", tax_exemption_law="")
+		reference_row = frappe._dict(tax_amount=200.0, description="Consulting")
+
+		append_row_as_charges(items, tax, reference_row, summary)
+
+		self.assertEqual(len(items), 1)
+		row = items[0]
+		self.assertEqual(row.tax_rate, 22.0)
+		self.assertEqual(row.tax_amount, 44.0)  # 200 * 22 / 100
+		self.assertEqual(row.taxable_amount, 200.0)
+		self.assertEqual(row.item_code, "Consulting")
+		self.assertEqual(row.item_tax_rate, {"VAT - IT": 22.0})
+		self.assertEqual(summary["22.0"]["tax_amount"], 44.0)
+
+	def test_get_unamended_name(self):
+		# a doc missing the naming attributes is returned unchanged
+		plain = types.SimpleNamespace(name="ACC-SINV-2026-00001")
+		self.assertEqual(get_unamended_name(plain), "ACC-SINV-2026-00001")
+
+		# an amended doc drops the trailing amendment suffix
+		amended = frappe._dict(
+			name="ACC-SINV-2026-00001-1",
+			naming_series="ACC-SINV-.YYYY.-",
+			amended_from="ACC-SINV-2026-00001",
+		)
+		self.assertEqual(get_unamended_name(amended), "ACC-SINV-2026-00001")
+
+		# an original (non-amended) doc keeps its name
+		original = frappe._dict(
+			name="ACC-SINV-2026-00001", naming_series="ACC-SINV-.YYYY.-", amended_from=None
+		)
+		self.assertEqual(get_unamended_name(original), "ACC-SINV-2026-00001")