From 59a9d7a8d4f9620cdcf783b9ef94a99acbe20873 Mon Sep 17 00:00:00 2001 From: Morgan Fainberg Date: Thu, 3 Jul 2014 09:23:18 -0700 Subject: [PATCH] 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 --- tempest/api/identity/admin/v3/test_trusts.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tempest/api/identity/admin/v3/test_trusts.py b/tempest/api/identity/admin/v3/test_trusts.py index 8e3a7d166c..ac5efa1215 100644 --- a/tempest/api/identity/admin/v3/test_trusts.py +++ b/tempest/api/identity/admin/v3/test_trusts.py @@ -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,