Merge "tests: Remove unnecessary nesting of volume resources"

5 files changed, 24 insertions, 28 deletions

diff --git a/openstackclient/tests/unit/common/test_availability_zone.py b/openstackclient/tests/unit/common/test_availability_zone.py
index e5348ec3..096038ca 100644
--- a/openstackclient/tests/unit/common/test_availability_zone.py
+++ b/openstackclient/tests/unit/common/test_availability_zone.py
@@ -78,7 +78,7 @@ def _build_network_az_datalist(network_az, long_datalist=False):
 class TestAvailabilityZone(utils.TestCommand):
 
     def setUp(self):
-        super(TestAvailabilityZone, self).setUp()
+        super().setUp()
 
         compute_client = compute_fakes.FakeComputev2Client(
             endpoint=fakes.AUTH_URL,
@@ -113,8 +113,7 @@ class TestAvailabilityZoneList(TestAvailabilityZone):
 
     compute_azs = \
         compute_fakes.FakeAvailabilityZone.create_availability_zones()
-    volume_azs = \
-        volume_fakes.FakeAvailabilityZone.create_availability_zones(count=1)
+    volume_azs = volume_fakes.create_availability_zones(count=1)
     network_azs = network_fakes.create_availability_zones()
 
     short_columnslist = ('Zone Name', 'Zone Status')
@@ -128,7 +127,7 @@ class TestAvailabilityZoneList(TestAvailabilityZone):
     )
 
     def setUp(self):
-        super(TestAvailabilityZoneList, self).setUp()
+        super().setUp()
 
         self.compute_azs_mock.list.return_value = self.compute_azs
         self.volume_azs_mock.list.return_value = self.volume_azs
diff --git a/openstackclient/tests/unit/common/test_extension.py b/openstackclient/tests/unit/common/test_extension.py
index 5093cbbb..bd90b32d 100644
--- a/openstackclient/tests/unit/common/test_extension.py
+++ b/openstackclient/tests/unit/common/test_extension.py
@@ -26,7 +26,7 @@ from openstackclient.tests.unit.volume.v2 import fakes as volume_fakes
 class TestExtension(utils.TestCommand):
 
     def setUp(self):
-        super(TestExtension, self).setUp()
+        super().setUp()
 
         identity_client = identity_fakes.FakeIdentityv2Client(
             endpoint=fakes.AUTH_URL,
@@ -66,13 +66,13 @@ class TestExtensionList(TestExtension):
     long_columns = ('Name', 'Alias', 'Description', 'Namespace', 'Updated',
                     'Links')
 
-    volume_extension = volume_fakes.FakeExtension.create_one_extension()
+    volume_extension = volume_fakes.create_one_extension()
     identity_extension = identity_fakes.FakeExtension.create_one_extension()
     compute_extension = compute_fakes.FakeExtension.create_one_extension()
     network_extension = network_fakes.FakeExtension.create_one_extension()
 
     def setUp(self):
-        super(TestExtensionList, self).setUp()
+        super().setUp()
 
         self.identity_extensions_mock.list.return_value = [
             self.identity_extension]
@@ -310,7 +310,7 @@ class TestExtensionShow(TestExtension):
     )
 
     def setUp(self):
-        super(TestExtensionShow, self).setUp()
+        super().setUp()
 
         self.cmd = extension.ShowExtension(self.app, None)
 
diff --git a/openstackclient/tests/unit/common/test_limits.py b/openstackclient/tests/unit/common/test_limits.py
index d73db2cb..e3cdcf45 100644
--- a/openstackclient/tests/unit/common/test_limits.py
+++ b/openstackclient/tests/unit/common/test_limits.py
@@ -33,7 +33,7 @@ class TestComputeLimits(compute_fakes.TestComputev2):
     ]
 
     def setUp(self):
-        super(TestComputeLimits, self).setUp()
+        super().setUp()
         self.app.client_manager.volume_endpoint_enabled = False
         self.compute = self.app.client_manager.compute
 
@@ -87,7 +87,7 @@ class TestVolumeLimits(volume_fakes.TestVolume):
     ]
 
     def setUp(self):
-        super(TestVolumeLimits, self).setUp()
+        super().setUp()
         self.app.client_manager.compute_endpoint_enabled = False
         self.volume = self.app.client_manager.volume
 
diff --git a/openstackclient/tests/unit/common/test_project_purge.py b/openstackclient/tests/unit/common/test_project_purge.py
index 5199093c..26333d70 100644
--- a/openstackclient/tests/unit/common/test_project_purge.py
+++ b/openstackclient/tests/unit/common/test_project_purge.py
@@ -26,7 +26,7 @@ from openstackclient.tests.unit.volume.v2 import fakes as volume_fakes
 class TestProjectPurgeInit(tests_utils.TestCommand):
 
     def setUp(self):
-        super(TestProjectPurgeInit, self).setUp()
+        super().setUp()
         compute_client = compute_fakes.FakeComputev2Client(
             endpoint=fakes.AUTH_URL,
             token=fakes.AUTH_TOKEN,
@@ -71,12 +71,12 @@ class TestProjectPurge(TestProjectPurgeInit):
     project = identity_fakes.FakeProject.create_one_project()
     server = compute_fakes.FakeServer.create_one_server()
     image = image_fakes.create_one_image()
-    volume = volume_fakes.FakeVolume.create_one_volume()
-    backup = volume_fakes.FakeBackup.create_one_backup()
-    snapshot = volume_fakes.FakeSnapshot.create_one_snapshot()
+    volume = volume_fakes.create_one_volume()
+    backup = volume_fakes.create_one_backup()
+    snapshot = volume_fakes.create_one_snapshot()
 
     def setUp(self):
-        super(TestProjectPurge, self).setUp()
+        super().setUp()
         self.projects_mock.get.return_value = self.project
         self.projects_mock.delete.return_value = None
         self.images_mock.list.return_value = [self.image]
diff --git a/openstackclient/tests/unit/common/test_quota.py b/openstackclient/tests/unit/common/test_quota.py
index 0900200e..682799b3 100644
--- a/openstackclient/tests/unit/common/test_quota.py
+++ b/openstackclient/tests/unit/common/test_quota.py
@@ -176,12 +176,12 @@ class TestQuotaList(TestQuota):
         )
 
         self.volume_quotas = [
-            volume_fakes.FakeQuota.create_one_vol_quota(),
-            volume_fakes.FakeQuota.create_one_vol_quota(),
+            volume_fakes.create_one_vol_quota(),
+            volume_fakes.create_one_vol_quota(),
         ]
         self.volume_default_quotas = [
-            volume_fakes.FakeQuota.create_one_default_vol_quota(),
-            volume_fakes.FakeQuota.create_one_default_vol_quota(),
+            volume_fakes.create_one_default_vol_quota(),
+            volume_fakes.create_one_default_vol_quota(),
         ]
         self.volume = self.app.client_manager.volume
         self.volume.quotas.defaults = mock.Mock(
@@ -280,8 +280,7 @@ class TestQuotaList(TestQuota):
             sorted(detailed_reference_data), sorted(ret_quotas))
 
     def test_quota_list_details_volume(self):
-        detailed_quota = (
-            volume_fakes.FakeQuota.create_one_detailed_quota())
+        detailed_quota = volume_fakes.create_one_detailed_quota()
 
         detailed_column_header = (
             'Resource',
@@ -567,7 +566,7 @@ class TestQuotaList(TestQuota):
         self.volume.quotas.get = mock.Mock(
             side_effect=[
                 self.volume_quotas[0],
-                volume_fakes.FakeQuota.create_one_default_vol_quota(),
+                volume_fakes.create_one_default_vol_quota(),
             ],
         )
 
@@ -591,7 +590,7 @@ class TestQuotaList(TestQuota):
         self.volume.quotas.get = mock.Mock(
             side_effect=[
                 self.volume_quotas[0],
-                volume_fakes.FakeQuota.create_one_default_vol_quota(),
+                volume_fakes.create_one_default_vol_quota(),
             ],
         )
 
@@ -1046,10 +1045,9 @@ class TestQuotaShow(TestQuota):
             loaded=True,
         )
 
-        self.volume_quota = volume_fakes.FakeQuota.create_one_vol_quota()
+        self.volume_quota = volume_fakes.create_one_vol_quota()
         self.volume_quotas_mock.get.return_value = self.volume_quota
-        self.volume_default_quota = \
-            volume_fakes.FakeQuota.create_one_default_vol_quota()
+        self.volume_default_quota = volume_fakes.create_one_default_vol_quota()
         self.volume_quotas_mock.defaults.return_value = \
             self.volume_default_quota
         self.volume_quotas_class_mock.get.return_value = FakeQuotaResource(
@@ -1217,8 +1215,7 @@ class TestQuotaShow(TestQuota):
         self.compute_quota = \
             compute_fakes.FakeQuota.create_one_comp_detailed_quota()
         self.compute_quotas_mock.get.return_value = self.compute_quota
-        self.volume_quota = \
-            volume_fakes.FakeQuota.create_one_detailed_quota()
+        self.volume_quota = volume_fakes.create_one_detailed_quota()
         self.volume_quotas_mock.get.return_value = self.volume_quota
         self.network.get_quota.return_value = \
             network_fakes.FakeQuota.create_one_net_detailed_quota()