From 6b99a54cbe02a1859d97475ea6681df2b266b6e1 Mon Sep 17 00:00:00 2001 From: Zane Bitter Date: Fri, 16 Feb 2018 13:19:22 -0500 Subject: [PATCH] Unit tests: Fix broken Monasca client test The context is held by a weak reference in the ClientPlugin, so don't let it go out of scope in the caller. This test was added by 36e53d3c2b973c45a4dc7af7af39b0110437de48. Change-Id: I6a313970f1c0604e94d008eb12aeaa94da1d25c4 Closes-Bug: #1750024 --- heat/tests/clients/test_monasca_client.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/heat/tests/clients/test_monasca_client.py b/heat/tests/clients/test_monasca_client.py index e9ddb26eb9..adc4cec061 100644 --- a/heat/tests/clients/test_monasca_client.py +++ b/heat/tests/clients/test_monasca_client.py @@ -51,8 +51,8 @@ class MonascaClientPluginTest(common.HeatTestCase): @mock.patch.object(monascaclient.client, '_session') def test_client_uses_session(self, mock_session): - monasca_client = client_plugin.MonascaClientPlugin( - context=mock.MagicMock()) + context = mock.MagicMock() + monasca_client = client_plugin.MonascaClientPlugin(context=context) self.assertIsNotNone(monasca_client._create())