Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 1 | # vim: tabstop=4 shiftwidth=4 softtabstop=4 |
| 2 | |
| 3 | # Copyright 2012 OpenStack, LLC |
| 4 | # All Rights Reserved. |
| 5 | # |
| 6 | # Licensed under the Apache License, Version 2.0 (the "License"); you may |
| 7 | # not use this file except in compliance with the License. You may obtain |
| 8 | # a copy of the License at |
| 9 | # |
| 10 | # http://www.apache.org/licenses/LICENSE-2.0 |
| 11 | # |
| 12 | # Unless required by applicable law or agreed to in writing, software |
| 13 | # distributed under the License is distributed on an "AS IS" BASIS, WITHOUT |
| 14 | # WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the |
| 15 | # License for the specific language governing permissions and limitations |
| 16 | # under the License. |
| 17 | |
Rohit Karajgi | d2d3f79 | 2012-05-14 10:28:43 -0700 | [diff] [blame] | 18 | import unittest2 as unittest |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 19 | |
Attila Fazekas | 40ec123 | 2013-01-08 11:45:32 +0100 | [diff] [blame^] | 20 | from nose.plugins.attrib import attr |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 21 | from tempest.common.utils.data_utils import rand_name |
Matthew Treinish | a83a16e | 2012-12-07 13:44:02 -0500 | [diff] [blame] | 22 | from tempest import exceptions |
Vincent Hou | 6b8a7b7 | 2012-08-25 01:24:33 +0800 | [diff] [blame] | 23 | from tempest.tests.identity import base |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 24 | |
| 25 | |
Vincent Hou | 6b8a7b7 | 2012-08-25 01:24:33 +0800 | [diff] [blame] | 26 | class TenantsTestBase(object): |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 27 | |
Rohit Karajgi | d2d3f79 | 2012-05-14 10:28:43 -0700 | [diff] [blame] | 28 | def test_list_tenants_by_unauthorized_user(self): |
Sean Dague | 46c4a2b | 2013-01-03 17:54:17 -0500 | [diff] [blame] | 29 | # Non-admin user should not be able to list tenants |
Rohit Karajgi | d2d3f79 | 2012-05-14 10:28:43 -0700 | [diff] [blame] | 30 | self.assertRaises(exceptions.Unauthorized, |
Zhongyue Luo | 79d8d36 | 2012-09-25 13:49:27 +0800 | [diff] [blame] | 31 | self.non_admin_client.list_tenants) |
Rohit Karajgi | d2d3f79 | 2012-05-14 10:28:43 -0700 | [diff] [blame] | 32 | |
| 33 | def test_list_tenant_request_without_token(self): |
Sean Dague | 46c4a2b | 2013-01-03 17:54:17 -0500 | [diff] [blame] | 34 | # Request to list tenants without a valid token should fail |
Rohit Karajgi | d2d3f79 | 2012-05-14 10:28:43 -0700 | [diff] [blame] | 35 | token = self.client.get_auth() |
| 36 | self.client.delete_token(token) |
| 37 | self.assertRaises(exceptions.Unauthorized, self.client.list_tenants) |
| 38 | self.client.clear_auth() |
| 39 | |
Attila Fazekas | 40ec123 | 2013-01-08 11:45:32 +0100 | [diff] [blame^] | 40 | def test_tenant_list_delete(self): |
Sean Dague | 46c4a2b | 2013-01-03 17:54:17 -0500 | [diff] [blame] | 41 | # Create several tenants and delete them |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 42 | tenants = [] |
Attila Fazekas | 40ec123 | 2013-01-08 11:45:32 +0100 | [diff] [blame^] | 43 | for _ in xrange(3): |
| 44 | resp, tenant = self.client.create_tenant(rand_name('tenant-new')) |
| 45 | self.data.tenants.append(tenant) |
| 46 | tenants.append(tenant) |
| 47 | tenant_ids = map(lambda x: x['id'], tenants) |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 48 | resp, body = self.client.list_tenants() |
Attila Fazekas | 40ec123 | 2013-01-08 11:45:32 +0100 | [diff] [blame^] | 49 | self.assertTrue(resp['status'].startswith('2')) |
| 50 | found = [tenant for tenant in body if tenant['id'] in tenant_ids] |
| 51 | self.assertEqual(len(found), len(tenants), 'Tenants not created') |
| 52 | |
| 53 | for tenant in tenants: |
| 54 | resp, body = self.client.delete_tenant(tenant['id']) |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 55 | self.assertTrue(resp['status'].startswith('2')) |
Attila Fazekas | 40ec123 | 2013-01-08 11:45:32 +0100 | [diff] [blame^] | 56 | self.data.tenants.remove(tenant) |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 57 | |
| 58 | resp, body = self.client.list_tenants() |
Attila Fazekas | 40ec123 | 2013-01-08 11:45:32 +0100 | [diff] [blame^] | 59 | found = [tenant for tenant in body if tenant['id'] in tenant_ids] |
| 60 | self.assertFalse(any(found), 'Tenants failed to delete') |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 61 | |
Attila Fazekas | 40ec123 | 2013-01-08 11:45:32 +0100 | [diff] [blame^] | 62 | @attr(type='negative') |
Rohit Karajgi | d2d3f79 | 2012-05-14 10:28:43 -0700 | [diff] [blame] | 63 | def test_tenant_delete_by_unauthorized_user(self): |
Sean Dague | 46c4a2b | 2013-01-03 17:54:17 -0500 | [diff] [blame] | 64 | # Non-admin user should not be able to delete a tenant |
Rohit Karajgi | d2d3f79 | 2012-05-14 10:28:43 -0700 | [diff] [blame] | 65 | tenant_name = rand_name('tenant-') |
| 66 | resp, tenant = self.client.create_tenant(tenant_name) |
Attila Fazekas | 40ec123 | 2013-01-08 11:45:32 +0100 | [diff] [blame^] | 67 | self.data.tenants.append(tenant) |
Rohit Karajgi | d2d3f79 | 2012-05-14 10:28:43 -0700 | [diff] [blame] | 68 | self.assertRaises(exceptions.Unauthorized, |
Zhongyue Luo | 79d8d36 | 2012-09-25 13:49:27 +0800 | [diff] [blame] | 69 | self.non_admin_client.delete_tenant, tenant['id']) |
Rohit Karajgi | d2d3f79 | 2012-05-14 10:28:43 -0700 | [diff] [blame] | 70 | |
Attila Fazekas | 40ec123 | 2013-01-08 11:45:32 +0100 | [diff] [blame^] | 71 | @attr(type='negative') |
Rohit Karajgi | d2d3f79 | 2012-05-14 10:28:43 -0700 | [diff] [blame] | 72 | def test_tenant_delete_request_without_token(self): |
Sean Dague | 46c4a2b | 2013-01-03 17:54:17 -0500 | [diff] [blame] | 73 | # Request to delete a tenant without a valid token should fail |
Rohit Karajgi | d2d3f79 | 2012-05-14 10:28:43 -0700 | [diff] [blame] | 74 | tenant_name = rand_name('tenant-') |
| 75 | resp, tenant = self.client.create_tenant(tenant_name) |
Attila Fazekas | 40ec123 | 2013-01-08 11:45:32 +0100 | [diff] [blame^] | 76 | self.data.tenants.append(tenant) |
Rohit Karajgi | d2d3f79 | 2012-05-14 10:28:43 -0700 | [diff] [blame] | 77 | token = self.client.get_auth() |
| 78 | self.client.delete_token(token) |
| 79 | self.assertRaises(exceptions.Unauthorized, self.client.delete_tenant, |
Zhongyue Luo | 79d8d36 | 2012-09-25 13:49:27 +0800 | [diff] [blame] | 80 | tenant['id']) |
Rohit Karajgi | d2d3f79 | 2012-05-14 10:28:43 -0700 | [diff] [blame] | 81 | self.client.clear_auth() |
| 82 | |
Attila Fazekas | 40ec123 | 2013-01-08 11:45:32 +0100 | [diff] [blame^] | 83 | @attr(type='negative') |
Rohit Karajgi | d2d3f79 | 2012-05-14 10:28:43 -0700 | [diff] [blame] | 84 | def test_delete_non_existent_tenant(self): |
Sean Dague | 46c4a2b | 2013-01-03 17:54:17 -0500 | [diff] [blame] | 85 | # Attempt to delete a non existent tenant should fail |
Rohit Karajgi | d2d3f79 | 2012-05-14 10:28:43 -0700 | [diff] [blame] | 86 | self.assertRaises(exceptions.NotFound, self.client.delete_tenant, |
Zhongyue Luo | 79d8d36 | 2012-09-25 13:49:27 +0800 | [diff] [blame] | 87 | 'junk_tenant_123456abc') |
Rohit Karajgi | d2d3f79 | 2012-05-14 10:28:43 -0700 | [diff] [blame] | 88 | |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 89 | def test_tenant_create_with_description(self): |
Sean Dague | 46c4a2b | 2013-01-03 17:54:17 -0500 | [diff] [blame] | 90 | # Create tenant with a description |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 91 | tenant_name = rand_name('tenant-') |
| 92 | tenant_desc = rand_name('desc-') |
| 93 | resp, body = self.client.create_tenant(tenant_name, |
| 94 | description=tenant_desc) |
Attila Fazekas | 40ec123 | 2013-01-08 11:45:32 +0100 | [diff] [blame^] | 95 | tenant = body |
| 96 | self.data.tenants.append(tenant) |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 97 | st1 = resp['status'] |
| 98 | tenant_id = body['id'] |
| 99 | desc1 = body['description'] |
| 100 | self.assertTrue(st1.startswith('2')) |
| 101 | self.assertEqual(desc1, tenant_desc, 'Description should have ' |
Zhongyue Luo | 79d8d36 | 2012-09-25 13:49:27 +0800 | [diff] [blame] | 102 | 'been sent in response for create') |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 103 | resp, body = self.client.get_tenant(tenant_id) |
| 104 | desc2 = body['description'] |
| 105 | self.assertEqual(desc2, tenant_desc, 'Description does not appear' |
Zhongyue Luo | 79d8d36 | 2012-09-25 13:49:27 +0800 | [diff] [blame] | 106 | 'to be set') |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 107 | self.client.delete_tenant(tenant_id) |
Attila Fazekas | 40ec123 | 2013-01-08 11:45:32 +0100 | [diff] [blame^] | 108 | self.data.tenants.remove(tenant) |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 109 | |
| 110 | def test_tenant_create_enabled(self): |
Sean Dague | 46c4a2b | 2013-01-03 17:54:17 -0500 | [diff] [blame] | 111 | # Create a tenant that is enabled |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 112 | tenant_name = rand_name('tenant-') |
| 113 | resp, body = self.client.create_tenant(tenant_name, enabled=True) |
Attila Fazekas | 40ec123 | 2013-01-08 11:45:32 +0100 | [diff] [blame^] | 114 | tenant = body |
| 115 | self.data.tenants.append(tenant) |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 116 | tenant_id = body['id'] |
| 117 | st1 = resp['status'] |
| 118 | en1 = body['enabled'] |
| 119 | self.assertTrue(st1.startswith('2')) |
| 120 | self.assertTrue(en1, 'Enable should be True in response') |
| 121 | resp, body = self.client.get_tenant(tenant_id) |
| 122 | en2 = body['enabled'] |
| 123 | self.assertTrue(en2, 'Enable should be True in lookup') |
| 124 | self.client.delete_tenant(tenant_id) |
Attila Fazekas | 40ec123 | 2013-01-08 11:45:32 +0100 | [diff] [blame^] | 125 | self.data.tenants.remove(tenant) |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 126 | |
| 127 | def test_tenant_create_not_enabled(self): |
Sean Dague | 46c4a2b | 2013-01-03 17:54:17 -0500 | [diff] [blame] | 128 | # Create a tenant that is not enabled |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 129 | tenant_name = rand_name('tenant-') |
| 130 | resp, body = self.client.create_tenant(tenant_name, enabled=False) |
Attila Fazekas | 40ec123 | 2013-01-08 11:45:32 +0100 | [diff] [blame^] | 131 | tenant = body |
| 132 | self.data.tenants.append(tenant) |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 133 | tenant_id = body['id'] |
| 134 | st1 = resp['status'] |
| 135 | en1 = body['enabled'] |
| 136 | self.assertTrue(st1.startswith('2')) |
Vincent Hou | 6b8a7b7 | 2012-08-25 01:24:33 +0800 | [diff] [blame] | 137 | self.assertEqual('false', str(en1).lower(), |
| 138 | 'Enable should be False in response') |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 139 | resp, body = self.client.get_tenant(tenant_id) |
| 140 | en2 = body['enabled'] |
Vincent Hou | 6b8a7b7 | 2012-08-25 01:24:33 +0800 | [diff] [blame] | 141 | self.assertEqual('false', str(en2).lower(), |
| 142 | 'Enable should be False in lookup') |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 143 | self.client.delete_tenant(tenant_id) |
Attila Fazekas | 40ec123 | 2013-01-08 11:45:32 +0100 | [diff] [blame^] | 144 | self.data.tenants.remove(tenant) |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 145 | |
Attila Fazekas | 40ec123 | 2013-01-08 11:45:32 +0100 | [diff] [blame^] | 146 | @attr(type='negative') |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 147 | def test_tenant_create_duplicate(self): |
Sean Dague | 46c4a2b | 2013-01-03 17:54:17 -0500 | [diff] [blame] | 148 | # Tenant names should be unique |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 149 | tenant_name = rand_name('tenant-dup-') |
| 150 | resp, body = self.client.create_tenant(tenant_name) |
Attila Fazekas | 40ec123 | 2013-01-08 11:45:32 +0100 | [diff] [blame^] | 151 | tenant = body |
| 152 | self.data.tenants.append(tenant) |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 153 | tenant1_id = body.get('id') |
| 154 | |
| 155 | try: |
| 156 | resp, body = self.client.create_tenant(tenant_name) |
| 157 | # this should have raised an exception |
| 158 | self.fail('Should not be able to create a duplicate tenant name') |
| 159 | except exceptions.Duplicate: |
| 160 | pass |
Attila Fazekas | 40ec123 | 2013-01-08 11:45:32 +0100 | [diff] [blame^] | 161 | self.client.delete_tenant(tenant1_id) |
| 162 | self.data.tenants.remove(tenant) |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 163 | |
Attila Fazekas | 40ec123 | 2013-01-08 11:45:32 +0100 | [diff] [blame^] | 164 | @attr(type='negative') |
Rohit Karajgi | d2d3f79 | 2012-05-14 10:28:43 -0700 | [diff] [blame] | 165 | def test_create_tenant_by_unauthorized_user(self): |
Sean Dague | 46c4a2b | 2013-01-03 17:54:17 -0500 | [diff] [blame] | 166 | # Non-admin user should not be authorized to create a tenant |
Rohit Karajgi | d2d3f79 | 2012-05-14 10:28:43 -0700 | [diff] [blame] | 167 | tenant_name = rand_name('tenant-') |
| 168 | self.assertRaises(exceptions.Unauthorized, |
Zhongyue Luo | 79d8d36 | 2012-09-25 13:49:27 +0800 | [diff] [blame] | 169 | self.non_admin_client.create_tenant, tenant_name) |
Rohit Karajgi | d2d3f79 | 2012-05-14 10:28:43 -0700 | [diff] [blame] | 170 | |
Attila Fazekas | 40ec123 | 2013-01-08 11:45:32 +0100 | [diff] [blame^] | 171 | @attr(type='negative') |
Rohit Karajgi | d2d3f79 | 2012-05-14 10:28:43 -0700 | [diff] [blame] | 172 | def test_create_tenant_request_without_token(self): |
Sean Dague | 46c4a2b | 2013-01-03 17:54:17 -0500 | [diff] [blame] | 173 | # Create tenant request without a token should not be authorized |
Rohit Karajgi | d2d3f79 | 2012-05-14 10:28:43 -0700 | [diff] [blame] | 174 | tenant_name = rand_name('tenant-') |
| 175 | token = self.client.get_auth() |
| 176 | self.client.delete_token(token) |
| 177 | self.assertRaises(exceptions.Unauthorized, self.client.create_tenant, |
Zhongyue Luo | 79d8d36 | 2012-09-25 13:49:27 +0800 | [diff] [blame] | 178 | tenant_name) |
Rohit Karajgi | d2d3f79 | 2012-05-14 10:28:43 -0700 | [diff] [blame] | 179 | self.client.clear_auth() |
| 180 | |
Attila Fazekas | 40ec123 | 2013-01-08 11:45:32 +0100 | [diff] [blame^] | 181 | @attr(type='negative') |
Rohit Karajgi | d2d3f79 | 2012-05-14 10:28:43 -0700 | [diff] [blame] | 182 | def test_create_tenant_with_empty_name(self): |
Sean Dague | 46c4a2b | 2013-01-03 17:54:17 -0500 | [diff] [blame] | 183 | # Tenant name should not be empty |
Rohit Karajgi | d2d3f79 | 2012-05-14 10:28:43 -0700 | [diff] [blame] | 184 | self.assertRaises(exceptions.BadRequest, self.client.create_tenant, |
| 185 | name='') |
| 186 | |
Rohit Karajgi | d2d3f79 | 2012-05-14 10:28:43 -0700 | [diff] [blame] | 187 | def test_create_tenants_name_length_over_64(self): |
Sean Dague | 46c4a2b | 2013-01-03 17:54:17 -0500 | [diff] [blame] | 188 | # Tenant name length should not be greater than 64 characters |
David Kranz | 28e35c5 | 2012-07-10 10:14:38 -0400 | [diff] [blame] | 189 | tenant_name = 'a' * 65 |
Rohit Karajgi | d2d3f79 | 2012-05-14 10:28:43 -0700 | [diff] [blame] | 190 | self.assertRaises(exceptions.BadRequest, self.client.create_tenant, |
Zhongyue Luo | 79d8d36 | 2012-09-25 13:49:27 +0800 | [diff] [blame] | 191 | tenant_name) |
Rohit Karajgi | d2d3f79 | 2012-05-14 10:28:43 -0700 | [diff] [blame] | 192 | |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 193 | def test_tenant_update_name(self): |
Sean Dague | 46c4a2b | 2013-01-03 17:54:17 -0500 | [diff] [blame] | 194 | # Update name attribute of a tenant |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 195 | t_name1 = rand_name('tenant-') |
| 196 | resp, body = self.client.create_tenant(t_name1) |
Attila Fazekas | 40ec123 | 2013-01-08 11:45:32 +0100 | [diff] [blame^] | 197 | tenant = body |
| 198 | self.data.tenants.append(tenant) |
| 199 | |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 200 | t_id = body['id'] |
| 201 | resp1_name = body['name'] |
| 202 | |
| 203 | t_name2 = rand_name('tenant2-') |
| 204 | resp, body = self.client.update_tenant(t_id, name=t_name2) |
| 205 | st2 = resp['status'] |
| 206 | resp2_name = body['name'] |
| 207 | self.assertTrue(st2.startswith('2')) |
| 208 | self.assertNotEqual(resp1_name, resp2_name) |
| 209 | |
| 210 | resp, body = self.client.get_tenant(t_id) |
| 211 | resp3_name = body['name'] |
| 212 | |
| 213 | self.assertNotEqual(resp1_name, resp3_name) |
| 214 | self.assertEqual(t_name1, resp1_name) |
| 215 | self.assertEqual(resp2_name, resp3_name) |
| 216 | |
| 217 | self.client.delete_tenant(t_id) |
Attila Fazekas | 40ec123 | 2013-01-08 11:45:32 +0100 | [diff] [blame^] | 218 | self.data.tenants.remove(tenant) |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 219 | |
| 220 | def test_tenant_update_desc(self): |
Sean Dague | 46c4a2b | 2013-01-03 17:54:17 -0500 | [diff] [blame] | 221 | # Update description attribute of a tenant |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 222 | t_name = rand_name('tenant-') |
| 223 | t_desc = rand_name('desc-') |
| 224 | resp, body = self.client.create_tenant(t_name, description=t_desc) |
Attila Fazekas | 40ec123 | 2013-01-08 11:45:32 +0100 | [diff] [blame^] | 225 | tenant = body |
| 226 | self.data.tenants.append(tenant) |
| 227 | |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 228 | t_id = body['id'] |
| 229 | resp1_desc = body['description'] |
| 230 | |
| 231 | t_desc2 = rand_name('desc2-') |
| 232 | resp, body = self.client.update_tenant(t_id, description=t_desc2) |
| 233 | st2 = resp['status'] |
| 234 | resp2_desc = body['extra']['description'] |
| 235 | self.assertTrue(st2.startswith('2')) |
| 236 | self.assertNotEqual(resp1_desc, resp2_desc) |
| 237 | |
| 238 | resp, body = self.client.get_tenant(t_id) |
| 239 | resp3_desc = body['description'] |
| 240 | |
| 241 | self.assertNotEqual(resp1_desc, resp3_desc) |
| 242 | self.assertEqual(t_desc, resp1_desc) |
| 243 | self.assertEqual(resp2_desc, resp3_desc) |
| 244 | |
| 245 | self.client.delete_tenant(t_id) |
Attila Fazekas | 40ec123 | 2013-01-08 11:45:32 +0100 | [diff] [blame^] | 246 | self.data.tenants.remove(tenant) |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 247 | |
| 248 | def test_tenant_update_enable(self): |
Sean Dague | 46c4a2b | 2013-01-03 17:54:17 -0500 | [diff] [blame] | 249 | # Update the enabled attribute of a tenant |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 250 | t_name = rand_name('tenant-') |
| 251 | t_en = False |
| 252 | resp, body = self.client.create_tenant(t_name, enabled=t_en) |
Attila Fazekas | 40ec123 | 2013-01-08 11:45:32 +0100 | [diff] [blame^] | 253 | tenant = body |
| 254 | self.data.tenants.append(tenant) |
| 255 | |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 256 | t_id = body['id'] |
| 257 | resp1_en = body['enabled'] |
| 258 | |
| 259 | t_en2 = True |
| 260 | resp, body = self.client.update_tenant(t_id, enabled=t_en2) |
| 261 | st2 = resp['status'] |
| 262 | resp2_en = body['extra']['enabled'] |
| 263 | self.assertTrue(st2.startswith('2')) |
| 264 | self.assertNotEqual(resp1_en, resp2_en) |
| 265 | |
| 266 | resp, body = self.client.get_tenant(t_id) |
| 267 | resp3_en = body['enabled'] |
| 268 | |
| 269 | self.assertNotEqual(resp1_en, resp3_en) |
Vincent Hou | 6b8a7b7 | 2012-08-25 01:24:33 +0800 | [diff] [blame] | 270 | self.assertEqual('false', str(resp1_en).lower()) |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 271 | self.assertEqual(resp2_en, resp3_en) |
| 272 | |
| 273 | self.client.delete_tenant(t_id) |
Attila Fazekas | 40ec123 | 2013-01-08 11:45:32 +0100 | [diff] [blame^] | 274 | self.data.tenants.remove(tenant) |
Vincent Hou | 6b8a7b7 | 2012-08-25 01:24:33 +0800 | [diff] [blame] | 275 | |
| 276 | |
| 277 | class TenantsTestJSON(base.BaseIdentityAdminTestJSON, |
| 278 | TenantsTestBase): |
| 279 | |
| 280 | @classmethod |
| 281 | def setUpClass(cls): |
| 282 | super(TenantsTestJSON, cls).setUpClass() |
Vincent Hou | 6b8a7b7 | 2012-08-25 01:24:33 +0800 | [diff] [blame] | 283 | |
| 284 | |
Zhongyue Luo | e0884a3 | 2012-09-25 17:24:17 +0800 | [diff] [blame] | 285 | class TenantsTestXML(base.BaseIdentityAdminTestXML, TenantsTestBase): |
Vincent Hou | 6b8a7b7 | 2012-08-25 01:24:33 +0800 | [diff] [blame] | 286 | |
| 287 | @classmethod |
| 288 | def setUpClass(cls): |
| 289 | super(TenantsTestXML, cls).setUpClass() |