Skip to content

[ADD] l10n_ar_tax: added test cases#1377

Open
jue-adhoc wants to merge 1 commit intoingadhoc:19.0from
adhoc-dev:19.0-h-113889-jue-2
Open

[ADD] l10n_ar_tax: added test cases#1377
jue-adhoc wants to merge 1 commit intoingadhoc:19.0from
adhoc-dev:19.0-h-113889-jue-2

Conversation

@jue-adhoc
Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings April 21, 2026 16:06
@roboadhoc
Copy link
Copy Markdown
Contributor

Pull request status dashboard

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Este PR agrega cobertura de tests para el comportamiento de map_tax() en posiciones fiscales argentinas cuando la FP solo configura percepciones/retenciones (l10n_ar_tax_ids) y no tiene mapeos explícitos de IVA (tax_ids), evitando sustituciones no deseadas por delegación al domestic FP.

Changes:

  • Nuevo archivo de tests que cubre casos de map_tax() para FPs “perception-only” y FPs con mapeo explícito de impuestos.
  • Inclusión del nuevo módulo de tests en l10n_ar_tax/tests/__init__.py.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
l10n_ar_tax/tests/test_map_tax_fiscal_position.py Añade tests de regresión/validación para map_tax() y escenarios con domestic FP y facturas.
l10n_ar_tax/tests/init.py Importa el nuevo archivo de tests para que sea ejecutado por la suite.

Comment thread l10n_ar_tax/tests/test_map_tax_fiscal_position.py Outdated
Comment thread l10n_ar_tax/tests/test_map_tax_fiscal_position.py
Comment thread l10n_ar_tax/tests/test_map_tax_fiscal_position.py Outdated
@jue-adhoc jue-adhoc force-pushed the 19.0-h-113889-jue-2 branch 3 times, most recently from d342267 to a1a4f87 Compare April 22, 2026 18:46
@jue-adhoc jue-adhoc requested a review from Copilot April 22, 2026 18:47
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

Comment thread l10n_ar_tax/tests/test_map_tax_fiscal_position.py Outdated
Comment thread l10n_ar_tax/tests/test_map_tax_fiscal_position.py Outdated
Comment thread l10n_ar_tax/tests/test_map_tax_fiscal_position.py Outdated
@jue-adhoc jue-adhoc force-pushed the 19.0-h-113889-jue-2 branch from 9ae5068 to a1a4f87 Compare April 23, 2026 10:52
@jue-adhoc jue-adhoc force-pushed the 19.0-h-113889-jue-2 branch from a1a4f87 to 65bf0c8 Compare April 23, 2026 10:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants