odoo/odoo#186394
Created by Marketing, Thibault Delavallee (tde)
Merged
at ecb9e69e75b42557f9f7c4c2f92cd6f8f234620d
Statuses:
- legal/cla: Contributor License Agreement check
- ci/runbot: Odoo Test Suite
- ci/upgrade_enterprise: Test upgrades for enterprise master
- ci/template: Contact runbot team on discord for help.
- ci/style: Optional style check. Ignore it only if strictly necessary.
- ci/security: Required security check. Can only be ignored by security team.
- label
- odoo-dev:16.0-mail-access-tde
- head
- 00f88ae7554ec6d093d00c1be8a336823714a6cb
- merged
- 1 year ago by Marketing, Thibault Delavallee (tde)
| odoo/odoo | |
|---|---|
| 16.0 | #186394 |
| 17.0 | #189120 |
| saas-17.2 | #189181 |
| saas-17.4 | #189202 |
| 18.0 | #189226 |
| saas-18.2 | |
| saas-18.3 | |
| saas-18.4 | |
| 19.0 | |
| master | #189253 |
[IMP] test_mail: make access tests independent from discuss models
Improve access related tests for mail.message and their sub models. Currently
tests are dependent on discuss.channel model, and not always written in a
clear way. Understanding the purpose of each test is not crystal clear.
We now use a test model with simple rules, defined in test_mail addon. It
allows to simulate models with
- public access;
- portal access;
- internal access;
- admin access;
using ACLs and ir.rules. Tests are rewritten to be more concise and precise
and use new models.
This is done in stable to keep coherency in tests codebase. It also eases
writing bugfixes as tests won't have to be modified during forward port
process. Finally it allows to backport improvements or fixes in later
versions if it applies to previous versions.