Skip to content

[IMP] tests: no empty records - #2878

Closed
abdrahmanrashed wants to merge 1 commit into
odoo:19.0from
odoo-dev:19.0-tests-no-empty-records-aras
Closed

[IMP] tests: no empty records#2878
abdrahmanrashed wants to merge 1 commit into
odoo:19.0from
odoo-dev:19.0-tests-no-empty-records-aras

Conversation

@abdrahmanrashed

Copy link
Copy Markdown
Contributor

This commit adds a test to ensure no records exists without any fields in it

task-6515434

@robodoo

robodoo commented Aug 28, 2026

Copy link
Copy Markdown
Collaborator

Pull request status dashboard

@vava-odoo vava-odoo left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

you can now fix both ci:

  • industry with the failing files 🙂
  • style with you logger formatting

@abdrahmanrashed
abdrahmanrashed force-pushed the 19.0-tests-no-empty-records-aras branch 2 times, most recently from 21a45fd to 56a1aeb Compare September 1, 2026 08:01

@vava-odoo vava-odoo left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

almost there, only need to apply it on physical_therapy as well

Image

Comment thread mental_therapy/data/res_config_settings.xml
This commit adds a test to ensure no records exists without any fields in it

task-6515434
@abdrahmanrashed
abdrahmanrashed force-pushed the 19.0-tests-no-empty-records-aras branch from 56a1aeb to cefab78 Compare September 1, 2026 09:28
@vava-odoo

Copy link
Copy Markdown
Collaborator

@robodoo r+

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants