[15.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

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

Merging for normalizing the dependencies:

/ocabot merge patch

OCA-git-bot commented 1 year ago

This PR looks fantastic, let's merge it! Prepared branch 15.0-ocabot-merge-pr-1713-by-pedrobaeza-bump-patch, awaiting test results.

OCA-git-bot commented 1 year ago

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