search

OCA / l10n-brazil

Localização brasileira oficial do Odoo.
https://odoo-community.org/psc-teams/brazil-66
GNU Affero General Public License v3.0
241 stars 245 forks source link

[15.0][l10n_br_fiscal] Oca port from 14.0 to 15.0 pr 3109 #3194

Closed rvalyi closed 3 months ago

rvalyi commented 3 months ago

port de #3109

OCA-git-bot commented 3 months ago

Hi @renatonlima, some modules you are maintaining are being modified, check this out!

rvalyi commented 3 months ago

passou os testes nao:

2024-07-22 02:02:14,333 388 INFO odoo odoo.addons.l10n_br_fiscal.tests.test_fiscal_document_generic: Starting TestFiscalDocumentGeneric.test_nfe_same_state ... 
2024-07-22 02:02:14,752 388 INFO odoo odoo.addons.l10n_br_fiscal.tests.test_fiscal_document_generic: ====================================================================== 
2024-07-22 02:02:14,752 388 ERROR odoo odoo.addons.l10n_br_fiscal.tests.test_fiscal_document_generic: FAIL: TestFiscalDocumentGeneric.test_nfe_same_state
Traceback (most recent call last):
  File "/__w/l10n-brazil/l10n-brazil/l10n_br_fiscal/tests/test_fiscal_document_generic.py", line 181, in test_nfe_same_state
    self.assertEqual(self.nfe_same_state.amount_price_gross, 3965)
AssertionError: 430.5 != 3965
antoniospneto commented 3 months ago

wow, e tá bem grande a diferença com o valor esperado, vou dar uma conferida

antoniospneto commented 3 months ago

@rvalyi @marcelsavegnago

Pelo que vi, o motivo é que faltou fazer o port da PR #3016

rvalyi commented 3 months ago

@antoniospneto eu dei um rebase mas deu o mesmo erro. Talvez seria bom arrumar esse para finalizar a migraçao do l10n_br_account para a v16

antoniospneto commented 3 months ago

@rvalyi, encontrei o erro. Os dados de demonstração usados nesses testes tiveram os valores alterados. O registro "product.product_product_27" tinha um list_price de 3645.00 na versão 14.0, que mudou para 110.50 nas versões 15.0/16.0.

Vou refatorar esse teste na 14.0 para que não dependa desse registro específico. Após isso, você poderá fazer um cherry-pick aqui, e assim evitamos a necessidade de fazer alterações adicionais e aumentamos o diff com as outras versões.

antoniospneto commented 3 months ago

@rvalyi se puder incluir o cherry-pick do commit 8ebf3970e990c05d77692caa68df612f922e96ad Conforme correção feita aqui: https://github.com/OCA/l10n-brazil/pull/3244

OCA-git-bot commented 3 months ago

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

rvalyi commented 3 months ago

/ocabot merge patch

OCA-git-bot commented 3 months ago

Hey, thanks for contributing! Proceeding to merge this for you. Prepared branch 15.0-ocabot-merge-pr-3194-by-rvalyi-bump-patch, awaiting test results.

OCA-git-bot commented 3 months ago

Congratulations, your PR was merged at 79e3399852a41b1022fc0fd34a8988be2e0c6228. Thanks a lot for contributing to OCA. ❤️