From b4f15f15a10f93278e869f6ab7e776e7450000bb Mon Sep 17 00:00:00 2001 From: Tiago Mello Date: Wed, 26 Dec 2012 21:03:47 -0200 Subject: Adds test for 'itemNotFound' errors in 'Delete server' Implements a test for 'itemNotFound' errors when deleting a server. Fixes Bug 1089368 Change-Id: I201486bb5775f856048ac1f5e4f3c852fb38e091 --- nova/tests/api/openstack/compute/test_servers.py | 13 ++++++++++--- 1 file changed, 10 insertions(+), 3 deletions(-) diff --git a/nova/tests/api/openstack/compute/test_servers.py b/nova/tests/api/openstack/compute/test_servers.py index 15be74d15..d32640bf2 100644 --- a/nova/tests/api/openstack/compute/test_servers.py +++ b/nova/tests/api/openstack/compute/test_servers.py @@ -1520,9 +1520,9 @@ class ServersControllerTest(test.TestCase): self.assertEqual(s['hostId'], host_ids[i % 2]) self.assertEqual(s['name'], 'server%d' % (i + 1)) - def test_delete_server_instance(self): + def _delete_server_instance(self, uuid=FAKE_UUID): fakes.stub_out_instance_quota(self.stubs, 0, 10) - req = fakes.HTTPRequest.blank('/v2/fake/servers/%s' % FAKE_UUID) + req = fakes.HTTPRequest.blank('/v2/fake/servers/%s' % uuid) req.method = 'DELETE' self.server_delete_called = False @@ -1534,10 +1534,17 @@ class ServersControllerTest(test.TestCase): self.server_delete_called = True self.stubs.Set(db, 'instance_destroy', instance_destroy_mock) - self.controller.delete(req, FAKE_UUID) + self.controller.delete(req, uuid) + def test_delete_server_instance(self): + self._delete_server_instance() self.assertEqual(self.server_delete_called, True) + def test_delete_server_instance_not_found(self): + self.assertRaises(webob.exc.HTTPNotFound, + self._delete_server_instance, + uuid='non-existent-uuid') + def test_delete_server_instance_while_building(self): fakes.stub_out_instance_quota(self.stubs, 0, 10) req = fakes.HTTPRequest.blank('/v2/fake/servers/%s' % FAKE_UUID) -- cgit