From 3dc593307de202532e1d438a7c65d661892c979e Mon Sep 17 00:00:00 2001 From: yaroslavmt Date: Fri, 1 Sep 2017 07:21:19 -0400 Subject: [PATCH] Add test GET for member url in the Assignment API Also add test GET and HEAD requests member_url for tests with not exist user. Change-Id: I00b40026bd2893e7976d97482ea06ba2b056935f Closes-Bug: #1703245 --- keystone/tests/unit/test_v3_assignment.py | 23 +++++++++++++++++++++++ 1 file changed, 23 insertions(+) diff --git a/keystone/tests/unit/test_v3_assignment.py b/keystone/tests/unit/test_v3_assignment.py index 8a2f8e50b6..3a965b4cba 100644 --- a/keystone/tests/unit/test_v3_assignment.py +++ b/keystone/tests/unit/test_v3_assignment.py @@ -117,6 +117,7 @@ class AssignmentTestCase(test_v3.RestfulTestCase, self.put(member_url) self.head(member_url) + self.get(member_url, expected_status=http_client.NO_CONTENT) r = self.get(collection_url) self.assertValidRoleListResponse(r, ref=role, resource_url=collection_url, @@ -146,6 +147,8 @@ class AssignmentTestCase(test_v3.RestfulTestCase, 'role_id': self.role_id} self.put(member_url, expected_status=http_client.NOT_FOUND) + self.head(member_url, expected_status=http_client.NOT_FOUND) + self.get(member_url, expected_status=http_client.NOT_FOUND) def test_crud_user_domain_role_grants(self): time = datetime.datetime.utcnow() @@ -160,6 +163,7 @@ class AssignmentTestCase(test_v3.RestfulTestCase, self.put(member_url) self.head(member_url) + self.get(member_url, expected_status=http_client.NO_CONTENT) r = self.get(collection_url) self.assertValidRoleListResponse(r, ref=self.role, resource_url=collection_url) @@ -193,6 +197,8 @@ class AssignmentTestCase(test_v3.RestfulTestCase, 'role_id': self.role_id} self.put(member_url, expected_status=http_client.NOT_FOUND) + self.head(member_url, expected_status=http_client.NOT_FOUND) + self.get(member_url, expected_status=http_client.NOT_FOUND) def test_crud_group_project_role_grants(self): time = datetime.datetime.utcnow() @@ -207,6 +213,7 @@ class AssignmentTestCase(test_v3.RestfulTestCase, self.put(member_url) self.head(member_url) + self.get(member_url, expected_status=http_client.NO_CONTENT) r = self.get(collection_url) self.assertValidRoleListResponse(r, ref=self.role, resource_url=collection_url) @@ -241,6 +248,8 @@ class AssignmentTestCase(test_v3.RestfulTestCase, 'role_id': self.role_id} self.put(member_url, expected_status=http_client.NOT_FOUND) + self.head(member_url, expected_status=http_client.NOT_FOUND) + self.get(member_url, expected_status=http_client.NOT_FOUND) def test_crud_group_domain_role_grants(self): time = datetime.datetime.utcnow() @@ -255,6 +264,7 @@ class AssignmentTestCase(test_v3.RestfulTestCase, self.put(member_url) self.head(member_url) + self.get(member_url, expected_status=http_client.NO_CONTENT) r = self.get(collection_url) self.assertValidRoleListResponse(r, ref=self.role, resource_url=collection_url) @@ -289,6 +299,8 @@ class AssignmentTestCase(test_v3.RestfulTestCase, 'role_id': self.role_id} self.put(member_url, expected_status=http_client.NOT_FOUND) + self.head(member_url, expected_status=http_client.NOT_FOUND) + self.get(member_url, expected_status=http_client.NOT_FOUND) def _create_new_user_and_assign_role_on_project(self): """Create a new user and assign user a role on a project.""" @@ -306,6 +318,7 @@ class AssignmentTestCase(test_v3.RestfulTestCase, self.put(member_url) # Check the user has the role assigned self.head(member_url) + self.get(member_url, expected_status=http_client.NO_CONTENT) return member_url, user_ref def test_delete_user_before_removing_role_assignment_succeeds(self): @@ -394,6 +407,7 @@ class AssignmentTestCase(test_v3.RestfulTestCase, # check the grant that was just created self.head(member_url) + self.get(member_url, expected_status=http_client.NO_CONTENT) resp = self.get(collection_url) self.assertValidRoleListResponse(resp, ref=self.role, resource_url=collection_url) @@ -424,6 +438,7 @@ class AssignmentTestCase(test_v3.RestfulTestCase, # check the grant that was just created self.head(member_url) + self.get(member_url, expected_status=http_client.NO_CONTENT) resp = self.get(collection_url) self.assertValidRoleListResponse(resp, ref=self.role, resource_url=collection_url) @@ -454,6 +469,7 @@ class AssignmentTestCase(test_v3.RestfulTestCase, # check the grant that was just created self.head(member_url) + self.get(member_url, expected_status=http_client.NO_CONTENT) resp = self.get(collection_url) self.assertValidRoleListResponse(resp, ref=self.role, resource_url=collection_url) @@ -484,6 +500,7 @@ class AssignmentTestCase(test_v3.RestfulTestCase, # check the grant that was just created self.head(member_url) + self.get(member_url, expected_status=http_client.NO_CONTENT) resp = self.get(collection_url) self.assertValidRoleListResponse(resp, ref=self.role, resource_url=collection_url) @@ -1517,6 +1534,7 @@ class AssignmentInheritanceTestCase(test_v3.RestfulTestCase, # Check we can read it back self.head(member_url) + self.get(member_url, expected_status=http_client.NO_CONTENT) r = self.get(collection_url) self.assertValidRoleListResponse(r, ref=role_list[0], resource_url=collection_url) @@ -1576,6 +1594,7 @@ class AssignmentInheritanceTestCase(test_v3.RestfulTestCase, self.put(member_url) self.head(member_url) + self.get(member_url, expected_status=http_client.NO_CONTENT) r = self.get(collection_url) self.assertValidRoleListResponse(r, ref=role_list[3], resource_url=collection_url) @@ -1821,6 +1840,7 @@ class AssignmentInheritanceTestCase(test_v3.RestfulTestCase, self.put(member_url) self.head(member_url) + self.get(member_url, expected_status=http_client.NO_CONTENT) r = self.get(collection_url) self.assertValidRoleListResponse(r, ref=role_list[3], resource_url=collection_url) @@ -1904,6 +1924,7 @@ class AssignmentInheritanceTestCase(test_v3.RestfulTestCase, self.put(member_url) self.head(member_url) + self.get(member_url, expected_status=http_client.NO_CONTENT) r = self.get(collection_url) self.assertValidRoleListResponse(r, ref=role_list[3], resource_url=collection_url) @@ -1995,6 +2016,7 @@ class AssignmentInheritanceTestCase(test_v3.RestfulTestCase, self.put(member_url) self.head(member_url) + self.get(member_url, expected_status=http_client.NO_CONTENT) r = self.get(collection_url) self.assertValidRoleListResponse(r, ref=role_list[3], resource_url=collection_url) @@ -2010,6 +2032,7 @@ class AssignmentInheritanceTestCase(test_v3.RestfulTestCase, self.put(member_url) self.head(member_url) + self.get(member_url, expected_status=http_client.NO_CONTENT) r = self.get(collection_url) self.assertValidRoleListResponse(r, ref=role_list[4], resource_url=collection_url)