OCA / l10n-brazil

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

[14.0][REF] l10n_br_fiscal: Manter os preços no teste unitário. #3244

Closed antoniospneto closed 1 month ago

antoniospneto commented 1 month ago

Pequena refatoração nos testes, para que os preços definido nos dados de demonstração se mantenham.

Antes, ao chamar o onchange do produto nos testes, o preço definido na nfe estavam sendo descartados e o preço era atualizado conforme estava no cadastro do produto.

Dessa forma, conseguimos manter o mesmo teste sem quebrar na versão 15/16.0 pois lá o preço nesses produtos estão diferentes.

OCA-git-bot commented 1 month ago

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

mileo commented 1 month ago

/ocabot merge minor

OCA-git-bot commented 1 month ago

What a great day to merge this nice PR. Let's do it! Prepared branch 14.0-ocabot-merge-pr-3244-by-mileo-bump-minor, awaiting test results.

OCA-git-bot commented 1 month ago

@mileo your merge command was aborted due to failed check(s), which you can inspect on this commit of 14.0-ocabot-merge-pr-3244-by-mileo-bump-minor.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

rvalyi commented 1 month ago

@antoniospneto pode dar um rebase?

rvalyi commented 1 month ago

/ocabot merge patch

OCA-git-bot commented 1 month ago

What a great day to merge this nice PR. Let's do it! Prepared branch 14.0-ocabot-merge-pr-3244-by-rvalyi-bump-patch, awaiting test results.

OCA-git-bot commented 1 month ago

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