diff options
| author | ZhiQiang Fan <zhiqiang.fan@huawei.com> | 2014-05-27 12:26:40 +0800 |
|---|---|---|
| committer | ZhiQiang Fan <zhiqiang.fan@huawei.com> | 2014-05-27 12:26:40 +0800 |
| commit | 7390d78d95b52a85c6158c34e02189527dc212c0 (patch) | |
| tree | d9f8defe983b61d6ee7a12d332e868bc9b1a4eb9 | |
| parent | cb11c9bb3dd9781e6b3066c0db04667c8175db31 (diff) | |
| download | python-ceilometerclient-7390d78d95b52a85c6158c34e02189527dc212c0.tar.gz | |
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
| -rw-r--r-- | ceilometerclient/tests/test_shell.py | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/ceilometerclient/tests/test_shell.py b/ceilometerclient/tests/test_shell.py index 82329b4..cf52840 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'] |