Merge "Do not use any() to check whether a list is empty"
This commit is contained in:
commit
4dfef5cd41
@ -68,7 +68,7 @@ class FlavorsV2TestJSON(base.BaseV2ComputeTest):
|
||||
|
||||
params = {'marker': flavor_id}
|
||||
flavors = self.flavors_client.list_flavors(**params)['flavors']
|
||||
self.assertFalse(any([i for i in flavors if i['id'] == flavor_id]),
|
||||
self.assertEmpty([i for i in flavors if i['id'] == flavor_id],
|
||||
'The list of flavors did not start after the marker.')
|
||||
|
||||
@decorators.idempotent_id('6db2f0c0-ddee-4162-9c84-0703d3dd1107')
|
||||
@ -80,7 +80,7 @@ class FlavorsV2TestJSON(base.BaseV2ComputeTest):
|
||||
params = {'marker': flavor_id}
|
||||
flavors = self.flavors_client.list_flavors(detail=True,
|
||||
**params)['flavors']
|
||||
self.assertFalse(any([i for i in flavors if i['id'] == flavor_id]),
|
||||
self.assertEmpty([i for i in flavors if i['id'] == flavor_id],
|
||||
'The list of flavors did not start after the marker.')
|
||||
|
||||
@decorators.idempotent_id('3df2743e-3034-4e57-a4cb-b6527f6eac79')
|
||||
@ -92,7 +92,7 @@ class FlavorsV2TestJSON(base.BaseV2ComputeTest):
|
||||
params = {self._min_disk: flavor['disk'] + 1}
|
||||
flavors = self.flavors_client.list_flavors(detail=True,
|
||||
**params)['flavors']
|
||||
self.assertFalse(any([i for i in flavors if i['id'] == flavor_id]))
|
||||
self.assertEmpty([i for i in flavors if i['id'] == flavor_id])
|
||||
|
||||
@decorators.idempotent_id('09fe7509-b4ee-4b34-bf8b-39532dc47292')
|
||||
def test_list_flavors_detailed_filter_by_min_ram(self):
|
||||
@ -103,7 +103,7 @@ class FlavorsV2TestJSON(base.BaseV2ComputeTest):
|
||||
params = {self._min_ram: flavor['ram'] + 1}
|
||||
flavors = self.flavors_client.list_flavors(detail=True,
|
||||
**params)['flavors']
|
||||
self.assertFalse(any([i for i in flavors if i['id'] == flavor_id]))
|
||||
self.assertEmpty([i for i in flavors if i['id'] == flavor_id])
|
||||
|
||||
@decorators.idempotent_id('10645a4d-96f5-443f-831b-730711e11dd4')
|
||||
def test_list_flavors_filter_by_min_disk(self):
|
||||
@ -113,7 +113,7 @@ class FlavorsV2TestJSON(base.BaseV2ComputeTest):
|
||||
|
||||
params = {self._min_disk: flavor['disk'] + 1}
|
||||
flavors = self.flavors_client.list_flavors(**params)['flavors']
|
||||
self.assertFalse(any([i for i in flavors if i['id'] == flavor_id]))
|
||||
self.assertEmpty([i for i in flavors if i['id'] == flavor_id])
|
||||
|
||||
@decorators.idempotent_id('935cf550-e7c8-4da6-8002-00f92d5edfaa')
|
||||
def test_list_flavors_filter_by_min_ram(self):
|
||||
@ -123,4 +123,4 @@ class FlavorsV2TestJSON(base.BaseV2ComputeTest):
|
||||
|
||||
params = {self._min_ram: flavor['ram'] + 1}
|
||||
flavors = self.flavors_client.list_flavors(**params)['flavors']
|
||||
self.assertFalse(any([i for i in flavors if i['id'] == flavor_id]))
|
||||
self.assertEmpty([i for i in flavors if i['id'] == flavor_id])
|
||||
|
@ -129,9 +129,9 @@ class ListImageFiltersTestJSON(base.BaseV2ComputeTest):
|
||||
params = {'status': 'ACTIVE'}
|
||||
images = self.client.list_images(**params)['images']
|
||||
|
||||
self.assertTrue(any([i for i in images if i['id'] == self.image1_id]))
|
||||
self.assertTrue(any([i for i in images if i['id'] == self.image2_id]))
|
||||
self.assertTrue(any([i for i in images if i['id'] == self.image3_id]))
|
||||
self.assertNotEmpty([i for i in images if i['id'] == self.image1_id])
|
||||
self.assertNotEmpty([i for i in images if i['id'] == self.image2_id])
|
||||
self.assertNotEmpty([i for i in images if i['id'] == self.image3_id])
|
||||
|
||||
@decorators.idempotent_id('33163b73-79f5-4d07-a7ea-9213bcc468ff')
|
||||
def test_list_images_filter_by_name(self):
|
||||
@ -140,9 +140,9 @@ class ListImageFiltersTestJSON(base.BaseV2ComputeTest):
|
||||
params = {'name': self.image1['name']}
|
||||
images = self.client.list_images(**params)['images']
|
||||
|
||||
self.assertTrue(any([i for i in images if i['id'] == self.image1_id]))
|
||||
self.assertFalse(any([i for i in images if i['id'] == self.image2_id]))
|
||||
self.assertFalse(any([i for i in images if i['id'] == self.image3_id]))
|
||||
self.assertNotEmpty([i for i in images if i['id'] == self.image1_id])
|
||||
self.assertEmpty([i for i in images if i['id'] == self.image2_id])
|
||||
self.assertEmpty([i for i in images if i['id'] == self.image3_id])
|
||||
|
||||
@decorators.idempotent_id('9f238683-c763-45aa-b848-232ec3ce3105')
|
||||
@testtools.skipUnless(CONF.compute_feature_enabled.snapshot,
|
||||
@ -152,14 +152,13 @@ class ListImageFiltersTestJSON(base.BaseV2ComputeTest):
|
||||
params = {'server': self.server1['id']}
|
||||
images = self.client.list_images(**params)['images']
|
||||
|
||||
self.assertTrue(any([i for i in images
|
||||
if i['id'] == self.snapshot1_id]),
|
||||
"Failed to find image %s in images. Got images %s" %
|
||||
(self.image1_id, images))
|
||||
self.assertTrue(any([i for i in images
|
||||
if i['id'] == self.snapshot2_id]))
|
||||
self.assertFalse(any([i for i in images
|
||||
if i['id'] == self.snapshot3_id]))
|
||||
self.assertNotEmpty([i for i in images
|
||||
if i['id'] == self.snapshot1_id],
|
||||
"Failed to find image %s in images. "
|
||||
"Got images %s" % (self.image1_id, images))
|
||||
self.assertNotEmpty([i for i in images
|
||||
if i['id'] == self.snapshot2_id])
|
||||
self.assertEmpty([i for i in images if i['id'] == self.snapshot3_id])
|
||||
|
||||
@decorators.idempotent_id('05a377b8-28cf-4734-a1e6-2ab5c38bf606')
|
||||
@testtools.skipUnless(CONF.compute_feature_enabled.snapshot,
|
||||
@ -173,12 +172,12 @@ class ListImageFiltersTestJSON(base.BaseV2ComputeTest):
|
||||
params = {'server': link['href']}
|
||||
images = self.client.list_images(**params)['images']
|
||||
|
||||
self.assertFalse(any([i for i in images
|
||||
if i['id'] == self.snapshot1_id]))
|
||||
self.assertFalse(any([i for i in images
|
||||
if i['id'] == self.snapshot2_id]))
|
||||
self.assertTrue(any([i for i in images
|
||||
if i['id'] == self.snapshot3_id]))
|
||||
self.assertEmpty([i for i in images
|
||||
if i['id'] == self.snapshot1_id])
|
||||
self.assertEmpty([i for i in images
|
||||
if i['id'] == self.snapshot2_id])
|
||||
self.assertNotEmpty([i for i in images
|
||||
if i['id'] == self.snapshot3_id])
|
||||
|
||||
@decorators.idempotent_id('e3356918-4d3e-4756-81d5-abc4524ba29f')
|
||||
@testtools.skipUnless(CONF.compute_feature_enabled.snapshot,
|
||||
@ -188,14 +187,13 @@ class ListImageFiltersTestJSON(base.BaseV2ComputeTest):
|
||||
params = {'type': 'snapshot'}
|
||||
images = self.client.list_images(**params)['images']
|
||||
|
||||
self.assertTrue(any([i for i in images
|
||||
if i['id'] == self.snapshot1_id]))
|
||||
self.assertTrue(any([i for i in images
|
||||
if i['id'] == self.snapshot2_id]))
|
||||
self.assertTrue(any([i for i in images
|
||||
if i['id'] == self.snapshot3_id]))
|
||||
self.assertFalse(any([i for i in images
|
||||
if i['id'] == self.image_ref]))
|
||||
self.assertNotEmpty([i for i in images
|
||||
if i['id'] == self.snapshot1_id])
|
||||
self.assertNotEmpty([i for i in images
|
||||
if i['id'] == self.snapshot2_id])
|
||||
self.assertNotEmpty([i for i in images
|
||||
if i['id'] == self.snapshot3_id])
|
||||
self.assertEmpty([i for i in images if i['id'] == self.image_ref])
|
||||
|
||||
@decorators.idempotent_id('3a484ca9-67ba-451e-b494-7fcf28d32d62')
|
||||
def test_list_images_limit_results(self):
|
||||
@ -212,8 +210,8 @@ class ListImageFiltersTestJSON(base.BaseV2ComputeTest):
|
||||
# Filter by the image's created time
|
||||
params = {'changes-since': self.image3['created']}
|
||||
images = self.client.list_images(**params)['images']
|
||||
found = any([i for i in images if i['id'] == self.image3_id])
|
||||
self.assertTrue(found)
|
||||
found = [i for i in images if i['id'] == self.image3_id]
|
||||
self.assertNotEmpty(found)
|
||||
|
||||
@decorators.idempotent_id('9b0ea018-6185-4f71-948a-a123a107988e')
|
||||
def test_list_images_with_detail_filter_by_status(self):
|
||||
@ -222,9 +220,9 @@ class ListImageFiltersTestJSON(base.BaseV2ComputeTest):
|
||||
params = {'status': 'ACTIVE'}
|
||||
images = self.client.list_images(detail=True, **params)['images']
|
||||
|
||||
self.assertTrue(any([i for i in images if i['id'] == self.image1_id]))
|
||||
self.assertTrue(any([i for i in images if i['id'] == self.image2_id]))
|
||||
self.assertTrue(any([i for i in images if i['id'] == self.image3_id]))
|
||||
self.assertNotEmpty([i for i in images if i['id'] == self.image1_id])
|
||||
self.assertNotEmpty([i for i in images if i['id'] == self.image2_id])
|
||||
self.assertNotEmpty([i for i in images if i['id'] == self.image3_id])
|
||||
|
||||
@decorators.idempotent_id('644ea267-9bd9-4f3b-af9f-dffa02396a17')
|
||||
def test_list_images_with_detail_filter_by_name(self):
|
||||
@ -233,9 +231,9 @@ class ListImageFiltersTestJSON(base.BaseV2ComputeTest):
|
||||
params = {'name': self.image1['name']}
|
||||
images = self.client.list_images(detail=True, **params)['images']
|
||||
|
||||
self.assertTrue(any([i for i in images if i['id'] == self.image1_id]))
|
||||
self.assertFalse(any([i for i in images if i['id'] == self.image2_id]))
|
||||
self.assertFalse(any([i for i in images if i['id'] == self.image3_id]))
|
||||
self.assertNotEmpty([i for i in images if i['id'] == self.image1_id])
|
||||
self.assertEmpty([i for i in images if i['id'] == self.image2_id])
|
||||
self.assertEmpty([i for i in images if i['id'] == self.image3_id])
|
||||
|
||||
@decorators.idempotent_id('ba2fa9a9-b672-47cc-b354-3b4c0600e2cb')
|
||||
def test_list_images_with_detail_limit_results(self):
|
||||
@ -257,12 +255,12 @@ class ListImageFiltersTestJSON(base.BaseV2ComputeTest):
|
||||
params = {'server': link['href']}
|
||||
images = self.client.list_images(detail=True, **params)['images']
|
||||
|
||||
self.assertFalse(any([i for i in images
|
||||
if i['id'] == self.snapshot1_id]))
|
||||
self.assertFalse(any([i for i in images
|
||||
if i['id'] == self.snapshot2_id]))
|
||||
self.assertTrue(any([i for i in images
|
||||
if i['id'] == self.snapshot3_id]))
|
||||
self.assertEmpty([i for i in images
|
||||
if i['id'] == self.snapshot1_id])
|
||||
self.assertEmpty([i for i in images
|
||||
if i['id'] == self.snapshot2_id])
|
||||
self.assertNotEmpty([i for i in images
|
||||
if i['id'] == self.snapshot3_id])
|
||||
|
||||
@decorators.idempotent_id('888c0cc0-7223-43c5-9db0-b125fd0a393b')
|
||||
@testtools.skipUnless(CONF.compute_feature_enabled.snapshot,
|
||||
@ -273,14 +271,13 @@ class ListImageFiltersTestJSON(base.BaseV2ComputeTest):
|
||||
images = self.client.list_images(detail=True, **params)['images']
|
||||
self.client.show_image(self.image_ref)
|
||||
|
||||
self.assertTrue(any([i for i in images
|
||||
if i['id'] == self.snapshot1_id]))
|
||||
self.assertTrue(any([i for i in images
|
||||
if i['id'] == self.snapshot2_id]))
|
||||
self.assertTrue(any([i for i in images
|
||||
if i['id'] == self.snapshot3_id]))
|
||||
self.assertFalse(any([i for i in images
|
||||
if i['id'] == self.image_ref]))
|
||||
self.assertNotEmpty([i for i in images
|
||||
if i['id'] == self.snapshot1_id])
|
||||
self.assertNotEmpty([i for i in images
|
||||
if i['id'] == self.snapshot2_id])
|
||||
self.assertNotEmpty([i for i in images
|
||||
if i['id'] == self.snapshot3_id])
|
||||
self.assertEmpty([i for i in images if i['id'] == self.image_ref])
|
||||
|
||||
@decorators.idempotent_id('7d439e18-ac2e-4827-b049-7e18004712c4')
|
||||
def test_list_images_with_detail_filter_by_changes_since(self):
|
||||
@ -290,4 +287,4 @@ class ListImageFiltersTestJSON(base.BaseV2ComputeTest):
|
||||
# Filter by the image's created time
|
||||
params = {'changes-since': self.image1['created']}
|
||||
images = self.client.list_images(detail=True, **params)['images']
|
||||
self.assertTrue(any([i for i in images if i['id'] == self.image1_id]))
|
||||
self.assertNotEmpty([i for i in images if i['id'] == self.image1_id])
|
||||
|
@ -44,12 +44,12 @@ class ListImagesTestJSON(base.BaseV2ComputeTest):
|
||||
def test_list_images(self):
|
||||
# The list of all images should contain the image
|
||||
images = self.client.list_images()['images']
|
||||
found = any([i for i in images if i['id'] == self.image_ref])
|
||||
self.assertTrue(found)
|
||||
found = [i for i in images if i['id'] == self.image_ref]
|
||||
self.assertNotEmpty(found)
|
||||
|
||||
@decorators.idempotent_id('9f94cb6b-7f10-48c5-b911-a0b84d7d4cd6')
|
||||
def test_list_images_with_detail(self):
|
||||
# Detailed list of all images should contain the expected images
|
||||
images = self.client.list_images(detail=True)['images']
|
||||
found = any([i for i in images if i['id'] == self.image_ref])
|
||||
self.assertTrue(found)
|
||||
found = [i for i in images if i['id'] == self.image_ref]
|
||||
self.assertNotEmpty(found)
|
||||
|
@ -159,8 +159,8 @@ class SecurityGroupRulesTestJSON(base.BaseSecurityGroupsTest):
|
||||
# Get rules of the created Security Group
|
||||
rules = self.security_groups_client.show_security_group(
|
||||
securitygroup_id)['security_group']['rules']
|
||||
self.assertTrue(any([i for i in rules if i['id'] == rule1_id]))
|
||||
self.assertTrue(any([i for i in rules if i['id'] == rule2_id]))
|
||||
self.assertNotEmpty([i for i in rules if i['id'] == rule1_id])
|
||||
self.assertNotEmpty([i for i in rules if i['id'] == rule2_id])
|
||||
|
||||
@decorators.idempotent_id('fc5c5acf-2091-43a6-a6ae-e42760e9ffaf')
|
||||
@test.services('network')
|
||||
|
@ -97,16 +97,16 @@ class ServersTestJSON(base.BaseV2ComputeTest):
|
||||
# The created server should be in the list of all servers
|
||||
body = self.client.list_servers()
|
||||
servers = body['servers']
|
||||
found = any([i for i in servers if i['id'] == self.server['id']])
|
||||
self.assertTrue(found)
|
||||
found = [i for i in servers if i['id'] == self.server['id']]
|
||||
self.assertNotEmpty(found)
|
||||
|
||||
@decorators.idempotent_id('585e934c-448e-43c4-acbf-d06a9b899997')
|
||||
def test_list_servers_with_detail(self):
|
||||
# The created server should be in the detailed list of all servers
|
||||
body = self.client.list_servers(detail=True)
|
||||
servers = body['servers']
|
||||
found = any([i for i in servers if i['id'] == self.server['id']])
|
||||
self.assertTrue(found)
|
||||
found = [i for i in servers if i['id'] == self.server['id']]
|
||||
self.assertNotEmpty(found)
|
||||
|
||||
@decorators.idempotent_id('cbc0f52f-05aa-492b-bdc1-84b575ca294b')
|
||||
@testtools.skipUnless(CONF.validation.run_validation,
|
||||
|
@ -54,7 +54,7 @@ class RolesTestJSON(base.BaseIdentityV2AdminTest):
|
||||
"""Return a list of all roles."""
|
||||
body = self.roles_client.list_roles()['roles']
|
||||
found = [role for role in body if role in self.roles]
|
||||
self.assertTrue(any(found))
|
||||
self.assertNotEmpty(found)
|
||||
self.assertEqual(len(found), len(self.roles))
|
||||
|
||||
@decorators.idempotent_id('c62d909d-6c21-48c0-ae40-0a0760e6db5e')
|
||||
@ -68,13 +68,13 @@ class RolesTestJSON(base.BaseIdentityV2AdminTest):
|
||||
|
||||
body = self.roles_client.list_roles()['roles']
|
||||
found = [role for role in body if role['name'] == role_name]
|
||||
self.assertTrue(any(found))
|
||||
self.assertNotEmpty(found)
|
||||
|
||||
body = self.roles_client.delete_role(found[0]['id'])
|
||||
|
||||
body = self.roles_client.list_roles()['roles']
|
||||
found = [role for role in body if role['name'] == role_name]
|
||||
self.assertFalse(any(found))
|
||||
self.assertEmpty(found)
|
||||
|
||||
@decorators.idempotent_id('db6870bd-a6ed-43be-a9b1-2f10a5c9994f')
|
||||
def test_get_role_by_id(self):
|
||||
|
@ -37,7 +37,7 @@ class TenantsTestJSON(base.BaseIdentityV2AdminTest):
|
||||
|
||||
body = self.tenants_client.list_tenants()['tenants']
|
||||
found = [tenant for tenant in body if tenant['id'] in tenant_ids]
|
||||
self.assertFalse(any(found), 'Tenants failed to delete')
|
||||
self.assertEmpty(found, 'Tenants failed to delete')
|
||||
|
||||
@decorators.idempotent_id('d25e9f24-1310-4d29-b61b-d91299c21d6d')
|
||||
def test_tenant_create_with_description(self):
|
||||
|
Loading…
Reference in New Issue
Block a user