From 77bed6f5ab386adee3bb0eeeea2c19565971905f Mon Sep 17 00:00:00 2001 From: Mohammed Naser Date: Thu, 6 Apr 2017 16:38:59 -0400 Subject: [PATCH] Remove un-used Keystone test path This patch removes very old and unused Keystone test path, we do this indirectly by our simple sanity tests (and hopefully) by using Tempest. Change-Id: I55916d08942df97990515c76914b799d1f327565 --- tests/clients.py | 69 ------------------------------------------ tests/test_keystone.py | 27 ----------------- 2 files changed, 96 deletions(-) delete mode 100644 tests/clients.py delete mode 100644 tests/test_keystone.py diff --git a/tests/clients.py b/tests/clients.py deleted file mode 100644 index a845579b1b..0000000000 --- a/tests/clients.py +++ /dev/null @@ -1,69 +0,0 @@ -# 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 keystoneclient.v2_0 import client as ksclient -import logging - -logging.basicConfig(level=logging.WARNING) -LOG = logging.getLogger(__name__) - - -class OpenStackClients(object): - - def __init__(self): - self._connected_clients = {} - self._supported_clients = self.__class__.__subclasses__() - self.client = None - - def get_client(self, name): - if name in self._connected_clients: - return self._connected_clients[name] - try: - aclass = next(s for s in self._supported_clients if name in - s.__name__) - sclient = aclass() - connected_client = sclient.create() - self._connected_clients[name] = connected_client - return connected_client - - except StopIteration: - LOG.warning("Requested client %s not found", name) - raise - - def create(self): - pass - - -class KeystoneClient(OpenStackClients): - - def __init__(self): - super(KeystoneClient, self).__init__() - # TODO(Jeff Peeler): this shouldn't be hard coded - self.creds = {'auth_url': 'http://10.0.0.4:5000/v2.0', - 'username': 'admin', - 'password': 'steakfordinner', - 'tenant_name': 'admin'} - - def create(self): - if self.client is None: - self.client = ksclient.Client(**self.creds) - return self.client - - -if __name__ == '__main__': - # TODO(Jeff Peeler): mox this - client_mgr = OpenStackClients() - ks = client_mgr.get_client('KeystoneClient') - LOG.info(ks) - ks2 = client_mgr.get_client('KeystoneClient') - LOG.info(ks2) diff --git a/tests/test_keystone.py b/tests/test_keystone.py deleted file mode 100644 index dbbba88ad5..0000000000 --- a/tests/test_keystone.py +++ /dev/null @@ -1,27 +0,0 @@ -# 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 tests import clients -import testtools - - -# TODO(jeffrey4l): remove this skip when this test can passed. -@testtools.skip -class KeystoneTest(testtools.TestCase): - def setUp(self): - super(KeystoneTest, self).setUp() - self.kc = clients.OpenStackClients().get_client('KeystoneClient') - - def test_tenants(self): - result = self.kc.tenants.list() - # only admin tenant - self.assertEqual(1, len(result))