[16.0][FIX+IMP] account_chart_update: Don't depend on l10n_generic_coa + TransactionCase

OCA / account-financial-tools

Odoo Accountant Financial Tools and Utils

GNU Affero General Public License v3.0

308 stars 750 forks source link

Closed pedrobaeza closed 1 year ago

pedrobaeza commented 1 year ago

Forward-port of #1713

Depending on l10n_generic_coa is the lazy option for not putting proper initialization data on the test, and it also couples the tests to the external module changes.
Switch to TransactionCase, for populating once the company, CoA, etc.
Speed up a bit the tests, removing superflual mail operations.

@Tecnativa

pedrobaeza commented 1 year ago

And fast-tracking as well:

/ocabot merge patch

OCA-git-bot commented 1 year ago

Hey, thanks for contributing! Proceeding to merge this for you. Prepared branch 16.0-ocabot-merge-pr-1714-by-pedrobaeza-bump-patch, awaiting test results.

OCA-git-bot commented 1 year ago

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