From add0c02edf266da8b691b17d9e06c2dd34e5a4cb Mon Sep 17 00:00:00 2001 From: Kiril Vangelovski Date: Mon, 8 Aug 2022 13:40:41 +0200 Subject: [PATCH] [FIX] base_user_role: tests - include `trans_implied_ids` The unlink tests miss to add `trans_implied_ids` which is accounted for when reseting the groups. --- base_user_role/tests/test_user_role.py | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/base_user_role/tests/test_user_role.py b/base_user_role/tests/test_user_role.py index 9f63f9f0..800add0d 100644 --- a/base_user_role/tests/test_user_role.py +++ b/base_user_role/tests/test_user_role.py @@ -137,7 +137,9 @@ class TestUserRole(TransactionCase): def test_role_unlink(self): # Get role1 groups - role1_group_ids = self.role1_id.implied_ids.ids + role1_group_ids = ( + self.role1_id.implied_ids.ids + self.role1_id.trans_implied_ids.ids + ) role1_group_ids.append(self.role1_id.group_id.id) role1_group_ids = sorted(set(role1_group_ids)) @@ -161,7 +163,9 @@ class TestUserRole(TransactionCase): def test_role_line_unlink(self): # Get role1 groups - role1_group_ids = self.role1_id.implied_ids.ids + role1_group_ids = ( + self.role1_id.implied_ids.ids + self.role1_id.trans_implied_ids.ids + ) role1_group_ids.append(self.role1_id.group_id.id) role1_group_ids = sorted(set(role1_group_ids))