From 60dd53504529a26c3fa47593b3aac1fccc7ed5d2 Mon Sep 17 00:00:00 2001 From: Mark McClain Date: Tue, 28 Mar 2017 15:47:00 -0400 Subject: [PATCH] Capture to ensure CVX error is logged Change-Id: I93dceda0eb04f3af64b09d0cfa00383acc5241df --- .../tests/unit/ml2/test_arista_mechanism_driver.py | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/networking_arista/tests/unit/ml2/test_arista_mechanism_driver.py b/networking_arista/tests/unit/ml2/test_arista_mechanism_driver.py index aab523d..3551dee 100644 --- a/networking_arista/tests/unit/ml2/test_arista_mechanism_driver.py +++ b/networking_arista/tests/unit/ml2/test_arista_mechanism_driver.py @@ -1558,9 +1558,12 @@ class NegativeRPCWrapperTestCase(base.BaseTestCase): ndb = db_lib.NeutronNets() drv = arista_ml2.AristaRPCWrapperEapi(ndb) - drv._server = mock.MagicMock() - drv._server.runCmds.side_effect = Exception('server error') - self.assertRaises(arista_exc.AristaRpcError, drv.get_tenants) + drv._send_api_request = mock.MagicMock( + side_effect=Exception('server error') + ) + with mock.patch.object(arista_ml2.LOG, 'error') as log_err: + self.assertRaises(arista_exc.AristaRpcError, drv.get_tenants) + log_err.assert_called_once_with(mock.ANY) class RealNetStorageAristaDriverTestCase(testlib_api.SqlTestCase):