Add a skip for bug #1334368
Trust checking for 204 response is incorrect to support httpd based deployment because mod_wsgi could translate a HEAD call to GET. According to HTTP spec HEAD needs to respond with the same response as GET. This disables the trust check for HTTP 204 for changeId: I13ce159cbe9739d4bf5d321fc4bd069245f32734 Once the changes to Icehouse and Juno (master) are accepted, this can be modified to look for HTTP 200 and re-enabled. Change-Id: I5e7d7dab2fc1432888bf8c691cae9f2109ac2fec Partial-Bug: #1334368
This commit is contained in:
parent
e3e9da70c6
commit
59a9d7a8d4
|
@ -155,6 +155,7 @@ class BaseTrustsV3Test(base.BaseIdentityV3AdminTest):
|
|||
self.assertNotIn('v3/roles/%s' % self.not_delegated_role_id,
|
||||
role['links']['self'])
|
||||
|
||||
@test.skip_because(bug='1334368')
|
||||
def check_trust_roles(self):
|
||||
# Check we find the delegated role
|
||||
resp, roles_get = self.trustor_client.get_trust_roles(
|
||||
|
@ -170,7 +171,7 @@ class BaseTrustsV3Test(base.BaseIdentityV3AdminTest):
|
|||
|
||||
resp, role_get = self.trustor_client.check_trust_role(
|
||||
self.trust_id, self.delegated_role_id)
|
||||
self.assertEqual('204', resp['status'])
|
||||
self.assertEqual('200', resp['status'])
|
||||
|
||||
# And that we don't find not_delegated_role
|
||||
self.assertRaises(exceptions.NotFound,
|
||||
|
|
Loading…
Reference in New Issue