diff options
| author | Roman Bogorodskiy <bogorodskiy@gmail.com> | 2016-09-22 02:06:53 -0400 |
|---|---|---|
| committer | Roman Bogorodskiy <bogorodskiy@gmail.com> | 2016-09-24 21:18:22 +0300 |
| commit | dc9e4491779c0347f4eb770de03827b967541bf3 (patch) | |
| tree | 35bc5eec3696c48f6657e6e7c822a1b54fd57d38 /keystoneclient/tests | |
| parent | 659ee35babe95cea0bf43e2330681d27608956af (diff) | |
| download | python-keystoneclient-dc9e4491779c0347f4eb770de03827b967541bf3.tar.gz | |
Fix non-ascii attributes
It's possible to set and get custom attributes through the API, and
it's also possible to use any Unicode strings, not only ASCII range.
It works perfectly fine when accessing the API directly using
e.g. curl.
However, keystoneclient stumbles on the non-ascii keys because
it tries to set this as an attribute for the resource class and fails
because Python 2.7 does not support non-ascii identifiers:
https://docs.python.org/2.7/reference/lexical_analysis.html#identifiers
So change the logic to skip setting non-ascii attributes; they are still
available in the dict representation.
Closes-Bug: #1626403
Change-Id: I267188cdb1d303e3d0fb6bd3229b606f4fe9b2d8
Diffstat (limited to 'keystoneclient/tests')
| -rw-r--r-- | keystoneclient/tests/unit/test_base.py | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/keystoneclient/tests/unit/test_base.py b/keystoneclient/tests/unit/test_base.py index 9814d3d..f6ca651 100644 --- a/keystoneclient/tests/unit/test_base.py +++ b/keystoneclient/tests/unit/test_base.py @@ -1,3 +1,4 @@ +# -*- coding: utf-8 -*- # 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 @@ -96,6 +97,15 @@ class BaseTest(utils.TestCase): r = HumanReadable(None, {"name": "1 of !"}) self.assertEqual(r.human_id, "1-of") + def test_non_ascii_attr(self): + r_dict = {"name": "foobar", + u"тест": "1234", + u"тест2": u"привет мир"} + + r = base.Resource(None, r_dict) + self.assertEqual(r.name, "foobar") + self.assertEqual(r.to_dict(), r_dict) + class ManagerTest(utils.TestCase): body = {"hello": {"hi": 1}} |
