Renames switchToRbacRole to toggle_rbac_role
This is a trivial change that renames switchToRbacRole to
toggle_rbac_role.
This change is needed because:
- switchToRbacRole is camel case which is inconsistent with
all the snake-case variables names throughout Patrole and
OpenStack
- It violates Python's naming conventions [0].
[0] See pep8 styling guideline:
"Function names should be lowercase, with words separated
by underscores as necessary to improve readability. mixedCase
is allowed only in contexts where that's already the prevailing
style."
And for variable names:
"Use the function naming rules: lowercase with words separated
by underscores as necessary to improve readability."
Change-Id: Ib6dd50ba02c25d585e2ca0c9383771f9337212b8
Depends-On: Ie1c4b6a2801f10c8e83f6d3f88fe0364d69317fadiff --git a/patrole_tempest_plugin/rbac_rule_validation.py b/patrole_tempest_plugin/rbac_rule_validation.py
index 9b959d7..ec63119 100644
--- a/patrole_tempest_plugin/rbac_rule_validation.py
+++ b/patrole_tempest_plugin/rbac_rule_validation.py
@@ -123,7 +123,7 @@
(CONF.rbac.rbac_test_role, rule))
finally:
caller_ref.rbac_utils.switch_role(caller_ref,
- switchToRbacRole=False)
+ toggle_rbac_role=False)
return wrapper
return decorator
diff --git a/patrole_tempest_plugin/rbac_utils.py b/patrole_tempest_plugin/rbac_utils.py
index 70fe0b7..4bfa7fe 100644
--- a/patrole_tempest_plugin/rbac_utils.py
+++ b/patrole_tempest_plugin/rbac_utils.py
@@ -42,7 +42,7 @@
@six.add_metaclass(Singleton)
class RbacUtils(object):
- # References the last value of `switch_to_rbac_role` that was passed to
+ # References the last value of `toggle_rbac_role` that was passed to
# `switch_role`. Used for ensuring that `switch_role` is correctly used
# in a test file, so that false positives are prevented. The key used
# to index into the dictionary is the module path plus class name, which is
@@ -51,7 +51,7 @@
admin_role_id = None
rbac_role_id = None
- def switch_role(self, test_obj, switchToRbacRole=False):
+ def switch_role(self, test_obj, toggle_rbac_role=False):
self.user_id = test_obj.auth_provider.credentials.user_id
self.project_id = test_obj.auth_provider.credentials.tenant_id
self.token = test_obj.auth_provider.get_token()
@@ -62,15 +62,15 @@
else:
self.roles_client = test_obj.os_admin.roles_client
- LOG.debug('Switching role to: %s', switchToRbacRole)
+ LOG.debug('Switching role to: %s', toggle_rbac_role)
try:
if not self.admin_role_id or not self.rbac_role_id:
self._get_roles()
- rbac_utils._validate_switch_role(self, test_obj, switchToRbacRole)
+ rbac_utils._validate_switch_role(self, test_obj, toggle_rbac_role)
- if switchToRbacRole:
+ if toggle_rbac_role:
self._add_role_to_user(self.rbac_role_id)
else:
self._add_role_to_user(self.admin_role_id)
@@ -116,7 +116,7 @@
return False
- def _validate_switch_role(self, test_obj, switchToRbacRole):
+ def _validate_switch_role(self, test_obj, toggle_rbac_role):
"""Validates that the rbac role passed to `switch_role` is legal.
Throws an error for the following improper usages of `switch_role`:
@@ -124,9 +124,9 @@
* `switch_role` is never called in a test file, except in tearDown
* `switch_role` is called with the same boolean value twice
"""
- if not isinstance(switchToRbacRole, bool):
+ if not isinstance(toggle_rbac_role, bool):
raise rbac_exceptions.RbacResourceSetupFailed(
- 'switchToRbacRole must be a boolean value.')
+ 'toggle_rbac_role must be a boolean value.')
# The unique key is the combination of module path plus class name.
class_name = test_obj.__name__ if isinstance(test_obj, type) else \
@@ -136,19 +136,19 @@
self.switch_role_history.setdefault(key, None)
- if self.switch_role_history[key] == switchToRbacRole:
+ if self.switch_role_history[key] == toggle_rbac_role:
# If the test was skipped, then this is a legitimate use case,
# so do not throw an exception.
exc_value = sys.exc_info()[1]
if not isinstance(exc_value, testtools.TestCase.skipException):
self.switch_role_history[key] = False
- error_message = '`switchToRbacRole` must not be called with '\
+ error_message = '`toggle_rbac_role` must not be called with '\
'the same bool value twice. Make sure that you included '\
'a rbac_utils.switch_role method call inside the test.'
LOG.error(error_message)
raise rbac_exceptions.RbacResourceSetupFailed(error_message)
else:
- self.switch_role_history[key] = switchToRbacRole
+ self.switch_role_history[key] = toggle_rbac_role
def _get_roles(self):
available_roles = self.roles_client.list_roles()
diff --git a/patrole_tempest_plugin/tests/api/compute/admin/test_admin_password_rbac.py b/patrole_tempest_plugin/tests/api/compute/admin/test_admin_password_rbac.py
index 561bdae..2c9809e 100644
--- a/patrole_tempest_plugin/tests/api/compute/admin/test_admin_password_rbac.py
+++ b/patrole_tempest_plugin/tests/api/compute/admin/test_admin_password_rbac.py
@@ -51,7 +51,7 @@
service="nova", rule="os_compute_api:os-admin-password")
@decorators.idempotent_id('908a7d59-3a66-441c-94cf-38e57ed14956')
def test_change_server_password(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.change_password(
self.server_id,
adminPass=data_utils.rand_password())
diff --git a/patrole_tempest_plugin/tests/api/compute/admin/test_admin_server_actions_rbac.py b/patrole_tempest_plugin/tests/api/compute/admin/test_admin_server_actions_rbac.py
index 89190fc..2f0d03e 100644
--- a/patrole_tempest_plugin/tests/api/compute/admin/test_admin_server_actions_rbac.py
+++ b/patrole_tempest_plugin/tests/api/compute/admin/test_admin_server_actions_rbac.py
@@ -47,7 +47,7 @@
rule="os_compute_api:os-admin-actions:reset_state")
@decorators.idempotent_id('ae84dd0b-f364-462e-b565-3457f9c019ef')
def test_reset_server_state(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.reset_state(self.server_id, state='error')
self.addCleanup(self.client.reset_state,
self.server_id,
@@ -58,7 +58,7 @@
rule="os_compute_api:os-admin-actions:inject_network_info")
@decorators.idempotent_id('ce48c340-51c1-4cff-9b6e-0cc5ef008630')
def test_inject_network_info(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.inject_network_info(self.server_id)
@rbac_rule_validation.action(
@@ -66,5 +66,5 @@
rule="os_compute_api:os-admin-actions:reset_network")
@decorators.idempotent_id('2911a242-15c4-4fcb-80d5-80a8930661b0')
def test_reset_network(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.reset_network(self.server_id)
diff --git a/patrole_tempest_plugin/tests/api/compute/rbac_base.py b/patrole_tempest_plugin/tests/api/compute/rbac_base.py
index e5d7d9c..9da50c6 100644
--- a/patrole_tempest_plugin/tests/api/compute/rbac_base.py
+++ b/patrole_tempest_plugin/tests/api/compute/rbac_base.py
@@ -38,7 +38,7 @@
super(BaseV2ComputeRbacTest, cls).setup_clients()
cls.auth_provider = cls.os.auth_provider
cls.rbac_utils = rbac_utils()
- cls.rbac_utils.switch_role(cls, switchToRbacRole=False)
+ cls.rbac_utils.switch_role(cls, toggle_rbac_role=False)
class BaseV2ComputeAdminRbacTest(compute_base.BaseV2ComputeAdminTest):
@@ -57,7 +57,7 @@
super(BaseV2ComputeAdminRbacTest, cls).setup_clients()
cls.auth_provider = cls.os.auth_provider
cls.rbac_utils = rbac_utils()
- cls.rbac_utils.switch_role(cls, switchToRbacRole=False)
+ cls.rbac_utils.switch_role(cls, toggle_rbac_role=False)
@classmethod
def resource_setup(cls):
diff --git a/patrole_tempest_plugin/tests/api/compute/test_agents_rbac.py b/patrole_tempest_plugin/tests/api/compute/test_agents_rbac.py
index 76120f6..f355358 100644
--- a/patrole_tempest_plugin/tests/api/compute/test_agents_rbac.py
+++ b/patrole_tempest_plugin/tests/api/compute/test_agents_rbac.py
@@ -33,7 +33,7 @@
service="nova", rule="os_compute_api:os-agents")
@decorators.idempotent_id('d1bc6d97-07f5-4f45-ac29-1c619a6a7e27')
def test_list_agents_rbac(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.agents_client.list_agents()
@rbac_rule_validation.action(
@@ -44,7 +44,7 @@
params = {'hypervisor': 'kvm', 'os': 'win', 'architecture': 'x86',
'version': '7.0', 'url': 'xxx://xxxx/xxx/xxx',
'md5hash': 'add6bb58e139be103324d04d82d8f545'}
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
body = self.agents_client.create_agent(**params)['agent']
self.addCleanup(self.agents_client.delete_agent,
body['agent_id'])
diff --git a/patrole_tempest_plugin/tests/api/compute/test_aggregates_rbac.py b/patrole_tempest_plugin/tests/api/compute/test_aggregates_rbac.py
index 7c3d653..a180bc4 100644
--- a/patrole_tempest_plugin/tests/api/compute/test_aggregates_rbac.py
+++ b/patrole_tempest_plugin/tests/api/compute/test_aggregates_rbac.py
@@ -60,7 +60,7 @@
service="nova", rule="os_compute_api:os-aggregates:create")
@decorators.idempotent_id('ba754393-896e-434a-9704-452ff4a84f3f')
def test_create_aggregate_rbac(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self._create_aggregate()
@rbac_rule_validation.action(
@@ -68,14 +68,14 @@
@decorators.idempotent_id('8fb0b749-b120-4727-b3fb-bcfa3fa6f55b')
def test_show_aggregate_rbac(self):
aggregate_id = self._create_aggregate()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.aggregates_client.show_aggregate(aggregate_id)
@rbac_rule_validation.action(
service="nova", rule="os_compute_api:os-aggregates:index")
@decorators.idempotent_id('146284da-5dd6-4c97-b598-42b480f014c6')
def test_list_aggregate_rbac(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.aggregates_client.list_aggregates()['aggregates']
@rbac_rule_validation.action(
@@ -83,7 +83,7 @@
@decorators.idempotent_id('c94e0d69-99b6-477e-b301-2cd0e9d0ad81')
def test_update_aggregate_rbac(self):
aggregate_id = self._create_aggregate()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
new_name = data_utils.rand_name('aggregate')
self.aggregates_client.update_aggregate(aggregate_id, name=new_name)
@@ -92,7 +92,7 @@
@decorators.idempotent_id('5a50c5a6-0f12-4405-a1ce-2288ae895ea6')
def test_delete_aggregate_rbac(self):
aggregate_id = self._create_aggregate()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.aggregates_client.delete_aggregate(aggregate_id)
@rbac_rule_validation.action(
@@ -100,7 +100,7 @@
@decorators.idempotent_id('97e6e9df-5291-4faa-8147-755b2d1f1ce2')
def test_add_host_to_aggregate_rbac(self):
aggregate_id = self._create_aggregate()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self._add_host_to_aggregate(aggregate_id)
@rbac_rule_validation.action(
@@ -109,7 +109,7 @@
def test_remove_host_from_aggregate_rbac(self):
aggregate_id = self._create_aggregate()
host_name = self._add_host_to_aggregate(aggregate_id)
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.aggregates_client.remove_host(aggregate_id, host=host_name)
@rbac_rule_validation.action(
@@ -119,7 +119,7 @@
aggregate_id = self._create_aggregate()
rand_key = data_utils.rand_name('key')
rand_val = data_utils.rand_name('val')
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.aggregates_client.set_metadata(
aggregate_id,
metadata={rand_key: rand_val})
diff --git a/patrole_tempest_plugin/tests/api/compute/test_assisted_volume_snapshot_rbac.py b/patrole_tempest_plugin/tests/api/compute/test_assisted_volume_snapshot_rbac.py
index 9998bea..37e8573 100644
--- a/patrole_tempest_plugin/tests/api/compute/test_assisted_volume_snapshot_rbac.py
+++ b/patrole_tempest_plugin/tests/api/compute/test_assisted_volume_snapshot_rbac.py
@@ -49,7 +49,7 @@
RBAC test for assisted volume snapshot role-create
"""
self._create_and_attach()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.assisted_volume_snapshot_client.\
create_volume_attachments(self.volume['id'],
data_utils.rand_uuid())
@@ -68,6 +68,6 @@
snapshot_id = data_utils.rand_uuid()
self.assisted_volume_snapshot_client.\
create_volume_attachments(self.volume['id'], snapshot_id)
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.assisted_volume_snapshot_client.\
delete_volume_attachments(snapshot_id, self.volume['id'])
diff --git a/patrole_tempest_plugin/tests/api/compute/test_attach_interfaces_rbac.py b/patrole_tempest_plugin/tests/api/compute/test_attach_interfaces_rbac.py
index bfaeea6..7241c98 100644
--- a/patrole_tempest_plugin/tests/api/compute/test_attach_interfaces_rbac.py
+++ b/patrole_tempest_plugin/tests/api/compute/test_attach_interfaces_rbac.py
@@ -72,7 +72,7 @@
service="nova",
rule="os_compute_api:os-attach-interfaces")
def test_list_interfaces(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.list_interfaces(self.server['id'])['interfaceAttachments']
@decorators.idempotent_id('d2d3a24d-4738-4bce-a287-36d664746cde')
@@ -80,7 +80,7 @@
service="nova",
rule="os_compute_api:os-attach-interfaces:create")
def test_create_interface(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self._attach_interface_to_server()
@decorators.idempotent_id('55b05692-ed44-4608-a84c-cd4219c82799')
@@ -89,5 +89,5 @@
rule="os_compute_api:os-attach-interfaces:delete")
def test_delete_interface(self):
interface = self._attach_interface_to_server()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.delete_interface(self.server['id'], interface['port_id'])
diff --git a/patrole_tempest_plugin/tests/api/compute/test_availability_zone_rbac.py b/patrole_tempest_plugin/tests/api/compute/test_availability_zone_rbac.py
index 5078d43..c466ad2 100644
--- a/patrole_tempest_plugin/tests/api/compute/test_availability_zone_rbac.py
+++ b/patrole_tempest_plugin/tests/api/compute/test_availability_zone_rbac.py
@@ -33,12 +33,12 @@
"os-availability-zone:list")
@decorators.idempotent_id('cd34e7ea-d26e-4fa3-a8d0-f8883726ce3d')
def test_get_availability_zone_list_rbac(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.os.availability_zone_client.list_availability_zones()
@rbac_rule_validation.action(service="nova", rule="os_compute_api:"
"os-availability-zone:detail")
@decorators.idempotent_id('2f61c191-6ece-4f21-b487-39d749e3d38e')
def test_get_availability_zone_list_detail_rbac(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.os.availability_zone_client.list_availability_zones(detail=True)
diff --git a/patrole_tempest_plugin/tests/api/compute/test_config_drive_rbac.py b/patrole_tempest_plugin/tests/api/compute/test_config_drive_rbac.py
index e46d1ba..794e0d2 100644
--- a/patrole_tempest_plugin/tests/api/compute/test_config_drive_rbac.py
+++ b/patrole_tempest_plugin/tests/api/compute/test_config_drive_rbac.py
@@ -40,7 +40,7 @@
service="nova",
rule="os_compute_api:os-config-drive")
def test_create_test_server_with_config_drive(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
# NOTE(felipemonteiro): This policy action is always enforced,
# regardless whether the config_drive flag is set to true or false.
# However, it has been explicitly set to true below, in case that this
diff --git a/patrole_tempest_plugin/tests/api/compute/test_deferred_delete_rbac.py b/patrole_tempest_plugin/tests/api/compute/test_deferred_delete_rbac.py
index 79d2c52..fe9cb11 100644
--- a/patrole_tempest_plugin/tests/api/compute/test_deferred_delete_rbac.py
+++ b/patrole_tempest_plugin/tests/api/compute/test_deferred_delete_rbac.py
@@ -45,7 +45,7 @@
service="nova",
rule="os_compute_api:os-deferred-delete")
def test_force_delete_server(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
# Force-deleting a server enforces os-deferred-delete according to the
# following API: https://github.com/openstack/nova/blob/master/nova/api
# /openstack/compute/deferred_delete.py
diff --git a/patrole_tempest_plugin/tests/api/compute/test_flavor_access_rbac.py b/patrole_tempest_plugin/tests/api/compute/test_flavor_access_rbac.py
index 1704644..5c3f55a 100644
--- a/patrole_tempest_plugin/tests/api/compute/test_flavor_access_rbac.py
+++ b/patrole_tempest_plugin/tests/api/compute/test_flavor_access_rbac.py
@@ -56,7 +56,7 @@
def test_show_flavor(self):
# NOTE(felipemonteiro): show_flavor enforces the specified policy
# action, but only works if a public flavor is passed.
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.show_flavor(self.public_flavor_id)['flavor']
@decorators.idempotent_id('39cb5c8f-9990-436f-9282-fc76a41d9bac')
@@ -64,7 +64,7 @@
service="nova",
rule="os_compute_api:os-flavor-access:add_tenant_access")
def test_add_flavor_access(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.add_flavor_access(
flavor_id=self.flavor_id, tenant_id=self.tenant_id)[
'flavor_access']
@@ -81,6 +81,6 @@
self.addCleanup(test_utils.call_and_ignore_notfound_exc,
self.client.remove_flavor_access,
flavor_id=self.flavor_id, tenant_id=self.tenant_id)
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.remove_flavor_access(
flavor_id=self.flavor_id, tenant_id=self.tenant_id)
diff --git a/patrole_tempest_plugin/tests/api/compute/test_flavor_extra_specs_rbac.py b/patrole_tempest_plugin/tests/api/compute/test_flavor_extra_specs_rbac.py
index d742bec..0cf789b 100644
--- a/patrole_tempest_plugin/tests/api/compute/test_flavor_extra_specs_rbac.py
+++ b/patrole_tempest_plugin/tests/api/compute/test_flavor_extra_specs_rbac.py
@@ -64,7 +64,7 @@
rule="os_compute_api:os-flavor-extra-specs:show")
def test_show_flavor_extra_spec(self):
key = self._set_flavor_extra_spec()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.show_flavor_extra_spec(self.flavor['id'], key)[key]
@decorators.idempotent_id('fcffeca2-ed04-4e85-bf93-02fb5643f22b')
@@ -72,7 +72,7 @@
service="nova",
rule="os_compute_api:os-flavor-extra-specs:create")
def test_set_flavor_extra_spec(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self._set_flavor_extra_spec()
@decorators.idempotent_id('42b85279-6bfa-4f58-b7a2-258c284f03c5')
@@ -81,7 +81,7 @@
rule="os_compute_api:os-flavor-extra-specs:update")
def test_update_flavor_extra_spec(self):
key = self._set_flavor_extra_spec()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
update_val = data_utils.rand_name('val')
self.client.update_flavor_extra_spec(self.flavor['id'], key,
**{key: update_val})[key]
@@ -92,7 +92,7 @@
rule="os_compute_api:os-flavor-extra-specs:delete")
def test_unset_flavor_extra_spec(self):
key = self._set_flavor_extra_spec()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.unset_flavor_extra_spec(self.flavor['id'], key)
@decorators.idempotent_id('02c3831a-3ce9-476e-a722-d805ac2da621')
@@ -101,5 +101,5 @@
rule="os_compute_api:os-flavor-extra-specs:index")
def test_list_flavor_extra_specs(self):
self._set_flavor_extra_spec()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.list_flavor_extra_specs(self.flavor['id'])['extra_specs']
diff --git a/patrole_tempest_plugin/tests/api/compute/test_flavor_rxtx_rbac.py b/patrole_tempest_plugin/tests/api/compute/test_flavor_rxtx_rbac.py
index c8af2c2..bc33d3b 100644
--- a/patrole_tempest_plugin/tests/api/compute/test_flavor_rxtx_rbac.py
+++ b/patrole_tempest_plugin/tests/api/compute/test_flavor_rxtx_rbac.py
@@ -39,6 +39,6 @@
service="nova",
rule="os_compute_api:os-flavor-rxtx")
def test_create_flavor_rxtx(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
# Enforces os_compute_api:os-flavor-rxtx.
self.client.list_flavors(detail=True)['flavors']
diff --git a/patrole_tempest_plugin/tests/api/compute/test_floating_ip_pools_rbac.py b/patrole_tempest_plugin/tests/api/compute/test_floating_ip_pools_rbac.py
index acf7208..6d78878 100644
--- a/patrole_tempest_plugin/tests/api/compute/test_floating_ip_pools_rbac.py
+++ b/patrole_tempest_plugin/tests/api/compute/test_floating_ip_pools_rbac.py
@@ -48,5 +48,5 @@
service="nova",
rule="os_compute_api:os-floating-ip-pools")
def test_list_floating_ip_pools(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.list_floating_ip_pools()['floating_ip_pools']
diff --git a/patrole_tempest_plugin/tests/api/compute/test_floating_ips_bulk_rbac.py b/patrole_tempest_plugin/tests/api/compute/test_floating_ips_bulk_rbac.py
index e726298..ab9d551 100644
--- a/patrole_tempest_plugin/tests/api/compute/test_floating_ips_bulk_rbac.py
+++ b/patrole_tempest_plugin/tests/api/compute/test_floating_ips_bulk_rbac.py
@@ -48,5 +48,5 @@
service="nova",
rule="os_compute_api:os-floating-ips-bulk")
def test_list_floating_ips_bulk(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.list_floating_ips_bulk()['floating_ip_info']
diff --git a/patrole_tempest_plugin/tests/api/compute/test_floating_ips_rbac.py b/patrole_tempest_plugin/tests/api/compute/test_floating_ips_rbac.py
index 073b989..784a82d 100644
--- a/patrole_tempest_plugin/tests/api/compute/test_floating_ips_rbac.py
+++ b/patrole_tempest_plugin/tests/api/compute/test_floating_ips_rbac.py
@@ -48,5 +48,5 @@
service="nova",
rule="os_compute_api:os-floating-ips")
def test_list_floating_ips(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.list_floating_ips()['floating_ips']
diff --git a/patrole_tempest_plugin/tests/api/compute/test_hosts_rbac.py b/patrole_tempest_plugin/tests/api/compute/test_hosts_rbac.py
index 6532731..b03de11 100644
--- a/patrole_tempest_plugin/tests/api/compute/test_hosts_rbac.py
+++ b/patrole_tempest_plugin/tests/api/compute/test_hosts_rbac.py
@@ -41,5 +41,5 @@
service="nova",
rule="os_compute_api:os-hosts")
def test_list_hosts(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.list_hosts()['hosts']
diff --git a/patrole_tempest_plugin/tests/api/compute/test_hypervisor_rbac.py b/patrole_tempest_plugin/tests/api/compute/test_hypervisor_rbac.py
index d4b97a5..36ac916 100644
--- a/patrole_tempest_plugin/tests/api/compute/test_hypervisor_rbac.py
+++ b/patrole_tempest_plugin/tests/api/compute/test_hypervisor_rbac.py
@@ -40,5 +40,5 @@
service="nova",
rule="os_compute_api:os-hypervisors")
def test_list_hypervisors(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.list_hypervisors()['hypervisors']
diff --git a/patrole_tempest_plugin/tests/api/compute/test_images_rbac.py b/patrole_tempest_plugin/tests/api/compute/test_images_rbac.py
index 674eeb1..3a4f2d6 100644
--- a/patrole_tempest_plugin/tests/api/compute/test_images_rbac.py
+++ b/patrole_tempest_plugin/tests/api/compute/test_images_rbac.py
@@ -66,7 +66,7 @@
service="glance",
rule="get_images")
def test_list_images(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.list_images()
@decorators.idempotent_id('4365ae0f-15ee-4b54-a527-1679faaed140')
@@ -74,7 +74,7 @@
service="glance",
rule="get_images")
def test_list_images_with_details(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.list_images(detail=True)
@decorators.idempotent_id('886dfcae-51bf-4610-9e52-82d7189524c2')
@@ -82,7 +82,7 @@
service="glance",
rule="get_image")
def test_show_image_details(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.show_image(self.image['id'])
@decorators.idempotent_id('dbe09d4c-e615-48cb-b908-a06a0f410a8e')
@@ -94,7 +94,7 @@
self.addCleanup(self.client.delete_image_metadata_item,
self.image['id'], key='foo')
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.show_image_metadata_item(self.image['id'], key='foo')
@decorators.idempotent_id('59f66079-d564-47e8-81b0-03c2e84d339e')
@@ -102,7 +102,7 @@
service="glance",
rule="get_image")
def test_list_image_metadata(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.list_image_metadata(self.image['id'])
@decorators.idempotent_id('5888c7aa-0803-46d4-a3fb-5d4729465cd5')
@@ -115,7 +115,7 @@
self.addCleanup(test_utils.call_and_ignore_notfound_exc,
self.glance_image_client.delete_image, image['id'])
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.delete_image(image['id'])
@decorators.idempotent_id('575604aa-909f-4b1b-a5a5-cfae1f63044b')
@@ -123,7 +123,7 @@
service="glance",
rule="modify_image")
def test_create_image_metadata(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
# NOTE(felipemonteiro): Although the name of the client function
# appears wrong, it's actually correct: update_image_metadata does an
# http post.
@@ -137,7 +137,7 @@
service="glance",
rule="modify_image")
def test_update_image_metadata(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.set_image_metadata(self.image['id'], meta={'foo': 'bar'})
self.addCleanup(self.client.delete_image_metadata_item,
self.image['id'], key='foo')
@@ -147,7 +147,7 @@
service="glance",
rule="modify_image")
def test_update_image_metadata_item(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.set_image_metadata_item(self.image['id'],
meta={'foo': 'bar'}, key='foo')
self.addCleanup(self.client.delete_image_metadata_item,
@@ -163,5 +163,5 @@
self.client.delete_image_metadata_item,
self.image['id'], key='foo')
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.delete_image_metadata_item(self.image['id'], key='foo')
diff --git a/patrole_tempest_plugin/tests/api/compute/test_instance_actions_rbac.py b/patrole_tempest_plugin/tests/api/compute/test_instance_actions_rbac.py
index dcf3c90..bd1e5c8 100644
--- a/patrole_tempest_plugin/tests/api/compute/test_instance_actions_rbac.py
+++ b/patrole_tempest_plugin/tests/api/compute/test_instance_actions_rbac.py
@@ -46,7 +46,7 @@
service="nova",
rule="os_compute_api:os-instance-actions")
def test_list_instance_actions(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.list_instance_actions(self.server['id'])
@decorators.idempotent_id('eb04c439-4215-4029-9ccb-5b3c041bfc25')
@@ -54,7 +54,7 @@
service="nova",
rule="os_compute_api:os-instance-actions:events")
def test_get_instance_action(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
instance_action = self.client.show_instance_action(
self.server['id'], self.request_id)['instanceAction']
if 'events' not in instance_action:
diff --git a/patrole_tempest_plugin/tests/api/compute/test_instance_usages_audit_log_rbac.py b/patrole_tempest_plugin/tests/api/compute/test_instance_usages_audit_log_rbac.py
index e252054..7b116e4 100644
--- a/patrole_tempest_plugin/tests/api/compute/test_instance_usages_audit_log_rbac.py
+++ b/patrole_tempest_plugin/tests/api/compute/test_instance_usages_audit_log_rbac.py
@@ -40,6 +40,6 @@
@rbac_rule_validation.action(
service="nova", rule="os_compute_api:os-instance-usage-audit-log")
def test_list_instance_usage_audit_logs(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.list_instance_usage_audit_logs()
["instance_usage_audit_logs"]
diff --git a/patrole_tempest_plugin/tests/api/compute/test_ips_rbac.py b/patrole_tempest_plugin/tests/api/compute/test_ips_rbac.py
index 8ecc08c..96fa243 100644
--- a/patrole_tempest_plugin/tests/api/compute/test_ips_rbac.py
+++ b/patrole_tempest_plugin/tests/api/compute/test_ips_rbac.py
@@ -54,7 +54,7 @@
service="nova",
rule="os_compute_api:ips:index")
def test_list_addresses(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.list_addresses(self.server['id'])['addresses']
@decorators.idempotent_id('fa43e7e5-0db9-48eb-9c6b-c11eb766b8e4')
@@ -64,6 +64,6 @@
def test_list_addresses_by_network(self):
addresses = self.client.list_addresses(self.server['id'])['addresses']
address = next(iter(addresses))
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.list_addresses_by_network(
self.server['id'], address)[address]
diff --git a/patrole_tempest_plugin/tests/api/compute/test_keypairs_rbac.py b/patrole_tempest_plugin/tests/api/compute/test_keypairs_rbac.py
index df470c6..f7b2ace 100644
--- a/patrole_tempest_plugin/tests/api/compute/test_keypairs_rbac.py
+++ b/patrole_tempest_plugin/tests/api/compute/test_keypairs_rbac.py
@@ -41,7 +41,7 @@
service="nova",
rule="os_compute_api:os-keypairs:create")
def test_create_keypair(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self._create_keypair()
@decorators.idempotent_id('85a5eb99-40ec-4e77-9358-bee2cdf9d7df')
@@ -50,7 +50,7 @@
rule="os_compute_api:os-keypairs:show")
def test_show_keypair(self):
kp_name = self._create_keypair()['keypair']['name']
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.show_keypair(kp_name)
@decorators.idempotent_id('6bff9f1c-b809-43c1-8d63-61fbd19d49d3')
@@ -59,7 +59,7 @@
rule="os_compute_api:os-keypairs:delete")
def test_delete_keypair(self):
kp_name = self._create_keypair()['keypair']['name']
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.delete_keypair(kp_name)
@decorators.idempotent_id('6bb31346-ff7f-4b10-978e-170ac5fcfa3e')
@@ -67,5 +67,5 @@
service="nova",
rule="os_compute_api:os-keypairs:index")
def test_index_keypair(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.list_keypairs()
diff --git a/patrole_tempest_plugin/tests/api/compute/test_limits_rbac.py b/patrole_tempest_plugin/tests/api/compute/test_limits_rbac.py
index fb631e6..eb0e686 100644
--- a/patrole_tempest_plugin/tests/api/compute/test_limits_rbac.py
+++ b/patrole_tempest_plugin/tests/api/compute/test_limits_rbac.py
@@ -38,5 +38,5 @@
rule="os_compute_api:limits")
@decorators.idempotent_id('3fb60f83-9a5f-4fdd-89d9-26c3710844a1')
def test_show_limits(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.show_limits()
diff --git a/patrole_tempest_plugin/tests/api/compute/test_migrations_rbac.py b/patrole_tempest_plugin/tests/api/compute/test_migrations_rbac.py
index 251b750..0a21bcb 100644
--- a/patrole_tempest_plugin/tests/api/compute/test_migrations_rbac.py
+++ b/patrole_tempest_plugin/tests/api/compute/test_migrations_rbac.py
@@ -41,5 +41,5 @@
service="nova",
rule="os_compute_api:os-migrations:index")
def test_list_services(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.list_migrations()['migrations']
diff --git a/patrole_tempest_plugin/tests/api/compute/test_multinic_rbac.py b/patrole_tempest_plugin/tests/api/compute/test_multinic_rbac.py
index 158fcae..7e353be 100644
--- a/patrole_tempest_plugin/tests/api/compute/test_multinic_rbac.py
+++ b/patrole_tempest_plugin/tests/api/compute/test_multinic_rbac.py
@@ -64,6 +64,6 @@
network_id = self.interfaces_client.create_interface(
self.server['id'])['interfaceAttachment']['net_id']
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.add_fixed_ip(self.server['id'],
networkId=network_id)
diff --git a/patrole_tempest_plugin/tests/api/compute/test_quota_sets_rbac.py b/patrole_tempest_plugin/tests/api/compute/test_quota_sets_rbac.py
index a7f8447..826ae8d 100644
--- a/patrole_tempest_plugin/tests/api/compute/test_quota_sets_rbac.py
+++ b/patrole_tempest_plugin/tests/api/compute/test_quota_sets_rbac.py
@@ -60,7 +60,7 @@
default_quota_set.pop('id')
new_quota_set = {'injected_file_content_bytes': 20480}
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.update_quota_set(self.tenant_id,
force=True,
**new_quota_set)['quota_set']
@@ -72,7 +72,7 @@
service="nova",
rule="os_compute_api:os-quota-sets:defaults")
def test_show_default_quota_set(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.show_default_quota_set(self.tenant_id)['quota_set']
@decorators.idempotent_id('e8169ac4-c402-4864-894e-aba74e3a459c')
@@ -80,7 +80,7 @@
service="nova",
rule="os_compute_api:os-quota-sets:show")
def test_show_quota_set(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.show_quota_set(self.tenant_id)['quota_set']
@decorators.idempotent_id('4e240644-bf61-4872-9c32-8289ee2fdbbd')
@@ -94,7 +94,7 @@
self.addCleanup(test_utils.call_and_ignore_notfound_exc,
self.projects_client.delete_project, project_id)
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.delete_quota_set(project_id)
@decorators.idempotent_id('ac9184b6-f3b3-4e17-a632-4b92c6500f86')
@@ -102,5 +102,5 @@
service="nova",
rule="os_compute_api:os-quota-sets:detail")
def test_show_quota_set_details(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.show_quota_set(self.tenant_id, detail=True)['quota_set']
diff --git a/patrole_tempest_plugin/tests/api/compute/test_rescue_rbac.py b/patrole_tempest_plugin/tests/api/compute/test_rescue_rbac.py
index c5fe31a..acf9bd6 100644
--- a/patrole_tempest_plugin/tests/api/compute/test_rescue_rbac.py
+++ b/patrole_tempest_plugin/tests/api/compute/test_rescue_rbac.py
@@ -46,5 +46,5 @@
rule="os_compute_api:os-rescue")
@decorators.idempotent_id('fbbb2afc-ed0e-4552-887d-ac00fb5d436e')
def test_rescue_server(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.rescue_server(self.server['id'])
diff --git a/patrole_tempest_plugin/tests/api/compute/test_security_groups_rbac.py b/patrole_tempest_plugin/tests/api/compute/test_security_groups_rbac.py
index d65e7e3..f45632e 100644
--- a/patrole_tempest_plugin/tests/api/compute/test_security_groups_rbac.py
+++ b/patrole_tempest_plugin/tests/api/compute/test_security_groups_rbac.py
@@ -26,5 +26,5 @@
rule="os_compute_api:os-security-groups")
@decorators.idempotent_id('4ac58e49-48c1-4fca-a6c3-3f95fb99eb77')
def test_server_security_groups(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.security_groups_client.list_security_groups()
diff --git a/patrole_tempest_plugin/tests/api/compute/test_server_actions_rbac.py b/patrole_tempest_plugin/tests/api/compute/test_server_actions_rbac.py
index d466ded..a47db68 100644
--- a/patrole_tempest_plugin/tests/api/compute/test_server_actions_rbac.py
+++ b/patrole_tempest_plugin/tests/api/compute/test_server_actions_rbac.py
@@ -163,7 +163,7 @@
rule="os_compute_api:servers:stop")
@decorators.idempotent_id('ab4a17d2-166f-4a6d-9944-f17baa576cf2')
def test_stop_server(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self._test_stop_server()
@test.attr(type='slow')
@@ -173,7 +173,7 @@
@decorators.idempotent_id('8876bfa9-4d10-406e-a335-a57e451abb12')
def test_start_server(self):
self._test_stop_server()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self._test_start_server()
@test.attr(type='slow')
@@ -184,7 +184,7 @@
@testtools.skipUnless(CONF.compute_feature_enabled.resize,
'Resize is not available.')
def test_resize_server(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self._test_resize_server(self.flavor_ref_alt)
@test.attr(type='slow')
@@ -196,7 +196,7 @@
'Resize is not available.')
def test_revert_resize_server(self):
self._test_resize_server(self.flavor_ref_alt)
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self._test_revert_resize_server()
@test.attr(type='slow')
@@ -208,7 +208,7 @@
'Resize is not available.')
def test_confirm_resize_server(self):
self._test_resize_server(self.flavor_ref_alt)
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.addCleanup(
lambda: (self._test_resize_server(self.flavor_ref),
self._test_confirm_resize_server())
@@ -220,7 +220,7 @@
rule="os_compute_api:servers:rebuild")
@decorators.idempotent_id('54b1a30b-c96c-472c-9c83-ccaf6ec7e20b')
def test_rebuild_server(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.rebuild_server(self.server_id, self.image_ref)
waiters.wait_for_server_status(self.client, self.server_id,
'ACTIVE')
@@ -230,7 +230,7 @@
rule="os_compute_api:servers:reboot")
@decorators.idempotent_id('19f27856-56e1-44f8-8615-7257f6b85cbb')
def test_reboot_server(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.reboot_server(self.server_id, type='HARD')
waiters.wait_for_server_status(self.client, self.server_id,
'ACTIVE')
@@ -240,7 +240,7 @@
rule="os_compute_api:servers:index")
@decorators.idempotent_id('631f0d86-7607-4198-8312-9da2f05464a4')
def test_server_index(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.list_servers(minimal=True)
@rbac_rule_validation.action(
@@ -248,7 +248,7 @@
rule="os_compute_api:servers:detail")
@decorators.idempotent_id('96093480-3ce5-4a8b-b569-aed870379c24')
def test_server_detail(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.list_servers(detail=True)
@rbac_rule_validation.action(
@@ -256,7 +256,7 @@
rule="os_compute_api:servers:detail:get_all_tenants")
@decorators.idempotent_id('a9e5a1c0-acfe-49a2-b2b1-fd8b19d61f71')
def test_server_detail_all_tenants(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.list_servers(detail=True, all_tenants=1)
@rbac_rule_validation.action(
@@ -264,7 +264,7 @@
rule="os_compute_api:servers:index:get_all_tenants")
@decorators.idempotent_id('4b93ba56-69e6-41f5-82c4-84a5c4c42091')
def test_server_index_all_tenants(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.list_servers(minimal=True, all_tenants=1)
@rbac_rule_validation.action(
@@ -272,7 +272,7 @@
rule="os_compute_api:servers:show")
@decorators.idempotent_id('eaaf4f51-31b5-497f-8f0f-f527e5f70b83')
def test_show_server(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.show_server(self.server_id)
@rbac_rule_validation.action(
@@ -280,7 +280,7 @@
rule="os_compute_api:servers:create_image")
@decorators.idempotent_id('ba0ac859-99f4-4055-b5e0-e0905a44d331')
def test_create_image(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
# This function will also call show image
self.create_image_from_server(self.server_id,
@@ -292,7 +292,7 @@
@decorators.idempotent_id('8b869f73-49b3-4cc4-a0ce-ef64f8e1d6f9')
def test_create_image_volume_backed(self):
server = self._create_test_server_with_volume(self.volume_id)
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
# This function will also call show image
self.create_image_from_server(server['id'],
@@ -344,7 +344,7 @@
rule="os_compute_api:servers:show:host_status")
@decorators.idempotent_id('736da575-86f8-4b2a-9902-dd37dc9a409b')
def test_show_server_host_status(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
server = self.client.show_server(self.server_id)['server']
if 'host_status' not in server:
diff --git a/patrole_tempest_plugin/tests/api/compute/test_server_diagnostics_rbac.py b/patrole_tempest_plugin/tests/api/compute/test_server_diagnostics_rbac.py
index dd98b11..dd810a5 100644
--- a/patrole_tempest_plugin/tests/api/compute/test_server_diagnostics_rbac.py
+++ b/patrole_tempest_plugin/tests/api/compute/test_server_diagnostics_rbac.py
@@ -46,5 +46,5 @@
rule="os_compute_api:os-server-diagnostics")
@decorators.idempotent_id('5dabfcc4-bedb-417b-8247-b3ee7c5c0f3e')
def test_show_server_diagnostics(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.show_server_diagnostics(self.server['id'])
diff --git a/patrole_tempest_plugin/tests/api/compute/test_server_groups_rbac.py b/patrole_tempest_plugin/tests/api/compute/test_server_groups_rbac.py
index 6d1fdbd..b89750d 100644
--- a/patrole_tempest_plugin/tests/api/compute/test_server_groups_rbac.py
+++ b/patrole_tempest_plugin/tests/api/compute/test_server_groups_rbac.py
@@ -46,7 +46,7 @@
rule="os_compute_api:os-server-groups:create")
@decorators.idempotent_id('7f3eae94-6130-47e9-81ac-34009f55be2f')
def test_create_server_group(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.create_test_server_group()
@rbac_rule_validation.action(
@@ -55,7 +55,7 @@
@decorators.idempotent_id('832d9be3-632e-47b2-93d2-5897db43e3e2')
def test_delete_server_group(self):
server_group = self.create_test_server_group()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.delete_server_group(server_group['id'])
@rbac_rule_validation.action(
@@ -63,7 +63,7 @@
rule="os_compute_api:os-server-groups:index")
@decorators.idempotent_id('5eccd67f-5945-483b-b1c8-de851ebfc1c1')
def test_list_server_groups(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.list_server_groups()
@rbac_rule_validation.action(
@@ -72,5 +72,5 @@
@decorators.idempotent_id('62534e3f-7e99-4a3d-a08e-33e056460cf2')
def test_show_server_group(self):
server_group = self.create_test_server_group()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.show_server_group(server_group['id'])
diff --git a/patrole_tempest_plugin/tests/api/compute/test_server_migrations_rbac.py b/patrole_tempest_plugin/tests/api/compute/test_server_migrations_rbac.py
index 805143e..e5b1ea0 100644
--- a/patrole_tempest_plugin/tests/api/compute/test_server_migrations_rbac.py
+++ b/patrole_tempest_plugin/tests/api/compute/test_server_migrations_rbac.py
@@ -75,7 +75,7 @@
@decorators.idempotent_id('c6f1607c-9fed-4c00-807e-9ba675b98b1b')
def test_cold_migration(self):
server = self.create_test_server(wait_until="ACTIVE")
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.migrate_server(server['id'])
waiters.wait_for_server_status(self.admin_servers_client,
server['id'], 'VERIFY_RESIZE')
@@ -93,7 +93,7 @@
actual_host = self._get_host_for_server(server_id)
target_host = self._get_host_other_than(actual_host)
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.live_migrate_server(
server_id, host=target_host, block_migration=self.block_migration)
waiters.wait_for_server_status(self.admin_servers_client,
diff --git a/patrole_tempest_plugin/tests/api/compute/test_server_password_rbac.py b/patrole_tempest_plugin/tests/api/compute/test_server_password_rbac.py
index 5f90c97..0a0a660 100644
--- a/patrole_tempest_plugin/tests/api/compute/test_server_password_rbac.py
+++ b/patrole_tempest_plugin/tests/api/compute/test_server_password_rbac.py
@@ -45,7 +45,7 @@
service="nova",
rule="os_compute_api:os-server-password")
def test_delete_server_password(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.delete_password(self.server['id'])
@rbac_rule_validation.action(
@@ -53,5 +53,5 @@
rule="os_compute_api:os-server-password")
@decorators.idempotent_id('f677971a-7d20-493c-977f-6ff0a74b5b2c')
def test_get_server_password(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.show_password(self.server['id'])
diff --git a/patrole_tempest_plugin/tests/api/compute/test_server_rbac.py b/patrole_tempest_plugin/tests/api/compute/test_server_rbac.py
index e5a53c9..4157c05 100644
--- a/patrole_tempest_plugin/tests/api/compute/test_server_rbac.py
+++ b/patrole_tempest_plugin/tests/api/compute/test_server_rbac.py
@@ -110,7 +110,7 @@
rule="os_compute_api:servers:create")
@decorators.idempotent_id('4f34c73a-6ddc-4677-976f-71320fa855bd')
def test_create_server(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.create_test_server(wait_until='ACTIVE')
@rbac_rule_validation.action(
@@ -128,7 +128,7 @@
# The first key of the dictionary specifies the host name.
host = hosts[0].keys()[0]
availability_zone = 'nova:' + host
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.create_test_server(wait_until='ACTIVE',
availability_zone=availability_zone)
@@ -137,7 +137,7 @@
rule="os_compute_api:servers:create:attach_volume")
@decorators.idempotent_id('eeddac5e-15aa-454f-838d-db608aae4dd8')
def test_create_server_attach_volume(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self._create_test_server_with_volume(self.volume_id)
@rbac_rule_validation.action(
@@ -147,7 +147,7 @@
def test_create_server_attach_network(self):
network = self._create_network_resources()
network_id = {'uuid': network['id']}
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
server = self.create_test_server(wait_until='ACTIVE',
networks=[network_id])
@@ -164,7 +164,7 @@
@decorators.idempotent_id('062e3440-e873-4b41-9317-bf6d8be50c12')
def test_delete_server(self):
server = self.create_test_server(wait_until='ACTIVE')
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.delete_server(server['id'])
waiters.wait_for_server_termination(self.client, server['id'])
@@ -175,7 +175,7 @@
def test_update_server(self):
server = self.create_test_server(wait_until='ACTIVE')
new_name = data_utils.rand_name('server')
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
try:
self.client.update_server(server['id'], name=new_name)
except exceptions.ServerFault as e:
diff --git a/patrole_tempest_plugin/tests/api/compute/test_server_tags_rbac.py b/patrole_tempest_plugin/tests/api/compute/test_server_tags_rbac.py
index 6d88557..4de63ce 100644
--- a/patrole_tempest_plugin/tests/api/compute/test_server_tags_rbac.py
+++ b/patrole_tempest_plugin/tests/api/compute/test_server_tags_rbac.py
@@ -54,7 +54,7 @@
service="nova",
rule="os_compute_api:os-server-tags:index")
def test_list_tags(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.list_tags(self.server['id'])['tags']
@decorators.idempotent_id('9297c99e-94eb-429f-93cf-9b1838e33622')
@@ -63,7 +63,7 @@
rule="os_compute_api:os-server-tags:show")
def test_check_tag_existence(self):
tag_name = self._add_tag_to_server()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.check_tag_existence(self.server['id'], tag_name)
@decorators.idempotent_id('0d84ee94-d3ca-4635-8edf-b7f67ab8e4a3')
@@ -71,7 +71,7 @@
service="nova",
rule="os_compute_api:os-server-tags:update")
def test_update_tag(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self._add_tag_to_server()
@decorators.idempotent_id('115c2694-00aa-41ee-99f6-9eab4040c182')
@@ -80,7 +80,7 @@
rule="os_compute_api:os-server-tags:delete")
def test_delete_tag(self):
tag_name = self._add_tag_to_server()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.delete_tag(self.server['id'], tag_name)
@decorators.idempotent_id('a8e19b87-6580-4bc8-9933-e62561ff667d')
@@ -88,7 +88,7 @@
service="nova",
rule="os_compute_api:os-server-tags:update_all")
def test_update_all_tags(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
new_tag_name = data_utils.rand_name('tag')
self.client.update_all_tags(self.server['id'], [new_tag_name])['tags']
@@ -97,5 +97,5 @@
service="nova",
rule="os_compute_api:os-server-tags:delete_all")
def test_delete_all_tags(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.delete_all_tags(self.server['id'])
diff --git a/patrole_tempest_plugin/tests/api/compute/test_server_usage_rbac.py b/patrole_tempest_plugin/tests/api/compute/test_server_usage_rbac.py
index 27cbbf3..4493826 100644
--- a/patrole_tempest_plugin/tests/api/compute/test_server_usage_rbac.py
+++ b/patrole_tempest_plugin/tests/api/compute/test_server_usage_rbac.py
@@ -46,5 +46,5 @@
rule="os_compute_api:os-server-usage")
@decorators.idempotent_id('f0437ead-b9fb-462a-9f3d-ce53fac9d57a')
def test_show_server_diagnostics(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.show_server(self.server['id'])
diff --git a/patrole_tempest_plugin/tests/api/compute/test_server_virtual_interfaces_rbac.py b/patrole_tempest_plugin/tests/api/compute/test_server_virtual_interfaces_rbac.py
index c523e84..3e74a01 100644
--- a/patrole_tempest_plugin/tests/api/compute/test_server_virtual_interfaces_rbac.py
+++ b/patrole_tempest_plugin/tests/api/compute/test_server_virtual_interfaces_rbac.py
@@ -38,7 +38,7 @@
@decorators.idempotent_id('fc719ae3-0f73-4689-8378-1b841f0f2818')
def test_list_virtual_interfaces(self):
server = self.create_test_server(wait_until='ACTIVE')
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
try:
self.client.list_virtual_interfaces(server['id'])
except exceptions.ServerFault as e:
diff --git a/patrole_tempest_plugin/tests/api/compute/test_server_volume_attachments_rbac.py b/patrole_tempest_plugin/tests/api/compute/test_server_volume_attachments_rbac.py
index 7556234..ba33a0f 100644
--- a/patrole_tempest_plugin/tests/api/compute/test_server_volume_attachments_rbac.py
+++ b/patrole_tempest_plugin/tests/api/compute/test_server_volume_attachments_rbac.py
@@ -51,7 +51,7 @@
rule="os_compute_api:os-volumes-attachments:index")
@decorators.idempotent_id('529b668b-6edb-41d5-8886-d7dbd0614678')
def test_list_volume_attachments(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.list_volume_attachments(self.server['id'])[
'volumeAttachments']
@@ -60,7 +60,7 @@
rule="os_compute_api:os-volumes-attachments:create")
@decorators.idempotent_id('21c2c3fd-fbe8-41b1-8ef8-115ec47d54c1')
def test_create_volume_attachment(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.attach_volume(self.server, self.volume)
@rbac_rule_validation.action(
@@ -69,7 +69,7 @@
@decorators.idempotent_id('997df9c2-6e54-47b6-ab74-e4fdb500f385')
def test_show_volume_attachment(self):
attachment = self.attach_volume(self.server, self.volume)
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.show_volume_attachment(
self.server['id'], attachment['id'])
@@ -81,7 +81,7 @@
def test_update_volume_attachment(self):
attachment = self.attach_volume(self.server, self.volume)
alt_volume = self.create_volume()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.update_attached_volume(
self.server['id'], attachment['id'], volumeId=alt_volume['id'])
waiters.wait_for_volume_resource_status(self.volumes_client,
@@ -103,7 +103,7 @@
@decorators.idempotent_id('12b03e90-d087-46af-9c4d-507d021c4984')
def test_delete_volume_attachment(self):
self.attach_volume(self.server, self.volume)
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.detach_volume(self.server['id'], self.volume['id'])
waiters.wait_for_volume_resource_status(self.volumes_client,
self.volume['id'], 'available')
diff --git a/patrole_tempest_plugin/tests/api/compute/test_services_rbac.py b/patrole_tempest_plugin/tests/api/compute/test_services_rbac.py
index 01c19df..c25303f 100644
--- a/patrole_tempest_plugin/tests/api/compute/test_services_rbac.py
+++ b/patrole_tempest_plugin/tests/api/compute/test_services_rbac.py
@@ -41,5 +41,5 @@
rule="os_compute_api:os-services")
@decorators.idempotent_id('7472261b-9c6d-453a-bcb3-aecaa29ad281')
def test_list_services(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.list_services()['services']
diff --git a/patrole_tempest_plugin/tests/api/compute/test_simple_tenant_usage_rbac.py b/patrole_tempest_plugin/tests/api/compute/test_simple_tenant_usage_rbac.py
index 32bb407..93c179e 100644
--- a/patrole_tempest_plugin/tests/api/compute/test_simple_tenant_usage_rbac.py
+++ b/patrole_tempest_plugin/tests/api/compute/test_simple_tenant_usage_rbac.py
@@ -41,7 +41,7 @@
rule="os_compute_api:os-simple-tenant-usage:list")
@decorators.idempotent_id('2aef094f-0452-4df6-a66a-0ec22a92b16e')
def test_simple_tenant_usage_list(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.list_tenant_usages()
@rbac_rule_validation.action(
@@ -53,5 +53,5 @@
# the validation method in the API call throws an error.
self.create_test_server(wait_until='ACTIVE')['id']
tenant_id = self.auth_provider.credentials.tenant_id
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.show_tenant_usage(tenant_id=tenant_id)
diff --git a/patrole_tempest_plugin/tests/api/compute/test_suspend_server_rbac.py b/patrole_tempest_plugin/tests/api/compute/test_suspend_server_rbac.py
index c7860db..8106a1a 100644
--- a/patrole_tempest_plugin/tests/api/compute/test_suspend_server_rbac.py
+++ b/patrole_tempest_plugin/tests/api/compute/test_suspend_server_rbac.py
@@ -59,7 +59,7 @@
service="nova",
rule="os_compute_api:os-suspend-server:suspend")
def test_suspend_server(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.suspend_server(self.server['id'])
waiters.wait_for_server_status(self.client, self.server['id'],
'SUSPENDED')
@@ -72,7 +72,7 @@
self.client.suspend_server(self.server['id'])
waiters.wait_for_server_status(self.client, self.server['id'],
'SUSPENDED')
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.resume_server(self.server['id'])
waiters.wait_for_server_status(self.client,
self.server['id'],
diff --git a/patrole_tempest_plugin/tests/api/compute/test_tenant_networks_rbac.py b/patrole_tempest_plugin/tests/api/compute/test_tenant_networks_rbac.py
index d81b3c0..15a385a 100644
--- a/patrole_tempest_plugin/tests/api/compute/test_tenant_networks_rbac.py
+++ b/patrole_tempest_plugin/tests/api/compute/test_tenant_networks_rbac.py
@@ -56,5 +56,5 @@
service="nova",
rule="os_compute_api:os-tenant-networks")
def test_list_show_tenant_networks(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.list_tenant_networks()['networks']
diff --git a/patrole_tempest_plugin/tests/api/identity/v2/rbac_base.py b/patrole_tempest_plugin/tests/api/identity/v2/rbac_base.py
index 969631d..d5c8528 100644
--- a/patrole_tempest_plugin/tests/api/identity/v2/rbac_base.py
+++ b/patrole_tempest_plugin/tests/api/identity/v2/rbac_base.py
@@ -41,7 +41,7 @@
cls.tenants_client = cls.os.tenants_client
cls.users_client = cls.os.users_client
cls.rbac_utils = rbac_utils()
- cls.rbac_utils.switch_role(cls, switchToRbacRole=False)
+ cls.rbac_utils.switch_role(cls, toggle_rbac_role=False)
def _create_service(self):
name = data_utils.rand_name('service')
diff --git a/patrole_tempest_plugin/tests/api/identity/v2/test_endpoints_rbac.py b/patrole_tempest_plugin/tests/api/identity/v2/test_endpoints_rbac.py
index ba04d06..9364e2f 100644
--- a/patrole_tempest_plugin/tests/api/identity/v2/test_endpoints_rbac.py
+++ b/patrole_tempest_plugin/tests/api/identity/v2/test_endpoints_rbac.py
@@ -64,7 +64,7 @@
RBAC test for Identity Admin 2.0 create_endpoint
"""
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self._create_endpoint()
@rbac_rule_validation.action(service="keystone",
@@ -79,7 +79,7 @@
"""
endpoint = self._create_endpoint()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.endpoints_client.delete_endpoint(endpoint['endpoint']['id'])
@rbac_rule_validation.action(service="keystone",
@@ -93,5 +93,5 @@
RBAC test for Identity Admin 2.0 list_endpoint
"""
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.endpoints_client.list_endpoints()
diff --git a/patrole_tempest_plugin/tests/api/identity/v2/test_projects_rbac.py b/patrole_tempest_plugin/tests/api/identity/v2/test_projects_rbac.py
index dd0af46..a120562 100644
--- a/patrole_tempest_plugin/tests/api/identity/v2/test_projects_rbac.py
+++ b/patrole_tempest_plugin/tests/api/identity/v2/test_projects_rbac.py
@@ -35,7 +35,7 @@
RBAC test for Identity 2.0 create_tenant
"""
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self._create_tenant()
@rbac_rule_validation.action(service="keystone",
@@ -50,7 +50,7 @@
"""
tenant = self._create_tenant()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.tenants_client.update_tenant(tenant['id'],
description="Changed description")
@@ -66,7 +66,7 @@
"""
tenant = self._create_tenant()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.tenants_client.delete_tenant(tenant['id'])
@rbac_rule_validation.action(service="keystone",
@@ -82,7 +82,7 @@
tenant = self._create_tenant()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.tenants_client.show_tenant(tenant['id'])
@rbac_rule_validation.action(service="keystone",
@@ -95,7 +95,7 @@
RBAC test for Identity 2.0 list_tenants
"""
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.tenants_client.list_tenants()
@rbac_rule_validation.action(service="keystone",
@@ -110,5 +110,5 @@
"""
tenant = self._create_tenant()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.tenants_client.list_tenant_users(tenant['id'])
diff --git a/patrole_tempest_plugin/tests/api/identity/v2/test_roles_rbac.py b/patrole_tempest_plugin/tests/api/identity/v2/test_roles_rbac.py
index aeb8d05..4196cdd 100644
--- a/patrole_tempest_plugin/tests/api/identity/v2/test_roles_rbac.py
+++ b/patrole_tempest_plugin/tests/api/identity/v2/test_roles_rbac.py
@@ -63,7 +63,7 @@
RBAC test for Identity Admin 2.0 role-create
"""
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self._create_role()
@rbac_rule_validation.action(service="keystone",
@@ -78,7 +78,7 @@
"""
role = self._create_role()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.roles_client.delete_role(role['id'])
@rbac_rule_validation.action(service="keystone",
@@ -93,7 +93,7 @@
"""
role = self._create_role()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.roles_client.show_role(role['id'])
@rbac_rule_validation.action(service="keystone",
@@ -106,7 +106,7 @@
RBAC test for Identity Admin 2.0 role-list
"""
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.roles_client.list_roles()
@rbac_rule_validation.action(service="keystone",
@@ -120,7 +120,7 @@
RBAC test for Identity Admin 2.0 create_user_role_on_project
"""
tenant, user, role = self._create_tenant_user_and_role()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self._create_role_on_project(tenant, user, role)
@rbac_rule_validation.action(service="keystone",
@@ -136,7 +136,7 @@
tenant, user, role = self._create_tenant_user_and_role()
self._create_role_on_project(tenant, user, role)
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.roles_client.delete_role_from_user_on_project(
tenant['id'], user['id'], role['id'])
@@ -153,6 +153,6 @@
tenant = self._create_tenant()
user = self._create_user(tenantid=tenant['id'])
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.roles_client.list_user_roles_on_project(
tenant['id'], user['id'])
diff --git a/patrole_tempest_plugin/tests/api/identity/v2/test_services_rbac.py b/patrole_tempest_plugin/tests/api/identity/v2/test_services_rbac.py
index bc3dae8..e9b59be 100644
--- a/patrole_tempest_plugin/tests/api/identity/v2/test_services_rbac.py
+++ b/patrole_tempest_plugin/tests/api/identity/v2/test_services_rbac.py
@@ -38,7 +38,7 @@
RBAC test for Identity Admin 2.0 create_service
"""
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self._create_service()
@rbac_rule_validation.action(service="keystone",
@@ -52,7 +52,7 @@
"""
service_id = self._create_service()['OS-KSADM:service']['id']
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.services_client.delete_service(service_id)
@rbac_rule_validation.action(service="keystone",
@@ -66,7 +66,7 @@
"""
service_id = self._create_service()['OS-KSADM:service']['id']
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.services_client.show_service(service_id)
@rbac_rule_validation.action(service="keystone",
@@ -78,5 +78,5 @@
RBAC test for Identity Admin 2.0 list_service
"""
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.services_client.list_services()
diff --git a/patrole_tempest_plugin/tests/api/identity/v2/test_users_rbac.py b/patrole_tempest_plugin/tests/api/identity/v2/test_users_rbac.py
index da5cdce..3222b61 100644
--- a/patrole_tempest_plugin/tests/api/identity/v2/test_users_rbac.py
+++ b/patrole_tempest_plugin/tests/api/identity/v2/test_users_rbac.py
@@ -27,7 +27,7 @@
admin_only=True)
@decorators.idempotent_id('0f148510-63bf-11e6-1342-080044d0d904')
def test_create_user(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self._create_user()
@rbac_rule_validation.action(service="keystone",
@@ -37,7 +37,7 @@
def test_update_user(self):
user = self._create_user()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.users_client.update_user(user['id'], email="changedUser@xyz.com")
@rbac_rule_validation.action(service="keystone",
@@ -47,7 +47,7 @@
def test_update_user_enabled(self):
user = self._create_user()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.users_client.update_user_enabled(user['id'], enabled=True)
@rbac_rule_validation.action(service="keystone",
@@ -57,7 +57,7 @@
def test_delete_user(self):
user = self._create_user()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.users_client.delete_user(user['id'])
@rbac_rule_validation.action(service="keystone",
@@ -65,7 +65,7 @@
admin_only=True)
@decorators.idempotent_id('0f148510-63bf-11e6-1342-080044d0d907')
def test_list_users(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.users_client.list_users()
@rbac_rule_validation.action(service="keystone",
@@ -75,7 +75,7 @@
def test_show_user(self):
user = self._create_user()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.users_client.show_user(user['id'])
@rbac_rule_validation.action(service="keystone",
@@ -85,6 +85,6 @@
def test_update_user_password(self):
user = self._create_user()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.users_client.update_user_password(
user['id'], password=data_utils.rand_password())
diff --git a/patrole_tempest_plugin/tests/api/identity/v3/rbac_base.py b/patrole_tempest_plugin/tests/api/identity/v3/rbac_base.py
index 44504cf..6a0a309 100644
--- a/patrole_tempest_plugin/tests/api/identity/v3/rbac_base.py
+++ b/patrole_tempest_plugin/tests/api/identity/v3/rbac_base.py
@@ -40,7 +40,7 @@
cls.auth_provider = cls.os.auth_provider
cls.rbac_utils = rbac_utils()
- cls.rbac_utils.switch_role(cls, switchToRbacRole=False)
+ cls.rbac_utils.switch_role(cls, toggle_rbac_role=False)
cls.creds_client = cls.os.credentials_client
cls.consumers_client = cls.os.oauth_consumers_client
diff --git a/patrole_tempest_plugin/tests/api/identity/v3/test_credentials_rbac.py b/patrole_tempest_plugin/tests/api/identity/v3/test_credentials_rbac.py
index 1262c2e..ad6feaa 100644
--- a/patrole_tempest_plugin/tests/api/identity/v3/test_credentials_rbac.py
+++ b/patrole_tempest_plugin/tests/api/identity/v3/test_credentials_rbac.py
@@ -35,7 +35,7 @@
def test_create_credential(self):
project = self.setup_test_project()
user = self.setup_test_user(project_id=project['id'])
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.setup_test_credential(user=user)
@rbac_rule_validation.action(service="keystone",
@@ -46,7 +46,7 @@
new_keys = [data_utils.rand_uuid_hex(),
data_utils.rand_uuid_hex()]
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.creds_client.update_credential(
credential['id'],
credential=credential,
@@ -60,7 +60,7 @@
def test_delete_credential(self):
credential = self._create_user_project_and_credential()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.creds_client.delete_credential(credential['id'])
@rbac_rule_validation.action(service="keystone",
@@ -69,12 +69,12 @@
def test_show_credential(self):
credential = self._create_user_project_and_credential()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.creds_client.show_credential(credential['id'])
@rbac_rule_validation.action(service="keystone",
rule="identity:list_credentials")
@decorators.idempotent_id('3de303e2-12a7-4811-805a-f18906472038')
def test_list_credentials(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.creds_client.list_credentials()
diff --git a/patrole_tempest_plugin/tests/api/identity/v3/test_endpoints_rbac.py b/patrole_tempest_plugin/tests/api/identity/v3/test_endpoints_rbac.py
index a18a056..8c67e5f 100644
--- a/patrole_tempest_plugin/tests/api/identity/v3/test_endpoints_rbac.py
+++ b/patrole_tempest_plugin/tests/api/identity/v3/test_endpoints_rbac.py
@@ -28,7 +28,7 @@
@decorators.idempotent_id('6bdaecd4-0843-4ed6-ab64-3a57ab0cd127')
def test_create_endpoint(self):
service = self.setup_test_service()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.setup_test_endpoint(service=service)
@rbac_rule_validation.action(service="keystone",
@@ -38,7 +38,7 @@
endpoint = self.setup_test_endpoint()
new_url = data_utils.rand_url()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.endpoints_client.update_endpoint(
endpoint["id"],
url=new_url)
@@ -49,7 +49,7 @@
def test_delete_endpoint(self):
endpoint = self.setup_test_endpoint()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.endpoints_client.delete_endpoint(endpoint['id'])
@rbac_rule_validation.action(service="keystone",
@@ -58,12 +58,12 @@
def test_show_endpoint(self):
endpoint = self.setup_test_endpoint()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.endpoints_client.show_endpoint(endpoint['id'])
@rbac_rule_validation.action(service="keystone",
rule="identity:list_endpoints")
@decorators.idempotent_id('6bdaecd4-0843-4ed6-ab64-3a57ab0cd131')
def test_list_endpoints(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.endpoints_client.list_endpoints()
diff --git a/patrole_tempest_plugin/tests/api/identity/v3/test_groups_rbac.py b/patrole_tempest_plugin/tests/api/identity/v3/test_groups_rbac.py
index 5b5c079..1a96010 100644
--- a/patrole_tempest_plugin/tests/api/identity/v3/test_groups_rbac.py
+++ b/patrole_tempest_plugin/tests/api/identity/v3/test_groups_rbac.py
@@ -33,7 +33,7 @@
rule="identity:create_group")
@decorators.idempotent_id('88377f51-9074-4d64-a22f-f8931d048c9a')
def test_create_group(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.setup_test_group()
@rbac_rule_validation.action(service="keystone",
@@ -43,7 +43,7 @@
group = self.setup_test_group()
new_group_name = data_utils.rand_name('group')
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.groups_client.update_group(group['id'],
name=new_group_name)
@@ -53,7 +53,7 @@
def test_delete_group(self):
group = self.setup_test_group()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.groups_client.delete_group(group['id'])
@rbac_rule_validation.action(service="keystone",
@@ -62,14 +62,14 @@
def test_show_group(self):
group = self.setup_test_group()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.groups_client.show_group(group['id'])
@rbac_rule_validation.action(service="keystone",
rule="identity:list_groups")
@decorators.idempotent_id('c4d0f76b-735f-4fd0-868b-0006bc420ff4')
def test_list_groups(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.groups_client.list_groups()
@rbac_rule_validation.action(service="keystone",
@@ -79,7 +79,7 @@
group = self.setup_test_group()
user = self.setup_test_user()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.groups_client.add_group_user(group['id'], user['id'])
@rbac_rule_validation.action(service="keystone",
@@ -88,7 +88,7 @@
def test_remove_user_group(self):
group_id, user_id = self._create_user_and_add_to_new_group()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.groups_client.delete_group_user(group_id, user_id)
@rbac_rule_validation.action(service="keystone",
@@ -97,7 +97,7 @@
def test_list_user_group(self):
group = self.setup_test_group()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.groups_client.list_group_users(group['id'])
@rbac_rule_validation.action(service="keystone",
@@ -106,5 +106,5 @@
def test_check_user_group(self):
group_id, user_id = self._create_user_and_add_to_new_group()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.groups_client.check_group_user_existence(group_id, user_id)
diff --git a/patrole_tempest_plugin/tests/api/identity/v3/test_oauth_consumers_rbac.py b/patrole_tempest_plugin/tests/api/identity/v3/test_oauth_consumers_rbac.py
index 62a8084..1032303 100644
--- a/patrole_tempest_plugin/tests/api/identity/v3/test_oauth_consumers_rbac.py
+++ b/patrole_tempest_plugin/tests/api/identity/v3/test_oauth_consumers_rbac.py
@@ -36,7 +36,7 @@
rule="identity:create_consumer")
@decorators.idempotent_id('0f148510-63bf-11e6-4522-080044d0d970')
def test_create_consumer(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self._create_consumer()
@rbac_rule_validation.action(service="keystone",
@@ -45,7 +45,7 @@
def test_delete_consumer(self):
consumer = self._create_consumer()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.consumers_client.delete_consumer(consumer['id'])
@rbac_rule_validation.action(service="keystone",
@@ -55,7 +55,7 @@
consumer = self._create_consumer()
new_description = data_utils.rand_name('test_update_consumer')
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.consumers_client.update_consumer(consumer['id'],
new_description)
@@ -65,12 +65,12 @@
def test_show_consumer(self):
consumer = self._create_consumer()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.consumers_client.show_consumer(consumer['id'])
@rbac_rule_validation.action(service="keystone",
rule="identity:list_consumers")
@decorators.idempotent_id('0f148510-63bf-11e6-4522-080044d0d975')
def test_list_consumers(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.consumers_client.list_consumers()
diff --git a/patrole_tempest_plugin/tests/api/identity/v3/test_policies_rbac.py b/patrole_tempest_plugin/tests/api/identity/v3/test_policies_rbac.py
index 792ddaa..d60a1ab 100644
--- a/patrole_tempest_plugin/tests/api/identity/v3/test_policies_rbac.py
+++ b/patrole_tempest_plugin/tests/api/identity/v3/test_policies_rbac.py
@@ -26,7 +26,7 @@
rule="identity:create_policy")
@decorators.idempotent_id('de2f7ecb-fbf0-41f3-abf4-b97b5e082fd5')
def test_create_policy(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.setup_test_policy()
@rbac_rule_validation.action(service="keystone",
@@ -36,7 +36,7 @@
policy = self.setup_test_policy()
new_policy_type = data_utils.rand_name('policy_type')
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.policies_client.update_policy(policy['id'],
type=new_policy_type)
@@ -46,7 +46,7 @@
def test_delete_policy(self):
policy = self.setup_test_policy()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.policies_client.delete_policy(policy['id'])
@rbac_rule_validation.action(service="keystone",
@@ -55,12 +55,12 @@
def test_show_policy(self):
policy = self.setup_test_policy()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.policies_client.show_policy(policy['id'])
@rbac_rule_validation.action(service="keystone",
rule="identity:list_policies")
@decorators.idempotent_id('35a56161-4054-4237-8a78-7ce805dce202')
def test_list_policies(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.policies_client.list_policies()['policies']
diff --git a/patrole_tempest_plugin/tests/api/identity/v3/test_projects_rbac.py b/patrole_tempest_plugin/tests/api/identity/v3/test_projects_rbac.py
index fbbc81b..3f65076 100644
--- a/patrole_tempest_plugin/tests/api/identity/v3/test_projects_rbac.py
+++ b/patrole_tempest_plugin/tests/api/identity/v3/test_projects_rbac.py
@@ -27,7 +27,7 @@
rule="identity:create_project")
@decorators.idempotent_id('0f148510-63bf-11e6-1564-080044d0d904')
def test_create_project(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.setup_test_project()
@rbac_rule_validation.action(service="keystone",
@@ -37,7 +37,7 @@
project = self.setup_test_project()
new_desc = data_utils.rand_name('description')
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.projects_client.update_project(project['id'],
description=new_desc)
@@ -47,7 +47,7 @@
def test_delete_project(self):
project = self.setup_test_project()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.projects_client.delete_project(project['id'])
@rbac_rule_validation.action(service="keystone",
@@ -56,12 +56,12 @@
def test_show_project(self):
project = self.setup_test_project()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.projects_client.show_project(project['id'])
@rbac_rule_validation.action(service="keystone",
rule="identity:list_projects")
@decorators.idempotent_id('0f148510-63bf-11e6-1564-080044d0d908')
def test_list_projects(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.projects_client.list_projects()
diff --git a/patrole_tempest_plugin/tests/api/identity/v3/test_regions_rbac.py b/patrole_tempest_plugin/tests/api/identity/v3/test_regions_rbac.py
index 9e4886a..3eefdc4 100644
--- a/patrole_tempest_plugin/tests/api/identity/v3/test_regions_rbac.py
+++ b/patrole_tempest_plugin/tests/api/identity/v3/test_regions_rbac.py
@@ -26,7 +26,7 @@
rule="identity:create_region")
@decorators.idempotent_id('6bdaecd4-0843-4ed6-ab64-3a57ab0cd119')
def test_create_region(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.setup_test_region()
@rbac_rule_validation.action(service="keystone",
@@ -36,7 +36,7 @@
region = self.setup_test_region()
new_description = data_utils.rand_name('test_update_region')
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.regions_client.update_region(region['id'],
description=new_description)
@@ -46,7 +46,7 @@
def test_delete_region(self):
region = self.setup_test_region()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.regions_client.delete_region(region['id'])
@rbac_rule_validation.action(service="keystone",
@@ -55,12 +55,12 @@
def test_show_region(self):
region = self.setup_test_region()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.regions_client.show_region(region['id'])
@rbac_rule_validation.action(service="keystone",
rule="identity:list_regions")
@decorators.idempotent_id('6bdaecd4-0843-4ed6-ab64-3a57ab0cd123')
def test_list_regions(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.regions_client.list_regions()
diff --git a/patrole_tempest_plugin/tests/api/identity/v3/test_role_assignments_rbac.py b/patrole_tempest_plugin/tests/api/identity/v3/test_role_assignments_rbac.py
index 878edcb..c683f57 100644
--- a/patrole_tempest_plugin/tests/api/identity/v3/test_role_assignments_rbac.py
+++ b/patrole_tempest_plugin/tests/api/identity/v3/test_role_assignments_rbac.py
@@ -31,7 +31,7 @@
@rbac_rule_validation.action(service="keystone",
rule="identity:list_role_assignments")
def test_list_role_assignments(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.list_role_assignments()['role_assignments']
@decorators.idempotent_id('36c7a990-857e-415c-8717-38d7200a9894')
@@ -41,7 +41,7 @@
def test_list_role_assignments_for_tree(self):
project = self.setup_test_project()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.list_role_assignments(
include_subtree=True, **{'scope.project.id': project['id']})[
'role_assignments']
diff --git a/patrole_tempest_plugin/tests/api/identity/v3/test_roles_rbac.py b/patrole_tempest_plugin/tests/api/identity/v3/test_roles_rbac.py
index 40f7e3c..b033c8e 100644
--- a/patrole_tempest_plugin/tests/api/identity/v3/test_roles_rbac.py
+++ b/patrole_tempest_plugin/tests/api/identity/v3/test_roles_rbac.py
@@ -37,7 +37,7 @@
rule="identity:create_role")
@decorators.idempotent_id('0f148510-63bf-11e6-1395-080044d0d904')
def test_create_role(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.setup_test_role()
@rbac_rule_validation.action(service="keystone",
@@ -46,7 +46,7 @@
def test_update_role(self):
new_role_name = data_utils.rand_name('test_update_role')
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.roles_client.update_role(self.role['id'],
name=new_role_name)
@@ -56,28 +56,28 @@
def test_delete_role(self):
role = self.setup_test_role()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.roles_client.delete_role(role['id'])
@rbac_rule_validation.action(service="keystone",
rule="identity:get_role")
@decorators.idempotent_id('0f148510-63bf-11e6-1395-080044d0d907')
def test_show_role(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.roles_client.show_role(self.role['id'])
@rbac_rule_validation.action(service="keystone",
rule="identity:list_roles")
@decorators.idempotent_id('0f148510-63bf-11e6-1395-080044d0d908')
def test_list_roles(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.roles_client.list_roles()
@rbac_rule_validation.action(service="keystone",
rule="identity:create_grant")
@decorators.idempotent_id('0f148510-63bf-11e6-1395-080044d0d909')
def test_create_user_role_on_project(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.roles_client.create_user_role_on_project(
self.project['id'],
self.user['id'],
@@ -102,7 +102,7 @@
self.user['id'],
self.role['id'])
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.roles_client.check_user_role_existence_on_project(
self.project['id'],
self.user['id'],
@@ -122,7 +122,7 @@
self.user['id'],
self.role['id'])
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.roles_client.delete_role_from_user_on_project(
self.project['id'],
self.user['id'],
@@ -132,7 +132,7 @@
rule="identity:list_grants")
@decorators.idempotent_id('0f148510-63bf-11e6-1395-080044d0d90b')
def test_list_user_roles_on_project(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.roles_client.list_user_roles_on_project(
self.project['id'],
self.user['id'])
@@ -141,7 +141,7 @@
rule="identity:create_grant")
@decorators.idempotent_id('0f148510-63bf-11e6-1395-080044d0d90c')
def test_create_group_role_on_project(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.roles_client.create_group_role_on_project(
self.project['id'],
self.group['id'],
@@ -166,7 +166,7 @@
self.group['id'],
self.role['id'])
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.roles_client.delete_role_from_group_on_project(
self.project['id'],
self.group['id'],
@@ -176,7 +176,7 @@
rule="identity:list_grants")
@decorators.idempotent_id('0f148510-63bf-11e6-1395-080044d0d90e')
def test_list_group_roles_on_project(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.roles_client.list_group_roles_on_project(
self.project['id'],
self.group['id'])
@@ -185,7 +185,7 @@
rule="identity:create_grant")
@decorators.idempotent_id('0f148510-63bf-11e6-1395-080044d0d90f')
def test_create_user_role_on_domain(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.roles_client.create_user_role_on_domain(
self.domain['id'],
self.user['id'],
@@ -210,7 +210,7 @@
self.user['id'],
self.role['id'])
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.roles_client.delete_role_from_user_on_domain(
self.domain['id'],
self.user['id'],
@@ -220,7 +220,7 @@
rule="identity:list_grants")
@decorators.idempotent_id('0f148510-63bf-11e6-1395-080044d0d911')
def test_list_user_roles_on_domain(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.roles_client.list_user_roles_on_domain(
self.domain['id'],
self.user['id'])
@@ -229,7 +229,7 @@
rule="identity:create_grant")
@decorators.idempotent_id('0f148510-63bf-11e6-1395-080044d0d912')
def test_create_group_role_on_domain(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.roles_client.create_group_role_on_domain(
self.domain['id'],
self.group['id'],
@@ -254,7 +254,7 @@
self.group['id'],
self.role['id'])
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.roles_client.delete_role_from_group_on_domain(
self.domain['id'],
self.group['id'],
@@ -264,7 +264,7 @@
rule="identity:list_grants")
@decorators.idempotent_id('0f148510-63bf-11e6-1395-080044d0d914')
def test_list_group_roles_on_domain(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.roles_client.list_group_roles_on_domain(
self.domain['id'],
self.group['id'])
@@ -273,7 +273,7 @@
@rbac_rule_validation.action(service="keystone",
rule="identity:create_implied_role")
def test_create_role_inference_rule(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.roles_client.create_role_inference_rule(
self.role['id'], self.implies_role['id'])['role_inference']
self.addCleanup(self.roles_client.delete_role_inference_rule,
@@ -288,7 +288,7 @@
self.addCleanup(self.roles_client.delete_role_inference_rule,
self.role['id'], self.implies_role['id'])
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.roles_client.show_role_inference_rule(
self.role['id'], self.implies_role['id'])['role_inference']
@@ -296,7 +296,7 @@
@rbac_rule_validation.action(service="keystone",
rule="identity:list_implied_roles")
def test_list_role_inferences_rules(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.roles_client.list_role_inferences_rules(self.role['id'])[
'role_inference']
@@ -309,7 +309,7 @@
self.addCleanup(self.roles_client.delete_role_inference_rule,
self.role['id'], self.implies_role['id'])
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.roles_client.check_role_inference_rule(
self.role['id'], self.implies_role['id'])
@@ -323,7 +323,7 @@
self.roles_client.delete_role_inference_rule,
self.role['id'], self.implies_role['id'])
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.roles_client.delete_role_inference_rule(
self.role['id'], self.implies_role['id'])
@@ -331,5 +331,5 @@
@rbac_rule_validation.action(service="keystone",
rule="identity:list_role_inference_rules")
def test_list_all_role_inference_rules(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.roles_client.list_all_role_inference_rules()['role_inferences']
diff --git a/patrole_tempest_plugin/tests/api/identity/v3/test_services_rbac.py b/patrole_tempest_plugin/tests/api/identity/v3/test_services_rbac.py
index 1316be0..f5bd99b 100644
--- a/patrole_tempest_plugin/tests/api/identity/v3/test_services_rbac.py
+++ b/patrole_tempest_plugin/tests/api/identity/v3/test_services_rbac.py
@@ -26,7 +26,7 @@
rule="identity:create_service")
@decorators.idempotent_id('9a4bb317-f0bb-4005-8df0-4b672885b7c8')
def test_create_service(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.setup_test_service()
@rbac_rule_validation.action(service="keystone",
@@ -36,7 +36,7 @@
service = self.setup_test_service()
new_name = data_utils.rand_name('service')
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.services_client.update_service(service['id'],
service=service,
name=new_name,
@@ -48,7 +48,7 @@
def test_delete_service(self):
service = self.setup_test_service()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.services_client.delete_service(service['id'])
@rbac_rule_validation.action(service="keystone",
@@ -57,12 +57,12 @@
def test_show_service(self):
service = self.setup_test_service()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.services_client.show_service(service['id'])
@rbac_rule_validation.action(service="keystone",
rule="identity:list_services")
@decorators.idempotent_id('706e6bea-3385-4718-919c-0b5121395806')
def test_list_services(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.services_client.list_services()
diff --git a/patrole_tempest_plugin/tests/api/identity/v3/test_users_rbac.py b/patrole_tempest_plugin/tests/api/identity/v3/test_users_rbac.py
index b07e982..e61b162 100644
--- a/patrole_tempest_plugin/tests/api/identity/v3/test_users_rbac.py
+++ b/patrole_tempest_plugin/tests/api/identity/v3/test_users_rbac.py
@@ -32,7 +32,7 @@
rule="identity:create_user")
@decorators.idempotent_id('0f148510-63bf-11e6-4522-080044d0d904')
def test_create_user(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.setup_test_user()
@rbac_rule_validation.action(service="keystone",
@@ -42,7 +42,7 @@
user = self.setup_test_user()
new_email = data_utils.rand_name('user_email')
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.users_client.update_user(user['id'],
name=user['name'],
email=new_email)
@@ -53,21 +53,21 @@
def test_delete_user(self):
user = self.setup_test_user()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.users_client.delete_user(user['id'])
@rbac_rule_validation.action(service="keystone",
rule="identity:list_users")
@decorators.idempotent_id('0f148510-63bf-11e6-4522-080044d0d907')
def test_list_users(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.users_client.list_users()
@rbac_rule_validation.action(service="keystone",
rule="identity:get_user")
@decorators.idempotent_id('0f148510-63bf-11e6-4522-080044d0d908')
def test_show_own_user(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.users_client.show_user(self.default_user_id)
@rbac_rule_validation.action(service="keystone",
@@ -77,7 +77,7 @@
original_password = data_utils.rand_password()
user = self.setup_test_user(password=original_password)
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.users_client.update_user_password(
user['id'],
original_password=original_password,
@@ -87,12 +87,12 @@
rule="identity:list_groups_for_user")
@decorators.idempotent_id('bd5946d4-46d2-423d-a800-a3e7aabc18b3')
def test_list_own_user_group(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.users_client.list_user_groups(self.default_user_id)
@rbac_rule_validation.action(service="keystone",
rule="identity:list_user_projects")
@decorators.idempotent_id('0f148510-63bf-11e6-1564-080044d0d909')
def test_list_own_user_projects(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.users_client.list_user_projects(self.default_user_id)
diff --git a/patrole_tempest_plugin/tests/api/image/rbac_base.py b/patrole_tempest_plugin/tests/api/image/rbac_base.py
index 25521f3..7266079 100644
--- a/patrole_tempest_plugin/tests/api/image/rbac_base.py
+++ b/patrole_tempest_plugin/tests/api/image/rbac_base.py
@@ -35,7 +35,7 @@
super(BaseV1ImageRbacTest, cls).setup_clients()
cls.auth_provider = cls.os.auth_provider
cls.rbac_utils = rbac_utils()
- cls.rbac_utils.switch_role(cls, switchToRbacRole=False)
+ cls.rbac_utils.switch_role(cls, toggle_rbac_role=False)
class BaseV2ImageRbacTest(image_base.BaseV2ImageTest):
@@ -54,4 +54,4 @@
super(BaseV2ImageRbacTest, cls).setup_clients()
cls.auth_provider = cls.os.auth_provider
cls.rbac_utils = rbac_utils()
- cls.rbac_utils.switch_role(cls, switchToRbacRole=False)
+ cls.rbac_utils.switch_role(cls, toggle_rbac_role=False)
diff --git a/patrole_tempest_plugin/tests/api/image/v1/test_images_member_rbac.py b/patrole_tempest_plugin/tests/api/image/v1/test_images_member_rbac.py
index d431eee..643ca73 100644
--- a/patrole_tempest_plugin/tests/api/image/v1/test_images_member_rbac.py
+++ b/patrole_tempest_plugin/tests/api/image/v1/test_images_member_rbac.py
@@ -45,7 +45,7 @@
"""
image = self.create_image()
# Toggle role and add image member
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.image_member_client.create_image_member(image['id'],
self.alt_tenant_id)
@@ -60,7 +60,7 @@
self.image_member_client.create_image_member(image['id'],
self.alt_tenant_id)
# Toggle role and delete image member
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.image_member_client.delete_image_member(image['id'],
self.alt_tenant_id)
@@ -75,5 +75,5 @@
self.image_member_client.create_image_member(image['id'],
self.alt_tenant_id)
# Toggle role and delete image member
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.image_member_client.list_image_members(image['id'])
diff --git a/patrole_tempest_plugin/tests/api/image/v1/test_images_rbac.py b/patrole_tempest_plugin/tests/api/image/v1/test_images_rbac.py
index 251c372..ae4e4d2 100644
--- a/patrole_tempest_plugin/tests/api/image/v1/test_images_rbac.py
+++ b/patrole_tempest_plugin/tests/api/image/v1/test_images_rbac.py
@@ -36,7 +36,7 @@
"""
properties = {'prop1': 'val1'}
image_name = data_utils.rand_name('image')
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.create_image(name=image_name,
container_format='bare',
disk_format='raw',
@@ -58,7 +58,7 @@
is_public=False,
properties=properties)
image_id = body['id']
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.delete_image(image_id)
@rbac_rule_validation.action(service="glance", rule="download_image")
@@ -80,7 +80,7 @@
image_file = moves.cStringIO(data_utils.random_bytes())
self.client.update_image(image_id, data=image_file)
# Toggle role and get created image
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.show_image(image_id)
@rbac_rule_validation.action(service="glance", rule="get_image")
@@ -102,7 +102,7 @@
image_file = moves.cStringIO(data_utils.random_bytes())
self.client.update_image(image_id, data=image_file)
# Toggle role and get created image
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.check_image(image_id)
@rbac_rule_validation.action(service="glance", rule="get_images")
@@ -112,7 +112,7 @@
RBAC test for the glance get_images policy.
"""
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.list_images()
@rbac_rule_validation.action(service="glance", rule="modify_image")
@@ -131,7 +131,7 @@
properties=properties)
image_id = body.get('id')
properties = {'prop1': 'val2'}
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.update_image(image_id, headers=properties)
@rbac_rule_validation.action(service="glance", rule="publicize_image")
@@ -143,7 +143,7 @@
"""
image_name = data_utils.rand_name('image')
properties = {'prop1': 'val1'}
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.create_image(name=image_name,
container_format='bare',
disk_format='raw',
diff --git a/patrole_tempest_plugin/tests/api/image/v2/test_image_namespace_objects_rbac.py b/patrole_tempest_plugin/tests/api/image/v2/test_image_namespace_objects_rbac.py
index 4370b6d..e7c7199 100644
--- a/patrole_tempest_plugin/tests/api/image/v2/test_image_namespace_objects_rbac.py
+++ b/patrole_tempest_plugin/tests/api/image/v2/test_image_namespace_objects_rbac.py
@@ -35,7 +35,7 @@
RBAC test for the glance add_metadef_object policy
"""
namespace = self.create_namespace()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
# create a md object, it will be cleaned automatically after
# cleanup of namespace
object_name = data_utils.rand_name('test-object')
@@ -55,7 +55,7 @@
RBAC test for the glance get_metadef_objects policy
"""
namespace = self.create_namespace()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
# list md objects
self.namespace_objects_client.list_namespace_objects(
namespace['namespace'])
@@ -78,7 +78,7 @@
namespace['namespace'], object_name)
# Toggle role and modify object
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
new_name = "Object New Name"
self.namespace_objects_client.update_namespace_object(
namespace['namespace'], object_name, name=new_name)
@@ -100,7 +100,7 @@
self.namespace_objects_client.delete_namespace_object,
namespace['namespace'], object_name)
# Toggle role and get object
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.namespace_objects_client.show_namespace_object(
namespace['namespace'],
object_name)
diff --git a/patrole_tempest_plugin/tests/api/image/v2/test_image_namespace_property_rbac.py b/patrole_tempest_plugin/tests/api/image/v2/test_image_namespace_property_rbac.py
index e1858f4..946837e 100644
--- a/patrole_tempest_plugin/tests/api/image/v2/test_image_namespace_property_rbac.py
+++ b/patrole_tempest_plugin/tests/api/image/v2/test_image_namespace_property_rbac.py
@@ -40,7 +40,7 @@
RBAC test for the glance add_metadef_property policy
"""
namespace = self.create_namespace()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
property_name = data_utils.rand_name('test-ns-property')
self.namespace_properties_client.create_namespace_property(
namespace=namespace['namespace'], type="string",
@@ -55,7 +55,7 @@
RBAC test for the glance get_metadef_properties policy
"""
namespace = self.create_namespace()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.namespace_properties_client.list_namespace_properties(
namespace=namespace['namespace'])
@@ -73,7 +73,7 @@
namespace=namespace['namespace'], type="string",
title=property_name, name=self.resource_name)
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.namespace_properties_client.show_namespace_properties(
namespace['namespace'], self.resource_name)
@@ -91,7 +91,7 @@
namespace=namespace['namespace'], type="string",
title=property_name, name=self.resource_name)
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.namespace_properties_client.update_namespace_properties(
namespace['namespace'], self.resource_name, type="string",
title=property_name, name=self.resource_name)
diff --git a/patrole_tempest_plugin/tests/api/image/v2/test_image_namespace_rbac.py b/patrole_tempest_plugin/tests/api/image/v2/test_image_namespace_rbac.py
index 779ba8f..1010796 100644
--- a/patrole_tempest_plugin/tests/api/image/v2/test_image_namespace_rbac.py
+++ b/patrole_tempest_plugin/tests/api/image/v2/test_image_namespace_rbac.py
@@ -35,7 +35,7 @@
RBAC test for the glance add_metadef_namespace policy
"""
namespace_name = data_utils.rand_name('test-ns')
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.namespaces_client.create_namespace(
namespace=namespace_name,
protected=False)
@@ -52,7 +52,7 @@
RBAC test for the glance get_metadef_namespaces policy
"""
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.namespaces_client.list_namespaces()
@rbac_rule_validation.action(service="glance",
@@ -67,7 +67,7 @@
body = self.namespaces_client.create_namespace(
namespace=namespace_name,
protected=False)
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.namespaces_client.update_namespace(body['namespace'],
description="My new "
"description")
diff --git a/patrole_tempest_plugin/tests/api/image/v2/test_image_namespace_resource_type.py b/patrole_tempest_plugin/tests/api/image/v2/test_image_namespace_resource_type.py
index e537414..87d88c0 100644
--- a/patrole_tempest_plugin/tests/api/image/v2/test_image_namespace_resource_type.py
+++ b/patrole_tempest_plugin/tests/api/image/v2/test_image_namespace_resource_type.py
@@ -34,7 +34,7 @@
RBAC test for the glance list_metadef_resource_type policy.
"""
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.resource_types_client.list_resource_types()
@rbac_rule_validation.action(service="glance",
@@ -54,6 +54,6 @@
self.namespaces_client.delete_namespace,
namespace_name)
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.resource_types_client.list_resource_type_association(
namespace_name)
diff --git a/patrole_tempest_plugin/tests/api/image/v2/test_images_member_rbac.py b/patrole_tempest_plugin/tests/api/image/v2/test_images_member_rbac.py
index c53480d..16d27f3 100644
--- a/patrole_tempest_plugin/tests/api/image/v2/test_images_member_rbac.py
+++ b/patrole_tempest_plugin/tests/api/image/v2/test_images_member_rbac.py
@@ -51,7 +51,7 @@
"""
image_id = self.create_image()['id']
# Toggle role and add image member
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.image_member_client.create_image_member(image_id,
member=self.alt_tenant_id)
@@ -68,7 +68,7 @@
self.image_member_client.create_image_member(image_id,
member=self.alt_tenant_id)
# Toggle role and delete image member
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.image_member_client.delete_image_member(image_id,
self.alt_tenant_id)
@@ -88,7 +88,7 @@
member=self.alt_tenant_id)
# Toggle role and get image member
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.image_member_client.show_image_member(
image_id,
self.alt_tenant_id)
@@ -110,7 +110,7 @@
image_id, self.tenant_id,
status='accepted')
# Toggle role and update member
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.image_member_client.update_image_member(
image_id, self.tenant_id,
status='pending')
@@ -128,5 +128,5 @@
self.image_member_client.create_image_member(image_id,
member=self.alt_tenant_id)
# Toggle role and list image members
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.image_member_client.list_image_members(image_id)
diff --git a/patrole_tempest_plugin/tests/api/image/v2/test_images_rbac.py b/patrole_tempest_plugin/tests/api/image/v2/test_images_rbac.py
index 8812262..456b6ec 100644
--- a/patrole_tempest_plugin/tests/api/image/v2/test_images_rbac.py
+++ b/patrole_tempest_plugin/tests/api/image/v2/test_images_rbac.py
@@ -45,7 +45,7 @@
"""
uuid = '00000000-1111-2222-3333-444455556666'
image_name = data_utils.rand_name('image')
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.create_image(name=image_name,
container_format='bare',
disk_format='raw',
@@ -69,7 +69,7 @@
visibility='private',
ramdisk_id=uuid)
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
# Try uploading an image file
image_file = moves.cStringIO(data_utils.random_bytes())
self.client.store_image_file(body['id'], image_file)
@@ -90,7 +90,7 @@
visibility='public')
image_id = body.get('id')
# Toggle role and delete created image
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.delete_image(image_id)
self.client.wait_for_resource_deletion(image_id)
@@ -111,7 +111,7 @@
visibility='private')
image_id = body.get('id')
# Toggle role and get created image
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.show_image(image_id)
@rbac_rule_validation.action(service="glance",
@@ -125,7 +125,7 @@
"""
# Toggle role and get created image
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.list_images()
@rbac_rule_validation.action(service="glance",
@@ -149,7 +149,7 @@
self.client.store_image_file(image_id, image_file)
# Toggle role and update created image
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
new_image_name = data_utils.rand_name('new-image')
body = self.client.update_image(image_id, [
dict(replace='/name', value=new_image_name)])
@@ -164,7 +164,7 @@
RBAC test for the glance publicize_image endpoint
"""
image_name = data_utils.rand_name('image')
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.create_image(name=image_name,
container_format='bare',
disk_format='raw',
@@ -191,7 +191,7 @@
image_file = moves.cStringIO(data_utils.random_bytes())
self.client.store_image_file(image_id=image_id, data=image_file)
# Toggling role and deacivate image
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.deactivate_image(image_id)
@rbac_rule_validation.action(service="glance",
@@ -216,5 +216,5 @@
image_file = moves.cStringIO(data_utils.random_bytes())
self.client.store_image_file(image_id=image_id, data=image_file)
# Toggling role and reactivate image
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.reactivate_image(image_id)
diff --git a/patrole_tempest_plugin/tests/api/network/rbac_base.py b/patrole_tempest_plugin/tests/api/network/rbac_base.py
index cad5498..6e7898f 100644
--- a/patrole_tempest_plugin/tests/api/network/rbac_base.py
+++ b/patrole_tempest_plugin/tests/api/network/rbac_base.py
@@ -37,4 +37,4 @@
super(BaseNetworkRbacTest, cls).setup_clients()
cls.auth_provider = cls.os.auth_provider
cls.rbac_utils = rbac_utils()
- cls.rbac_utils.switch_role(cls, switchToRbacRole=False)
+ cls.rbac_utils.switch_role(cls, toggle_rbac_role=False)
diff --git a/patrole_tempest_plugin/tests/api/network/test_floating_ips_rbac.py b/patrole_tempest_plugin/tests/api/network/test_floating_ips_rbac.py
index bd562ec..a875321 100644
--- a/patrole_tempest_plugin/tests/api/network/test_floating_ips_rbac.py
+++ b/patrole_tempest_plugin/tests/api/network/test_floating_ips_rbac.py
@@ -75,7 +75,7 @@
RBAC test for the neutron create_floatingip policy
"""
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self._create_floatingip()
@rbac_rule_validation.action(service="neutron",
@@ -88,7 +88,7 @@
"""
fip = str(netaddr.IPAddress(self.cidr) + 10)
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self._create_floatingip(floating_ip_address=fip)
@rbac_rule_validation.action(service="neutron",
@@ -100,7 +100,7 @@
RBAC test for the neutron update_floatingip policy
"""
floating_ip = self._create_floatingip()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
# Associate floating IP to the other port
self.floating_ips_client.update_floatingip(
@@ -116,7 +116,7 @@
RBAC test for the neutron get_floatingip policy
"""
floating_ip = self._create_floatingip()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
# Show floating IP
self.floating_ips_client.show_floatingip(floating_ip['id'])
@@ -130,6 +130,6 @@
RBAC test for the neutron delete_floatingip policy
"""
floating_ip = self._create_floatingip()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
# Delete the floating IP
self.floating_ips_client.delete_floatingip(floating_ip['id'])
diff --git a/patrole_tempest_plugin/tests/api/network/test_metering_label_rules_rbac.py b/patrole_tempest_plugin/tests/api/network/test_metering_label_rules_rbac.py
index 10a20f4..2f55e30 100644
--- a/patrole_tempest_plugin/tests/api/network/test_metering_label_rules_rbac.py
+++ b/patrole_tempest_plugin/tests/api/network/test_metering_label_rules_rbac.py
@@ -75,7 +75,7 @@
RBAC test for the neutron create_metering_label_rule policy
"""
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self._create_metering_label_rule(self.label)
@rbac_rule_validation.action(service="neutron",
@@ -88,7 +88,7 @@
RBAC test for the neutron get_metering_label_rule policy
"""
label_rule = self._create_metering_label_rule(self.label)
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.metering_label_rules_client.show_metering_label_rule(
label_rule['id'])
@@ -102,6 +102,6 @@
RBAC test for the neutron delete_metering_label_rule policy
"""
label_rule = self._create_metering_label_rule(self.label)
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.metering_label_rules_client.delete_metering_label_rule(
label_rule['id'])
diff --git a/patrole_tempest_plugin/tests/api/network/test_metering_labels_rbac.py b/patrole_tempest_plugin/tests/api/network/test_metering_labels_rbac.py
index eb17df4..091e31d 100644
--- a/patrole_tempest_plugin/tests/api/network/test_metering_labels_rbac.py
+++ b/patrole_tempest_plugin/tests/api/network/test_metering_labels_rbac.py
@@ -57,7 +57,7 @@
RBAC test for the neutron "create_metering_label" policy
"""
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self._create_metering_label()
@rbac_rule_validation.action(service="neutron",
@@ -70,7 +70,7 @@
RBAC test for the neutron "get_metering_label" policy
"""
label = self._create_metering_label()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.metering_labels_client.show_metering_label(label['id'])
@rbac_rule_validation.action(service="neutron",
@@ -83,5 +83,5 @@
RBAC test for the neutron "delete_metering_label" policy
"""
label = self._create_metering_label()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.metering_labels_client.delete_metering_label(label['id'])
diff --git a/patrole_tempest_plugin/tests/api/network/test_networks_multiprovider_rbac.py b/patrole_tempest_plugin/tests/api/network/test_networks_multiprovider_rbac.py
index 233f705..01cf0fd 100644
--- a/patrole_tempest_plugin/tests/api/network/test_networks_multiprovider_rbac.py
+++ b/patrole_tempest_plugin/tests/api/network/test_networks_multiprovider_rbac.py
@@ -56,7 +56,7 @@
RBAC test for the neutron create_network:segments policy
"""
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self._create_network_segments()
@rbac_rule_validation.action(service="neutron",
@@ -70,7 +70,7 @@
network = self._create_network_segments()
new_segments = [{"provider:network_type": "gre"}]
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.networks_client.update_network(network['id'],
segments=new_segments)
@@ -84,7 +84,7 @@
"""
network = self._create_network_segments()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
body = self.networks_client.show_network(network['id'],
fields='segments')
response_network = body['network']
diff --git a/patrole_tempest_plugin/tests/api/network/test_networks_rbac.py b/patrole_tempest_plugin/tests/api/network/test_networks_rbac.py
index 5d4e11a..9752db6 100644
--- a/patrole_tempest_plugin/tests/api/network/test_networks_rbac.py
+++ b/patrole_tempest_plugin/tests/api/network/test_networks_rbac.py
@@ -136,7 +136,7 @@
RBAC test for the neutron create_network policy
"""
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self._create_network()
@rbac_rule_validation.action(service="neutron",
@@ -148,7 +148,7 @@
RBAC test for the neutron create_network:shared policy
"""
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self._create_network(shared=True)
@rbac_rule_validation.action(service="neutron",
@@ -160,7 +160,7 @@
RBAC test for the neutron create_network:router:external policy
"""
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self._create_network(router_external=True)
@rbac_rule_validation.action(service="neutron",
@@ -172,7 +172,7 @@
RBAC test for the neutron create_network:provider:network_type policy
"""
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self._create_network(provider_network_type='vxlan')
@rbac_rule_validation.action(
@@ -185,7 +185,7 @@
RBAC test for the neutron create_network:provider:segmentation_id
"""
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self._create_network(provider_network_type='vxlan',
provider_segmentation_id=200)
@@ -198,7 +198,7 @@
RBAC test for the neutron update_network policy
"""
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
updated_network = self._update_network(admin=False)
self.assertEqual(updated_network['admin_state_up'], False)
@@ -215,7 +215,7 @@
RBAC test for the neutron update_network:shared policy
"""
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
updated_network = self._update_network(shared_network=True)
self.assertEqual(updated_network['shared'], True)
@@ -233,7 +233,7 @@
RBAC test for the neutron update_network:router:external policy
"""
network = self._create_network()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self._update_network(net_id=network['id'], router_external=True)
@rbac_rule_validation.action(service="neutron",
@@ -245,7 +245,7 @@
RBAC test for the neutron get_network policy
"""
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
# show a network that has been created during class setup
self.networks_client.show_network(self.admin_network['id'])
@@ -260,7 +260,7 @@
"""
post_body = {'fields': 'router:external'}
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.networks_client.show_network(self.admin_network['id'],
**post_body)
@@ -275,7 +275,7 @@
"""
post_body = {'fields': 'provider:network_type'}
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
body = self.networks_client.show_network(self.admin_network['id'],
**post_body)
showed_net = body['network']
@@ -294,7 +294,7 @@
"""
post_body = {'fields': 'provider:physical_network'}
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
body = self.networks_client.show_network(self.admin_network['id'],
**post_body)
showed_net = body['network']
@@ -313,7 +313,7 @@
"""
post_body = {'fields': 'provider:segmentation_id'}
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
body = self.networks_client.show_network(self.admin_network['id'],
**post_body)
showed_net = body['network']
@@ -334,7 +334,7 @@
RBAC test for the neutron delete_network policy
"""
network = self._create_network()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.networks_client.delete_network(network['id'])
@rbac_rule_validation.action(service="neutron",
@@ -349,7 +349,7 @@
network = self._create_network()
self.assertEqual('ACTIVE', network['status'])
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
# Create a subnet
self.create_subnet(network, enable_dhcp=False)
@@ -362,7 +362,7 @@
RBAC test for the neutron get_subnet policy
"""
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.subnets_client.show_subnet(self.admin_subnet['id'])
@rbac_rule_validation.action(service="neutron",
@@ -374,7 +374,7 @@
RBAC test for the neutron update_subnet policy
"""
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.subnets_client.update_subnet(self.admin_subnet['id'],
name="New_subnet")
@@ -394,6 +394,6 @@
# Create a subnet using admin privilege
subnet = self.create_subnet(network, enable_dhcp=False)
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
# Delete the subnet
self.subnets_client.delete_subnet(subnet['id'])
diff --git a/patrole_tempest_plugin/tests/api/network/test_ports_rbac.py b/patrole_tempest_plugin/tests/api/network/test_ports_rbac.py
index b59e1e5..33e0d2e 100644
--- a/patrole_tempest_plugin/tests/api/network/test_ports_rbac.py
+++ b/patrole_tempest_plugin/tests/api/network/test_ports_rbac.py
@@ -73,7 +73,7 @@
@decorators.idempotent_id('0ec8c551-625c-4864-8a52-85baa7c40f22')
def test_create_port(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
post_body = {'network_id': self.network['id']}
self._create_port(**post_body)
@@ -85,7 +85,7 @@
post_body = {'network_id': self.network['id'],
'binding:host_id': "rbac_test_host"}
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self._create_port(**post_body)
@rbac_rule_validation.action(service="neutron",
@@ -105,7 +105,7 @@
post_body = {'network_id': self.network['id'],
'fixed_ips': fixed_ips}
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self._create_port(**post_body)
@rbac_rule_validation.action(service="neutron",
@@ -116,7 +116,7 @@
post_body = {'network_id': self.network['id'],
'mac_address': data_utils.rand_mac_address()}
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self._create_port(**post_body)
@rbac_rule_validation.action(service="neutron",
@@ -129,7 +129,7 @@
post_body = {'network_id': self.network['id'],
'binding:profile': binding_profile}
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self._create_port(**post_body)
@rbac_rule_validation.action(service="neutron",
@@ -144,7 +144,7 @@
post_body = {'network_id': self.network['id'],
'allowed_address_pairs': allowed_address_pairs}
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self._create_port(**post_body)
@rbac_rule_validation.action(service="neutron",
@@ -152,7 +152,7 @@
expected_error_code=404)
@decorators.idempotent_id('a9d41cb8-78a2-4b97-985c-44e4064416f4')
def test_show_port(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.ports_client.show_port(self.port['id'])
@rbac_rule_validation.action(service="neutron",
@@ -163,7 +163,7 @@
# Verify specific fields of a port
fields = ['binding:vif_type']
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
retrieved_port = self.ports_client.show_port(
self.port['id'], fields=fields)['port']
@@ -180,7 +180,7 @@
# Verify specific fields of a port
fields = ['binding:vif_details']
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
retrieved_port = self.ports_client.show_port(
self.port['id'], fields=fields)['port']
@@ -200,7 +200,7 @@
'binding:host_id': data_utils.rand_name('host-id')}
port = self._create_port(**post_body)
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
retrieved_port = self.ports_client.show_port(
port['id'], fields=fields)['port']
@@ -221,7 +221,7 @@
'binding:profile': binding_profile}
port = self._create_port(**post_body)
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
retrieved_port = self.ports_client.show_port(
port['id'], fields=fields)['port']
@@ -236,7 +236,7 @@
def test_update_port(self):
port = self.create_port(self.network)
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.ports_client.update_port(port['id'], admin_state_up=False)
@rbac_rule_validation.action(service="neutron",
@@ -245,7 +245,7 @@
def test_update_port_mac_address(self):
port = self.create_port(self.network)
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.ports_client.update_port(
port['id'],
mac_address=data_utils.rand_mac_address())
@@ -267,7 +267,7 @@
1)
fixed_ips = [{'ip_address': ip_list[0]}]
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.ports_client.update_port(port['id'],
fixed_ips=fixed_ips)
@@ -277,7 +277,7 @@
def test_update_port_security_enabled(self):
port = self.create_port(self.network)
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.ports_client.update_port(port['id'],
security_groups=[])
@@ -293,7 +293,7 @@
updated_body = {'port_id': port['id'],
'binding:host_id': 'rbac_test_host_updated'}
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.ports_client.update_port(**updated_body)
@rbac_rule_validation.action(service="neutron",
@@ -311,7 +311,7 @@
updated_body = {'port_id': port['id'],
'binding:profile': new_binding_profile}
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.ports_client.update_port(**updated_body)
@rbac_rule_validation.action(service="neutron",
@@ -331,7 +331,7 @@
post_body = {'network_id': self.network['id']}
port = self._create_port(**post_body)
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.ports_client.update_port(port['id'],
allowed_address_pairs=address_pairs)
@@ -342,5 +342,5 @@
def test_delete_port(self):
port = self._create_port(network_id=self.network['id'])
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.ports_client.delete_port(port['id'])
diff --git a/patrole_tempest_plugin/tests/api/network/test_routers_rbac.py b/patrole_tempest_plugin/tests/api/network/test_routers_rbac.py
index eee9ff2..5ea1180 100644
--- a/patrole_tempest_plugin/tests/api/network/test_routers_rbac.py
+++ b/patrole_tempest_plugin/tests/api/network/test_routers_rbac.py
@@ -59,7 +59,7 @@
RBAC test for the neutron create_router policy
"""
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
router = self.routers_client.create_router()
self.addCleanup(self.routers_client.delete_router,
router['router']['id'])
@@ -78,7 +78,7 @@
external_gateway_info = {'network_id': self.network['id'],
'enable_snat': True}
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
router = self.routers_client.create_router(
name=name, external_gateway_info=external_gateway_info)
self.addCleanup(self.routers_client.delete_router,
@@ -108,7 +108,7 @@
'enable_snat': False,
'external_fixed_ips': [external_fixed_ips]}
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
router = self.routers_client.create_router(
name=name, external_gateway_info=external_gateway_info)
self.addCleanup(self.routers_client.delete_router,
@@ -123,7 +123,7 @@
RBAC test for the neutron get_router policy
"""
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.routers_client.show_router(self.router['id'])
@rbac_rule_validation.action(
@@ -135,7 +135,7 @@
RBAC test for the neutron update_router policy
"""
new_name = data_utils.rand_name('new-router-name')
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.routers_client.update_router(self.router['id'],
name=new_name)
@@ -148,7 +148,7 @@
RBAC test for the neutron
update_router:external_gateway_info policy
"""
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.routers_client.update_router(self.router['id'],
external_gateway_info={})
@@ -162,7 +162,7 @@
RBAC test for the neutron
update_router:external_gateway_info:network_id policy
"""
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.routers_client.update_router(
self.router['id'],
external_gateway_info={'network_id': self.network['id']})
@@ -181,7 +181,7 @@
RBAC test for the neutron
update_router:external_gateway_info:enable_snat policy
"""
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.routers_client.update_router(
self.router['id'],
external_gateway_info={'network_id': self.network['id'],
@@ -212,7 +212,7 @@
external_gateway_info = {'network_id': self.network['id'],
'external_fixed_ips': [external_fixed_ips]}
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.routers_client.update_router(
self.router['id'],
external_gateway_info=external_gateway_info)
@@ -231,7 +231,7 @@
RBAC test for the neutron delete_router policy
"""
router = self.create_router()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.routers_client.delete_router(router['id'])
@rbac_rule_validation.action(service="neutron",
@@ -247,7 +247,7 @@
subnet = self.create_subnet(network)
router = self.create_router()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.routers_client.add_router_interface(
router['id'], subnet_id=subnet['id'])
self.addCleanup(
@@ -277,7 +277,7 @@
router['id'],
subnet_id=subnet['id'])
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.routers_client.remove_router_interface(
router['id'],
subnet_id=subnet['id'])
diff --git a/patrole_tempest_plugin/tests/api/network/test_security_groups_rbac.py b/patrole_tempest_plugin/tests/api/network/test_security_groups_rbac.py
index 289c669..e04600e 100644
--- a/patrole_tempest_plugin/tests/api/network/test_security_groups_rbac.py
+++ b/patrole_tempest_plugin/tests/api/network/test_security_groups_rbac.py
@@ -74,7 +74,7 @@
@decorators.idempotent_id('db7003ce-5717-4e5b-afc7-befa35e8c67f')
def test_create_security_group(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self._create_security_group()
@rbac_rule_validation.action(service="neutron",
@@ -83,7 +83,7 @@
@decorators.idempotent_id('56335e77-aef2-4b54-86c7-7f772034b585')
def test_show_security_groups(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.security_groups_client.show_security_group(
self.secgroup['id'])
@@ -96,7 +96,7 @@
# Create a security group
secgroup_id = self._create_security_group()['id']
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.security_groups_client.delete_security_group(secgroup_id)
@rbac_rule_validation.action(service="neutron",
@@ -108,7 +108,7 @@
# Create a security group
secgroup_id = self._create_security_group()['id']
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.security_groups_client.update_security_group(
secgroup_id,
description="test description")
@@ -118,7 +118,7 @@
@decorators.idempotent_id('fbaf8d96-ed3e-49af-b24c-5fb44f05bbb7')
def test_list_security_groups(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.security_groups_client.list_security_groups()
@rbac_rule_validation.action(service="neutron",
@@ -126,7 +126,7 @@
@decorators.idempotent_id('953d78df-00cd-416f-9cbd-b7cb4ea65772')
def test_create_security_group_rule(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self._create_security_group_rule()
@rbac_rule_validation.action(service="neutron",
@@ -136,7 +136,7 @@
def test_delete_security_group_rule(self):
sec_group_rule = self._create_security_group_rule()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.security_group_rules_client.delete_security_group_rule(
sec_group_rule['id'])
@@ -147,7 +147,7 @@
def test_show_security_group_rule(self):
sec_group_rule = self._create_security_group_rule()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.security_group_rules_client.show_security_group_rule(
sec_group_rule['id'])
@@ -156,5 +156,5 @@
@decorators.idempotent_id('05739ab6-fa35-11e6-bc64-92361f002671')
def test_list_security_group_rules(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.security_group_rules_client.list_security_group_rules()
diff --git a/patrole_tempest_plugin/tests/api/network/test_subnetpools_rbac.py b/patrole_tempest_plugin/tests/api/network/test_subnetpools_rbac.py
index 3667212..e7682df 100644
--- a/patrole_tempest_plugin/tests/api/network/test_subnetpools_rbac.py
+++ b/patrole_tempest_plugin/tests/api/network/test_subnetpools_rbac.py
@@ -62,7 +62,7 @@
RBAC test for the neutron create_subnetpool policy
"""
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self._create_subnetpool()
@rbac_rule_validation.action(service="neutron",
@@ -73,7 +73,7 @@
RBAC test for the neutron create_subnetpool:shared policy
"""
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self._create_subnetpool(shared=True)
@rbac_rule_validation.action(service="neutron",
@@ -86,7 +86,7 @@
RBAC test for the neutron get_subnetpool policy
"""
subnetpool = self._create_subnetpool()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.subnetpools_client.show_subnetpool(subnetpool['id'])
@rbac_rule_validation.action(service="neutron",
@@ -98,7 +98,7 @@
RBAC test for the neutron update_subnetpool policy
"""
subnetpool = self._create_subnetpool()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.subnetpools_client.update_subnetpool(subnetpool['id'],
min_prefixlen=24)
@@ -112,5 +112,5 @@
RBAC test for the neutron delete_subnetpool policy
"""
subnetpool = self._create_subnetpool()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.subnetpools_client.delete_subnetpool(subnetpool['id'])
diff --git a/patrole_tempest_plugin/tests/api/orchestration/rbac_base.py b/patrole_tempest_plugin/tests/api/orchestration/rbac_base.py
index 734007f..b426ecd 100644
--- a/patrole_tempest_plugin/tests/api/orchestration/rbac_base.py
+++ b/patrole_tempest_plugin/tests/api/orchestration/rbac_base.py
@@ -35,4 +35,4 @@
super(BaseOrchestrationRbacTest, cls).setup_clients()
cls.auth_provider = cls.os.auth_provider
cls.rbac_utils = rbac_utils()
- cls.rbac_utils.switch_role(cls, switchToRbacRole=False)
+ cls.rbac_utils.switch_role(cls, toggle_rbac_role=False)
diff --git a/patrole_tempest_plugin/tests/api/orchestration/test_resource_types_rbac.py b/patrole_tempest_plugin/tests/api/orchestration/test_resource_types_rbac.py
index d2cc5d0..a348512 100644
--- a/patrole_tempest_plugin/tests/api/orchestration/test_resource_types_rbac.py
+++ b/patrole_tempest_plugin/tests/api/orchestration/test_resource_types_rbac.py
@@ -44,20 +44,20 @@
@rbac_rule_validation.action(service="heat",
rule="stacks:list_resource_types")
def test_list_resource_types(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.list_resource_types()['resource_types']
@decorators.idempotent_id('8b0290f9-0d53-479e-8e4d-3d865b0107a4')
@rbac_rule_validation.action(service="heat",
rule="stacks:generate_template")
def test_show_resource_type_template(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.show_resource_type_template(self.resource_type_name)
@decorators.idempotent_id('2cdcd47f-6abe-43af-b736-c188df27dd38')
@rbac_rule_validation.action(service="heat",
rule="stacks:resource_schema")
def test_show_resource_type_schema(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.show_resource_type(self.resource_type_name)[
'resource_type']
diff --git a/patrole_tempest_plugin/tests/api/orchestration/test_soft_config_rbac.py b/patrole_tempest_plugin/tests/api/orchestration/test_soft_config_rbac.py
index fa3840c..d219ace 100644
--- a/patrole_tempest_plugin/tests/api/orchestration/test_soft_config_rbac.py
+++ b/patrole_tempest_plugin/tests/api/orchestration/test_soft_config_rbac.py
@@ -32,35 +32,35 @@
rule="software_configs:show")
@decorators.idempotent_id('b2e7c98c-e17b-4f37-82f3-5d21eff86e79')
def test_get_software_config(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.show_software_config(self.config['id'])
@rbac_rule_validation.action(service="heat",
rule="software_deployments:metadata")
@decorators.idempotent_id('defa34ab-9d1f-4b14-8613-34e964c0c478')
def test_get_deployment_metadata(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.show_software_deployment_metadata(self.server_id)
@rbac_rule_validation.action(service="heat",
rule="software_deployments:index")
@decorators.idempotent_id('2a4dcb91-1803-4749-9cb7-5b69ba668b18')
def test_get_deployment_list(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.list_software_deployments()
@rbac_rule_validation.action(service="heat",
rule="software_deployments:show")
@decorators.idempotent_id('d4e627bc-88a3-4189-8092-151f22ed989d')
def test_software_show_deployment(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.show_software_deployment(self.deployment_id)
@rbac_rule_validation.action(service="heat",
rule="software_deployments:update")
@decorators.idempotent_id('90e8958c-6fa7-4515-b6d7-6d6952979f8c')
def test_software_deployment_update(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
new_action = data_utils.rand_name('ACTION')
new_status = data_utils.rand_name('STATUS')
new_reason = data_utils.rand_name('REASON')
@@ -77,7 +77,7 @@
rule="software_deployments:create")
@decorators.idempotent_id('9175fe7b-4210-4c1d-acbb-954998a9fd77')
def test_software_deployment_create(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self._deployment_create(self.config['id'])
@rbac_rule_validation.action(service="heat",
@@ -85,14 +85,14 @@
@decorators.idempotent_id('20f4683d-7316-4d88-a6ea-1ee6013da908')
def test_software_deployment_delete(self):
deploy_id = self._deployment_create(self.config['id'])
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.delete_software_deploy(deploy_id)
@rbac_rule_validation.action(service="heat",
rule="software_configs:create")
@decorators.idempotent_id('c8fb1c73-fcb6-46c2-9510-8ef0083c9620')
def test_config_create(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self._config_create('e')
@rbac_rule_validation.action(service="heat",
@@ -100,7 +100,7 @@
@decorators.idempotent_id('f4f784ea-9878-4306-bc5f-041ba5307ce5')
def test_config_delete(self):
configuration = self._config_create('d')
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.delete_software_config(configuration['id'])
def _config_create(self, suffix):
diff --git a/patrole_tempest_plugin/tests/api/volume/admin/test_qos_rbac.py b/patrole_tempest_plugin/tests/api/volume/admin/test_qos_rbac.py
index 29c334f..74230a7 100644
--- a/patrole_tempest_plugin/tests/api/volume/admin/test_qos_rbac.py
+++ b/patrole_tempest_plugin/tests/api/volume/admin/test_qos_rbac.py
@@ -35,7 +35,7 @@
service="cinder", rule="volume_extension:qos_specs_manage:create")
@decorators.idempotent_id('4f9f45f0-b379-4577-a279-cec3e917cbec')
def test_create_qos_with_consumer(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
# Create a qos
self.create_test_qos_specs()
@@ -45,7 +45,7 @@
def test_delete_qos_with_consumer(self):
# Create a qos
qos = self.create_test_qos_specs()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
# Delete a qos
self.client.delete_qos(qos['id'])
@@ -55,7 +55,7 @@
def test_get_qos(self):
# Create a qos
qos = self.create_test_qos_specs()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
# Get a qos
self.client.show_qos(qos['id'])['qos_specs']
@@ -63,7 +63,7 @@
rule="volume_extension:qos_specs_manage:read")
@decorators.idempotent_id('546b8bb1-04a4-4387-9506-a538a7f3cd6a')
def test_list_qos(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
# list all qos
self.client.list_qos()['qos_specs']
@@ -73,7 +73,7 @@
def test_set_qos_key(self):
# Create a qos
qos = self.create_test_qos_specs()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
# set key
self.client.set_qos_key(qos['id'], iops_bytes='500')['qos_specs']
@@ -85,7 +85,7 @@
qos = self.create_test_qos_specs()
# Set key
self.client.set_qos_key(qos['id'], iops_bytes='500')['qos_specs']
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
# Unset key
keys = ['iops_bytes']
self.client.unset_qos_key(qos['id'], keys)
@@ -101,7 +101,7 @@
qos = self.create_test_qos_specs()
# create a test volume-type
vol_type = self.create_volume_type()['id']
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
# associate the qos-specs with volume-types
self.client.associate_qos(qos['id'], vol_type)
self.addCleanup(self.client.disassociate_qos, qos['id'], vol_type)
@@ -116,7 +116,7 @@
# associate the qos-specs with volume-types
self.client.associate_qos(qos['id'], vol_type)
self.addCleanup(self.client.disassociate_qos, qos['id'], vol_type)
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
# get the association of the qos-specs
self.client.show_association_qos(qos['id'])
@@ -131,7 +131,7 @@
self.client.associate_qos(qos['id'], vol_type)
self.addCleanup(test_utils.call_and_ignore_notfound_exc,
self.client.disassociate_qos, qos['id'], vol_type)
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
# disassociate a volume-type with qos-specs
self.client.disassociate_qos(qos['id'], vol_type)
operation = 'disassociate'
@@ -149,7 +149,7 @@
self.client.associate_qos(qos['id'], vol_type)
self.addCleanup(test_utils.call_and_ignore_notfound_exc,
self.client.disassociate_qos, qos['id'], vol_type)
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
# disassociate all volume-types from qos-specs
self.client.disassociate_all_qos(qos['id'])
operation = 'disassociate-all'
diff --git a/patrole_tempest_plugin/tests/api/volume/admin/test_volume_quotas_rbac.py b/patrole_tempest_plugin/tests/api/volume/admin/test_volume_quotas_rbac.py
index 8e054ac..28adfa7 100644
--- a/patrole_tempest_plugin/tests/api/volume/admin/test_volume_quotas_rbac.py
+++ b/patrole_tempest_plugin/tests/api/volume/admin/test_volume_quotas_rbac.py
@@ -43,7 +43,7 @@
rule="volume_extension:quotas:show")
@decorators.idempotent_id('b3c7177e-b6b1-4d0f-810a-fc95606964dd')
def test_list_default_quotas(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.show_default_quota_set(
self.demo_tenant_id)['quota_set']
@@ -55,7 +55,7 @@
'volumes': 11,
'snapshots': 11}
# Update limits for all quota resources
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.update_quota_set(
self.demo_tenant_id,
**new_quota_set)['quota_set']
diff --git a/patrole_tempest_plugin/tests/api/volume/admin/test_volumes_backup_admin_rbac.py b/patrole_tempest_plugin/tests/api/volume/admin/test_volumes_backup_admin_rbac.py
index a89aa87..e3d2438 100644
--- a/patrole_tempest_plugin/tests/api/volume/admin/test_volumes_backup_admin_rbac.py
+++ b/patrole_tempest_plugin/tests/api/volume/admin/test_volumes_backup_admin_rbac.py
@@ -59,7 +59,7 @@
# Create a temp backup
backup = self.create_backup(volume_id=self.volume['id'])
# Export Backup
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.backups_client.export_backup(
backup['id'])['backup-record']
@@ -77,7 +77,7 @@
new_url = self._modify_backup_url(
export_backup['backup_url'], {'id': new_id})
# Import the temp backup
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
import_backup = self.backups_client.import_backup(
backup_service=export_backup['backup_service'],
backup_url=new_url)['backup']
diff --git a/patrole_tempest_plugin/tests/api/volume/rbac_base.py b/patrole_tempest_plugin/tests/api/volume/rbac_base.py
index 3c7cd68..d47f8ff 100644
--- a/patrole_tempest_plugin/tests/api/volume/rbac_base.py
+++ b/patrole_tempest_plugin/tests/api/volume/rbac_base.py
@@ -35,7 +35,7 @@
super(BaseVolumeRbacTest, cls).setup_clients()
cls.auth_provider = cls.os.auth_provider
cls.rbac_utils = rbac_utils()
- cls.rbac_utils.switch_role(cls, switchToRbacRole=False)
+ cls.rbac_utils.switch_role(cls, toggle_rbac_role=False)
class BaseVolumeAdminRbacTest(vol_base.BaseVolumeAdminTest):
@@ -54,7 +54,7 @@
super(BaseVolumeAdminRbacTest, cls).setup_clients()
cls.auth_provider = cls.os.auth_provider
cls.rbac_utils = rbac_utils()
- cls.rbac_utils.switch_role(cls, switchToRbacRole=False)
+ cls.rbac_utils.switch_role(cls, toggle_rbac_role=False)
version_checker = {
1: [cls.os.volume_hosts_client, cls.os.volume_types_client],
2: [cls.os.volume_hosts_v2_client, cls.os.volume_types_v2_client],
diff --git a/patrole_tempest_plugin/tests/api/volume/test_availability_zone_rbac.py b/patrole_tempest_plugin/tests/api/volume/test_availability_zone_rbac.py
index 711c7d6..95b5949 100644
--- a/patrole_tempest_plugin/tests/api/volume/test_availability_zone_rbac.py
+++ b/patrole_tempest_plugin/tests/api/volume/test_availability_zone_rbac.py
@@ -33,5 +33,5 @@
rule="volume:availability_zone_list")
@decorators.idempotent_id('8cfd920c-4b6c-402d-b6e2-ede86bedc702')
def test_get_availability_zone_list(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.list_availability_zones()
diff --git a/patrole_tempest_plugin/tests/api/volume/test_extensions_rbac.py b/patrole_tempest_plugin/tests/api/volume/test_extensions_rbac.py
index c9c2b63..3304452 100644
--- a/patrole_tempest_plugin/tests/api/volume/test_extensions_rbac.py
+++ b/patrole_tempest_plugin/tests/api/volume/test_extensions_rbac.py
@@ -28,7 +28,7 @@
rule="volume:list_extensions")
@decorators.idempotent_id('7f2dcc41-e850-493f-a400-82db4e2b50c0')
def test_list_extensions(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.volumes_extension_client.list_extensions()
diff --git a/patrole_tempest_plugin/tests/api/volume/test_snapshots_actions_rbac.py b/patrole_tempest_plugin/tests/api/volume/test_snapshots_actions_rbac.py
index 7b5064a..b070a10 100644
--- a/patrole_tempest_plugin/tests/api/volume/test_snapshots_actions_rbac.py
+++ b/patrole_tempest_plugin/tests/api/volume/test_snapshots_actions_rbac.py
@@ -54,7 +54,7 @@
def test_reset_snapshot_status(self):
# Reset snapshot status to error
status = 'error'
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.\
reset_snapshot_status(self.snapshot['id'], status)
@@ -68,7 +68,7 @@
# and force delete temp snapshot
temp_snapshot = self.create_snapshot(self.volume['id'])
# Force delete the snapshot
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.force_delete_snapshot(temp_snapshot['id'])
self.client.wait_for_resource_deletion(temp_snapshot['id'])
diff --git a/patrole_tempest_plugin/tests/api/volume/test_snapshots_metadata_rbac.py b/patrole_tempest_plugin/tests/api/volume/test_snapshots_metadata_rbac.py
index e78c492..bf48716 100644
--- a/patrole_tempest_plugin/tests/api/volume/test_snapshots_metadata_rbac.py
+++ b/patrole_tempest_plugin/tests/api/volume/test_snapshots_metadata_rbac.py
@@ -53,7 +53,7 @@
@decorators.idempotent_id('c9cbec1c-edfe-46b8-825b-7b6ac0a58c25')
def test_create_snapshot_metadata(self):
# Create metadata for the snapshot
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self._create_test_snapshot_metadata()
@rbac_rule_validation.action(service="cinder",
@@ -63,7 +63,7 @@
# Create volume and snapshot metadata
self._create_test_snapshot_metadata()
# Get metadata for the snapshot
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.snapshots_client.show_snapshot_metadata(
self.snapshot_id)
@@ -75,7 +75,7 @@
# Create volume and snapshot metadata
self._create_test_snapshot_metadata()
# Get metadata for the snapshot
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
# Get the metadata of the snapshot
self.snapshots_client.show_snapshot_metadata(
self.snapshot_id)['metadata']
diff --git a/patrole_tempest_plugin/tests/api/volume/test_volume_actions_rbac.py b/patrole_tempest_plugin/tests/api/volume/test_volume_actions_rbac.py
index 5c181f9..e1f9430 100644
--- a/patrole_tempest_plugin/tests/api/volume/test_volume_actions_rbac.py
+++ b/patrole_tempest_plugin/tests/api/volume/test_volume_actions_rbac.py
@@ -66,7 +66,7 @@
@decorators.idempotent_id('f97b10e4-2eed-4f8b-8632-71c02cb9fe42')
def test_attach_volume_to_instance(self):
server = self._create_server()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
# Attach the volume
self._attach_volume(server)
@@ -77,7 +77,7 @@
# Attach the volume
server = self._create_server()
self._attach_volume(server)
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
# Detach the volume
self._detach_volume()
@@ -88,7 +88,7 @@
def test_volume_upload(self):
self.image_client = self.os.image_client
image_name = data_utils.rand_name('image')
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
body = self.client.upload_volume(
self.volume['id'], image_name=image_name,
disk_format=CONF.volume.disk_format)['os-volume_upload_image']
@@ -103,7 +103,7 @@
@decorators.idempotent_id('2750717a-f250-4e41-9e09-02624aad6ff8')
def test_volume_readonly_update(self):
volume = self.create_volume()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
# Update volume readonly
self.client.update_volume_readonly(volume['id'], readonly=True)
diff --git a/patrole_tempest_plugin/tests/api/volume/test_volume_basic_crud.py b/patrole_tempest_plugin/tests/api/volume/test_volume_basic_crud.py
index 70f53fd..6bc9e4e 100644
--- a/patrole_tempest_plugin/tests/api/volume/test_volume_basic_crud.py
+++ b/patrole_tempest_plugin/tests/api/volume/test_volume_basic_crud.py
@@ -29,7 +29,7 @@
rule="volume:create")
@decorators.idempotent_id('426b08ef-6394-4d06-9128-965d5a6c38ef')
def test_create_volume(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.create_volume()
@rbac_rule_validation.action(service="cinder",
@@ -37,14 +37,14 @@
@decorators.idempotent_id('6de9f9c2-509f-4558-867b-af21c7163be4')
def test_delete_volume(self):
volume = self.create_volume()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.volumes_client.delete_volume(volume['id'])
@rbac_rule_validation.action(service="cinder", rule="volume:get")
@decorators.idempotent_id('c4c3fdd5-b1b1-49c3-b977-a9f40ee9257a')
def test_get_volume(self):
volume = self.create_volume()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.volumes_client.show_volume(volume['id'])
@rbac_rule_validation.action(service="cinder",
@@ -52,7 +52,7 @@
@decorators.idempotent_id('e3ab7906-b04b-4c45-aa11-1104d302f940')
def test_volume_list(self):
# Get a list of Volumes
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.volumes_client.list_volumes()
@rbac_rule_validation.action(
@@ -61,7 +61,7 @@
@decorators.idempotent_id('3d48ca91-f02b-4616-a69d-4a8b296c8529')
def test_volume_list_image_metadata(self):
# Get a list of Volumes
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.volumes_client.list_volumes(detail=True)
@rbac_rule_validation.action(service="cinder", rule="volume:update")
@@ -69,7 +69,7 @@
def test_update_volume(self):
volume = self.create_volume()
new_name = data_utils.rand_name('volume')
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.volumes_client.update_volume(volume['id'],
name=new_name)
diff --git a/patrole_tempest_plugin/tests/api/volume/test_volume_hosts_rbac.py b/patrole_tempest_plugin/tests/api/volume/test_volume_hosts_rbac.py
index 60130ee..d4d010b 100644
--- a/patrole_tempest_plugin/tests/api/volume/test_volume_hosts_rbac.py
+++ b/patrole_tempest_plugin/tests/api/volume/test_volume_hosts_rbac.py
@@ -25,5 +25,5 @@
rule="volume_extension:hosts")
@decorators.idempotent_id('64e837f5-5452-4e26-b934-c721ea7a8644')
def test_list_hosts(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.volume_hosts_client.list_hosts()
diff --git a/patrole_tempest_plugin/tests/api/volume/test_volume_metadata_rbac.py b/patrole_tempest_plugin/tests/api/volume/test_volume_metadata_rbac.py
index 08de7b5..1c73e68 100644
--- a/patrole_tempest_plugin/tests/api/volume/test_volume_metadata_rbac.py
+++ b/patrole_tempest_plugin/tests/api/volume/test_volume_metadata_rbac.py
@@ -45,7 +45,7 @@
@decorators.idempotent_id('232bbb8b-4c29-44dc-9077-b1398c20b738')
def test_create_volume_metadata(self):
volume = self.create_volume()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self._add_metadata(volume)
@rbac_rule_validation.action(service="cinder",
@@ -54,7 +54,7 @@
def test_get_volume_metadata(self):
volume = self.create_volume()
self._add_metadata(volume)
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.volumes_client.show_volume_metadata(volume['id'])['metadata']
@rbac_rule_validation.action(service="cinder",
@@ -63,7 +63,7 @@
def test_delete_volume_metadata(self):
volume = self.create_volume()
self._add_metadata(volume)
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.volumes_client.delete_volume_metadata_item(volume['id'],
"key1")
@@ -75,7 +75,7 @@
self._add_metadata(volume)
# Metadata to update
update_item = {"key3": "value3_update"}
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.volumes_client.update_volume_metadata_item(
volume['id'], "key3", update_item)['meta']
diff --git a/patrole_tempest_plugin/tests/api/volume/test_volume_transfers_rbac.py b/patrole_tempest_plugin/tests/api/volume/test_volume_transfers_rbac.py
index 358733f..5505084 100644
--- a/patrole_tempest_plugin/tests/api/volume/test_volume_transfers_rbac.py
+++ b/patrole_tempest_plugin/tests/api/volume/test_volume_transfers_rbac.py
@@ -59,7 +59,7 @@
rule="volume:create_transfer")
@decorators.idempotent_id('25413af4-468d-48ff-94ca-4436f8526b3e')
def test_create_volume_transfer(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self._create_transfer()
@rbac_rule_validation.action(service="cinder",
@@ -67,14 +67,14 @@
@decorators.idempotent_id('7a0925d3-ed97-4c25-8299-e5cdabe2eb55')
def test_get_volume_transfer(self):
transfer = self._create_transfer()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.show_volume_transfer(transfer['id'])
@rbac_rule_validation.action(service="cinder",
rule="volume:get_all_transfers")
@decorators.idempotent_id('02a06f2b-5040-49e2-b2b7-619a7db59603')
def test_list_volume_transfers(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.list_volume_transfers()
@rbac_rule_validation.action(service="cinder",
@@ -82,7 +82,7 @@
@decorators.idempotent_id('987f2a11-d657-4984-a6c9-28f06c1cd014')
def test_accept_volume_transfer(self):
transfer = self._create_transfer()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.accept_volume_transfer(transfer['id'],
auth_key=transfer['auth_key'])
@@ -91,7 +91,7 @@
@decorators.idempotent_id('4672187e-7fff-454b-832a-5c8865dda868')
def test_delete_volume_transfer(self):
transfer = self._create_transfer()
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.delete_volume_transfer(transfer['id'])
diff --git a/patrole_tempest_plugin/tests/api/volume/test_volume_types_extra_specs_rbac.py b/patrole_tempest_plugin/tests/api/volume/test_volume_types_extra_specs_rbac.py
index e045794..6978166 100644
--- a/patrole_tempest_plugin/tests/api/volume/test_volume_types_extra_specs_rbac.py
+++ b/patrole_tempest_plugin/tests/api/volume/test_volume_types_extra_specs_rbac.py
@@ -31,6 +31,6 @@
vol_type = self.create_volume_type()
# List Volume types extra specs.
extra_specs = {"spec1": "val1"}
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.volume_types_client.create_volume_type_extra_specs(
vol_type['id'], extra_specs)
diff --git a/patrole_tempest_plugin/tests/api/volume/test_volumes_backup_rbac.py b/patrole_tempest_plugin/tests/api/volume/test_volumes_backup_rbac.py
index 20991fe..6e1b9f1 100644
--- a/patrole_tempest_plugin/tests/api/volume/test_volumes_backup_rbac.py
+++ b/patrole_tempest_plugin/tests/api/volume/test_volumes_backup_rbac.py
@@ -55,7 +55,7 @@
rule="backup:create")
@decorators.idempotent_id('6887ec94-0bcf-4ab7-b30f-3808a4b5a2a5')
def test_volume_backup_create(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self._create_backup(volume_id=self.volume['id'])
@test.attr(type="slow")
@@ -66,14 +66,14 @@
# Create a temp backup
backup = self._create_backup(volume_id=self.volume['id'])
# Get a given backup
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.backups_client.show_backup(backup['id'])
@rbac_rule_validation.action(service="cinder",
rule="backup:get_all")
@decorators.idempotent_id('4d18f0f0-7e01-4007-b622-dedc859b22f6')
def test_volume_backup_list(self):
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.backups_client.list_backups()
@test.attr(type="slow")
@@ -84,7 +84,7 @@
# Create a temp backup
backup = self._create_backup(volume_id=self.volume['id'])
# Restore backup
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
restore = self.backups_client.restore_backup(backup['id'])['restore']
waiters.wait_for_volume_resource_status(
self.backups_client, restore['backup_id'], 'available')
@@ -96,7 +96,7 @@
def test_volume_backup_delete(self):
# Create a temp backup
backup = self._create_backup(volume_id=self.volume['id'])
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
# Delete backup
self.backups_client.delete_backup(backup['id'])
self.backups_client.wait_for_resource_deletion(backup['id'])
diff --git a/patrole_tempest_plugin/tests/api/volume/test_volumes_extend_rbac.py b/patrole_tempest_plugin/tests/api/volume/test_volumes_extend_rbac.py
index 64493c5..72afd69 100644
--- a/patrole_tempest_plugin/tests/api/volume/test_volumes_extend_rbac.py
+++ b/patrole_tempest_plugin/tests/api/volume/test_volumes_extend_rbac.py
@@ -36,7 +36,7 @@
def test_volume_extend(self):
# Extend volume test
extend_size = int(self.volume['size']) + 1
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.volumes_client.extend_volume(self.volume['id'],
new_size=extend_size)
waiters.wait_for_volume_resource_status(
diff --git a/patrole_tempest_plugin/tests/api/volume/test_volumes_rbac.py b/patrole_tempest_plugin/tests/api/volume/test_volumes_rbac.py
index f6e79dc..b10f5b3 100644
--- a/patrole_tempest_plugin/tests/api/volume/test_volumes_rbac.py
+++ b/patrole_tempest_plugin/tests/api/volume/test_volumes_rbac.py
@@ -39,7 +39,7 @@
def test_volume_reset_status(self):
volume = self.create_volume()
# Test volume reset status : available->error->available
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.reset_volume_status(volume['id'], status='error')
self.client.reset_volume_status(volume['id'], status='available')
@@ -51,7 +51,7 @@
volume = self.create_volume()
self.client.reset_volume_status(volume['id'], status='error')
# Test force delete when status of volume is error
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.force_delete_volume(volume['id'])
self.client.wait_for_resource_deletion(volume['id'])
diff --git a/patrole_tempest_plugin/tests/api/volume/test_volumes_snapshots_rbac.py b/patrole_tempest_plugin/tests/api/volume/test_volumes_snapshots_rbac.py
index 9374cd9..ce443e2 100644
--- a/patrole_tempest_plugin/tests/api/volume/test_volumes_snapshots_rbac.py
+++ b/patrole_tempest_plugin/tests/api/volume/test_volumes_snapshots_rbac.py
@@ -58,7 +58,7 @@
@decorators.idempotent_id('ac7b2ee5-fbc0-4360-afc2-de8fa4881ede')
def test_snapshot_create(self):
# Create a temp snapshot
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.create_snapshot(self.volume['id'])
@rbac_rule_validation.action(service="cinder",
@@ -66,7 +66,7 @@
@decorators.idempotent_id('93a11b40-1ba8-44d6-a196-f8d97220f796')
def test_snapshot_get(self):
# Get the snapshot
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.show_snapshot(self.snapshot
['id'])['snapshot']
@@ -77,7 +77,7 @@
new_desc = 'This is the new description of snapshot.'
params = {'description': new_desc}
# Updates snapshot with new values
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self.client.update_snapshot(
self.snapshot['id'], **params)['snapshot']
@@ -88,7 +88,7 @@
"""list snapshots with params."""
# Verify list snapshots by display_name filter
params = {'name': self.snapshot['name']}
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
self._list_by_param_values(params)
@rbac_rule_validation.action(service="cinder",
@@ -97,7 +97,7 @@
def test_snapshot_delete(self):
# Create a temp snapshot
temp_snapshot = self.create_snapshot(self.volume['id'])
- self.rbac_utils.switch_role(self, switchToRbacRole=True)
+ self.rbac_utils.switch_role(self, toggle_rbac_role=True)
# Delete the snapshot
self.client.delete_snapshot(temp_snapshot['id'])
diff --git a/patrole_tempest_plugin/tests/unit/test_rbac_utils.py b/patrole_tempest_plugin/tests/unit/test_rbac_utils.py
index 692c0b9..feccfe5 100644
--- a/patrole_tempest_plugin/tests/unit/test_rbac_utils.py
+++ b/patrole_tempest_plugin/tests/unit/test_rbac_utils.py
@@ -174,7 +174,7 @@
self.rbac_utils.switch_role, self.mock_test_obj,
False)
self.assertIn(
- '`switchToRbacRole` must not be called with the same bool value '
+ '`toggle_rbac_role` must not be called with the same bool value '
'twice. Make sure that you included a rbac_utils.switch_role '
'method call inside the test.', str(e))
@@ -187,7 +187,7 @@
self.rbac_utils.switch_role, self.mock_test_obj,
True)
self.assertIn(
- '`switchToRbacRole` must not be called with the same bool value '
+ '`toggle_rbac_role` must not be called with the same bool value '
'twice. Make sure that you included a rbac_utils.switch_role '
'method call inside the test.', str(e))