Skip to content

Commit 335bfdc

Browse files
committed
Check the default role list before adding a new one
The default role list now adds "manager" by default. This new default role was added in Keystone 24.0.0.0rc1, that includes [1]. [1]https://review.opendev.org/c/openstack/keystone/+/822601 Story: #2010923 Task: #48787 Change-Id: I3eeca1a0296137eef777770bff4993a56e2fcc5c
1 parent 2737e06 commit 335bfdc

1 file changed

Lines changed: 11 additions & 3 deletions

File tree

openstackclient/tests/functional/identity/v3/test_role.py

Lines changed: 11 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -146,11 +146,19 @@ def test_role_remove(self):
146146
self.assertEqual(0, len(remove_raw_output))
147147

148148
def test_implied_role_list(self):
149+
raw_output = self.openstack('implied role list')
150+
default_roles = self.parse_listing(raw_output)
151+
self.assert_table_structure(
152+
default_roles, self.IMPLIED_ROLE_LIST_HEADERS
153+
)
154+
149155
self._create_dummy_implied_role()
150156
raw_output = self.openstack('implied role list')
151-
items = self.parse_listing(raw_output)
152-
self.assert_table_structure(items, self.IMPLIED_ROLE_LIST_HEADERS)
153-
self.assertEqual(3, len(items))
157+
current_roles = self.parse_listing(raw_output)
158+
self.assert_table_structure(
159+
current_roles, self.IMPLIED_ROLE_LIST_HEADERS
160+
)
161+
self.assertEqual(len(default_roles) + 1, len(current_roles))
154162

155163
def test_implied_role_create(self):
156164
role_name = self._create_dummy_role()

0 commit comments

Comments
 (0)