openstack/tempest
Code Issues Proposed changes

Define the v3 Users Client as a library

Browse Source 
Migrate the v3 API Users Client to tempest.lib and add unit tests
for the client.

Partially implements blueprint consistent-service-method-names

Change-Id: I32441765d1f9028f5ddaaddf045f534181b44ad3
This commit is contained in:
Dustin Schoenbrun
2016-09-01 16:28:42 -04:00
committed by Martin Kopec
parent 90990cd586
commit 6c74229962
5 changed files with 211 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
releasenotes/notes/add-new-identity-clients-as-library-5f7ndha733nwdsn9.yaml
View File

@@ -7,3 +7,4 @@ features:
* identity_client(v2)
* groups_client(v3)
* users_client(v3)

0
tempest/services/identity/v3/json/users_clients.py → tempest/lib/services/identity/v3/users_client.py
View File

6
tempest/services/identity/v3/__init__.py
View File

@@ -19,16 +19,16 @@ from tempest.lib.services.identity.v3.projects_client import ProjectsClient
from tempest.lib.services.identity.v3.regions_client import RegionsClient
from tempest.lib.services.identity.v3.services_client import ServicesClient
from tempest.lib.services.identity.v3.token_client import V3TokenClient
from tempest.lib.services.identity.v3.users_client import UsersClient
from tempest.services.identity.v3.json.credentials_client import \
CredentialsClient
from tempest.services.identity.v3.json.domains_client import DomainsClient
from tempest.services.identity.v3.json.identity_client import IdentityClient
from tempest.services.identity.v3.json.roles_client import RolesClient
from tempest.services.identity.v3.json.trusts_client import TrustsClient
from tempest.services.identity.v3.json.users_clients import UsersClient
__all__ = ['EndPointsClient', 'GroupsClient', 'PoliciesClient',
'ProjectsClient', 'RegionsClient', 'ServicesClient',
'V3TokenClient', 'CredentialsClient', 'DomainsClient',
'IdentityClient', 'RolesClient', 'TrustsClient', 'UsersClient', ]
'V3TokenClient', 'UsersClient', 'CredentialsClient',
'DomainsClient', 'IdentityClient', 'RolesClient', 'TrustsClient', ]

3
tempest/tests/common/test_dynamic_creds.py
View File

@@ -31,11 +31,12 @@ from tempest.lib.services.identity.v2 import users_client as v2_users_client
from tempest.lib.services.identity.v3 import projects_client as \
v3_projects_client
from tempest.lib.services.identity.v3 import token_client as v3_token_client
from tempest.lib.services.identity.v3 import users_client as \
v3_users_client
from tempest.lib.services.network import routers_client
from tempest.services.identity.v3.json import domains_client
from tempest.services.identity.v3.json import identity_client as v3_iden_client
from tempest.services.identity.v3.json import roles_client as v3_roles_client
from tempest.services.identity.v3.json import users_clients as v3_users_client
from tempest.tests import base
from tempest.tests import fake_config
from tempest.tests.lib import fake_http

205
tempest/tests/lib/services/identity/v3/test_users_client.py Normal file
View File

@@ -0,0 +1,205 @@
# Copyright 2016 Red Hat, Inc.
#
# Licensed under the Apache License, Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
# You may obtain a copy of the License at
#
# http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS,
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
# See the License for the specific language governing permissions and
# limitations under the License.
from tempest.lib.services.identity.v3 import users_client
from tempest.tests.lib import fake_auth_provider
from tempest.tests.lib.services import base
class TestUsersClient(base.BaseServiceTest):
FAKE_CREATE_USER = {
'user': {
'default_project_id': '95f8c3f8e7b54409a418fc30717f9ae0',
'domain_id': '8347b31afc3545c4b311cb4cce788a08',
'enabled': True,
'name': 'Tempest User',
'password': 'TempestPassword',
}
}
FAKE_USER_INFO = {
'user': {
'default_project_id': '95f8c3f8e7b54409a418fc30717f9ae0',
'domain_id': '8347b31afc3545c4b311cb4cce788a08',
'enabled': True,
'id': '817fb3c23fd7465ba6d7fe1b1320121d',
'links': {
'self': 'http://example.com/identity',
},
'name': 'Tempest User',
'password_expires_at': '2016-11-06T15:32:17.000000',
}
}
FAKE_USER_LIST = {
'links': {
'next': None,
'previous': None,
'self': 'http://example.com/identity/v3/users',
},
'users': [
{
'domain_id': 'TempestDomain',
'enabled': True,
'id': '817fb3c23fd7465ba6d7fe1b1320121d',
'links': {
'self': 'http://example.com/identity/v3/users/' +
'817fb3c23fd7465ba6d7fe1b1320121d',
},
'name': 'Tempest User',
'password_expires_at': '2016-11-06T15:32:17.000000',
},
{
'domain_id': 'TempestDomain',
'enabled': True,
'id': 'bdbfb1e2f1344be197e90a778379cca1',
'links': {
'self': 'http://example.com/identity/v3/users/' +
'bdbfb1e2f1344be197e90a778379cca1',
},
'name': 'Tempest User',
'password_expires_at': None,
},
]
}
FAKE_GROUP_LIST = {
'links': {
'self': 'http://example.com/identity/v3/groups',
'previous': None,
'next': None,
},
'groups': [
{
'description': 'Tempest Group One Description',
'domain_id': 'TempestDomain',
'id': '1c92f3453ed34291a074b87493455b8f',
'links': {
'self': 'http://example.com/identity/v3/groups/' +
'1c92f3453ed34291a074b87493455b8f'
},
'name': 'Tempest Group One',
},
{
'description': 'Tempest Group Two Description',
'domain_id': 'TempestDomain',
'id': 'ce9e7dafed3b4877a7d4466ed730a9ee',
'links': {
'self': 'http://example.com/identity/v3/groups/' +
'ce9e7dafed3b4877a7d4466ed730a9ee'
},
'name': 'Tempest Group Two',
},
]
}
def setUp(self):
super(TestUsersClient, self).setUp()
fake_auth = fake_auth_provider.FakeAuthProvider()
self.client = users_client.UsersClient(fake_auth, 'identity',
'regionOne')
def _test_create_user(self, bytes_body=False):
self.check_service_client_function(
self.client.create_user,
'tempest.lib.common.rest_client.RestClient.post',
self.FAKE_CREATE_USER,
bytes_body,
status=201,
)
def _test_show_user(self, bytes_body=False):
self.check_service_client_function(
self.client.show_user,
'tempest.lib.common.rest_client.RestClient.get',
self.FAKE_USER_INFO,
bytes_body,
user_id='817fb3c23fd7465ba6d7fe1b1320121d',
)
def _test_list_users(self, bytes_body=False):
self.check_service_client_function(
self.client.list_users,
'tempest.lib.common.rest_client.RestClient.get',
self.FAKE_USER_LIST,
bytes_body,
)
def _test_update_user(self, bytes_body=False):
self.check_service_client_function(
self.client.update_user,
'tempest.lib.common.rest_client.RestClient.patch',
self.FAKE_USER_INFO,
bytes_body,
user_id='817fb3c23fd7465ba6d7fe1b1320121d',
name='NewName',
)
def _test_list_user_groups(self, bytes_body=False):
self.check_service_client_function(
self.client.list_user_groups,
'tempest.lib.common.rest_client.RestClient.get',
self.FAKE_GROUP_LIST,
bytes_body,
user_id='817fb3c23fd7465ba6d7fe1b1320121d',
)
def test_create_user_with_string_body(self):
self._test_create_user()
def test_create_user_with_bytes_body(self):
self._test_create_user(bytes_body=True)
def test_show_user_with_string_body(self):
self._test_show_user()
def test_show_user_with_bytes_body(self):
self._test_show_user(bytes_body=True)
def test_list_users_with_string_body(self):
self._test_list_users()
def test_list_users_with_bytes_body(self):
self._test_list_users(bytes_body=True)
def test_update_user_with_string_body(self):
self._test_update_user()
def test_update_user_with_bytes_body(self):
self._test_update_user(bytes_body=True)
def test_list_user_groups_with_string_body(self):
self._test_list_user_groups()
def test_list_user_groups_with_bytes_body(self):
self._test_list_user_groups(bytes_body=True)
def test_delete_user(self):
self.check_service_client_function(
self.client.delete_user,
'tempest.lib.common.rest_client.RestClient.delete',
{},
user_id='817fb3c23fd7465ba6d7fe1b1320121d',
status=204,
)
def test_change_user_password(self):
self.check_service_client_function(
self.client.update_user_password,
'tempest.lib.common.rest_client.RestClient.post',
{},
status=204,
user_id='817fb3c23fd7465ba6d7fe1b1320121d',
password='NewTempestPassword',
original_password='OldTempestPassword')