x
/
mogan
Code Issues Proposed changes

Merge "add test for get_one() when server_uuid not found"

This commit is contained in:
Zuul 2018-12-18 09:08:10 +00:00 committed by Gerrit Code Review
parent b23eec0419 03eba771f4
commit 54a5145b4a
1 changed files with 16 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
mogan/tests/unit/api/v1/test_server.py
View File

@ -140,6 +140,14 @@ class TestServerAuthorization(v1_test.APITestV1):
headers=headers)
self.assertNotIn('node', resp)
def test_server_get_one_by_owner_with_wrong_uuid(self):
# not admin but the owner
self.context.tenant = self.server1.project_id
headers = self.gen_headers(self.context, roles="no-admin")
resp = self.get_json('/servers/%s' % uuidutils.generate_uuid(),
expect_errors=True, headers=headers)
self.assertEqual(resp.status_code, 404)
def test_server_get_one_by_admin(self):
# when the evil tenant is admin, he can do everything.
self.context.tenant = self.evil_project
@ -148,6 +156,14 @@ class TestServerAuthorization(v1_test.APITestV1):
headers=headers)
self.assertIn('node', resp)
def test_server_get_one_by_admin_with_wrong_uuid(self):
# when the evil tenant is admin, he can do everything.
self.context.tenant = self.evil_project
headers = self.gen_headers(self.context, roles="admin")
resp = self.get_json('/servers/%s' % uuidutils.generate_uuid(),
expect_errors=True, headers=headers)
self.assertEqual(resp.status_code, 404)
def test_server_get_one_unauthorized(self):
# not admin and not the owner
self.context.tenant = self.evil_project