From c383dcd43ecc85e05f437c2f81cded7ba54292a4 Mon Sep 17 00:00:00 2001 From: Russell Bryant Date: Wed, 28 Nov 2012 17:47:45 -0500 Subject: Fix args to AggregateError exception. This patch adds some arguments to the AggregateError exception when raised from a test. This didn't actually break the test, but it threw me off for a little bit. I had done something to break this test, so when I went to look at why, there was an exception logged because of this mistake, which wasn't related to what the real problem was. To avoid this unnecessarily making it into the test log, pass some stuff so that building the exception message succeeds. Change-Id: I4460f4844c16877ef78b363842518b0ba96eb9aa --- nova/tests/test_xenapi.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/nova/tests/test_xenapi.py b/nova/tests/test_xenapi.py index b139f2d11..9ed066a93 100644 --- a/nova/tests/test_xenapi.py +++ b/nova/tests/test_xenapi.py @@ -2334,7 +2334,8 @@ class XenAPIAggregateTestCase(stubs.XenAPITestBase): def test_add_aggregate_host_raise_err(self): """Ensure the undo operation works correctly on add.""" def fake_driver_add_to_aggregate(context, aggregate, host, **_ignore): - raise exception.AggregateError + raise exception.AggregateError( + aggregate_id='', action='', reason='') self.stubs.Set(self.compute.driver, "add_to_aggregate", fake_driver_add_to_aggregate) metadata = {pool_states.POOL_FLAG: "XenAPI", -- cgit