Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[17.0] base_role_user : failing tests on using OCA CI #327

Open
bouktaibMohssin opened this issue Jan 7, 2025 · 0 comments
Open

[17.0] base_role_user : failing tests on using OCA CI #327

bouktaibMohssin opened this issue Jan 7, 2025 · 0 comments
Labels

Comments

@bouktaibMohssin
Copy link

bouktaibMohssin commented Jan 7, 2025

Module

base_role_user

Describe the bug

There is an issue with the ordering of lists in the base_role_user module that is causing a test failure. The error occurs when the test test_group_groups_into_role checks if the implied_ids.ids from a new role matches the user_group_ids. Although the lists contain the same elements, they are in a different order, which leads to an AssertionError.
image

To Reproduce

Affected versions: 17

Steps to reproduce the behavior:

  1. Run the tests using the OCA CI for the base_role_user module.

  2. Compare the two lists that are expected to be identical.
    Example:
    The lists are as follows:

    List 1 (from new_role.implied_ids.ids): [8, 28, 35, 60, 23, 45, 16, 26, 27, 9, 1, 13, 46, ... , 21]
    List 2 (from user_group_ids): [8, 28, 23, 45, 35, 60, 16, 26, 27, 9, 1, 13, 46, ... , 21]
    The first differing element is at index 2:

    List 1 has 35 at index 2, while List 2 has 23 at that position.

Expected behavior
The two lists (new_role.implied_ids.ids and user_group_ids) should be identical, with the elements appearing in the same order.

Additional context
test function that cause where the error comes from

self.assertEqual(new_role.implied_ids.ids, user_group_ids)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant