Prepare account_do Tryton package
This commit is contained in:
José Arturo García
@@ -0,0 +1,306 @@
|
||||
import unittest
|
||||
from collections import Counter
|
||||
from decimal import Decimal
|
||||
from pathlib import Path
|
||||
from xml.etree import ElementTree as ET
|
||||
|
||||
from trytond.modules.account_do.tax import TAX_KIND
|
||||
from trytond.pool import Pool
|
||||
from trytond.tests.test_tryton import ModuleTestCase, with_transaction
|
||||
from trytond.transaction import Transaction
|
||||
|
||||
MODULE_DIR = Path(__file__).resolve().parent.parent
|
||||
|
||||
|
||||
def _iter_xml_records(*filenames):
|
||||
for filename in filenames:
|
||||
path = MODULE_DIR / filename
|
||||
root = ET.parse(path).getroot()
|
||||
for record in root.findall('.//record'):
|
||||
values = {
|
||||
field.get('name'): (
|
||||
field.get('ref') or field.get('eval') or field.text or '')
|
||||
for field in record.findall('field')
|
||||
}
|
||||
yield filename, record.get('id'), record.get('model'), values
|
||||
|
||||
|
||||
class AccountDoTestCase(ModuleTestCase):
|
||||
"Test account_do module"
|
||||
module = 'account_do'
|
||||
|
||||
@with_transaction()
|
||||
def test_chart_template_is_loaded_by_default(self):
|
||||
'Test Dominican chart template is loaded without Spanish language setup'
|
||||
pool = Pool()
|
||||
ModelData = pool.get('ir.model.data')
|
||||
|
||||
self.assertTrue(ModelData.get_id('account_do', 'do_account_root'))
|
||||
|
||||
@with_transaction()
|
||||
def test_chart_creates_company_accounts_and_taxes(self):
|
||||
'Test Dominican chart creates usable accounts and taxes'
|
||||
pool = Pool()
|
||||
Account = pool.get('account.account')
|
||||
AccountTemplate = pool.get('account.account.template')
|
||||
Company = pool.get('company.company')
|
||||
Currency = pool.get('currency.currency')
|
||||
ModelData = pool.get('ir.model.data')
|
||||
Party = pool.get('party.party')
|
||||
Tax = pool.get('account.tax')
|
||||
TaxCode = pool.get('account.tax.code')
|
||||
TaxRule = pool.get('account.tax.rule')
|
||||
CreateChart = pool.get('account.create_chart', type='wizard')
|
||||
|
||||
currency, = Currency.create([{
|
||||
'name': 'Dominican Peso',
|
||||
'code': 'DOP',
|
||||
'symbol': 'RD$',
|
||||
'digits': 2,
|
||||
'rounding': '0.01',
|
||||
}])
|
||||
party, = Party.create([{
|
||||
'name': 'Empresa Dominicana',
|
||||
}])
|
||||
company, = Company.create([{
|
||||
'party': party.id,
|
||||
'currency': currency.id,
|
||||
}])
|
||||
template = AccountTemplate(ModelData.get_id(
|
||||
'account_do', 'do_account_root'))
|
||||
|
||||
session_id, _start, _end = CreateChart.create()
|
||||
chart = CreateChart(session_id)
|
||||
chart.account.account_template = template
|
||||
chart.account.company = company
|
||||
chart.transition_create_account()
|
||||
|
||||
receivable, = Account.search([
|
||||
('company', '=', company.id),
|
||||
('code', '=', '110201'),
|
||||
('type.receivable', '=', True),
|
||||
('party_required', '=', True),
|
||||
('closed', '!=', True),
|
||||
], limit=1)
|
||||
payable, = Account.search([
|
||||
('company', '=', company.id),
|
||||
('code', '=', '210101'),
|
||||
('type.payable', '=', True),
|
||||
('party_required', '=', True),
|
||||
('closed', '!=', True),
|
||||
], limit=1)
|
||||
for code in [
|
||||
'110401', '110406', '210201', '210202', '210203',
|
||||
'210206', '210207', '210208', '210209', '210210']:
|
||||
with self.subTest(code=code):
|
||||
self.assertTrue(Account.search([
|
||||
('company', '=', company.id),
|
||||
('code', '=', code),
|
||||
('closed', '!=', True),
|
||||
], limit=1))
|
||||
|
||||
self.assertGreaterEqual(len(Account.search([
|
||||
('company', '=', company.id),
|
||||
])), 200)
|
||||
self.assertGreaterEqual(len(Tax.search([
|
||||
('company', '=', company.id),
|
||||
])), 25)
|
||||
self.assertTrue(TaxCode.search([
|
||||
('company', '=', company.id),
|
||||
('name', '=', 'ITBIS — Balance Neto (Débito − Crédito)'),
|
||||
], limit=1))
|
||||
self.assertTrue(TaxRule.search([
|
||||
('company', '=', company.id),
|
||||
('kind', '=', 'sale'),
|
||||
], limit=1))
|
||||
self.assertTrue(TaxRule.search([
|
||||
('company', '=', company.id),
|
||||
('kind', '=', 'purchase'),
|
||||
], limit=1))
|
||||
self.assertTrue(Tax.search([
|
||||
('company', '=', company.id),
|
||||
('description', '=', 'ITBIS 18% Ventas'),
|
||||
('tax_kind', '=', 'itbis'),
|
||||
('rate', '=', Decimal('0.18')),
|
||||
], limit=1))
|
||||
self.assertTrue(Tax.search([
|
||||
('company', '=', company.id),
|
||||
('description', '=', 'Retención ITBIS Gran Contribuyente 30%'),
|
||||
('tax_kind', '=', 'itbis_withholding'),
|
||||
('rate', '=', Decimal('-0.054')),
|
||||
], limit=1))
|
||||
|
||||
chart.properties.company = company
|
||||
chart.properties.account_receivable = receivable
|
||||
chart.properties.account_payable = payable
|
||||
with Transaction().set_context(company=company.id):
|
||||
chart.transition_create_properties()
|
||||
CreateChart.delete(session_id)
|
||||
|
||||
|
||||
class AccountDoUnitTestCase(unittest.TestCase):
|
||||
|
||||
def test_xml_references_are_resolved_inside_module(self):
|
||||
records = list(_iter_xml_records(
|
||||
'account_chart_do.xml',
|
||||
'tax_do.xml',
|
||||
'tax_code_do.xml',
|
||||
'tax_rule_do.xml',
|
||||
))
|
||||
ids = {record_id for _, record_id, _, _ in records}
|
||||
missing = []
|
||||
for filename in [
|
||||
'account_chart_do.xml',
|
||||
'tax_do.xml',
|
||||
'tax_code_do.xml',
|
||||
'tax_rule_do.xml',
|
||||
]:
|
||||
root = ET.parse(MODULE_DIR / filename).getroot()
|
||||
for record in root.findall('.//record'):
|
||||
for field in record.findall('field'):
|
||||
ref = field.get('ref')
|
||||
if ref and '.' not in ref and ref not in ids:
|
||||
missing.append(
|
||||
(filename, record.get('id'), field.get('name'), ref))
|
||||
self.assertEqual(missing, [])
|
||||
|
||||
def test_chart_template_codes_are_unique(self):
|
||||
accounts = [
|
||||
values for _, _, model, values in _iter_xml_records(
|
||||
'account_chart_do.xml')
|
||||
if model == 'account.account.template'
|
||||
]
|
||||
codes = [values['code'] for values in accounts if values.get('code')]
|
||||
duplicates = [
|
||||
code for code, count in Counter(codes).items() if count > 1]
|
||||
self.assertEqual(duplicates, [])
|
||||
|
||||
def test_chart_root_and_statement_classification(self):
|
||||
accounts = {
|
||||
record_id: values
|
||||
for _, record_id, model, values in _iter_xml_records(
|
||||
'account_chart_do.xml')
|
||||
if model == 'account.account.template'
|
||||
}
|
||||
types = {
|
||||
record_id: values
|
||||
for _, record_id, model, values in _iter_xml_records(
|
||||
'account_chart_do.xml')
|
||||
if model == 'account.account.type.template'
|
||||
}
|
||||
|
||||
self.assertEqual([
|
||||
accounts[f'do_account_{code}']['code']
|
||||
for code in range(1, 8)
|
||||
], ['1', '2', '3', '4', '5', '6', '7'])
|
||||
self.assertEqual(
|
||||
types['do_type_retained_earnings']['statement'], 'balance')
|
||||
self.assertEqual(types['do_type_revenue']['statement'], 'income')
|
||||
self.assertEqual(types['do_type_expense']['statement'], 'income')
|
||||
self.assertEqual(
|
||||
accounts['do_account_110201']['type'], 'do_type_receivable')
|
||||
self.assertEqual(
|
||||
accounts['do_account_210101']['type'], 'do_type_payable')
|
||||
|
||||
def test_no_spanish_technical_other_tax_ids_remain(self):
|
||||
ids = {
|
||||
record_id
|
||||
for _, record_id, _, _ in _iter_xml_records(
|
||||
'tax_do.xml', 'tax_code_do.xml')
|
||||
}
|
||||
self.assertFalse([record_id for record_id in ids if 'otros' in record_id])
|
||||
|
||||
def test_tax_templates_are_classified(self):
|
||||
taxes = {
|
||||
record_id: values
|
||||
for _, record_id, model, values in _iter_xml_records('tax_do.xml')
|
||||
if model == 'account.tax.template'
|
||||
}
|
||||
self.assertEqual(len(taxes), 31)
|
||||
self.assertFalse([
|
||||
record_id for record_id, values in taxes.items()
|
||||
if not values.get('tax_kind')])
|
||||
self.assertEqual(
|
||||
taxes['do_tax_itbis_18_venta']['invoice_account'],
|
||||
'do_account_210201')
|
||||
self.assertEqual(
|
||||
taxes['do_tax_itbis_18_compra']['invoice_account'],
|
||||
'do_account_110401')
|
||||
self.assertEqual(
|
||||
taxes['do_tax_ret_itbis_30']['tax_kind'],
|
||||
'itbis_withholding')
|
||||
self.assertIn(
|
||||
"Decimal('-5.4')/100",
|
||||
taxes['do_tax_ret_itbis_30']['rate'])
|
||||
self.assertEqual(
|
||||
taxes['do_tax_cheques_020']['tax_kind'], 'others')
|
||||
self.assertIn(
|
||||
"Decimal('0.20')/100",
|
||||
taxes['do_tax_cheques_020']['rate'])
|
||||
|
||||
def test_tax_code_templates_use_expected_signs(self):
|
||||
codes = {
|
||||
record_id: values
|
||||
for _, record_id, model, values in _iter_xml_records(
|
||||
'tax_code_do.xml')
|
||||
if model == 'account.tax.code.template'
|
||||
}
|
||||
lines = {
|
||||
record_id: values
|
||||
for _, record_id, model, values in _iter_xml_records(
|
||||
'tax_code_do.xml')
|
||||
if model == 'account.tax.code.line.template'
|
||||
}
|
||||
for record_id in [
|
||||
'do_tc_itbis',
|
||||
'do_tc_isr',
|
||||
'do_tc_isc',
|
||||
'do_tc_cdt',
|
||||
'do_tc_others',
|
||||
]:
|
||||
self.assertEqual(codes[record_id].get('parent'), 'None')
|
||||
self.assertEqual(codes['do_tc_isr']['name'], 'ISR - Retenciones')
|
||||
self.assertEqual(
|
||||
lines['do_tcl_isr_int_pf_inv']['code'],
|
||||
'do_tc_isr_intereses_pf')
|
||||
self.assertEqual(
|
||||
lines['do_tcl_isr_int_pf_cr']['code'],
|
||||
'do_tc_isr_intereses_pf')
|
||||
self.assertEqual(
|
||||
lines['do_tcl_isr_int_pj_inv']['code'],
|
||||
'do_tc_isr_intereses')
|
||||
self.assertEqual(
|
||||
lines['do_tcl_isr_int_pj_cr']['code'],
|
||||
'do_tc_isr_intereses')
|
||||
self.assertEqual(
|
||||
lines['do_tcl_chq020_inv']['code'],
|
||||
'do_tc_others_checks_020')
|
||||
self.assertEqual(
|
||||
lines['do_tcl_chq020_cr']['code'],
|
||||
'do_tc_others_checks_020')
|
||||
self.assertEqual(lines['do_tcl_itbis18v_inv']['operator'], '+')
|
||||
self.assertEqual(lines['do_tcl_itbis18c_inv']['operator'], '-')
|
||||
self.assertEqual(lines['do_tcl_ret_itbis_inv']['operator'], '+')
|
||||
self.assertEqual(lines['do_tcl_ret_itbis_adq_inv']['operator'], '-')
|
||||
self.assertEqual(lines['do_tcl_prop_inv']['code'], 'do_tc_others_tip')
|
||||
|
||||
def test_tax_kind_contains_dominican_tax_categories(self):
|
||||
codes = {code for code, _ in TAX_KIND}
|
||||
self.assertEqual(codes, {
|
||||
'',
|
||||
'itbis',
|
||||
'itbis_withholding',
|
||||
'itbis_withholding_acquirer',
|
||||
'isr_withholding',
|
||||
'isc',
|
||||
'cdt',
|
||||
'tip',
|
||||
'others',
|
||||
})
|
||||
|
||||
def test_tax_kind_codes_are_unique(self):
|
||||
codes = [code for code, _ in TAX_KIND]
|
||||
self.assertEqual(len(codes), len(set(codes)))
|
||||
|
||||
|
||||
del ModuleTestCase
|
||||
Reference in New Issue
Block a user