Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[16.0][REF] l10n_br_fiscal, l10n_br_account: Cálculo e Visualização das Alíquotas Efetivas do Simples Nacional #3674

Draft
wants to merge 4 commits into
base: 16.0
Choose a base branch
from

Conversation

antoniospneto
Copy link
Contributor

Migração da PR #2822 (14.0)

@antoniospneto antoniospneto force-pushed the 16.0-ref-simples-nacional branch from 0ce049a to c3b88c6 Compare March 25, 2025 01:32
@antoniospneto antoniospneto force-pushed the 16.0-ref-simples-nacional branch from c3b88c6 to edb2468 Compare March 25, 2025 02:43
@rvalyi
Copy link
Member

rvalyi commented Mar 25, 2025

@antoniospneto vc viu o erro na NFe do Simples né?

 2025-03-25 02:48:51,760 437 INFO odoo odoo.addons.l10n_br_nfe.tests.test_nfe_serialize: XML file saved at /github/home/.local/share/Odoo/filestore/odoo/34/343543298062bbed0aaffd7f601a7358f967a3d1 
2025-03-25 02:48:51,776 437 INFO odoo odoo.addons.l10n_br_nfe.tests.test_nfe_serialize_sn: Diff with expected XML (if any): [UpdateTextIn(node='/*/*/*[4]/*[2]/*[1]/*/*[4]', text='0.37')] 
2025-03-25 02:48:51,776 437 INFO odoo odoo.addons.l10n_br_nfe.tests.test_nfe_serialize_sn: ====================================================================== 
2025-03-25 02:48:51,776 437 ERROR odoo odoo.addons.l10n_br_nfe.tests.test_nfe_serialize_sn: FAIL: TestNFeExportSN.test_serialize_xml
Traceback (most recent call last):
  File "/__w/l10n-brazil/l10n-brazil/l10n_br_nfe/tests/test_nfe_serialize_sn.py", line 24, in test_serialize_xml
    assert len(diff) == 0
AssertionError

Na v16 o Odoo remove os attachments criados no testes. Ou vc comenta isso no framework do teste do Odoo, ou vc da um print do XML e abre no meld para investigar e resolver.

cc @marcelsavegnago teria que usar a mesma tecnica para finalizar a migração do l10n_br_cte até na v16: #3663 (comment)

@antoniospneto
Copy link
Contributor Author

@rvalyi valeu, vou dar uma olhada nisso, eu tenho um caso de teste que preciso migrar do módulo fiscal pro fiscal_edi tbm, depois revisar umas coisas que o Renato tinha pedido na 14.0.

@rvalyi
Copy link
Member

rvalyi commented Mar 26, 2025

Aqui a gente ta ficando menos afogado nas migrações para a v16, então o @renatonlima deve conseguir participar mais agora tb.

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.

2 participants