Merge "Use six.python_2_unicode_compatible for NeutronException.__str__"

neutronclient/common/exceptions.py

@@ -13,6 +13,8 @@
 #    License for the specific language governing permissions and limitations
 #    under the License.
 
+import six
+
 from neutronclient._i18n import _
 
 """
@@ -29,6 +31,7 @@ Exceptions are classified into three categories:
 """
 
 
+@six.python_2_unicode_compatible
 class NeutronException(Exception):
     """Base Neutron Exception.
 

neutronclient/tests/unit/test_exceptions.py

@@ -0,0 +1,36 @@
+# All Rights Reserved.
+#
+#    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 fixtures
+import testtools
+
+from neutronclient.common import exceptions
+from neutronclient.i18n import _
+
+
+class TestExceptions(testtools.TestCase):
+
+    def test_exception_print_with_unicode(self):
+        class TestException(exceptions.NeutronException):
+            message = _('Exception with %(reason)s')
+
+        multibyte_unicode_string = u'\uff21\uff22\uff23'
+        e = TestException(reason=multibyte_unicode_string)
+
+        fixture = fixtures.StringStream('stdout')
+        self.useFixture(fixture)
+        with fixtures.MonkeyPatch('sys.stdout', fixture.stream):
+            print(e)
+        self.assertEqual('Exception with %s' % multibyte_unicode_string,
+                         fixture.getDetails().get('stdout').as_text())