summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJohn Vrbanac <john.vrbanac@rackspace.com>2015-04-08 00:00:19 -0500
committerThierry Carrez <thierry@openstack.org>2015-04-18 14:38:48 +0200
commit0285e859b621c4445fc431286370ad60bebc6b5e (patch)
treead55c2c5b4f60177ed64c4699893a5bcbcd1eddc
parent2919366867af335d59913764a55ca8e95569947d (diff)
downloadpython-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.py19
-rw-r--r--barbicanclient/secrets.py10
-rwxr-xr-xfunctionaltests/post_test_hook.sh3
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