From 261b4111d481562760321bfc83d64ba35e981b5b Mon Sep 17 00:00:00 2001 From: Brian Waldon Date: Mon, 26 Sep 2011 13:57:34 -0400 Subject: Explicit errors on confirm/revertResize failures Fixing bug 856527 Change-Id: Ib9be618596ade2e9a899ecdc10ec9f61bf06958a --- nova/tests/api/openstack/test_server_actions.py | 28 +++++++++++++++++++++++++ 1 file changed, 28 insertions(+) (limited to 'nova/tests') diff --git a/nova/tests/api/openstack/test_server_actions.py b/nova/tests/api/openstack/test_server_actions.py index ae40b3c95..b6852093e 100644 --- a/nova/tests/api/openstack/test_server_actions.py +++ b/nova/tests/api/openstack/test_server_actions.py @@ -272,6 +272,20 @@ class ServerActionsTest(test.TestCase): res = req.get_response(fakes.wsgi_app()) self.assertEqual(res.status_int, 400) + def test_confirm_resize_migration_not_found(self): + req = self.webreq('/1/action', 'POST', dict(confirmResize=None)) + + def confirm_resize_mock(*args): + raise exception.MigrationNotFoundByStatus(instance_id=1, + status='finished') + + self.stubs.Set(nova.compute.api.API, + 'confirm_resize', + confirm_resize_mock) + + res = req.get_response(fakes.wsgi_app()) + self.assertEqual(res.status_int, 400) + def test_revert_resize_server(self): req = self.webreq('/1/action', 'POST', dict(revertResize=None)) @@ -299,6 +313,20 @@ class ServerActionsTest(test.TestCase): res = req.get_response(fakes.wsgi_app()) self.assertEqual(res.status_int, 400) + def test_revert_resize_migration_not_found(self): + req = self.webreq('/1/action', 'POST', dict(revertResize=None)) + + def revert_resize_mock(*args): + raise exception.MigrationNotFoundByStatus(instance_id=1, + status='finished') + + self.stubs.Set(nova.compute.api.API, + 'revert_resize', + revert_resize_mock) + + res = req.get_response(fakes.wsgi_app()) + self.assertEqual(res.status_int, 400) + def test_create_backup(self): """The happy path for creating backups""" self.flags(allow_admin_api=True) -- cgit