diff options
author | Zack Feldstein <zfeldstein@Zachs-MacBook-Pro.local> | 2015-12-14 10:44:07 -0600 |
---|---|---|
committer | zack feldstein <zfeldstein@bluebox.net> | 2015-12-15 19:02:27 -0600 |
commit | d8c5f7afb319075667659e70dc0360a37ae5420b (patch) | |
tree | 5dc9983f600367a2e316ea997dcd173d35274935 /tempest | |
parent | b5548a62597ead1579a86854884ac6a19fd83053 (diff) | |
download | tempest-d8c5f7afb319075667659e70dc0360a37ae5420b.tar.gz |
Fix password not strong enough failures
Use rand_password method to create users with more complex
passwords.
Closes-Bug: 1448217
Depends-On: I86bf157f1bdb44f5fc579dc5317784fe31df8521
Change-Id: I57649f5aac9b1abe1a9961d4b35479372ebee519
Diffstat (limited to 'tempest')
-rw-r--r-- | tempest/api/compute/admin/test_quotas.py | 2 | ||||
-rw-r--r-- | tempest/api/identity/admin/v2/test_tokens.py | 4 | ||||
-rw-r--r-- | tempest/api/identity/admin/v2/test_users.py | 14 | ||||
-rw-r--r-- | tempest/api/identity/admin/v2/test_users_negative.py | 2 | ||||
-rw-r--r-- | tempest/api/identity/admin/v3/test_credentials.py | 2 | ||||
-rw-r--r-- | tempest/api/identity/admin/v3/test_groups.py | 3 | ||||
-rw-r--r-- | tempest/api/identity/admin/v3/test_list_users.py | 2 | ||||
-rw-r--r-- | tempest/api/identity/admin/v3/test_projects.py | 2 | ||||
-rw-r--r-- | tempest/api/identity/admin/v3/test_roles.py | 2 | ||||
-rw-r--r-- | tempest/api/identity/admin/v3/test_tokens.py | 4 | ||||
-rw-r--r-- | tempest/api/identity/admin/v3/test_trusts.py | 2 | ||||
-rw-r--r-- | tempest/api/identity/admin/v3/test_users.py | 8 | ||||
-rw-r--r-- | tempest/api/identity/admin/v3/test_users_negative.py | 2 | ||||
-rw-r--r-- | tempest/api/identity/base.py | 4 |
14 files changed, 28 insertions, 25 deletions
diff --git a/tempest/api/compute/admin/test_quotas.py b/tempest/api/compute/admin/test_quotas.py index fa1c66638..2907e2608 100644 --- a/tempest/api/compute/admin/test_quotas.py +++ b/tempest/api/compute/admin/test_quotas.py @@ -111,7 +111,7 @@ class QuotasAdminTestJSON(base.BaseV2ComputeAdminTest): # Verify that GET shows the updated quota set of user user_name = data_utils.rand_name('cpu_quota_user') - password = data_utils.rand_name('password') + password = data_utils.rand_password() email = user_name + '@testmail.tm' user = self.identity_utils.create_user(username=user_name, password=password, diff --git a/tempest/api/identity/admin/v2/test_tokens.py b/tempest/api/identity/admin/v2/test_tokens.py index cb9250b3f..0daa66d6c 100644 --- a/tempest/api/identity/admin/v2/test_tokens.py +++ b/tempest/api/identity/admin/v2/test_tokens.py @@ -24,7 +24,7 @@ class TokensTestJSON(base.BaseIdentityV2AdminTest): def test_create_get_delete_token(self): # get a token by username and password user_name = data_utils.rand_name(name='user') - user_password = data_utils.rand_name(name='pass') + user_password = data_utils.rand_password() # first:create a tenant tenant_name = data_utils.rand_name(name='tenant') tenant = self.tenants_client.create_tenant(tenant_name)['tenant'] @@ -59,7 +59,7 @@ class TokensTestJSON(base.BaseIdentityV2AdminTest): # Create a user. user_name = data_utils.rand_name(name='user') - user_password = data_utils.rand_name(name='pass') + user_password = data_utils.rand_password() tenant_id = None # No default tenant so will get unscoped token. email = '' user = self.client.create_user(user_name, user_password, diff --git a/tempest/api/identity/admin/v2/test_users.py b/tempest/api/identity/admin/v2/test_users.py index 13d6137d3..9cf1ce0b6 100644 --- a/tempest/api/identity/admin/v2/test_users.py +++ b/tempest/api/identity/admin/v2/test_users.py @@ -26,7 +26,7 @@ class UsersTestJSON(base.BaseIdentityV2AdminTest): def resource_setup(cls): super(UsersTestJSON, cls).resource_setup() cls.alt_user = data_utils.rand_name('test_user') - cls.alt_password = data_utils.rand_name('pass') + cls.alt_password = data_utils.rand_password() cls.alt_email = cls.alt_user + '@testmail.tm' @test.attr(type='smoke') @@ -132,15 +132,16 @@ class UsersTestJSON(base.BaseIdentityV2AdminTest): self.data.setup_test_tenant() user_ids = list() fetched_user_ids = list() + password1 = data_utils.rand_password() alt_tenant_user1 = data_utils.rand_name('tenant_user1') - user1 = self.client.create_user(alt_tenant_user1, 'password1', + user1 = self.client.create_user(alt_tenant_user1, password1, self.data.tenant['id'], 'user1@123')['user'] user_ids.append(user1['id']) self.data.users.append(user1) - + password2 = data_utils.rand_password() alt_tenant_user2 = data_utils.rand_name('tenant_user2') - user2 = self.client.create_user(alt_tenant_user2, 'password2', + user2 = self.client.create_user(alt_tenant_user2, password2, self.data.tenant['id'], 'user2@123')['user'] user_ids.append(user2['id']) @@ -173,7 +174,8 @@ class UsersTestJSON(base.BaseIdentityV2AdminTest): role['id'])['role'] alt_user2 = data_utils.rand_name('second_user') - second_user = self.client.create_user(alt_user2, 'password1', + alt_password2 = data_utils.rand_password() + second_user = self.client.create_user(alt_user2, alt_password2, self.data.tenant['id'], 'user2@123')['user'] user_ids.append(second_user['id']) @@ -198,7 +200,7 @@ class UsersTestJSON(base.BaseIdentityV2AdminTest): # Test case to check if updating of user password is successful. self.data.setup_test_user() # Updating the user with new password - new_pass = data_utils.rand_name('pass') + new_pass = data_utils.rand_password() update_user = self.client.update_user_password( self.data.user['id'], new_pass)['user'] self.assertEqual(update_user['id'], self.data.user['id']) diff --git a/tempest/api/identity/admin/v2/test_users_negative.py b/tempest/api/identity/admin/v2/test_users_negative.py index 0c7afe804..8fa5a3673 100644 --- a/tempest/api/identity/admin/v2/test_users_negative.py +++ b/tempest/api/identity/admin/v2/test_users_negative.py @@ -28,7 +28,7 @@ class UsersNegativeTestJSON(base.BaseIdentityV2AdminTest): def resource_setup(cls): super(UsersNegativeTestJSON, cls).resource_setup() cls.alt_user = data_utils.rand_name('test_user') - cls.alt_password = data_utils.rand_name('pass') + cls.alt_password = data_utils.rand_password() cls.alt_email = cls.alt_user + '@testmail.tm' @test.attr(type=['negative']) diff --git a/tempest/api/identity/admin/v3/test_credentials.py b/tempest/api/identity/admin/v3/test_credentials.py index 5e1c3cc44..b81bff758 100644 --- a/tempest/api/identity/admin/v3/test_credentials.py +++ b/tempest/api/identity/admin/v3/test_credentials.py @@ -29,7 +29,7 @@ class CredentialsTestJSON(base.BaseIdentityV3AdminTest): u_name = data_utils.rand_name('user') u_desc = '%s description' % u_name u_email = '%s@testmail.tm' % u_name - u_password = data_utils.rand_name('pass') + u_password = data_utils.rand_password() for i in range(2): cls.project = cls.client.create_project( data_utils.rand_name('project'), diff --git a/tempest/api/identity/admin/v3/test_groups.py b/tempest/api/identity/admin/v3/test_groups.py index 260ea54eb..e02202356 100644 --- a/tempest/api/identity/admin/v3/test_groups.py +++ b/tempest/api/identity/admin/v3/test_groups.py @@ -67,7 +67,8 @@ class GroupsV3TestJSON(base.BaseIdentityV3AdminTest): users = [] for i in range(3): name = data_utils.rand_name('User') - user = self.client.create_user(name)['user'] + password = data_utils.rand_password() + user = self.client.create_user(name, password)['user'] users.append(user) self.addCleanup(self.client.delete_user, user['id']) self.groups_client.add_group_user(group['id'], user['id']) diff --git a/tempest/api/identity/admin/v3/test_list_users.py b/tempest/api/identity/admin/v3/test_list_users.py index b7f37d494..ca91ce57c 100644 --- a/tempest/api/identity/admin/v3/test_list_users.py +++ b/tempest/api/identity/admin/v3/test_list_users.py @@ -34,7 +34,7 @@ class UsersV3TestJSON(base.BaseIdentityV3AdminTest): def resource_setup(cls): super(UsersV3TestJSON, cls).resource_setup() alt_user = data_utils.rand_name('test_user') - alt_password = data_utils.rand_name('pass') + alt_password = data_utils.rand_password() cls.alt_email = alt_user + '@testmail.tm' cls.data.setup_test_domain() # Create user with Domain diff --git a/tempest/api/identity/admin/v3/test_projects.py b/tempest/api/identity/admin/v3/test_projects.py index d39fd5f06..af9497c7c 100644 --- a/tempest/api/identity/admin/v3/test_projects.py +++ b/tempest/api/identity/admin/v3/test_projects.py @@ -161,7 +161,7 @@ class ProjectsTestJSON(base.BaseIdentityV3AdminTest): u_name = data_utils.rand_name('user') u_desc = u_name + 'description' u_email = u_name + '@testmail.tm' - u_password = data_utils.rand_name('pass') + u_password = data_utils.rand_password() user = self.client.create_user( u_name, description=u_desc, password=u_password, email=u_email, project_id=project['id'])['user'] diff --git a/tempest/api/identity/admin/v3/test_roles.py b/tempest/api/identity/admin/v3/test_roles.py index 3be264393..f194d9cb3 100644 --- a/tempest/api/identity/admin/v3/test_roles.py +++ b/tempest/api/identity/admin/v3/test_roles.py @@ -31,7 +31,7 @@ class RolesV3TestJSON(base.BaseIdentityV3AdminTest): u_name = data_utils.rand_name('user') u_desc = '%s description' % u_name u_email = '%s@testmail.tm' % u_name - cls.u_password = data_utils.rand_name('pass') + cls.u_password = data_utils.rand_password() cls.domain = cls.client.create_domain( data_utils.rand_name('domain'), description=data_utils.rand_name('domain-desc'))['domain'] diff --git a/tempest/api/identity/admin/v3/test_tokens.py b/tempest/api/identity/admin/v3/test_tokens.py index 7d33d4a13..f5b20d526 100644 --- a/tempest/api/identity/admin/v3/test_tokens.py +++ b/tempest/api/identity/admin/v3/test_tokens.py @@ -29,7 +29,7 @@ class TokensV3TestJSON(base.BaseIdentityV3AdminTest): u_name = data_utils.rand_name('user') u_desc = '%s-description' % u_name u_email = '%s@testmail.tm' % u_name - u_password = data_utils.rand_name('pass') + u_password = data_utils.rand_password() user = self.client.create_user( u_name, description=u_desc, password=u_password, email=u_email)['user'] @@ -60,7 +60,7 @@ class TokensV3TestJSON(base.BaseIdentityV3AdminTest): # Create a user. user_name = data_utils.rand_name(name='user') - user_password = data_utils.rand_name(name='pass') + user_password = data_utils.rand_password() user = self.client.create_user(user_name, password=user_password)['user'] self.addCleanup(self.client.delete_user, user['id']) diff --git a/tempest/api/identity/admin/v3/test_trusts.py b/tempest/api/identity/admin/v3/test_trusts.py index a8b0af9f3..ee6aa01ea 100644 --- a/tempest/api/identity/admin/v3/test_trusts.py +++ b/tempest/api/identity/admin/v3/test_trusts.py @@ -55,7 +55,7 @@ class BaseTrustsV3Test(base.BaseIdentityV3AdminTest): self.trustor_username = data_utils.rand_name('user') u_desc = self.trustor_username + 'description' u_email = self.trustor_username + '@testmail.xx' - self.trustor_password = data_utils.rand_name('pass') + self.trustor_password = data_utils.rand_password() user = self.client.create_user( self.trustor_username, description=u_desc, diff --git a/tempest/api/identity/admin/v3/test_users.py b/tempest/api/identity/admin/v3/test_users.py index e8d4a3644..6dbd4434b 100644 --- a/tempest/api/identity/admin/v3/test_users.py +++ b/tempest/api/identity/admin/v3/test_users.py @@ -29,7 +29,7 @@ class UsersV3TestJSON(base.BaseIdentityV3AdminTest): u_name = data_utils.rand_name('user') u_desc = u_name + 'description' u_email = u_name + '@testmail.tm' - u_password = data_utils.rand_name('pass') + u_password = data_utils.rand_password() user = self.client.create_user( u_name, description=u_desc, password=u_password, email=u_email, enabled=False)['user'] @@ -69,13 +69,13 @@ class UsersV3TestJSON(base.BaseIdentityV3AdminTest): def test_update_user_password(self): # Creating User to check password updation u_name = data_utils.rand_name('user') - original_password = data_utils.rand_name('pass') + original_password = data_utils.rand_password() user = self.client.create_user( u_name, password=original_password)['user'] # Delete the User at the end all test methods self.addCleanup(self.client.delete_user, user['id']) # Update user with new password - new_password = data_utils.rand_name('pass1') + new_password = data_utils.rand_password() self.client.update_user_password(user['id'], password=new_password, original_password=original_password) # TODO(lbragstad): Sleeping after the response status has been checked @@ -109,7 +109,7 @@ class UsersV3TestJSON(base.BaseIdentityV3AdminTest): u_name = data_utils.rand_name('user') u_desc = u_name + 'description' u_email = u_name + '@testmail.tm' - u_password = data_utils.rand_name('pass') + u_password = data_utils.rand_password() user_body = self.client.create_user( u_name, description=u_desc, password=u_password, email=u_email, enabled=False, project_id=u_project['id'])['user'] diff --git a/tempest/api/identity/admin/v3/test_users_negative.py b/tempest/api/identity/admin/v3/test_users_negative.py index d40a5b95d..4c80bdaee 100644 --- a/tempest/api/identity/admin/v3/test_users_negative.py +++ b/tempest/api/identity/admin/v3/test_users_negative.py @@ -28,7 +28,7 @@ class UsersNegativeTest(base.BaseIdentityV3AdminTest): # Attempt to create a user in a non-existent domain should fail u_name = data_utils.rand_name('user') u_email = u_name + '@testmail.tm' - u_password = data_utils.rand_name('pass') + u_password = data_utils.rand_password() self.assertRaises(lib_exc.NotFound, self.client.create_user, u_name, u_password, email=u_email, diff --git a/tempest/api/identity/base.py b/tempest/api/identity/base.py index 03a0391cc..8359b8f8a 100644 --- a/tempest/api/identity/base.py +++ b/tempest/api/identity/base.py @@ -212,7 +212,7 @@ class DataGenerator(object): """Set up a test user.""" self.setup_test_tenant() self.test_user = data_utils.rand_name('test_user') - self.test_password = data_utils.rand_name('pass') + self.test_password = data_utils.rand_password() self.test_email = self.test_user + '@testmail.tm' self.user = self.client.create_user(self.test_user, self.test_password, @@ -239,7 +239,7 @@ class DataGenerator(object): """Set up a test v3 user.""" self.setup_test_project() self.test_user = data_utils.rand_name('test_user') - self.test_password = data_utils.rand_name('pass') + self.test_password = data_utils.rand_password() self.test_email = self.test_user + '@testmail.tm' self.v3_user = self.client.create_user( self.test_user, |