python-openstackclient 9.0.0__py3-none-any.whl → 10.0.0__py3-none-any.whl

openstackclient/tests/functional/base.py

@@ -10,11 +10,13 @@
 #    License for the specific language governing permissions and limitations
 #    under the License.
 
+from collections.abc import Iterable, Mapping, Sequence
 import json
 import logging
 import os
 import shlex
 import subprocess
+from typing import Any, Literal, cast, overload
 
 from tempest.lib.cli import output_parser
 from tempest.lib import exceptions
@@ -24,7 +26,7 @@ ADMIN_CLOUD = os.environ.get('OS_ADMIN_CLOUD', 'devstack-admin')
 LOG = logging.getLogger(__name__)
 
 
-def execute(cmd, *, fail_ok=False):
+def execute(cmd: str, *, fail_ok: bool = False) -> str:
     """Executes specified command for the given action."""
     LOG.debug('Executing: %s', cmd)
     cmdlist = shlex.split(cmd)
@@ -53,15 +55,37 @@ def execute(cmd, *, fail_ok=False):
 
 
 class TestCase(testtools.TestCase):
+    @overload
     @classmethod
     def openstack(
         cls,
-        cmd,
+        cmd: str,
         *,
-        cloud=ADMIN_CLOUD,
-        fail_ok=False,
-        parse_output=False,
-    ):
+        cloud: str | None = ADMIN_CLOUD,
+        fail_ok: bool = False,
+        parse_output: Literal[False] = False,
+    ) -> str: ...
+
+    @overload
+    @classmethod
+    def openstack(
+        cls,
+        cmd: str,
+        *,
+        cloud: str | None = ADMIN_CLOUD,
+        fail_ok: bool = False,
+        parse_output: Literal[True] = ...,
+    ) -> Any: ...
+
+    @classmethod
+    def openstack(
+        cls,
+        cmd: str,
+        *,
+        cloud: str | None = ADMIN_CLOUD,
+        fail_ok: bool = False,
+        parse_output: bool = False,
+    ) -> str | Any:
         """Executes openstackclient command for the given action
 
         :param cmd: A string representation of the command to execute.
@@ -92,7 +116,7 @@ class TestCase(testtools.TestCase):
             format_args.append('-f json')
 
         output = execute(
-            ' '.join(['openstack'] + auth_args + [cmd] + format_args),
+            ' '.join(['openstack', *auth_args, cmd, *format_args]),
             fail_ok=fail_ok,
         )
 
@@ -106,7 +130,9 @@ class TestCase(testtools.TestCase):
             return output
 
     @classmethod
-    def is_service_enabled(cls, service, version=None):
+    def is_service_enabled(
+        cls, service: str, version: str | None = None
+    ) -> bool:
         """Ask client cloud if service is available
 
         :param service: The service name or type. This should be either an
@@ -126,7 +152,9 @@ class TestCase(testtools.TestCase):
         return bool(ret)
 
     @classmethod
-    def is_extension_enabled(cls, alias, *, service='network'):
+    def is_extension_enabled(
+        cls, alias: str, *, service: str = 'network'
+    ) -> bool:
         """Ask client cloud if extension is enabled"""
         extensions = cls.openstack(
             f'extension list --{service}',
@@ -135,38 +163,44 @@ class TestCase(testtools.TestCase):
         return alias in [x['Alias'] for x in extensions]
 
     @classmethod
-    def get_openstack_configuration_value(cls, configuration):
+    def get_openstack_configuration_value(cls, configuration: str) -> str:
         opts = cls.get_opts([configuration])
         return cls.openstack('configuration show ' + opts)
 
     @classmethod
-    def get_opts(cls, fields, output_format='value'):
+    def get_opts(cls, fields: list[str], output_format: str = 'value') -> str:
         return ' -f {} {}'.format(
             output_format, ' '.join(['-c ' + it for it in fields])
         )
 
     @classmethod
-    def assertOutput(cls, expected, actual):
+    def assertOutput(cls, expected: str, actual: str) -> None:
         if expected != actual:
             raise Exception(expected + ' != ' + actual)
 
     @classmethod
-    def assertInOutput(cls, expected, actual):
+    def assertInOutput(cls, expected: str, actual: str) -> None:
         if expected not in actual:
             raise Exception(expected + ' not in ' + actual)
 
     @classmethod
-    def assertsOutputNotNone(cls, observed):
+    def assertsOutputNotNone(cls, observed: Any) -> None:
         if observed is None:
             raise Exception('No output observed')
 
-    def assert_table_structure(self, items, field_names):
+    def assert_table_structure(
+        self, items: Iterable[Mapping[str, Any]], field_names: Sequence[str]
+    ) -> None:
         """Verify that all items have keys listed in field_names."""
         for item in items:
             for field in field_names:
                 self.assertIn(field, item)
 
-    def assert_show_fields(self, show_output, field_names):
+    def assert_show_fields(
+        self,
+        show_output: Iterable[Mapping[str, Any]],
+        field_names: Sequence[str],
+    ) -> None:
         """Verify that all items have keys listed in field_names."""
 
         # field_names = ['name', 'description']
@@ -186,17 +220,18 @@ class TestCase(testtools.TestCase):
             o.update(item)
         return o
 
-    def parse_show(self, raw_output):
+    def parse_show(self, raw_output: str) -> list[dict[str, Any]]:
         """Return list of dicts with item values parsed from cli output."""
 
         items = []
         table_ = output_parser.table(raw_output)
         for row in table_['values']:
             item = {}
-            item[row[0]] = row[1]
+            item[str(row[0])] = row[1]
             items.append(item)
         return items
 
-    def parse_listing(self, raw_output):
+    def parse_listing(self, raw_output: str) -> list[dict[str, Any]]:
         """Return list of dicts with basic item parsed from cli output."""
-        return output_parser.listing(raw_output)
+        # need to add hints to tempest
+        return cast(list[dict[str, Any]], output_parser.listing(raw_output))

openstackclient/tests/functional/common/test_extension.py

@@ -13,6 +13,8 @@
 #    License for the specific language governing permissions and limitations
 #    under the License.
 
+from typing import ClassVar
+
 from tempest.lib import exceptions as tempest_exc
 
 from openstackclient.tests.functional import base
@@ -21,6 +23,8 @@ from openstackclient.tests.functional import base
 class ExtensionTests(base.TestCase):
     """Functional tests for extension"""
 
+    haz_network: ClassVar[bool]
+
     @classmethod
     def setUpClass(cls):
         super().setUpClass()

openstackclient/tests/functional/common/test_quota.py

@@ -10,6 +10,7 @@
 #    License for the specific language governing permissions and limitations
 #    under the License.
 
+from typing import ClassVar
 import uuid
 
 from tempest.lib.common.utils import data_utils
@@ -25,7 +26,8 @@ class QuotaTests(base.TestCase):
     test runs as these may run in parallel and otherwise step on each other.
     """
 
-    PROJECT_NAME: str
+    haz_network: ClassVar[bool]
+    PROJECT_NAME: ClassVar[str]
 
     @classmethod
     def setUpClass(cls):

openstackclient/tests/functional/compute/v2/common.py

@@ -12,6 +12,7 @@
 #
 
 import time
+from typing import cast
 import uuid
 
 from tempest.lib import exceptions
@@ -35,18 +36,18 @@ class ComputeTestCase(base.TestCase):
 
     @classmethod
     def get_flavor(cls) -> str:
+        valid_flavors = ['m1.tiny', 'cirros256']
         # NOTE(rtheis): Get cirros256 or m1.tiny flavors since functional
         #               tests may create other flavors.
         flavors = cls.openstack("flavor list", parse_output=True)
-        server_flavor = None
         for flavor in flavors:
             if flavor['Name'] in ['m1.tiny', 'cirros256']:
-                server_flavor = flavor['Name']
-                break
-
-        assert server_flavor is not None
+                return cast(str, flavor['Name'])
 
-        return server_flavor
+        raise Exception(
+            f'Failed to find a suitable flavor. Required one of: '
+            f'{", ".join(valid_flavors)}'
+        )
 
     @classmethod
     def get_image(cls) -> str:
@@ -54,18 +55,17 @@ class ComputeTestCase(base.TestCase):
         #               create other images.  Image may be named '-uec' or
         #               '-disk'.
         images = cls.openstack("image list", parse_output=True)
-        server_image = None
         for image in images:
             if image['Name'].startswith('cirros-') and (
                 image['Name'].endswith('-uec')
                 or image['Name'].endswith('-disk')
             ):
-                server_image = image['Name']
-                break
+                return cast(str, image['Name'])
 
-        assert server_image is not None
-
-        return server_image
+        raise Exception(
+            'Failed to find a suitable image. Required one matching one of '
+            'the following patterns: cirros-*-uec, cirros-*-disk'
+        )
 
     @classmethod
     def get_network(cls) -> str:
@@ -78,7 +78,8 @@ class ComputeTestCase(base.TestCase):
             )
         except exceptions.CommandFailed:
             return ''
-        return '--nic net-id=' + cmd_output['id']
+
+        return '='.join(('--nic net-id', cmd_output['id']))
 
     def server_create(self, name=None, cleanup=True):
         """Create server, with cleanup"""

openstackclient/tests/functional/compute/v2/test_flavor.py

@@ -10,6 +10,7 @@
 #    License for the specific language governing permissions and limitations
 #    under the License.
 
+from typing import ClassVar
 import uuid
 
 from openstackclient.tests.functional import base
@@ -19,6 +20,7 @@ class FlavorTests(base.TestCase):
     """Functional tests for flavor."""
 
     PROJECT_NAME = uuid.uuid4().hex
+    PROJECT_ID: ClassVar[str]
 
     @classmethod
     def setUpClass(cls):
@@ -28,7 +30,7 @@ class FlavorTests(base.TestCase):
             "project create --enable " + cls.PROJECT_NAME,
             parse_output=True,
         )
-        cls.project_id = cmd_output["id"]
+        cls.PROJECT_ID = cmd_output["id"]
 
     @classmethod
     def tearDownClass(cls):

openstackclient/tests/functional/compute/v2/test_server.py

@@ -13,6 +13,7 @@
 import itertools
 import json
 import time
+from typing import ClassVar
 import uuid
 
 from tempest.lib import exceptions
@@ -25,6 +26,8 @@ from openstackclient.tests.functional.volume.v2 import common as volume_common
 class ServerTests(common.ComputeTestCase):
     """Functional tests for openstack server commands"""
 
+    haz_network: ClassVar[bool]
+
     @classmethod
     def setUpClass(cls):
         super().setUpClass()

openstackclient/tests/functional/identity/v2/common.py

@@ -11,6 +11,7 @@
 #    under the License.
 
 import os
+from typing import ClassVar
 import unittest
 
 import fixtures
@@ -57,19 +58,22 @@ class IdentityTests(base.TestCase):
     CATALOG_LIST_HEADERS = ['Name', 'Type', 'Endpoints']
     ENDPOINT_LIST_HEADERS = ['ID', 'Region', 'Service Name', 'Service Type']
 
+    PROJECT_NAME: ClassVar[str]
+    PROJECT_DESCRIPTION: ClassVar[str]
+
     @classmethod
     def setUpClass(cls):
         super().setUpClass()
         # create dummy project
-        cls.project_name = data_utils.rand_name('TestProject')
-        cls.project_description = data_utils.rand_name('description')
+        cls.PROJECT_NAME = data_utils.rand_name('TestProject')
+        cls.PROJECT_DESCRIPTION = data_utils.rand_name('description')
         try:
             cls.openstack(
                 '--os-identity-api-version 2 '
                 'project create '
-                f'--description {cls.project_description} '
+                f'--description {cls.PROJECT_DESCRIPTION} '
                 '--enable '
-                f'{cls.project_name}'
+                f'{cls.PROJECT_NAME}'
             )
         except tempest_exceptions.CommandFailed:
             # Good chance this is due to Identity v2 admin not being enabled
@@ -83,7 +87,7 @@ class IdentityTests(base.TestCase):
         try:
             cls.openstack(
                 '--os-identity-api-version 2 '
-                f'project delete {cls.project_name}'
+                f'project delete {cls.PROJECT_NAME}'
             )
         finally:
             super().tearDownClass()
@@ -125,7 +129,7 @@ class IdentityTests(base.TestCase):
         email = data_utils.rand_name() + '@example.com'
         raw_output = self.openstack(
             'user create '
-            f'--project {self.project_name} '
+            f'--project {self.PROJECT_NAME} '
             f'--password {password} '
             f'--email {email} '
             '--enable '

openstackclient/tests/functional/identity/v2/test_role.py

@@ -39,14 +39,14 @@ class RoleTests(common.IdentityTests):
         username = self._create_dummy_user()
         raw_output = self.openstack(
             'role add '
-            f'--project {self.project_name} '
+            f'--project {self.PROJECT_NAME} '
             f'--user {username} '
             f'{role_name}'
         )
         self.addCleanup(
             self.openstack,
             'role remove '
-            f'--project {self.project_name} '
+            f'--project {self.PROJECT_NAME} '
             f'--user {username} '
             f'{role_name}',
         )
@@ -58,13 +58,13 @@ class RoleTests(common.IdentityTests):
         username = self._create_dummy_user()
         add_raw_output = self.openstack(
             'role add '
-            f'--project {self.project_name} '
+            f'--project {self.PROJECT_NAME} '
             f'--user {username} '
             f'{role_name}'
         )
         del_raw_output = self.openstack(
             'role remove '
-            f'--project {self.project_name} '
+            f'--project {self.PROJECT_NAME} '
             f'--user {username} '
             f'{role_name}'
         )

openstackclient/tests/functional/identity/v3/common.py

@@ -11,6 +11,7 @@
 #    under the License.
 
 import os
+from typing import ClassVar
 
 import fixtures
 from tempest.lib.common.utils import data_utils
@@ -147,30 +148,35 @@ class IdentityTests(base.TestCase):
         'Region ID',
     ]
 
+    DOMAIN_NAME: ClassVar[str]
+    DOMAIN_DESCRIPTION: ClassVar[str]
+    PROJECT_NAME: ClassVar[str]
+    PROJECT_DESCRIPTION: ClassVar[str]
+
     @classmethod
     def setUpClass(cls):
         super().setUpClass()
         # create dummy domain
-        cls.domain_name = data_utils.rand_name('TestDomain')
-        cls.domain_description = data_utils.rand_name('description')
+        cls.DOMAIN_NAME = data_utils.rand_name('TestDomain')
+        cls.DOMAIN_DESCRIPTION = data_utils.rand_name('description')
         cls.openstack(
             '--os-identity-api-version 3 '
             'domain create '
-            f'--description {cls.domain_description} '
+            f'--description {cls.DOMAIN_DESCRIPTION} '
             '--enable '
-            f'{cls.domain_name}'
+            f'{cls.DOMAIN_NAME}'
         )
 
         # create dummy project
-        cls.project_name = data_utils.rand_name('TestProject')
-        cls.project_description = data_utils.rand_name('description')
+        cls.PROJECT_NAME = data_utils.rand_name('TestProject')
+        cls.PROJECT_DESCRIPTION = data_utils.rand_name('description')
         cls.openstack(
             '--os-identity-api-version 3 '
             'project create '
-            f'--domain {cls.domain_name} '
-            f'--description {cls.project_description} '
+            f'--domain {cls.DOMAIN_NAME} '
+            f'--description {cls.PROJECT_DESCRIPTION} '
             '--enable '
-            f'{cls.project_name}'
+            f'{cls.PROJECT_NAME}'
         )
 
     @classmethod
@@ -179,15 +185,15 @@ class IdentityTests(base.TestCase):
             # delete dummy project
             cls.openstack(
                 '--os-identity-api-version 3 '
-                f'project delete {cls.project_name}'
+                f'project delete {cls.PROJECT_NAME}'
             )
             # disable and delete dummy domain
             cls.openstack(
                 '--os-identity-api-version 3 '
-                f'domain set --disable {cls.domain_name}'
+                f'domain set --disable {cls.DOMAIN_NAME}'
             )
             cls.openstack(
-                f'--os-identity-api-version 3 domain delete {cls.domain_name}'
+                f'--os-identity-api-version 3 domain delete {cls.DOMAIN_NAME}'
             )
         finally:
             super().tearDownClass()
@@ -213,9 +219,9 @@ class IdentityTests(base.TestCase):
         description = data_utils.rand_name('description')
         raw_output = self.openstack(
             'user create '
-            f'--domain {self.domain_name} '
-            f'--project {self.project_name} '
-            f'--project-domain {self.domain_name} '
+            f'--domain {self.DOMAIN_NAME} '
+            f'--project {self.PROJECT_NAME} '
+            f'--project-domain {self.DOMAIN_NAME} '
             f'--password {password} '
             f'--email {email} '
             f'--description {description} '
@@ -262,14 +268,14 @@ class IdentityTests(base.TestCase):
         description = data_utils.rand_name('description')
         raw_output = self.openstack(
             'group create '
-            f'--domain {self.domain_name} '
+            f'--domain {self.DOMAIN_NAME} '
             f'--description {description} '
             f'{group_name}'
         )
         if add_clean_up:
             self.addCleanup(
                 self.openstack,
-                f'group delete --domain {self.domain_name} {group_name}',
+                f'group delete --domain {self.DOMAIN_NAME} {group_name}',
             )
         items = self.parse_show(raw_output)
         self.assert_show_fields(items, self.GROUP_FIELDS)
@@ -295,14 +301,14 @@ class IdentityTests(base.TestCase):
         project_description = data_utils.rand_name('description')
         self.openstack(
             'project create '
-            f'--domain {self.domain_name} '
+            f'--domain {self.DOMAIN_NAME} '
             f'--description {project_description} '
             f'--enable {project_name}'
         )
         if add_clean_up:
             self.addCleanup(
                 self.openstack,
-                f'project delete --domain {self.domain_name} {project_name}',
+                f'project delete --domain {self.DOMAIN_NAME} {project_name}',
             )
         return project_name
 

openstackclient/tests/functional/identity/v3/test_group.py

@@ -28,7 +28,7 @@ class GroupTests(common.IdentityTests):
 
     def test_group_list_with_domain(self):
         group_name = self._create_dummy_group()
-        raw_output = self.openstack(f'group list --domain {self.domain_name}')
+        raw_output = self.openstack(f'group list --domain {self.DOMAIN_NAME}')
         items = self.parse_listing(raw_output)
         self.assert_table_structure(items, common.BASIC_LIST_HEADERS)
         self.assertIn(group_name, raw_output)
@@ -36,14 +36,14 @@ class GroupTests(common.IdentityTests):
     def test_group_delete(self):
         group_name = self._create_dummy_group(add_clean_up=False)
         raw_output = self.openstack(
-            f'group delete --domain {self.domain_name} {group_name}'
+            f'group delete --domain {self.DOMAIN_NAME} {group_name}'
         )
         self.assertEqual(0, len(raw_output))
 
     def test_group_show(self):
         group_name = self._create_dummy_group()
         raw_output = self.openstack(
-            f'group show --domain {self.domain_name} {group_name}'
+            f'group show --domain {self.DOMAIN_NAME} {group_name}'
         )
         items = self.parse_show(raw_output)
         self.assert_show_fields(items, self.GROUP_FIELDS)
@@ -53,20 +53,20 @@ class GroupTests(common.IdentityTests):
         new_group_name = data_utils.rand_name('NewTestGroup')
         raw_output = self.openstack(
             'group set '
-            f'--domain {self.domain_name} '
+            f'--domain {self.DOMAIN_NAME} '
             f'--name {new_group_name} '
             f'{group_name}'
         )
         self.assertEqual(0, len(raw_output))
         raw_output = self.openstack(
-            f'group show --domain {self.domain_name} {new_group_name}'
+            f'group show --domain {self.DOMAIN_NAME} {new_group_name}'
         )
         group = self.parse_show_as_object(raw_output)
         self.assertEqual(new_group_name, group['name'])
         # reset group name to make sure it will be cleaned up
         raw_output = self.openstack(
             'group set '
-            f'--domain {self.domain_name} '
+            f'--domain {self.DOMAIN_NAME} '
             f'--name {group_name} '
             f'{new_group_name}'
         )
@@ -77,15 +77,15 @@ class GroupTests(common.IdentityTests):
         username = self._create_dummy_user()
         raw_output = self.openstack(
             'group add user '
-            f'--group-domain {self.domain_name} '
-            f'--user-domain {self.domain_name} '
+            f'--group-domain {self.DOMAIN_NAME} '
+            f'--user-domain {self.DOMAIN_NAME} '
             f'{group_name} {username}'
         )
         self.addCleanup(
             self.openstack,
             'group remove user '
-            f'--group-domain {self.domain_name} '
-            f'--user-domain {self.domain_name} '
+            f'--group-domain {self.DOMAIN_NAME} '
+            f'--user-domain {self.DOMAIN_NAME} '
             f'{group_name} {username}',
         )
         self.assertOutput('', raw_output)
@@ -95,22 +95,22 @@ class GroupTests(common.IdentityTests):
         username = self._create_dummy_user()
         raw_output = self.openstack(
             'group add user '
-            f'--group-domain {self.domain_name} '
-            f'--user-domain {self.domain_name} '
+            f'--group-domain {self.DOMAIN_NAME} '
+            f'--user-domain {self.DOMAIN_NAME} '
             f'{group_name} {username}'
         )
         self.addCleanup(
             self.openstack,
             'group remove user '
-            f'--group-domain {self.domain_name} '
-            f'--user-domain {self.domain_name} '
+            f'--group-domain {self.DOMAIN_NAME} '
+            f'--user-domain {self.DOMAIN_NAME} '
             f'{group_name} {username}',
         )
         self.assertOutput('', raw_output)
         raw_output = self.openstack(
             'group contains user '
-            f'--group-domain {self.domain_name} '
-            f'--user-domain {self.domain_name} '
+            f'--group-domain {self.DOMAIN_NAME} '
+            f'--user-domain {self.DOMAIN_NAME} '
             f'{group_name} {username}'
         )
         self.assertEqual(
@@ -123,14 +123,14 @@ class GroupTests(common.IdentityTests):
         username = self._create_dummy_user()
         add_raw_output = self.openstack(
             'group add user '
-            f'--group-domain {self.domain_name} '
-            f'--user-domain {self.domain_name} '
+            f'--group-domain {self.DOMAIN_NAME} '
+            f'--user-domain {self.DOMAIN_NAME} '
             f'{group_name} {username}'
         )
         remove_raw_output = self.openstack(
             'group remove user '
-            f'--group-domain {self.domain_name} '
-            f'--user-domain {self.domain_name} '
+            f'--group-domain {self.DOMAIN_NAME} '
+            f'--user-domain {self.DOMAIN_NAME} '
             f'{group_name} {username}'
         )
         self.assertOutput('', add_raw_output)

openstackclient/tests/functional/identity/v3/test_idp.py

@@ -56,7 +56,9 @@ class IdentityProviderTests(common.IdentityTests):
         raw_output = self.openstack(
             f'identity provider set '
             f'{identity_provider} '
-            f'--remote-id {new_remoteid}'
+            f'--authorization-ttl 7 '
+            f'--remote-id {new_remoteid} '
+            f'--remote-id {new_remoteid + "2nd"}'
         )
         self.assertEqual(0, len(raw_output))
         raw_output = self.openstack(