From 3870b8a763eb473abde37bd44ed3269919408094 Mon Sep 17 00:00:00 2001 From: Samuel de Medeiros Queiroz Date: Wed, 3 Aug 2016 09:46:58 -0300 Subject: [PATCH] Correct test_implied_roles That test is not supposed to test the count of roles. It is about role inference rules. This change removes the pure role checks, as it is not the main intent of the test. This also fixes a related bug where this test is failing intermittently. See the bug description for more details. Change-Id: I8ea1b6d8f344296486427a1f9ebc31c58314501e Closes-Bug: 1609398 --- keystoneclient/tests/functional/v3/test_implied_roles.py | 5 ----- 1 file changed, 5 deletions(-) diff --git a/keystoneclient/tests/functional/v3/test_implied_roles.py b/keystoneclient/tests/functional/v3/test_implied_roles.py index a514652d..b2f743c7 100644 --- a/keystoneclient/tests/functional/v3/test_implied_roles.py +++ b/keystoneclient/tests/functional/v3/test_implied_roles.py @@ -48,15 +48,10 @@ class TestImpliedRoles(base.V3ClientTestCase): super(TestImpliedRoles, self).setUp() def test_implied_roles(self): - - initial_role_count = len(self.client.roles.list()) initial_rule_count = len(self.client.roles.list_role_inferences()) self.create_roles() self.create_rules() - role_count = len(self.client.roles.list()) - self.assertEqual(initial_role_count + len(role_defs), - role_count) rule_count = len(self.client.roles.list_role_inferences()) self.assertEqual(initial_rule_count + len(inference_rules), rule_count)