openstack/tempest
Code Issues Proposed changes

Remove unnecessary assertIn

Browse Source 
addCleanup method uses 'id' key for volume type deletion,
so there isn't need to validate that the key exist.

Change-Id: I0741de04dbcc329cf642e58987194c11db9e3a9d
This commit is contained in:
lkuchlan 2017-09-26 10:47:23 +03:00
parent 2812b5c57b
commit bbabe5485b
1 changed files with 2 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
tempest/scenario/manager.py
View File

@ -271,10 +271,8 @@ class ScenarioTest(tempest.test.BaseTestCase):
if backend_name:
extra_specs = {"volume_backend_name": backend_name}
body = client.create_volume_type(name=randomized_name,
extra_specs=extra_specs)
volume_type = body['volume_type']
self.assertIn('id', volume_type)
volume_type = client.create_volume_type(
name=randomized_name, extra_specs=extra_specs)['volume_type']
self.addCleanup(client.delete_volume_type, volume_type['id'])
return volume_type