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

openstackclient/tests/unit/network/v2/taas/{test_osc_tap_flow.py → test_tap_flow.py}

@@ -77,13 +77,9 @@ class TestCreateTapFlow(network_fakes.TestNetworkV2):
                 'direction': 'BOTH',
             },
         )
-        self.app.client_manager.network.create_tap_flow.return_value = (
-            fake_tap_flow
-        )
-        self.app.client_manager.network.find_port.return_value = fake_port
-        self.app.client_manager.network.find_tap_service.return_value = (
-            fake_tap_service
-        )
+        self.network_client.create_tap_flow.return_value = fake_tap_flow
+        self.network_client.find_port.return_value = fake_port
+        self.network_client.find_tap_service.return_value = fake_tap_service
         arg_list = [
             '--name',
             fake_tap_flow['name'],
@@ -103,8 +99,7 @@ class TestCreateTapFlow(network_fakes.TestNetworkV2):
 
         parsed_args = self.check_parser(self.cmd, arg_list, verify_list)
         columns, data = self.cmd.take_action(parsed_args)
-        mock_create_t_f = self.app.client_manager.network.create_tap_flow
-        mock_create_t_f.assert_called_once_with(
+        self.network_client.create_tap_flow.assert_called_once_with(
             **{
                 'name': fake_tap_flow['name'],
                 'source_port': fake_tap_flow['source_port'],
@@ -129,7 +124,7 @@ class TestListTapFlow(network_fakes.TestNetworkV2):
         fake_tap_flows = list(
             sdk_fakes.generate_fake_resources(_tap_flow.TapFlow, count=2)
         )
-        self.app.client_manager.network.tap_flows.return_value = fake_tap_flows
+        self.network_client.tap_flows.return_value = fake_tap_flows
         arg_list = []
         verify_list = []
 
@@ -137,7 +132,7 @@ class TestListTapFlow(network_fakes.TestNetworkV2):
 
         headers, data = self.cmd.take_action(parsed_args)
 
-        self.app.client_manager.network.tap_flows.assert_called_once()
+        self.network_client.tap_flows.assert_called_once()
         self.assertEqual(headers, list(headers_long))
         self.assertCountEqual(
             list(data),
@@ -151,7 +146,7 @@ class TestListTapFlow(network_fakes.TestNetworkV2):
 class TestDeleteTapFlow(network_fakes.TestNetworkV2):
     def setUp(self):
         super().setUp()
-        self.app.client_manager.network.find_tap_flow.side_effect = (
+        self.network_client.find_tap_flow.side_effect = (
             lambda name_or_id, ignore_missing: _tap_flow.TapFlow(id=name_or_id)
         )
         self.cmd = osc_tap_flow.DeleteTapFlow(self.app, None)
@@ -171,8 +166,9 @@ class TestDeleteTapFlow(network_fakes.TestNetworkV2):
 
         result = self.cmd.take_action(parsed_args)
 
-        mock_delete_tap_flow = self.app.client_manager.network.delete_tap_flow
-        mock_delete_tap_flow.assert_called_once_with(fake_tap_flow['id'])
+        self.network_client.delete_tap_flow.assert_called_once_with(
+            fake_tap_flow['id']
+        )
         self.assertIsNone(result)
 
 
@@ -190,7 +186,7 @@ class TestShowTapFlow(network_fakes.TestNetworkV2):
 
     def setUp(self):
         super().setUp()
-        self.app.client_manager.network.find_tap_flow.side_effect = (
+        self.network_client.find_tap_flow.side_effect = (
             lambda name_or_id, ignore_missing: _tap_flow.TapFlow(id=name_or_id)
         )
         self.cmd = osc_tap_flow.ShowTapFlow(self.app, None)
@@ -198,9 +194,7 @@ class TestShowTapFlow(network_fakes.TestNetworkV2):
     def test_show_tap_flow(self):
         """Test Show tap flow."""
         fake_tap_flow = sdk_fakes.generate_fake_resource(_tap_flow.TapFlow)
-        self.app.client_manager.network.get_tap_flow.return_value = (
-            fake_tap_flow
-        )
+        self.network_client.get_tap_flow.return_value = fake_tap_flow
         arg_list = [
             fake_tap_flow['id'],
         ]
@@ -212,7 +206,7 @@ class TestShowTapFlow(network_fakes.TestNetworkV2):
 
         headers, data = self.cmd.take_action(parsed_args)
 
-        self.app.client_manager.network.get_tap_flow.assert_called_once_with(
+        self.network_client.get_tap_flow.assert_called_once_with(
             fake_tap_flow['id']
         )
         self.assertEqual(self.columns, headers)
@@ -245,7 +239,7 @@ class TestUpdateTapFlow(network_fakes.TestNetworkV2):
     def setUp(self):
         super().setUp()
         self.cmd = osc_tap_flow.UpdateTapFlow(self.app, None)
-        self.app.client_manager.network.find_tap_flow.side_effect = (
+        self.network_client.find_tap_flow.side_effect = (
             lambda name_or_id, ignore_missing: _tap_flow.TapFlow(id=name_or_id)
         )
 
@@ -255,9 +249,7 @@ class TestUpdateTapFlow(network_fakes.TestNetworkV2):
         new_tap_flow = copy.deepcopy(fake_tap_flow)
         new_tap_flow['name'] = self._new_name
 
-        self.app.client_manager.network.update_tap_flow.return_value = (
-            new_tap_flow
-        )
+        self.network_client.update_tap_flow.return_value = new_tap_flow
 
         arg_list = [
             fake_tap_flow['id'],
@@ -270,7 +262,8 @@ class TestUpdateTapFlow(network_fakes.TestNetworkV2):
         columns, data = self.cmd.take_action(parsed_args)
         attrs = {'name': self._new_name}
 
-        mock_update_t_f = self.app.client_manager.network.update_tap_flow
-        mock_update_t_f.assert_called_once_with(new_tap_flow['id'], **attrs)
+        self.network_client.update_tap_flow.assert_called_once_with(
+            new_tap_flow['id'], **attrs
+        )
         self.assertEqual(self.columns, columns)
         self.assertEqual(_get_data(new_tap_flow, self.columns), data)

openstackclient/tests/unit/network/v2/taas/{test_osc_tap_mirror.py → test_tap_mirror.py}

@@ -64,11 +64,9 @@ class TestCreateTapMirror(network_fakes.TestNetworkV2):
         fake_tap_mirror = sdk_fakes.generate_fake_resource(
             tap_mirror.TapMirror, **{'port_id': port_id, 'directions': 'IN=99'}
         )
-        self.app.client_manager.network.create_tap_mirror.return_value = (
-            fake_tap_mirror
-        )
-        self.app.client_manager.network.find_port.return_value = fake_port
-        self.app.client_manager.network.find_tap_mirror.side_effect = (
+        self.network_client.create_tap_mirror.return_value = fake_tap_mirror
+        self.network_client.find_port.return_value = fake_port
+        self.network_client.find_tap_mirror.side_effect = (
             lambda _, name_or_id: {'id': name_or_id}
         )
         arg_list = [
@@ -96,13 +94,10 @@ class TestCreateTapMirror(network_fakes.TestNetworkV2):
         ]
 
         parsed_args = self.check_parser(self.cmd, arg_list, verify_list)
-        self.app.client_manager.network.find_tap_mirror.return_value = (
-            fake_tap_mirror
-        )
+        self.network_client.find_tap_mirror.return_value = fake_tap_mirror
 
         columns, data = self.cmd.take_action(parsed_args)
-        create_tap_m_mock = self.app.client_manager.network.create_tap_mirror
-        create_tap_m_mock.assert_called_once_with(
+        self.network_client.create_tap_mirror.assert_called_once_with(
             **{
                 'name': fake_tap_mirror['name'],
                 'port_id': fake_tap_mirror['port_id'],
@@ -128,9 +123,7 @@ class TestListTapMirror(network_fakes.TestNetworkV2):
         fake_tap_mirrors = list(
             sdk_fakes.generate_fake_resources(tap_mirror.TapMirror, count=4)
         )
-        self.app.client_manager.network.tap_mirrors.return_value = (
-            fake_tap_mirrors
-        )
+        self.network_client.tap_mirrors.return_value = fake_tap_mirrors
 
         arg_list = []
         verify_list = []
@@ -139,7 +132,7 @@ class TestListTapMirror(network_fakes.TestNetworkV2):
 
         headers, data = self.cmd.take_action(parsed_args)
 
-        self.app.client_manager.network.tap_mirrors.assert_called_once()
+        self.network_client.tap_mirrors.assert_called_once()
         self.assertEqual(headers, list(headers_long))
         self.assertCountEqual(
             list(data),
@@ -153,7 +146,7 @@ class TestListTapMirror(network_fakes.TestNetworkV2):
 class TestDeleteTapMirror(network_fakes.TestNetworkV2):
     def setUp(self):
         super().setUp()
-        self.app.client_manager.network.find_tap_mirror.side_effect = (
+        self.network_client.find_tap_mirror.side_effect = (
             lambda name_or_id, ignore_missing: tap_mirror.TapMirror(
                 id=name_or_id
             )
@@ -177,8 +170,9 @@ class TestDeleteTapMirror(network_fakes.TestNetworkV2):
         parsed_args = self.check_parser(self.cmd, arg_list, verify_list)
         result = self.cmd.take_action(parsed_args)
 
-        mock_delete_tap_m = self.app.client_manager.network.delete_tap_mirror
-        mock_delete_tap_m.assert_called_once_with(fake_tap_mirror['id'])
+        self.network_client.delete_tap_mirror.assert_called_once_with(
+            fake_tap_mirror['id']
+        )
         self.assertIsNone(result)
 
 
@@ -196,7 +190,7 @@ class TestShowTapMirror(network_fakes.TestNetworkV2):
 
     def setUp(self):
         super().setUp()
-        self.app.client_manager.network.find_tap_mirror.side_effect = (
+        self.network_client.find_tap_mirror.side_effect = (
             lambda name_or_id, ignore_missing: tap_mirror.TapMirror(
                 id=name_or_id
             )
@@ -209,9 +203,7 @@ class TestShowTapMirror(network_fakes.TestNetworkV2):
         fake_tap_mirror = sdk_fakes.generate_fake_resource(
             tap_mirror.TapMirror
         )
-        self.app.client_manager.network.get_tap_mirror.return_value = (
-            fake_tap_mirror
-        )
+        self.network_client.get_tap_mirror.return_value = fake_tap_mirror
         arg_list = [
             fake_tap_mirror['id'],
         ]
@@ -223,8 +215,9 @@ class TestShowTapMirror(network_fakes.TestNetworkV2):
 
         headers, data = self.cmd.take_action(parsed_args)
 
-        mock_get_tap_m = self.app.client_manager.network.get_tap_mirror
-        mock_get_tap_m.assert_called_once_with(fake_tap_mirror['id'])
+        self.network_client.get_tap_mirror.assert_called_once_with(
+            fake_tap_mirror['id']
+        )
         self.assertEqual(self.columns, headers)
         fake_data = _get_data(
             fake_tap_mirror, osc_tap_mirror._get_columns(fake_tap_mirror)[1]
@@ -248,7 +241,7 @@ class TestUpdateTapMirror(network_fakes.TestNetworkV2):
     def setUp(self):
         super().setUp()
         self.cmd = osc_tap_mirror.UpdateTapMirror(self.app, None)
-        self.app.client_manager.network.find_tap_mirror.side_effect = (
+        self.network_client.find_tap_mirror.side_effect = (
             lambda name_or_id, ignore_missing: tap_mirror.TapMirror(
                 id=name_or_id
             )
@@ -262,9 +255,7 @@ class TestUpdateTapMirror(network_fakes.TestNetworkV2):
         new_tap_mirror = copy.deepcopy(fake_tap_mirror)
         new_tap_mirror['name'] = self._new_name
 
-        self.app.client_manager.network.update_tap_mirror.return_value = (
-            new_tap_mirror
-        )
+        self.network_client.update_tap_mirror.return_value = new_tap_mirror
 
         arg_list = [
             fake_tap_mirror['id'],
@@ -277,8 +268,7 @@ class TestUpdateTapMirror(network_fakes.TestNetworkV2):
         columns, data = self.cmd.take_action(parsed_args)
         attrs = {'name': self._new_name}
 
-        mock_update_tap_m = self.app.client_manager.network.update_tap_mirror
-        mock_update_tap_m.assert_called_once_with(
+        self.network_client.update_tap_mirror.assert_called_once_with(
             fake_tap_mirror['id'], **attrs
         )
         self.assertEqual(self.columns, columns)

openstackclient/tests/unit/network/v2/taas/{test_osc_tap_service.py → test_tap_service.py}

@@ -65,11 +65,9 @@ class TestCreateTapService(network_fakes.TestNetworkV2):
         fake_tap_service = sdk_fakes.generate_fake_resource(
             tap_service.TapService, **{'port_id': port_id}
         )
-        self.app.client_manager.network.create_tap_service.return_value = (
-            fake_tap_service
-        )
-        self.app.client_manager.network.find_port.return_value = fake_port
-        self.app.client_manager.network.find_tap_service.side_effect = (
+        self.network_client.create_tap_service.return_value = fake_tap_service
+        self.network_client.find_port.return_value = fake_port
+        self.network_client.find_tap_service.side_effect = (
             lambda _, name_or_id: {'id': name_or_id}
         )
         arg_list = [
@@ -85,13 +83,10 @@ class TestCreateTapService(network_fakes.TestNetworkV2):
         ]
 
         parsed_args = self.check_parser(self.cmd, arg_list, verify_list)
-        self.app.client_manager.network.find_tap_service.return_value = (
-            fake_tap_service
-        )
+        self.network_client.find_tap_service.return_value = fake_tap_service
 
         columns, data = self.cmd.take_action(parsed_args)
-        create_tap_s_mock = self.app.client_manager.network.create_tap_service
-        create_tap_s_mock.assert_called_once_with(
+        self.network_client.create_tap_service.assert_called_once_with(
             **{
                 'name': fake_tap_service['name'],
                 'port_id': fake_tap_service['port_id'],
@@ -114,9 +109,7 @@ class TestListTapService(network_fakes.TestNetworkV2):
         fake_tap_services = list(
             sdk_fakes.generate_fake_resources(tap_service.TapService, count=4)
         )
-        self.app.client_manager.network.tap_services.return_value = (
-            fake_tap_services
-        )
+        self.network_client.tap_services.return_value = fake_tap_services
 
         arg_list = []
         verify_list = []
@@ -125,7 +118,7 @@ class TestListTapService(network_fakes.TestNetworkV2):
 
         headers, data = self.cmd.take_action(parsed_args)
 
-        self.app.client_manager.network.tap_services.assert_called_once()
+        self.network_client.tap_services.assert_called_once()
         self.assertEqual(headers, list(headers_long))
         self.assertCountEqual(
             list(data),
@@ -139,7 +132,7 @@ class TestListTapService(network_fakes.TestNetworkV2):
 class TestDeleteTapService(network_fakes.TestNetworkV2):
     def setUp(self):
         super().setUp()
-        self.app.client_manager.network.find_tap_service.side_effect = (
+        self.network_client.find_tap_service.side_effect = (
             lambda name_or_id, ignore_missing: tap_service.TapService(
                 id=name_or_id
             )
@@ -163,8 +156,9 @@ class TestDeleteTapService(network_fakes.TestNetworkV2):
         parsed_args = self.check_parser(self.cmd, arg_list, verify_list)
         result = self.cmd.take_action(parsed_args)
 
-        mock_delete_tap_s = self.app.client_manager.network.delete_tap_service
-        mock_delete_tap_s.assert_called_once_with(fake_tap_service['id'])
+        self.network_client.delete_tap_service.assert_called_once_with(
+            fake_tap_service['id']
+        )
         self.assertIsNone(result)
 
 
@@ -180,7 +174,7 @@ class TestShowTapService(network_fakes.TestNetworkV2):
 
     def setUp(self):
         super().setUp()
-        self.app.client_manager.network.find_tap_service.side_effect = (
+        self.network_client.find_tap_service.side_effect = (
             lambda name_or_id, ignore_missing: tap_service.TapService(
                 id=name_or_id
             )
@@ -193,9 +187,7 @@ class TestShowTapService(network_fakes.TestNetworkV2):
         fake_tap_service = sdk_fakes.generate_fake_resource(
             tap_service.TapService
         )
-        self.app.client_manager.network.get_tap_service.return_value = (
-            fake_tap_service
-        )
+        self.network_client.get_tap_service.return_value = fake_tap_service
         arg_list = [
             fake_tap_service['id'],
         ]
@@ -207,8 +199,9 @@ class TestShowTapService(network_fakes.TestNetworkV2):
 
         headers, data = self.cmd.take_action(parsed_args)
 
-        mock_get_tap_s = self.app.client_manager.network.get_tap_service
-        mock_get_tap_s.assert_called_once_with(fake_tap_service['id'])
+        self.network_client.get_tap_service.assert_called_once_with(
+            fake_tap_service['id']
+        )
         self.assertEqual(self.columns, headers)
         fake_data = _get_data(
             fake_tap_service, osc_tap_service._get_columns(fake_tap_service)[1]
@@ -231,7 +224,7 @@ class TestUpdateTapService(network_fakes.TestNetworkV2):
     def setUp(self):
         super().setUp()
         self.cmd = osc_tap_service.UpdateTapService(self.app, None)
-        self.app.client_manager.network.find_tap_service.side_effect = (
+        self.network_client.find_tap_service.side_effect = (
             lambda name_or_id, ignore_missing: tap_service.TapService(
                 id=name_or_id
             )
@@ -245,9 +238,7 @@ class TestUpdateTapService(network_fakes.TestNetworkV2):
         new_tap_service = copy.deepcopy(fake_tap_service)
         new_tap_service['name'] = self._new_name
 
-        self.app.client_manager.network.update_tap_service.return_value = (
-            new_tap_service
-        )
+        self.network_client.update_tap_service.return_value = new_tap_service
 
         arg_list = [
             fake_tap_service['id'],
@@ -260,8 +251,7 @@ class TestUpdateTapService(network_fakes.TestNetworkV2):
         columns, data = self.cmd.take_action(parsed_args)
         attrs = {'name': self._new_name}
 
-        mock_update_tap_s = self.app.client_manager.network.update_tap_service
-        mock_update_tap_s.assert_called_once_with(
+        self.network_client.update_tap_service.assert_called_once_with(
             fake_tap_service['id'], **attrs
         )
         self.assertEqual(self.columns, columns)

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

@@ -519,7 +519,7 @@ class TestUnsetAddressGroup(TestAddressGroup):
 
         parsed_args = self.check_parser(self.cmd, arglist, verifylist)
         result = self.cmd.take_action(parsed_args)
-        self.network_client.remove_addresses_from_address_group.assert_called_once_with(  # noqa: E501
+        self.network_client.remove_addresses_from_address_group.assert_called_once_with(
             self._address_group, ['10.0.0.2/32']
         )
         self.assertIsNone(result)
@@ -539,7 +539,7 @@ class TestUnsetAddressGroup(TestAddressGroup):
 
         parsed_args = self.check_parser(self.cmd, arglist, verifylist)
         result = self.cmd.take_action(parsed_args)
-        self.network_client.remove_addresses_from_address_group.assert_called_once_with(  # noqa: E501
+        self.network_client.remove_addresses_from_address_group.assert_called_once_with(
             self._address_group, ['10.0.0.2/32', '2001::/16']
         )
         self.assertIsNone(result)

openstackclient/tests/unit/network/v2/{test_floating_ip_network.py → test_floating_ip.py}

@@ -383,7 +383,7 @@ class TestDeleteFloatingIPNetwork(TestFloatingIPNetwork):
             self.cmd.take_action(parsed_args)
             self.fail('CommandError should be raised.')
         except exceptions.CommandError as e:
-            self.assertEqual('1 of 2 floating_ips failed to delete.', str(e))
+            self.assertEqual('1 of 2 floating IPs failed to delete.', str(e))
 
         self.network_client.find_ip.assert_any_call(
             self.floating_ips[0].id,
@@ -425,7 +425,8 @@ class TestListFloatingIPNetwork(TestFloatingIPNetwork):
         'Floating Network',
         'Project',
     )
-    columns_long = columns + (
+    columns_long = (
+        *columns,
         'Router',
         'Status',
         'Description',

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

@@ -39,14 +39,14 @@ class TestFloatingIPPortForwarding(network_fakes.TestNetworkV2):
 class TestCreateFloatingIPPortForwarding(TestFloatingIPPortForwarding):
     def setUp(self):
         super().setUp()
-        self.new_port_forwarding = network_fakes.FakeFloatingIPPortForwarding.create_one_port_forwarding(  # noqa: E501
+        self.new_port_forwarding = network_fakes.FakeFloatingIPPortForwarding.create_one_port_forwarding(
             attrs={
                 'internal_port_id': self.port.id,
                 'floatingip_id': self.floating_ip.id,
             }
         )
 
-        self.new_port_forwarding_with_ranges = network_fakes.FakeFloatingIPPortForwarding.create_one_port_forwarding(  # noqa: E501
+        self.new_port_forwarding_with_ranges = network_fakes.FakeFloatingIPPortForwarding.create_one_port_forwarding(
             use_range=True,
             attrs={
                 'internal_port_id': self.port.id,
@@ -144,15 +144,15 @@ class TestCreateFloatingIPPortForwarding(TestFloatingIPPortForwarding):
         parsed_args = self.check_parser(self.cmd, arglist, verifylist)
         columns, data = self.cmd.take_action(parsed_args)
 
-        self.network_client.create_floating_ip_port_forwarding.assert_called_once_with(  # noqa: E501
+        self.network_client.create_floating_ip_port_forwarding.assert_called_once_with(
             self.new_port_forwarding.floatingip_id,
             **{
-                'external_port_range': self.new_port_forwarding_with_ranges.external_port_range,  # noqa: E501
-                'internal_ip_address': self.new_port_forwarding_with_ranges.internal_ip_address,  # noqa: E501
-                'internal_port_range': self.new_port_forwarding_with_ranges.internal_port_range,  # noqa: E501
-                'internal_port_id': self.new_port_forwarding_with_ranges.internal_port_id,  # noqa: E501
+                'external_port_range': self.new_port_forwarding_with_ranges.external_port_range,
+                'internal_ip_address': self.new_port_forwarding_with_ranges.internal_ip_address,
+                'internal_port_range': self.new_port_forwarding_with_ranges.internal_port_range,
+                'internal_port_id': self.new_port_forwarding_with_ranges.internal_port_id,
                 'protocol': self.new_port_forwarding_with_ranges.protocol,
-                'description': self.new_port_forwarding_with_ranges.description,  # noqa: E501
+                'description': self.new_port_forwarding_with_ranges.description,
             },
         )
         self.assertEqual(self.columns, columns)
@@ -325,11 +325,11 @@ class TestCreateFloatingIPPortForwarding(TestFloatingIPPortForwarding):
         parsed_args = self.check_parser(self.cmd, arglist, verifylist)
         columns, data = self.cmd.take_action(parsed_args)
 
-        self.network_client.create_floating_ip_port_forwarding.assert_called_once_with(  # noqa: E501
+        self.network_client.create_floating_ip_port_forwarding.assert_called_once_with(
             self.new_port_forwarding.floatingip_id,
             **{
                 'external_port': self.new_port_forwarding.external_port,
-                'internal_ip_address': self.new_port_forwarding.internal_ip_address,  # noqa: E501
+                'internal_ip_address': self.new_port_forwarding.internal_ip_address,
                 'internal_port': self.new_port_forwarding.internal_port,
                 'internal_port_id': self.new_port_forwarding.internal_port_id,
                 'protocol': self.new_port_forwarding.protocol,
@@ -375,7 +375,7 @@ class TestDeleteFloatingIPPortForwarding(TestFloatingIPPortForwarding):
 
         result = self.cmd.take_action(parsed_args)
 
-        self.network_client.delete_floating_ip_port_forwarding.assert_called_once_with(  # noqa: E501
+        self.network_client.delete_floating_ip_port_forwarding.assert_called_once_with(
             self.floating_ip.id,
             self._port_forwarding[0].id,
             ignore_missing=False,
@@ -553,7 +553,7 @@ class TestSetFloatingIPPortForwarding(TestFloatingIPPortForwarding):
     # The Port Forwarding to set.
     def setUp(self):
         super().setUp()
-        self._port_forwarding = network_fakes.FakeFloatingIPPortForwarding.create_one_port_forwarding(  # noqa: E501
+        self._port_forwarding = network_fakes.FakeFloatingIPPortForwarding.create_one_port_forwarding(
             attrs={
                 'floatingip_id': self.floating_ip.id,
             }
@@ -675,7 +675,7 @@ class TestShowFloatingIPPortForwarding(TestFloatingIPPortForwarding):
 
     def setUp(self):
         super().setUp()
-        self._port_forwarding = network_fakes.FakeFloatingIPPortForwarding.create_one_port_forwarding(  # noqa: E501
+        self._port_forwarding = network_fakes.FakeFloatingIPPortForwarding.create_one_port_forwarding(
             attrs={
                 'floatingip_id': self.floating_ip.id,
             }

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

@@ -340,8 +340,8 @@ class TestListNetworkAgent(TestNetworkAgent):
         )
 
         # Add a column 'HA State' and corresponding data.
-        router_agent_columns = self.columns + ('HA State',)
-        router_agent_data = [d + ('',) for d in self.data]
+        router_agent_columns = (*self.columns, 'HA State')
+        router_agent_data = [(*d, '') for d in self.data]
 
         self.assertEqual(router_agent_columns, columns)
         self.assertEqual(len(router_agent_data), len(list(data)))
@@ -541,8 +541,9 @@ class TestShowNetworkAgent(TestNetworkAgent):
         'configuration',
         'created_at',
         'description',
-        'host',
+        'ha_chassis_priority',
         'ha_state',
+        'host',
         'id',
         'last_heartbeat_at',
         'resources_synced',
@@ -558,6 +559,9 @@ class TestShowNetworkAgent(TestNetworkAgent):
         format_columns.DictColumn(_network_agent.configuration),
         _network_agent.created_at,
         _network_agent.description,
+        # NOTE(ralonsoh): `ha_chassis_priority` column is still not supported
+        # by the API. See LP#2103521.
+        None,
         _network_agent.ha_state,
         _network_agent.host,
         _network_agent.id,
@@ -601,5 +605,5 @@ class TestShowNetworkAgent(TestNetworkAgent):
         self.network_client.get_agent.assert_called_once_with(
             self._network_agent.id
         )
-        self.assertEqual(set(self.columns), set(columns))
+        self.assertCountEqual(self.columns, columns)
         self.assertEqual(len(list(self.data)), len(list(data)))

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

@@ -167,7 +167,7 @@ class TestValidateAutoAllocatedTopology(TestAutoAllocatedTopology):
         ]
 
         parsed_args = self.check_parser(self.cmd, arglist, verifylist)
-        columns, data = self.cmd.take_action(parsed_args)
+        _columns, _data = self.cmd.take_action(parsed_args)
 
         self.network_client.validate_auto_allocated_topology.assert_called_with(
             None
@@ -185,7 +185,7 @@ class TestValidateAutoAllocatedTopology(TestAutoAllocatedTopology):
         ]
 
         parsed_args = self.check_parser(self.cmd, arglist, verifylist)
-        columns, data = self.cmd.take_action(parsed_args)
+        _columns, _data = self.cmd.take_action(parsed_args)
 
         self.network_client.validate_auto_allocated_topology.assert_called_with(
             self.project.id
@@ -206,7 +206,7 @@ class TestValidateAutoAllocatedTopology(TestAutoAllocatedTopology):
         ]
 
         parsed_args = self.check_parser(self.cmd, arglist, verifylist)
-        columns, data = self.cmd.take_action(parsed_args)
+        _columns, _data = self.cmd.take_action(parsed_args)
 
         self.network_client.validate_auto_allocated_topology.assert_called_with(
             self.project.id

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

@@ -71,7 +71,7 @@ class TestAddNetworkFlavorToProfile(TestNetworkFlavor):
         parsed_args = self.check_parser(self.cmd, arglist, verifylist)
         self.cmd.take_action(parsed_args)
 
-        self.network_client.associate_flavor_with_service_profile.assert_called_once_with(  # noqa: E501
+        self.network_client.associate_flavor_with_service_profile.assert_called_once_with(
             self.network_flavor, self.service_profile
         )
 
@@ -377,7 +377,7 @@ class TestRemoveNetworkFlavorFromProfile(TestNetworkFlavor):
         parsed_args = self.check_parser(self.cmd, arglist, verifylist)
         self.cmd.take_action(parsed_args)
 
-        self.network_client.disassociate_flavor_from_service_profile.assert_called_once_with(  # noqa: E501
+        self.network_client.disassociate_flavor_from_service_profile.assert_called_once_with(
             self.network_flavor, self.service_profile
         )
 

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

@@ -28,7 +28,7 @@ class TestQosPolicy(network_fakes.TestNetworkV2):
     def setUp(self):
         super().setUp()
         # Get a shortcut to the ProjectManager Mock
-        self.projects_mock = self.app.client_manager.identity.projects
+        self.projects_mock = self.identity_client.projects
 
 
 class TestCreateNetworkQosPolicy(TestQosPolicy):

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

@@ -226,7 +226,7 @@ class TestCreateNetworkQosRuleMinimumPacketRate(TestNetworkQosRule):
         parsed_args = self.check_parser(self.cmd, arglist, verifylist)
         columns, data = self.cmd.take_action(parsed_args)
 
-        self.network_client.create_qos_minimum_packet_rate_rule.assert_called_once_with(  # noqa: E501
+        self.network_client.create_qos_minimum_packet_rate_rule.assert_called_once_with(
             self.qos_policy.id,
             **{
                 'min_kpps': self.new_rule.min_kpps,
@@ -613,7 +613,7 @@ class TestDeleteNetworkQosRuleMinimumPacketRate(TestNetworkQosRule):
         self.network_client.find_qos_policy.assert_called_once_with(
             self.qos_policy.id, ignore_missing=False
         )
-        self.network_client.delete_qos_minimum_packet_rate_rule.assert_called_once_with(  # noqa: E501
+        self.network_client.delete_qos_minimum_packet_rate_rule.assert_called_once_with(
             self.new_rule.id, self.qos_policy.id
         )
         self.assertIsNone(result)

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

@@ -224,7 +224,7 @@ class TestCreateNetworkRBAC(TestNetworkRBAC):
         ]
         parsed_args = self.check_parser(self.cmd, arglist, verifylist)
 
-        columns, data = self.cmd.take_action(parsed_args)
+        _columns, _data = self.cmd.take_action(parsed_args)
 
         self.network_client.create_rbac_policy.assert_called_with(
             **{

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

@@ -250,7 +250,7 @@ class TestListNetworkSegment(TestNetworkSegment):
         'Network Type',
         'Segment',
     )
-    columns_long = columns + ('Physical Network',)
+    columns_long = (*columns, 'Physical Network')
 
     data = []
     for _network_segment in _network_segments: