Update foreign withholding and tax rules
This commit is contained in:
+31
-11
@@ -557,35 +557,35 @@ class AccountDoUnitTestCase(unittest.TestCase):
|
||||
expected = {
|
||||
'account_chart_do_en.xml': {
|
||||
'account.account.type.template': 36,
|
||||
'account.account.template': 281,
|
||||
'account.account.template': 282,
|
||||
},
|
||||
'tax_do_en.xml': {
|
||||
'account.tax.group': 7,
|
||||
'account.tax.template': 50,
|
||||
'account.tax.template': 51,
|
||||
},
|
||||
'tax_code_do_en.xml': {
|
||||
'account.tax.code.template': 44,
|
||||
'account.tax.code.line.template': 93,
|
||||
'account.tax.code.line.template': 95,
|
||||
},
|
||||
'tax_rule_do_en.xml': {
|
||||
'account.tax.rule.template': 23,
|
||||
'account.tax.rule.line.template': 30,
|
||||
'account.tax.rule.template': 28,
|
||||
'account.tax.rule.line.template': 31,
|
||||
},
|
||||
'account_chart_do_es_419.xml': {
|
||||
'account.account.type.template': 36,
|
||||
'account.account.template': 281,
|
||||
'account.account.template': 282,
|
||||
},
|
||||
'tax_do_es_419.xml': {
|
||||
'account.tax.group': 7,
|
||||
'account.tax.template': 50,
|
||||
'account.tax.template': 51,
|
||||
},
|
||||
'tax_code_do_es_419.xml': {
|
||||
'account.tax.code.template': 44,
|
||||
'account.tax.code.line.template': 93,
|
||||
'account.tax.code.line.template': 95,
|
||||
},
|
||||
'tax_rule_do_es_419.xml': {
|
||||
'account.tax.rule.template': 23,
|
||||
'account.tax.rule.line.template': 30,
|
||||
'account.tax.rule.template': 28,
|
||||
'account.tax.rule.line.template': 31,
|
||||
},
|
||||
}
|
||||
for filename, expected_counts in expected.items():
|
||||
@@ -829,7 +829,7 @@ class AccountDoUnitTestCase(unittest.TestCase):
|
||||
'account_chart_do_en.xml')
|
||||
if model == 'account.account.template'
|
||||
}
|
||||
self.assertEqual(len(taxes), 50)
|
||||
self.assertEqual(len(taxes), 51)
|
||||
for record_id, values in taxes.items():
|
||||
with self.subTest(record_id=record_id):
|
||||
self.assertEqual(
|
||||
@@ -991,6 +991,12 @@ class AccountDoUnitTestCase(unittest.TestCase):
|
||||
self.assertEqual(
|
||||
lines['do_trline_supp_ext_isr27']['group'],
|
||||
'do_tax_group_isr_ext')
|
||||
self.assertEqual(
|
||||
lines['do_trline_supp_ext_isr15']['tax'],
|
||||
'do_tax_ret_isr_ext_15')
|
||||
self.assertEqual(
|
||||
lines['do_trline_supp_ext_isr15']['start_date'],
|
||||
'datetime.date(2026, 7, 1)')
|
||||
self.assertEqual(
|
||||
lines['do_trline_bank_check_transfer_015']['origin_tax'],
|
||||
'do_tax_cheques_015')
|
||||
@@ -1010,5 +1016,19 @@ class AccountDoUnitTestCase(unittest.TestCase):
|
||||
lines['do_trline_bank_check_transfer_020']['start_date'],
|
||||
'datetime.date(2026, 7, 3)')
|
||||
|
||||
def test_tax_rule_lines_have_reachable_match_patterns(self):
|
||||
"""A rule must not contain two lines with the same match pattern."""
|
||||
patterns = {}
|
||||
for _, record_id, model, values in _iter_xml_records(
|
||||
'tax_rule_do_en.xml'):
|
||||
if model != 'account.tax.rule.line.template':
|
||||
continue
|
||||
pattern = tuple(values.get(field) for field in [
|
||||
'rule', 'group', 'origin_tax', 'start_date', 'end_date'])
|
||||
self.assertNotIn(
|
||||
pattern, patterns,
|
||||
msg=f'{record_id} is shadowed by {patterns.get(pattern)}')
|
||||
patterns[pattern] = record_id
|
||||
|
||||
|
||||
del ModuleTestCase
|
||||
|
||||
Reference in New Issue
Block a user