commit dc73cff02ab484f901876bb42cfdd4d1f266432b
author Felipe Monteiro <felipe.monteiro@att.com> Tue Aug 22 22:21:50 2017 -0400
committer Felipe Monteiro <felipe.monteiro@att.com> Tue Aug 22 22:21:50 2017 -0400
tree a5c80693d03f677c61c05a5176aaac0997e60af5
parent d6bde6096519a9fb6878a4fafb592c039420d422

Remove identity v3 change_password test

This commit removes the v3 change_password test because
it is currently getting skipped [0], because the policy
action does not exist in [1] and the API where it would
be enforced does not enforce it [2].

So the correct approach is to remove the test. Also renames
the class name to drop "Admin" substring as there is nothing
about this test class that requires admin credentials in any
way.

[0] http://logs.openstack.org/68/496068/4/check/gate-tempest-dsvm-patrole-admin-ubuntu-xenial/7069c22/console.html#_2017-08-23_00_50_18_947787
[1] https://github.com/openstack/keystone/blob/master/keystone/common/policies/user.py
[2] https://github.com/openstack/keystone/blob/4950c24bddaf98b79dfd9ee178ae1086cf48e322/keystone/identity/controllers.py#L279

Change-Id: I90496d3969b1ef69316a2254754622679bca54b5

patrole_tempest_plugin/tests/api/identity/v3/test_users_rbac.py

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 0c85240..5812f9e 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
@@ -20,11 +20,11 @@
 from patrole_tempest_plugin.tests.api.identity import rbac_base
 
 
-class IdentityUserV3AdminRbacTest(rbac_base.BaseIdentityV3RbacTest):
+class IdentityUserV3RbacTest(rbac_base.BaseIdentityV3RbacTest):
 
     @classmethod
     def resource_setup(cls):
-        super(IdentityUserV3AdminRbacTest, cls).resource_setup()
+        super(IdentityUserV3RbacTest, cls).resource_setup()
         cls.default_user_id = cls.os_primary.credentials.user_id
 
     @rbac_rule_validation.action(service="keystone",
@@ -71,19 +71,6 @@
         self.users_client.show_user(self.default_user_id)
 
     @rbac_rule_validation.action(service="keystone",
-                                 rule="identity:change_password")
-    @decorators.idempotent_id('0f148510-63bf-11e6-4522-080044d0d90a')
-    def test_change_password(self):
-        original_password = data_utils.rand_password()
-        user = self.setup_test_user(password=original_password)
-
-        self.rbac_utils.switch_role(self, toggle_rbac_role=True)
-        self.users_client.update_user_password(
-            user['id'],
-            original_password=original_password,
-            password=data_utils.rand_password())
-
-    @rbac_rule_validation.action(service="keystone",
                                  rule="identity:list_groups_for_user")
     @decorators.idempotent_id('bd5946d4-46d2-423d-a800-a3e7aabc18b3')
     def test_list_own_user_group(self):