search

OCA / edi-framework

GNU Affero General Public License v3.0
6 stars 29 forks source link

[16.0] [MIG] edi_exchange_template_oca: Migration to 16.0 #28

Closed etobella closed 10 months ago

etobella commented 10 months ago

Migration to 16

etobella commented 10 months ago

After some tests, this was the only way I found to fix it... somehow, odoo_test_helper is not restoring the original registry. This is not a problem on at_install tests, but might be a problem with post_install tests. Now it should be fixed as all the affected tests have been transformed as at_install

Another option would be to add a second action, but this way seems cleaner to me

etobella commented 10 months ago

@simahawk

simahawk commented 10 months ago

After some tests, this was the only way I found to fix it... somehow, odoo_test_helper is not restoring the original registry. This is not a problem on at_install tests, but might be a problem with post_install tests. Now it should be fixed as all the affected tests have been transformed as at_install

Another option would be to add a second action, but this way seems cleaner to me

You probably need this https://github.com/OCA/odoo-test-helper/pull/28

etobella commented 10 months ago

Thanks @simahawk It works like a charm locally. Let me test on github actions :smile:

OCA-git-bot commented 10 months ago

There's no issue in this repo with the title 'Migration to version 16.0' and the milestone 16.0, so not possible to add the comment.

OCA-git-bot commented 10 months ago

This PR looks fantastic, let's merge it! Prepared branch 16.0-ocabot-merge-pr-28-by-pedrobaeza-bump-nobump, awaiting test results.

OCA-git-bot commented 10 months ago

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