Skip to content

[19.0][IMP] tracking_manager_domain: add missing tests for _tm_post_message branches#3626

Closed
NICO-SOLUTIONS wants to merge 9 commits into
OCA:19.0from
NICO-SOLUTIONS:19.0-imp-tracking_manager_domain
Closed

[19.0][IMP] tracking_manager_domain: add missing tests for _tm_post_message branches#3626
NICO-SOLUTIONS wants to merge 9 commits into
OCA:19.0from
NICO-SOLUTIONS:19.0-imp-tracking_manager_domain

Conversation

@NICO-SOLUTIONS

@NICO-SOLUTIONS NICO-SOLUTIONS commented May 25, 2026

Copy link
Copy Markdown
Member

Cover missing cases:

  • empty tracking_domain (continue branch)
  • field removal via domain mismatch
  • missing records handling
  • full keep-all scenario

Ensures stability of tracking domain filtering logic.

Depends on:

@OCA-git-bot OCA-git-bot added series:19.0 mod:tracking_manager_domain Module tracking_manager_domain labels May 25, 2026
… branches

Cover missing cases:
- empty tracking_domain (continue branch)
- field removal via domain mismatch
- missing records handling
- full keep-all scenario

Ensures stability of tracking domain filtering logic.
@NICO-SOLUTIONS NICO-SOLUTIONS force-pushed the 19.0-imp-tracking_manager_domain branch from b5897ec to 81e848c Compare May 25, 2026 10:49
@NICO-SOLUTIONS NICO-SOLUTIONS changed the title [19.0][IMP] tracking_manager_domain: add missing test coverage for _tm_post_message branches [19.0][IMP] tracking_manager_domain: add missing tests for _tm_post_message branches May 25, 2026
@NICO-SOLUTIONS NICO-SOLUTIONS marked this pull request as ready for review May 25, 2026 10:53
@NICO-SOLUTIONS

Copy link
Copy Markdown
Member Author

@CRogos
This PR adds the mentioned test extension. Feel free to review.

@CRogos

CRogos commented May 26, 2026

Copy link
Copy Markdown
Contributor

I've added the tests to the migration.

@NICO-SOLUTIONS

Copy link
Copy Markdown
Member Author

@CRogos
alright!

@NICO-SOLUTIONS NICO-SOLUTIONS deleted the 19.0-imp-tracking_manager_domain branch May 29, 2026 10:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

mod:tracking_manager_domain Module tracking_manager_domain series:19.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants