
We assume the keystoneauth1.exceptions.HttpError.response has a field named `description`, however, it doesn't.[1] We should use `details` instead. [2] [1]. https://github.com/openstack/keystoneauth/blob/2.6.0/keystoneauth1/exceptions/http.py#L65-L85 [2]. https://github.com/openstack/keystoneauth/blob/2.6.0/keystoneauth1/exceptions/http.py#L391 Change-Id: I77846c6cafc0f73a1c6cf20a17fe44ba1284eef1 Closes-Bug: #1577290
40 lines
1.4 KiB
Python
40 lines
1.4 KiB
Python
# Copyright 2016 Hewlett Packard Enterprise Development Company, L.P.
|
|
#
|
|
# Licensed under the Apache License, Version 2.0 (the "License"); you may
|
|
# not use this file except in compliance with the License. You may obtain
|
|
# a copy of the License at
|
|
#
|
|
# http://www.apache.org/licenses/LICENSE-2.0
|
|
#
|
|
# Unless required by applicable law or agreed to in writing, software
|
|
# distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
|
|
# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
|
|
# License for the specific language governing permissions and limitations
|
|
# under the License.
|
|
|
|
import sys
|
|
|
|
from keystoneauth1 import exceptions
|
|
import mock
|
|
import six
|
|
import testtools
|
|
|
|
from aodhclient import shell
|
|
|
|
|
|
class CliTest(testtools.TestCase):
|
|
|
|
@mock.patch('sys.stderr', six.StringIO())
|
|
def test_cli_http_error_with_details(self):
|
|
shell.AodhShell().clean_up(
|
|
None, None, exceptions.HttpError('foo', details='bar'))
|
|
stderr_lines = sys.stderr.getvalue().splitlines()
|
|
self.assertEqual(1, len(stderr_lines))
|
|
self.assertEqual('bar', stderr_lines[0])
|
|
|
|
@mock.patch('sys.stderr', six.StringIO())
|
|
def test_cli_http_error_without_details(self):
|
|
shell.AodhShell().clean_up(None, None, exceptions.HttpError('foo'))
|
|
stderr_lines = sys.stderr.getvalue().splitlines()
|
|
self.assertEqual(0, len(stderr_lines))
|