Merge "Renames switchToRbacRole to toggle_rbac_role"
diff --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 59e1641..cb40c50 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
@@ -55,7 +55,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",
@@ -63,14 +63,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",
@@ -78,7 +78,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'])
@@ -87,7 +87,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))