[Trivial]Remove unused variables and methods

This is to remove unused variables and methods to keep code clean.

Change-Id: Ib0b1e6ecb23481103791b065a622b4b89190412e
This commit is contained in:
zhufl 2020-11-18 15:34:25 +08:00
parent 805f2a070b
commit bf5ed0b676
8 changed files with 10 additions and 16 deletions

View File

@ -37,7 +37,7 @@ class ApplicationCredentialsV3AdminTest(base.BaseApplicationCredentialsV3Test,
secret = app_cred['secret']
# Check that the application credential is functional
token_id, resp = self.non_admin_token.get_token(
_, resp = self.non_admin_token.get_token(
app_cred_id=app_cred['id'],
app_cred_secret=secret,
auth_data=True

View File

@ -51,7 +51,7 @@ class ApplicationCredentialsV3Test(base.BaseApplicationCredentialsV3Test):
self.assertNotIn('secret', app_cred)
# Check that the application credential is functional
token_id, resp = self.non_admin_token.get_token(
_, resp = self.non_admin_token.get_token(
app_cred_id=app_cred['id'],
app_cred_secret=secret,
auth_data=True

View File

@ -66,7 +66,7 @@ class FloatingIPTestJSON(base.BaseNetworkTest):
cls.create_router_interface(cls.router['id'], cls.subnet['id'])
# Create two ports one each for Creation and Updating of floatingIP
cls.ports = []
for i in range(2):
for _ in range(2):
port = cls.create_port(cls.network)
cls.ports.append(port)

View File

@ -92,7 +92,7 @@ class ContainerSyncTest(base.BaseObjectTest):
# create object in container
object_name = data_utils.rand_name(name='TestSyncObject')
data = object_name[::-1].encode() # Raw data, we need bytes
resp, _ = obj_client[0].create_object(cont[0], object_name, data)
obj_client[0].create_object(cont[0], object_name, data)
self.objects.append(object_name)
# wait until container contents list is not empty

View File

@ -32,9 +32,6 @@ class CrossdomainTest(base.BaseObjectTest):
cls.xml_end = "</cross-domain-policy>"
def setUp(self):
super(CrossdomainTest, self).setUp()
@decorators.idempotent_id('d1b8b031-b622-4010-82f9-ff78a9e915c7')
@utils.requires_ext(extension='crossdomain', service='object')
def test_get_crossdomain_policy(self):

View File

@ -21,9 +21,6 @@ from tempest.lib import decorators
class HealthcheckTest(base.BaseObjectTest):
"""Test healthcheck"""
def setUp(self):
super(HealthcheckTest, self).setUp()
@decorators.idempotent_id('db5723b1-f25c-49a9-bfeb-7b5640caf337')
def test_get_healthcheck(self):
"""Test getting healthcheck"""

View File

@ -770,11 +770,11 @@ class ObjectTest(base.BaseObjectTest):
headers = {}
headers['X-Copy-From'] = "%s/%s" % (str(self.container_name),
str(object_name))
resp, body = self.object_client.create_object(self.container_name,
object_name,
data=None,
metadata=metadata,
headers=headers)
resp, _ = self.object_client.create_object(self.container_name,
object_name,
data=None,
metadata=metadata,
headers=headers)
self.assertHeaders(resp, 'Object', 'PUT')
# check the content type

View File

@ -165,6 +165,6 @@ class ObjectSloTest(base.BaseObjectTest):
self.assertHeaders(resp, 'Object', 'DELETE')
resp, body = self.container_client.list_container_objects(
resp, _ = self.container_client.list_container_objects(
self.container_name)
self.assertEqual(int(resp['x-container-object-count']), 0)