Odoo Changelog — 2025-10-28

Tuesday, October 28, 2025

2 changes · saas-18.4

Refactoring(2)

Code cleanup and technical improvements

[REF] *: remove unnecessary 'describe' calls

Coresaas-18.4base_importhtml_builderhtml_editorpoint_of_salepos_eventpos_hr

This update simplifies automated test files by removing unnecessary wrapper labels that repeated the file name or could be misleading. It does not change product behavior, but it makes the test suite easier for developers to read and maintain.

Original PR description

This commit removes test files using a single 'describe' call to wrap its tests, with a name being either redundant with the test file itself or is too misleading.

Regular expression used to find isolated suites:
```js
/^\s*import.*(?:\n(?!describe\()(?!test\().*)+\ndescribe\("[\w\s.]+",.*(?:\n(?!describe\().*)+\n\}\);\n(?!\n)/
```

Tip for reviewers: tick **"Hide whitespace"** when reviewing diff
<img width="237" height="205" alt="image" src="https://github.com/user-attachments/assets/373f7cc6-5e0e-4599-968a-39aad8fac2f0" />


Enterprise: https://github.com/odoo/enterprise/pull/98046

---
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

Forward-Port-Of: odoo/odoo#233207
Forward-Port-Of: odoo/odoo#233006

#tests#maintenance#cleanup

[REF] *: remove unnecessary 'describe' calls

PR #98206

Enterprisesaas-18.4iotpos_appointmentpos_enterprisespreadsheet_editionspreadsheet_sale_managementvoip

This change simplifies automated test files by removing unnecessary grouping labels that duplicated file names or could be misleading. It does not change product behavior, but it makes tests easier for developers to review and maintain.

Original PR description

This commit removes test files using a single 'describe' call to wrap its tests, with a name being either redundant with the test file itself or is too misleading.

Regular expression used to find isolated suites:
```js
/^\s*import.*(?:\n(?!describe\()(?!test\().*)+\ndescribe\("[\w\s.]+",.*(?:\n(?!describe\().*)+\n\}\);\n(?!\n)/
```

Tip for reviewers: tick **"Hide whitespace"** when reviewing diff
<img width="237" height="205" alt="image" src="https://github.com/user-attachments/assets/373f7cc6-5e0e-4599-968a-39aad8fac2f0" />

Community: https://github.com/odoo/odoo/pull/233006

Forward-Port-Of: odoo/enterprise#98171
Forward-Port-Of: odoo/enterprise#98046

#tests#maintenance#cleanup

This commit removes test files using a single 'describe' call to wrap its tests, with a name being either redundant with the test file itself or is too misleading. Regular expression used to find isolated suites: ```js /^\s*import.*(?:\n(?!describe\()(?!test\().*)+\ndescribe\("[\w\s.]+",.*(?:\n(?!describe\().*)+\n\}\);\n(?!\n)/ ``` Tip for reviewers: tick **"Hide whitespace"** when reviewing diff <img width="237" height="205" alt="image" src="https://github.com/user-attachments/assets/373f7cc6-5e0e-4599-968a-39aad8fac2f0" /> Enterprise: https://github.com/odoo/enterprise/pull/98046 --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr Forward-Port-Of: odoo/odoo#233207 Forward-Port-Of: odoo/odoo#233006

This commit removes test files using a single 'describe' call to wrap its tests, with a name being either redundant with the test file itself or is too misleading. Regular expression used to find isolated suites: ```js /^\s*import.*(?:\n(?!describe\()(?!test\().*)+\ndescribe\("[\w\s.]+",.*(?:\n(?!describe\().*)+\n\}\);\n(?!\n)/ ``` Tip for reviewers: tick **"Hide whitespace"** when reviewing diff <img width="237" height="205" alt="image" src="https://github.com/user-attachments/assets/373f7cc6-5e0e-4599-968a-39aad8fac2f0" /> Community: https://github.com/odoo/odoo/pull/233006 Forward-Port-Of: odoo/enterprise#98171 Forward-Port-Of: odoo/enterprise#98046