Merge "Fix V3 image tests according to new image_client"
This commit is contained in:
commit
99b7e3208c
@ -344,9 +344,6 @@
|
||||
# password? (boolean value)
|
||||
#change_password=false
|
||||
|
||||
# Does the test environment support snapshots? (boolean value)
|
||||
#create_image=false
|
||||
|
||||
# Does the test environment support resizing? (boolean value)
|
||||
#resize=false
|
||||
|
||||
|
@ -13,7 +13,6 @@
|
||||
# License for the specific language governing permissions and limitations
|
||||
# under the License.
|
||||
|
||||
import testtools
|
||||
|
||||
from tempest.api.compute import base
|
||||
from tempest.common.utils import data_utils
|
||||
@ -61,8 +60,6 @@ class ImagesOneServerTestJSON(base.BaseV2ComputeTest):
|
||||
resp, flavor = self.flavors_client.get_flavor_details(flavor_id)
|
||||
return flavor['disk']
|
||||
|
||||
@testtools.skipUnless(CONF.compute_feature_enabled.create_image,
|
||||
'Environment unable to create images.')
|
||||
@test.attr(type='smoke')
|
||||
def test_create_delete_image(self):
|
||||
|
||||
|
@ -35,7 +35,7 @@ class ImagesNegativeV3Test(base.BaseV3ComputeTest):
|
||||
resp, body = self.servers_client.create_image(server_id, name, meta)
|
||||
image_id = data_utils.parse_image_id(resp['location'])
|
||||
self.addCleanup(self.client.delete_image, image_id)
|
||||
self.client.wait_for_image_status(image_id, 'ACTIVE')
|
||||
self.client.wait_for_image_status(image_id, 'active')
|
||||
return resp, body
|
||||
|
||||
@test.attr(type=['negative', 'gate'])
|
||||
|
@ -13,7 +13,6 @@
|
||||
# License for the specific language governing permissions and limitations
|
||||
# under the License.
|
||||
|
||||
import testtools
|
||||
|
||||
from tempest.api.compute import base
|
||||
from tempest.common.utils import data_utils
|
||||
@ -61,8 +60,6 @@ class ImagesOneServerV3Test(base.BaseV3ComputeTest):
|
||||
resp, flavor = self.flavors_client.get_flavor_details(flavor_id)
|
||||
return flavor['disk']
|
||||
|
||||
@testtools.skipUnless(CONF.compute_feature_enabled.create_image,
|
||||
'Environment unable to create images.')
|
||||
@test.attr(type='smoke')
|
||||
def test_create_delete_image(self):
|
||||
|
||||
@ -73,26 +70,26 @@ class ImagesOneServerV3Test(base.BaseV3ComputeTest):
|
||||
name, meta)
|
||||
self.assertEqual(202, resp.status)
|
||||
image_id = data_utils.parse_image_id(resp['location'])
|
||||
self.client.wait_for_image_status(image_id, 'ACTIVE')
|
||||
self.client.wait_for_image_status(image_id, 'active')
|
||||
|
||||
# Verify the image was created correctly
|
||||
resp, image = self.client.get_image(image_id)
|
||||
resp, image = self.client.get_image_meta(image_id)
|
||||
self.assertEqual(name, image['name'])
|
||||
self.assertEqual('test', image['metadata']['image_type'])
|
||||
self.assertEqual('test', image['properties']['image_type'])
|
||||
|
||||
resp, original_image = self.client.get_image(self.image_ref)
|
||||
resp, original_image = self.client.get_image_meta(self.image_ref)
|
||||
|
||||
# Verify minRAM is the same as the original image
|
||||
self.assertEqual(image['minRam'], original_image['minRam'])
|
||||
self.assertEqual(image['min_ram'], original_image['min_ram'])
|
||||
|
||||
# Verify minDisk is the same as the original image or the flavor size
|
||||
flavor_disk_size = self._get_default_flavor_disk_size(self.flavor_ref)
|
||||
self.assertIn(str(image['minDisk']),
|
||||
(str(original_image['minDisk']), str(flavor_disk_size)))
|
||||
self.assertIn(str(image['min_disk']),
|
||||
(str(original_image['min_disk']), str(flavor_disk_size)))
|
||||
|
||||
# Verify the image was deleted correctly
|
||||
resp, body = self.client.delete_image(image_id)
|
||||
self.assertEqual('204', resp['status'])
|
||||
self.assertEqual('200', resp['status'])
|
||||
self.client.wait_for_resource_deletion(image_id)
|
||||
|
||||
@test.attr(type=['gate'])
|
||||
|
@ -246,9 +246,6 @@ ComputeFeaturesGroup = [
|
||||
default=False,
|
||||
help="Does the test environment support changing the admin "
|
||||
"password?"),
|
||||
cfg.BoolOpt('create_image',
|
||||
default=False,
|
||||
help="Does the test environment support snapshots?"),
|
||||
cfg.BoolOpt('resize',
|
||||
default=False,
|
||||
help="Does the test environment support resizing?"),
|
||||
|
Loading…
Reference in New Issue
Block a user