Skip to content

Commit

Permalink
functional: Remove 'get_invalid_image'
Browse files Browse the repository at this point in the history
This only had one caller and doesn't belong in a utility class.

Change-Id: I8e163f9d08a4606b166c411e13aff2497142313c
Signed-off-by: Stephen Finucane <[email protected]>
  • Loading branch information
stephenfin committed Dec 6, 2019
1 parent d888d9b commit aff95de
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 5 deletions.
4 changes: 0 additions & 4 deletions nova/tests/functional/integrated_helpers.py
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,6 @@

import os_traits
from oslo_log import log as logging
from oslo_utils.fixture import uuidsentinel as uuids

from nova.compute import instance_actions
from nova.compute import utils as compute_utils
Expand Down Expand Up @@ -316,9 +315,6 @@ def get_unused_flavor_name_id(self):
return (generate_new_element(flavor_names, 'flavor'),
int(generate_new_element(flavor_ids, '', True)))

def get_invalid_image(self):
return uuids.fake

def _build_minimal_create_server_request(self, name=None, image_uuid=None,
flavor_id=None, networks=None,
az=None, host=None):
Expand Down
2 changes: 1 addition & 1 deletion nova/tests/functional/test_servers.py
Original file line number Diff line number Diff line change
Expand Up @@ -266,7 +266,7 @@ def test_create_and_delete_server(self):
self.api.post_server, post)

# With an invalid imageRef, this throws 500.
server[self._image_ref_parameter] = self.get_invalid_image()
server[self._image_ref_parameter] = uuids.fake
# TODO(justinsb): Check whatever the spec says should be thrown here
self.assertRaises(client.OpenStackApiException,
self.api.post_server, post)
Expand Down

0 comments on commit aff95de

Please sign in to comment.