Skip to content

[18.0][IMP] automation_oca: Change where the mock has to be loaded#44

Merged
OCA-git-bot merged 1 commit intoOCA:18.0from
Tecnativa:18.0-fix-automation_oca-tests
Feb 20, 2026
Merged

[18.0][IMP] automation_oca: Change where the mock has to be loaded#44
OCA-git-bot merged 1 commit intoOCA:18.0from
Tecnativa:18.0-fix-automation_oca-tests

Conversation

@victoralmau
Copy link
Member

@victoralmau victoralmau commented Feb 20, 2026

Change where the mock has to be loaded

With the changes introduced in odoo/odoo@de056cc#diff-b37c7fd5520c97a29ddc59495779e7be61a66e15e85603928e27b3affb9dc31f, an error was occurring because the change validation on the model was being performed before tearDownClass was executed. Therefore, it has been modified so that the mock changes are applied in each test and cleaned up after each test. This way, the error will no longer appear and the test will run normally.

@Tecnativa

With the changes introduced in odoo/odoo@de056cc#diff-b37c7fd5520c97a29ddc59495779e7be61a66e15e85603928e27b3affb9dc31f, an error was occurring because the change validation on the model was being performed before tearDownClass was executed. Therefore, it has been modified so that the mock changes are applied in each test and cleaned up after each test. This way, the error will no longer appear and the test will run normally.
@pedrobaeza pedrobaeza added this to the 18.0 milestone Feb 20, 2026
Copy link
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Another victim of the pocalypse.

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

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

@OCA-git-bot OCA-git-bot merged commit 4a21cb4 into OCA:18.0 Feb 20, 2026
7 checks passed
@OCA-git-bot
Copy link
Contributor

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

@pedrobaeza pedrobaeza deleted the 18.0-fix-automation_oca-tests branch February 20, 2026 12:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants