Open Stack

On Thu, Oct 31, 2013 at 09:40:46PM -0400, Adam Young wrote:
<snip>
> I think it is safe to say that the trusts API is broken in XML.  I
> added the following test:
> 
> diff --git a/keystone/tests/test_v3_auth.py b/keystone/tests/test_v3_auth.py
> index c0e191b..6a0c10c 100644
> --- a/keystone/tests/test_v3_auth.py
> +++ b/keystone/tests/test_v3_auth.py
> @@ -2238,3 +2238,7 @@ class TestTrustAuth(TestAuthInfo):
>          self.get('/OS-TRUST/trusts?trustor_user_id=%s' %
>                   self.user_id, expected_status=401,
>                   token=trust_token)
> +
> +
> +class TestTrustAuthXML(TestTrustAuth):
> +    content_type = 'xml'
> 
> And, when running it, I got:
> 
> 
> Ran 24 tests in 5.832s
> 
> FAILED (SKIP=1, errors=12)
> 
> 
> https://bugs.launchpad.net/keystone/+bug/1246941

Great (well not great that we have a bug, but great that all this effort
going into testing is finding some real bugs! :) )

I notice there's a review associated with that bug, but I can't view it -
if it's draft can you please add me to the reviewers list?

Looking forward to seeing the patch, as you said the unit test examples
should help me complete my Tempest patch.

Also note I raised:

https://bugs.launchpad.net/keystone/+bug/1246831

Which seems somewhat related to this (we get a 500 with the XML encoded
expires_at=None, which results in a zero-length string, so the trust
controller treats it as a valid timestamp instead of ignoring it)

I was planning to send a patch for the latter, but seems like it may
overlap with your XML fixes, so I'll hold off for now.

Thanks!

Steve