Skip to content

[17.0][FIX] password_security: Bypass password security checks for other modules tests - #999

Merged
OCA-git-bot merged 1 commit into
OCA:17.0from
Tecnativa:17.0-fix-password_security-tests
Sep 7, 2026
Merged

[17.0][FIX] password_security: Bypass password security checks for other modules tests#999
OCA-git-bot merged 1 commit into
OCA:17.0from
Tecnativa:17.0-fix-password_security-tests

Conversation

@Andrii9090-tecnativa

Copy link
Copy Markdown
Contributor

In the test cases of other modules, we use sample passwords for test users.
These tests fail when password_security module is installed.

@pedrobaeza @adasatorres-tecnativa can you review please

@Tecnativa

@OCA-git-bot OCA-git-bot added series:17.0 mod:password_security Module password_security labels Sep 7, 2026

@adasatorres-tecnativa adasatorres-tecnativa left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM, Code review

@pedrobaeza pedrobaeza added this to the 17.0 milestone Sep 7, 2026

@pedrobaeza pedrobaeza left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

/ocabot merge patch

@OCA-git-bot

Copy link
Copy Markdown
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 17.0-ocabot-merge-pr-999-by-pedrobaeza-bump-patch, awaiting test results.

@OCA-git-bot
OCA-git-bot merged commit 608594c into OCA:17.0 Sep 7, 2026
6 of 7 checks passed
@OCA-git-bot

Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at ec7edc8. Thanks a lot for contributing to OCA. ❤️

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants