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

openstackclient/tests/unit/network/v2/test_subnet.py

@@ -790,7 +790,7 @@ class TestDeleteSubnet(TestSubnet):
 
 class TestListSubnet(TestSubnet):
     # The subnets going to be listed up.
-    _subnet = network_fakes.FakeSubnet.create_subnets(count=3)
+    _subnets = network_fakes.FakeSubnet.create_subnets(count=3)
 
     columns = (
         'ID',
@@ -798,7 +798,8 @@ class TestListSubnet(TestSubnet):
         'Network',
         'Subnet',
     )
-    columns_long = columns + (
+    columns_long = (
+        *columns,
         'Project',
         'DHCP',
         'Name Servers',
@@ -811,7 +812,7 @@ class TestListSubnet(TestSubnet):
     )
 
     data = []
-    for subnet in _subnet:
+    for subnet in _subnets:
         data.append(
             (
                 subnet.id,
@@ -822,7 +823,7 @@ class TestListSubnet(TestSubnet):
         )
 
     data_long = []
-    for subnet in _subnet:
+    for subnet in _subnets:
         data_long.append(
             (
                 subnet.id,
@@ -847,7 +848,7 @@ class TestListSubnet(TestSubnet):
         # Get the command object to test
         self.cmd = subnet_v2.ListSubnet(self.app, None)
 
-        self.network_client.subnets.return_value = self._subnet
+        self.network_client.subnets.return_value = self._subnets
 
     def test_subnet_list_no_options(self):
         arglist = []
@@ -877,6 +878,30 @@ class TestListSubnet(TestSubnet):
         self.assertEqual(self.columns_long, columns)
         self.assertCountEqual(self.data_long, list(data))
 
+    def test_subnet_list_pagination(self):
+        arglist = [
+            '--marker',
+            self._subnets[0].id,
+            '--limit',
+            '1',
+        ]
+        verifylist = [
+            ('marker', self._subnets[0].id),
+            ('limit', 1),
+        ]
+        parsed_args = self.check_parser(self.cmd, arglist, verifylist)
+
+        columns, data = self.cmd.take_action(parsed_args)
+
+        self.network_client.subnets.assert_called_once_with(
+            **{
+                'marker': self._subnets[0].id,
+                'limit': 1,
+            }
+        )
+        self.assertEqual(self.columns, columns)
+        self.assertEqual(self.data, list(data))
+
     def test_subnet_list_ip_version(self):
         arglist = [
             '--ip-version',

openstackclient/tests/unit/network/v2/test_subnet_pool.py

@@ -471,7 +471,8 @@ class TestListSubnetPool(TestSubnetPool):
         'Name',
         'Prefixes',
     )
-    columns_long = columns + (
+    columns_long = (
+        *columns,
         'Default Prefix Length',
         'Address Scope',
         'Default Subnet Pool',
@@ -540,6 +541,30 @@ class TestListSubnetPool(TestSubnetPool):
         self.assertEqual(self.columns_long, columns)
         self.assertCountEqual(self.data_long, list(data))
 
+    def test_subnet_pool_list_pagination(self):
+        arglist = [
+            '--marker',
+            self._subnet_pools[0].id,
+            '--limit',
+            '1',
+        ]
+        verifylist = [
+            ('marker', self._subnet_pools[0].id),
+            ('limit', 1),
+        ]
+        parsed_args = self.check_parser(self.cmd, arglist, verifylist)
+
+        columns, data = self.cmd.take_action(parsed_args)
+
+        self.network_client.subnet_pools.assert_called_once_with(
+            **{
+                'marker': self._subnet_pools[0].id,
+                'limit': 1,
+            }
+        )
+        self.assertEqual(self.columns, columns)
+        self.assertEqual(self.data, list(data))
+
     def test_subnet_pool_list_no_share(self):
         arglist = [
             '--no-share',

openstackclient/tests/unit/object/v1/fakes.py

@@ -11,9 +11,8 @@
 #   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 keystoneauth1 import session
+from unittest import mock
 
 from openstackclient.api import object_store_v1 as object_store
 from openstackclient.tests.unit import utils
@@ -80,12 +79,14 @@ OBJECT_2 = {
 object_upload_name = 'test-object-name'
 
 
-class TestObjectv1(utils.TestCommand):
+class FakeClientMixin:
     def setUp(self):
         super().setUp()
 
-        self.app.client_manager.session = session.Session()
-        self.app.client_manager.object_store = object_store.APIv1(
-            session=self.app.client_manager.session,
-            endpoint=ENDPOINT,
+        self.app.client_manager.object_store = mock.Mock(
+            spec=object_store.APIv1
         )
+        self.object_store_client = self.app.client_manager.object_store
+
+
+class TestObjectV1(FakeClientMixin, utils.TestCommand): ...

openstackclient/tests/unit/object/v1/test_container.py

@@ -14,48 +14,21 @@
 #
 
 import copy
-from unittest import mock
 
-from openstackclient.api import object_store_v1 as object_store
 from openstackclient.object.v1 import container
 from openstackclient.tests.unit.object.v1 import fakes as object_fakes
 
 
-AUTH_TOKEN = "foobar"
-AUTH_URL = "http://0.0.0.0"
-
-
-class FakeClient:
-    def __init__(self, endpoint=None, **kwargs):
-        self.endpoint = AUTH_URL
-        self.token = AUTH_TOKEN
-
-
-class TestContainer(object_fakes.TestObjectv1):
-    columns = ('Name',)
-
+class TestContainerDelete(object_fakes.TestObjectV1):
     def setUp(self):
         super().setUp()
-        self.app.client_manager.object_store = object_store.APIv1(
-            session=mock.Mock(),
-            service_type="object-store",
-        )
-        self.api = self.app.client_manager.object_store
-
 
-@mock.patch('openstackclient.api.object_store_v1.APIv1.object_delete')
-@mock.patch('openstackclient.api.object_store_v1.APIv1.object_list')
-@mock.patch('openstackclient.api.object_store_v1.APIv1.container_delete')
-class TestContainerDelete(TestContainer):
-    def setUp(self):
-        super().setUp()
+        self.object_store_client.container_delete.return_value = None
 
         # Get the command object to test
         self.cmd = container.DeleteContainer(self.app, None)
 
-    def test_container_delete(self, c_mock, o_list_mock, o_delete_mock):
-        c_mock.return_value = None
-
+    def test_container_delete(self):
         arglist = [
             object_fakes.container_name,
         ]
@@ -68,16 +41,17 @@ class TestContainerDelete(TestContainer):
         self.assertIsNone(self.cmd.take_action(parsed_args))
 
         kwargs = {}
-        c_mock.assert_called_with(
+        self.object_store_client.container_delete.assert_called_with(
             container=object_fakes.container_name, **kwargs
         )
-        self.assertFalse(o_list_mock.called)
-        self.assertFalse(o_delete_mock.called)
+        self.object_store_client.object_list.assert_not_called()
+        self.object_store_client.object_delete.assert_not_called()
 
-    def test_recursive_delete(self, c_mock, o_list_mock, o_delete_mock):
-        c_mock.return_value = None
-        o_list_mock.return_value = [object_fakes.OBJECT]
-        o_delete_mock.return_value = None
+    def test_recursive_delete(self):
+        self.object_store_client.object_delete.return_value = None
+        self.object_store_client.object_list.return_value = [
+            object_fakes.OBJECT
+        ]
 
         arglist = [
             '--recursive',
@@ -91,20 +65,22 @@ class TestContainerDelete(TestContainer):
 
         self.assertIsNone(self.cmd.take_action(parsed_args))
 
-        kwargs = {}
-        c_mock.assert_called_with(
-            container=object_fakes.container_name, **kwargs
+        self.object_store_client.container_delete.assert_called_with(
+            container=object_fakes.container_name
         )
-        o_list_mock.assert_called_with(container=object_fakes.container_name)
-        o_delete_mock.assert_called_with(
+        self.object_store_client.object_list.assert_called_with(
+            container=object_fakes.container_name
+        )
+        self.object_store_client.object_delete.assert_called_with(
             container=object_fakes.container_name,
             object=object_fakes.OBJECT['name'],
         )
 
-    def test_r_delete(self, c_mock, o_list_mock, o_delete_mock):
-        c_mock.return_value = None
-        o_list_mock.return_value = [object_fakes.OBJECT]
-        o_delete_mock.return_value = None
+    def test_r_delete(self):
+        self.object_store_client.object_delete.return_value = None
+        self.object_store_client.object_list.return_value = [
+            object_fakes.OBJECT
+        ]
 
         arglist = [
             '-r',
@@ -118,27 +94,29 @@ class TestContainerDelete(TestContainer):
 
         self.assertIsNone(self.cmd.take_action(parsed_args))
 
-        kwargs = {}
-        c_mock.assert_called_with(
-            container=object_fakes.container_name, **kwargs
+        self.object_store_client.container_delete.assert_called_with(
+            container=object_fakes.container_name
         )
-        o_list_mock.assert_called_with(container=object_fakes.container_name)
-        o_delete_mock.assert_called_with(
+        self.object_store_client.object_list.assert_called_with(
+            container=object_fakes.container_name
+        )
+        self.object_store_client.object_delete.assert_called_with(
             container=object_fakes.container_name,
             object=object_fakes.OBJECT['name'],
         )
 
 
-@mock.patch('openstackclient.api.object_store_v1.APIv1.container_list')
-class TestContainerList(TestContainer):
+class TestContainerList(object_fakes.TestObjectV1):
+    columns = ('Name',)
+
     def setUp(self):
         super().setUp()
 
         # Get the command object to test
         self.cmd = container.ListContainer(self.app, None)
 
-    def test_object_list_containers_no_options(self, c_mock):
-        c_mock.return_value = [
+    def test_object_list_containers_no_options(self):
+        self.object_store_client.container_list.return_value = [
             copy.deepcopy(object_fakes.CONTAINER),
             copy.deepcopy(object_fakes.CONTAINER_3),
             copy.deepcopy(object_fakes.CONTAINER_2),
@@ -153,9 +131,7 @@ class TestContainerList(TestContainer):
         # containing the data to be listed.
         columns, data = self.cmd.take_action(parsed_args)
 
-        # Set expected values
-        kwargs = {}
-        c_mock.assert_called_with(**kwargs)
+        self.object_store_client.container_list.assert_called_with()
 
         self.assertEqual(self.columns, columns)
         datalist = (
@@ -165,8 +141,8 @@ class TestContainerList(TestContainer):
         )
         self.assertEqual(datalist, tuple(data))
 
-    def test_object_list_containers_prefix(self, c_mock):
-        c_mock.return_value = [
+    def test_object_list_containers_prefix(self):
+        self.object_store_client.container_list.return_value = [
             copy.deepcopy(object_fakes.CONTAINER),
             copy.deepcopy(object_fakes.CONTAINER_3),
         ]
@@ -185,11 +161,9 @@ class TestContainerList(TestContainer):
         # containing the data to be listed.
         columns, data = self.cmd.take_action(parsed_args)
 
-        # Set expected values
-        kwargs = {
-            'prefix': 'bit',
-        }
-        c_mock.assert_called_with(**kwargs)
+        self.object_store_client.container_list.assert_called_with(
+            prefix='bit',
+        )
 
         self.assertEqual(self.columns, columns)
         datalist = (
@@ -198,8 +172,8 @@ class TestContainerList(TestContainer):
         )
         self.assertEqual(datalist, tuple(data))
 
-    def test_object_list_containers_marker(self, c_mock):
-        c_mock.return_value = [
+    def test_object_list_containers_marker(self):
+        self.object_store_client.container_list.return_value = [
             copy.deepcopy(object_fakes.CONTAINER),
             copy.deepcopy(object_fakes.CONTAINER_3),
         ]
@@ -221,12 +195,10 @@ class TestContainerList(TestContainer):
         # containing the data to be listed.
         columns, data = self.cmd.take_action(parsed_args)
 
-        # Set expected values
-        kwargs = {
-            'marker': object_fakes.container_name,
-            'end_marker': object_fakes.container_name_3,
-        }
-        c_mock.assert_called_with(**kwargs)
+        self.object_store_client.container_list.assert_called_with(
+            marker=object_fakes.container_name,
+            end_marker=object_fakes.container_name_3,
+        )
 
         self.assertEqual(self.columns, columns)
         datalist = (
@@ -235,8 +207,8 @@ class TestContainerList(TestContainer):
         )
         self.assertEqual(datalist, tuple(data))
 
-    def test_object_list_containers_limit(self, c_mock):
-        c_mock.return_value = [
+    def test_object_list_containers_limit(self):
+        self.object_store_client.container_list.return_value = [
             copy.deepcopy(object_fakes.CONTAINER),
             copy.deepcopy(object_fakes.CONTAINER_3),
         ]
@@ -255,11 +227,9 @@ class TestContainerList(TestContainer):
         # containing the data to be listed.
         columns, data = self.cmd.take_action(parsed_args)
 
-        # Set expected values
-        kwargs = {
-            'limit': 2,
-        }
-        c_mock.assert_called_with(**kwargs)
+        self.object_store_client.container_list.assert_called_with(
+            limit=2,
+        )
 
         self.assertEqual(self.columns, columns)
         datalist = (
@@ -268,8 +238,8 @@ class TestContainerList(TestContainer):
         )
         self.assertEqual(datalist, tuple(data))
 
-    def test_object_list_containers_long(self, c_mock):
-        c_mock.return_value = [
+    def test_object_list_containers_long(self):
+        self.object_store_client.container_list.return_value = [
             copy.deepcopy(object_fakes.CONTAINER),
             copy.deepcopy(object_fakes.CONTAINER_3),
         ]
@@ -287,9 +257,7 @@ class TestContainerList(TestContainer):
         # containing the data to be listed.
         columns, data = self.cmd.take_action(parsed_args)
 
-        # Set expected values
-        kwargs = {}
-        c_mock.assert_called_with(**kwargs)
+        self.object_store_client.container_list.assert_called_with()
 
         collist = ('Name', 'Bytes', 'Count')
         self.assertEqual(collist, columns)
@@ -307,8 +275,8 @@ class TestContainerList(TestContainer):
         )
         self.assertEqual(datalist, tuple(data))
 
-    def test_object_list_containers_all(self, c_mock):
-        c_mock.return_value = [
+    def test_object_list_containers_all(self):
+        self.object_store_client.container_list.return_value = [
             copy.deepcopy(object_fakes.CONTAINER),
             copy.deepcopy(object_fakes.CONTAINER_2),
             copy.deepcopy(object_fakes.CONTAINER_3),
@@ -327,11 +295,9 @@ class TestContainerList(TestContainer):
         # containing the data to be listed.
         columns, data = self.cmd.take_action(parsed_args)
 
-        # Set expected values
-        kwargs = {
-            'full_listing': True,
-        }
-        c_mock.assert_called_with(**kwargs)
+        self.object_store_client.container_list.assert_called_with(
+            full_listing=True,
+        )
 
         self.assertEqual(self.columns, columns)
         datalist = (
@@ -342,16 +308,17 @@ class TestContainerList(TestContainer):
         self.assertEqual(datalist, tuple(data))
 
 
-@mock.patch('openstackclient.api.object_store_v1.APIv1.container_show')
-class TestContainerShow(TestContainer):
+class TestContainerShow(object_fakes.TestObjectV1):
     def setUp(self):
         super().setUp()
 
         # Get the command object to test
         self.cmd = container.ShowContainer(self.app, None)
 
-    def test_container_show(self, c_mock):
-        c_mock.return_value = copy.deepcopy(object_fakes.CONTAINER)
+    def test_container_show(self):
+        self.object_store_client.container_show.return_value = copy.deepcopy(
+            object_fakes.CONTAINER
+        )
 
         arglist = [
             object_fakes.container_name,
@@ -366,11 +333,8 @@ class TestContainerShow(TestContainer):
         # data to be shown.
         columns, data = self.cmd.take_action(parsed_args)
 
-        # Set expected values
-        kwargs = {}
-        # lib.container.show_container(api, url, container)
-        c_mock.assert_called_with(
-            container=object_fakes.container_name, **kwargs
+        self.object_store_client.container_show.assert_called_with(
+            container=object_fakes.container_name,
         )
 
         collist = ('bytes', 'count', 'name')

openstackclient/tests/unit/object/v1/test_container_all.py

@@ -13,17 +13,24 @@
 
 import copy
 
+from keystoneauth1 import session
 from requests_mock.contrib import fixture
 
+from openstackclient.api import object_store_v1 as object_store
 from openstackclient.object.v1 import container as container_cmds
 from openstackclient.tests.unit.object.v1 import fakes as object_fakes
 
 
-class TestContainerAll(object_fakes.TestObjectv1):
+class TestContainerAll(object_fakes.TestObjectV1):
     def setUp(self):
         super().setUp()
 
+        # these tests require a "real" client since we mock requests
         self.requests_mock = self.useFixture(fixture.Fixture())
+        self.app.client_manager.object_store = object_store.APIv1(
+            session=session.Session(),
+            endpoint=object_fakes.ENDPOINT,
+        )
 
 
 class TestContainerCreate(TestContainerAll):