From 4e43747037058522087e6eb71ef3bcce266d55cc Mon Sep 17 00:00:00 2001 From: Joe Gordon Date: Mon, 26 Nov 2012 22:55:19 +0000 Subject: Provide better error message for aggregate-create For an InvalidAggregateAction exception: Before: 'ERROR: There was a conflict when trying to complete your request (HTTP 409)' After: 'ERROR: Cannot perform action 'create_aggregate' on aggregate 'N/A'. Reason: invalid zone. (HTTP 400)' Fix bug 1083353 Change-Id: I26e30059fe5bbc30eecc52cccec4eba2a57d00db --- nova/tests/api/openstack/compute/contrib/test_aggregates.py | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) (limited to 'nova/tests') diff --git a/nova/tests/api/openstack/compute/contrib/test_aggregates.py b/nova/tests/api/openstack/compute/contrib/test_aggregates.py index a209fdce8..41a87ac6a 100644 --- a/nova/tests/api/openstack/compute/contrib/test_aggregates.py +++ b/nova/tests/api/openstack/compute/contrib/test_aggregates.py @@ -89,11 +89,15 @@ class AggregateTestCase(test.TestCase): def test_create_with_incorrect_availability_zone(self): def stub_create_aggregate(context, name, availability_zone): - raise exception.InvalidAggregateAction + raise exception.InvalidAggregateAction(action='create_aggregate', + aggregate_id="'N/A'", + reason='invalid zone') + self.stubs.Set(self.controller.api, "create_aggregate", stub_create_aggregate) - self.assertRaises(exc.HTTPConflict, self.controller.create, + self.assertRaises(exception.InvalidAggregateAction, + self.controller.create, self.req, {"aggregate": {"name": "test", "availability_zone": "nova_bad"}}) -- cgit