audrius/server-backend
1
0
Fork 0
mirror of https://github.com/OCA/server-backend.git synced 2025-02-18 09:52:42 +02:00
Code Issues Packages Projects Releases Wiki Activity

[FIX] base_user_role: Improve tests resiliency

Browse Source 
These 2 tests were checking the exact set of groups a user should have.

If these tests are ran in a database where a module is previously installed which adds more groups to the base role, these exact group sets would be inexact, although the behavior that is being tested was actually properly working.

With this patch, basically I'm testing if the user contains the groups from the roles, not the exact role set expected. It should work in integration scenarios.

@Tecnativa TT20468
This commit is contained in:
Jairo Llopis
2019-11-13 11:48:09 +00:00
committed by Pedro M. Baeza
parent 37a0e1c8ab
commit 644a08b80f
1 changed files with 24 additions and 22 deletions
Download Patch File Download Diff File Expand all files Collapse all files

46
base_user_role/tests/test_user_role.py
View File

@@ -95,52 +95,54 @@ class TestUserRole(TransactionCase):
self.assertEqual(user_group_ids, role_group_ids) self.assertEqual(user_group_ids, role_group_ids)
def test_role_unlink(self): def test_role_unlink(self):
# Get role1 groups # Get role1 and role2 groups
role1_group_ids = self.role1_id.implied_ids.ids role1_groups = self.role1_id.implied_ids | self.role1_id.group_id
role1_group_ids.append(self.role1_id.group_id.id) role2_groups = self.role2_id.implied_ids | self.role2_id.group_id
role1_group_ids = sorted(set(role1_group_ids))
# Configure the user with role1 and role2 # Configure the user with role1 and role2
self.user_id.write( self.user_id.write(
{'role_line_ids': [ {'role_line_ids': [
(0, 0, {'role_id': self.role1_id.id}), (0, 0, {'role_id': self.role1_id.id}),
(0, 0, {'role_id': self.role2_id.id}), (0, 0, {'role_id': self.role2_id.id}),
]}) ]})
# Check user has groups from role1 and role2
self.assertLessEqual(role1_groups, self.user_id.groups_id)
self.assertLessEqual(role2_groups, self.user_id.groups_id)
# Remove role2 # Remove role2
self.role2_id.unlink() self.role2_id.unlink()
user_group_ids = sorted(set([ # Check user has groups from only role1
group.id for group in self.user_id.groups_id])) self.assertLessEqual(role1_groups, self.user_id.groups_id)
self.assertEqual(user_group_ids, role1_group_ids) self.assertFalse(role2_groups <= self.user_id.groups_id)
# Remove role1 # Remove role1
self.role1_id.unlink() self.role1_id.unlink()
user_group_ids = sorted(set([ # Check user has no groups from role1 and role2
group.id for group in self.user_id.groups_id])) self.assertFalse(role1_groups <= self.user_id.groups_id)
self.assertEqual(user_group_ids, []) self.assertFalse(role2_groups <= self.user_id.groups_id)
def test_role_line_unlink(self): def test_role_line_unlink(self):
# Get role1 groups # Get role1 and role2 groups
role1_group_ids = self.role1_id.implied_ids.ids role1_groups = self.role1_id.implied_ids | self.role1_id.group_id
role1_group_ids.append(self.role1_id.group_id.id) role2_groups = self.role2_id.implied_ids | self.role2_id.group_id
role1_group_ids = sorted(set(role1_group_ids))
# Configure the user with role1 and role2 # Configure the user with role1 and role2
self.user_id.write( self.user_id.write(
{'role_line_ids': [ {'role_line_ids': [
(0, 0, {'role_id': self.role1_id.id}), (0, 0, {'role_id': self.role1_id.id}),
(0, 0, {'role_id': self.role2_id.id}), (0, 0, {'role_id': self.role2_id.id}),
]}) ]})
# Check user has groups from role1 and role2
self.assertLessEqual(role1_groups, self.user_id.groups_id)
self.assertLessEqual(role2_groups, self.user_id.groups_id)
# Remove role2 from the user # Remove role2 from the user
self.user_id.role_line_ids.filtered( self.user_id.role_line_ids.filtered(
lambda l: l.role_id.id == self.role2_id.id).unlink() lambda l: l.role_id.id == self.role2_id.id).unlink()
user_group_ids = sorted(set([ # Check user has groups from only role1
group.id for group in self.user_id.groups_id])) self.assertLessEqual(role1_groups, self.user_id.groups_id)
self.assertEqual(user_group_ids, role1_group_ids) self.assertFalse(role2_groups <= self.user_id.groups_id)
# Remove role1 from the user # Remove role1 from the user
self.user_id.role_line_ids.filtered( self.user_id.role_line_ids.filtered(
lambda l: l.role_id.id == self.role1_id.id).unlink() lambda l: l.role_id.id == self.role1_id.id).unlink()
user_group_ids = sorted(set([ # Check user has no groups from role1 and role2
group.id for group in self.user_id.groups_id])) self.assertFalse(role1_groups <= self.user_id.groups_id)
self.assertEqual(user_group_ids, []) self.assertFalse(role2_groups <= self.user_id.groups_id)
def test_default_user_roles(self): def test_default_user_roles(self):
self.default_user.write({ self.default_user.write({