diff options
| author | Tiago Mello <tmello@linux.vnet.ibm.com> | 2012-12-26 21:03:47 -0200 |
|---|---|---|
| committer | Tiago Mello <tmello@linux.vnet.ibm.com> | 2012-12-26 21:03:47 -0200 |
| commit | b4f15f15a10f93278e869f6ab7e776e7450000bb (patch) | |
| tree | 16edce389ad817d2fe83018f59f1f5b52b7e9be9 | |
| parent | 094d6b294bec70501f9bbdce3df63c8a328e06bf (diff) | |
| download | nova-b4f15f15a10f93278e869f6ab7e776e7450000bb.tar.gz nova-b4f15f15a10f93278e869f6ab7e776e7450000bb.tar.xz nova-b4f15f15a10f93278e869f6ab7e776e7450000bb.zip | |
Adds test for 'itemNotFound' errors in 'Delete server'
Implements a test for 'itemNotFound' errors when deleting
a server.
Fixes Bug 1089368
Change-Id: I201486bb5775f856048ac1f5e4f3c852fb38e091
| -rw-r--r-- | nova/tests/api/openstack/compute/test_servers.py | 13 |
1 files 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) |