Wednesday, September 4, 2024
2 changes · saas-17.4
Resolved issues and error corrections
This fix ensures a web test restores a temporary browser visibility setting after it runs. It helps prevent one automated test from affecting others, improving confidence in test results without changing user-facing behavior.
Original PR description
Before this commit, the tests of the feature introduced at odoo/odoo@544da24ebc6359895a548c79b3bca4164dcee45c did not clean their mock overrides. NAmely the visibility state of the document. This could be problematic if other test were to do the same thing. After this commit, we revert the mock overrides to make sure we don't alter the test infrastructure. Description of the issue/feature this PR addresses: Current behavior before PR: Desired behavior after PR is merged: --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr
This fix adjusts how message display updates are tracked in Odoo Mail, preventing unnecessary internal recalculations during rendering. It helps keep mail and discussion screens responsive while simplifying the related performance checks.
Original PR description
They were put to prevent some re-render of message. However the problem was the test helpers that rely on hooks that were triggered more than once per actual render. This commit fixes the issue by using `onMounted()`/`onPatched` rather than `onRendered`. This allows the remove the `eager: true` on `Message` model computed fields. Also, test `"posting new message should only render relevant part"` has been adapted to be simpler to what's actually being tested.