From 69959a6bbf0d1d1a9f84786af696a69444d14b86 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 2be3c436..a4803d99 100644 --- a/base_user_role/tests/test_user_role.py +++ b/base_user_role/tests/test_user_role.py @@ -105,7 +105,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)) @@ -129,7 +131,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))