diff options
author | John Vrbanac <john.vrbanac@rackspace.com> | 2015-04-08 00:00:19 -0500 |
---|---|---|
committer | Thierry Carrez <thierry@openstack.org> | 2015-04-18 14:38:48 +0200 |
commit | 0285e859b621c4445fc431286370ad60bebc6b5e (patch) | |
tree | ad55c2c5b4f60177ed64c4699893a5bcbcd1eddc | |
parent | 2919366867af335d59913764a55ca8e95569947d (diff) | |
download | python-barbicanclient-0285e859b621c4445fc431286370ad60bebc6b5e.tar.gz |
Fixing the broken functional tests
Adding creator_id for Secret and Order types
Adding sub_status and sub_status_message for Order types
This is a result of these attributes being added to the API.
This change isn't aimed to support these fields, but rather just
accepts the data so that it won't bork the tests. Actual support
for using these fields should be addressed in a follow-up patch.
Fixes bug 1441848
(cherry picked from commit f0fe464dd3b01db6effa302da710f1024408dbd7)
Change-Id: I2340b7dd1dae3806418fa99bdaad73860dda5fbc
-rw-r--r-- | barbicanclient/orders.py | 19 | ||||
-rw-r--r-- | barbicanclient/secrets.py | 10 | ||||
-rwxr-xr-x | functionaltests/post_test_hook.sh | 3 |
3 files changed, 23 insertions, 9 deletions
diff --git a/barbicanclient/orders.py b/barbicanclient/orders.py index 8cfde36..25aa4d1 100644 --- a/barbicanclient/orders.py +++ b/barbicanclient/orders.py @@ -89,12 +89,16 @@ class Order(object): def __init__(self, api, type, status=None, created=None, updated=None, meta=None, order_ref=None, error_status_code=None, - error_reason=None): + error_reason=None, sub_status=None, sub_status_message=None, + creator_id=None): super(Order, self).__init__() self._api = api self._type = type self._status = status + self._sub_status = sub_status + self._sub_status_message = sub_status_message + self._creator_id = creator_id if created: self._created = parse_isotime(created) @@ -218,7 +222,8 @@ class KeyOrder(Order, KeyOrderFormatter): def __init__(self, api, name=None, algorithm=None, bit_length=None, mode=None, expiration=None, payload_content_type=None, status=None, created=None, updated=None, order_ref=None, - secret_ref=None, error_status_code=None, error_reason=None): + secret_ref=None, error_status_code=None, error_reason=None, + sub_status=None, sub_status_message=None, creator_id=None): super(KeyOrder, self).__init__( api, self._type, status=status, created=created, updated=updated, meta={ @@ -226,7 +231,8 @@ class KeyOrder(Order, KeyOrderFormatter): 'expiration': expiration, 'payload_content_type': payload_content_type }, order_ref=order_ref, error_status_code=error_status_code, - error_reason=error_reason) + error_reason=error_reason, sub_status=sub_status, + sub_status_message=sub_status_message, creator_id=creator_id) self._secret_ref = secret_ref if mode: self._meta['mode'] = mode @@ -259,8 +265,8 @@ class AsymmetricOrder(Order, AsymmetricOrderFormatter): def __init__(self, api, name=None, algorithm=None, bit_length=None, pass_phrase=None, expiration=None, payload_content_type=None, status=None, created=None, updated=None, order_ref=None, - container_ref=None, error_status_code=None, - error_reason=None): + container_ref=None, error_status_code=None, error_reason=None, + sub_status=None, sub_status_message=None, creator_id=None): super(AsymmetricOrder, self).__init__( api, self._type, status=status, created=created, updated=updated, meta={ @@ -268,7 +274,8 @@ class AsymmetricOrder(Order, AsymmetricOrderFormatter): 'expiration': expiration, 'payload_content_type': payload_content_type }, order_ref=order_ref, error_status_code=error_status_code, - error_reason=error_reason) + error_reason=error_reason, sub_status=sub_status, + sub_status_message=sub_status_message, creator_id=creator_id) self._container_ref = container_ref if pass_phrase: self._meta['pass_phrase'] = pass_phrase diff --git a/barbicanclient/secrets.py b/barbicanclient/secrets.py index 9db8023..ee989fb 100644 --- a/barbicanclient/secrets.py +++ b/barbicanclient/secrets.py @@ -82,7 +82,8 @@ class Secret(SecretFormatter): bit_length=None, mode=None, payload=None, payload_content_type=None, payload_content_encoding=None, secret_ref=None, created=None, updated=None, - content_types=None, status=None, secret_type=None): + content_types=None, status=None, secret_type=None, + creator_id=None): """ Secret objects should not be instantiated directly. You should use the `create` or `get` methods of the @@ -102,7 +103,8 @@ class Secret(SecretFormatter): created=created, updated=updated, content_types=content_types, - status=status + status=status, + creator_id=creator_id ) @property @@ -320,7 +322,8 @@ class Secret(SecretFormatter): bit_length=None, mode=None, payload=None, payload_content_type=None, payload_content_encoding=None, created=None, - updated=None, content_types=None, status=None): + updated=None, content_types=None, status=None, + creator_id=None): self._name = name self._algorithm = algorithm self._bit_length = bit_length @@ -328,6 +331,7 @@ class Secret(SecretFormatter): self._payload = payload self._payload_content_encoding = payload_content_encoding self._expiration = expiration + self._creator_id = creator_id if self._expiration: self._expiration = parse_isotime(self._expiration) if self._secret_ref: diff --git a/functionaltests/post_test_hook.sh b/functionaltests/post_test_hook.sh index 2ab490a..d150635 100755 --- a/functionaltests/post_test_hook.sh +++ b/functionaltests/post_test_hook.sh @@ -17,5 +17,8 @@ # Install packages from test-requirements.txt sudo pip install -r /opt/stack/new/python-barbicanclient/test-requirements.txt +# Make sure we're running the trunk client before we run the tests +sudo pip install -U /opt/stack/new/python-barbicanclient + cd /opt/stack/new/python-barbicanclient/functionaltests sudo ./run_tests.sh |