From 7390d78d95b52a85c6158c34e02189527dc212c0 Mon Sep 17 00:00:00 2001 From: ZhiQiang Fan Date: Tue, 27 May 2014 12:26:40 +0800 Subject: [PATCH] Avoid unnecessary stderr message when run test Currently, if we run `tox -epy27`, we will get an useless stderr message 'HTTPUnauthorized (HTTP 401)', which is triggered on purpose by a test. It should be mocked because it provides no such useful message. Change-Id: I009310257741a34fe966d6d745de3c6cf58e0feb Closes-Bug: #1323480 --- ceilometerclient/tests/test_shell.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/ceilometerclient/tests/test_shell.py b/ceilometerclient/tests/test_shell.py index 82329b43..cf528405 100644 --- a/ceilometerclient/tests/test_shell.py +++ b/ceilometerclient/tests/test_shell.py @@ -109,8 +109,9 @@ class ShellTest(utils.BaseTestCase): args = ['-d', 'event-list'] self.assertRaises(exc.CommandError, ceilometer_shell.main, args) + @mock.patch('sys.stderr') @mock.patch.object(ksclient, 'Client') - def test_no_debug_switch_no_raises_errors(self, mock_ksclient): + def test_no_debug_switch_no_raises_errors(self, mock_ksclient, __): mock_ksclient.side_effect = exc.HTTPUnauthorized("FAIL") self.make_env() args = ['event-list']