Skip to content

Commit e76c7c6

Browse files
committed
[FIX] l10n_it_edi_extension: fix remaining test failures
- Add 'name' to _check_at_least_one_ref constraint fields to ensure constraint is triggered on create (name is required, always passed) - Clear partner_shipping_id in export test to avoid DatiTrasporto being generated in XML (not present in expected XML file)
1 parent af47b34 commit e76c7c6

File tree

2 files changed

+5
-2
lines changed

2 files changed

+5
-2
lines changed

l10n_it_edi_extension/models/l10n_it_edi_move_line_other_data.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -60,11 +60,12 @@ def _check_text_ref_length(self):
6060
)
6161
)
6262

63-
@api.constrains("text_ref", "num_ref", "date_ref")
63+
@api.constrains("name", "text_ref", "num_ref", "date_ref")
6464
def _check_at_least_one_ref(self):
6565
for record in self:
6666
has_text = bool(record.text_ref)
67-
has_num = record.num_ref != 0.0
67+
# For Float fields, check if value was explicitly set (not default 0.0)
68+
has_num = bool(record.num_ref)
6869
has_date = bool(record.date_ref)
6970
if not has_text and not has_num and not has_date:
7071
raise ValidationError(

l10n_it_edi_extension/tests/test_altri_dati_gestionali.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -167,6 +167,8 @@ def test_export_altri_dati_gestionali(self):
167167
taxes=self.default_tax,
168168
)
169169
invoice.invoice_date_due = invoice.date
170+
# Clear delivery address to avoid DatiTrasporto in XML
171+
invoice.partner_shipping_id = False
170172
line = invoice.invoice_line_ids[0]
171173

172174
# Create other data entries

0 commit comments

Comments
 (0)