Skip to content

Commit dc5bd94

Browse files
Jenkinsopenstack-gerrit
authored andcommitted
Merge "Use FakeImage in server test cases."
2 parents e4cbc44 + a4696dc commit dc5bd94

1 file changed

Lines changed: 16 additions & 24 deletions

File tree

openstackclient/tests/compute/v2/test_server.py

Lines changed: 16 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -107,11 +107,7 @@ def setUp(self):
107107

108108
self.servers_mock.create.return_value = self.new_server
109109

110-
self.image = fakes.FakeResource(
111-
None,
112-
copy.deepcopy(image_fakes.IMAGE),
113-
loaded=True,
114-
)
110+
self.image = image_fakes.FakeImage.create_one_image()
115111
self.cimages_mock.get.return_value = self.image
116112

117113
self.flavor = compute_fakes.FakeFlavor.create_one_flavor()
@@ -549,13 +545,9 @@ def setUp(self):
549545
# This is the return value for utils.find_resource()
550546
self.servers_mock.get.return_value = self.server
551547

552-
self.servers_mock.create_image.return_value = image_fakes.image_id
553-
554-
self.images_mock.get.return_value = fakes.FakeResource(
555-
None,
556-
copy.deepcopy(image_fakes.IMAGE),
557-
loaded=True,
558-
)
548+
self.image = image_fakes.FakeImage.create_one_image()
549+
self.images_mock.get.return_value = self.image
550+
self.servers_mock.create_image.return_value = self.image.id
559551

560552
# Get the command object to test
561553
self.cmd = server.CreateServerImage(self.app, None)
@@ -581,12 +573,12 @@ def test_server_image_create_no_options(self):
581573
collist = ('id', 'name', 'owner', 'protected', 'tags', 'visibility')
582574
self.assertEqual(collist, columns)
583575
datalist = (
584-
image_fakes.image_id,
585-
image_fakes.image_name,
586-
image_fakes.image_owner,
587-
image_fakes.image_protected,
588-
image_fakes.image_tags,
589-
image_fakes.image_visibility,
576+
self.image.id,
577+
self.image.name,
578+
self.image.owner,
579+
self.image.protected,
580+
self.image.tags,
581+
self.image.visibility,
590582
)
591583
self.assertEqual(datalist, data)
592584

@@ -613,12 +605,12 @@ def test_server_image_create_name(self):
613605
collist = ('id', 'name', 'owner', 'protected', 'tags', 'visibility')
614606
self.assertEqual(collist, columns)
615607
datalist = (
616-
image_fakes.image_id,
617-
image_fakes.image_name,
618-
image_fakes.image_owner,
619-
image_fakes.image_protected,
620-
image_fakes.image_tags,
621-
image_fakes.image_visibility,
608+
self.image.id,
609+
self.image.name,
610+
self.image.owner,
611+
self.image.protected,
612+
self.image.tags,
613+
self.image.visibility,
622614
)
623615
self.assertEqual(datalist, data)
624616

0 commit comments

Comments
 (0)