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 | |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 18 | from tempest.common.utils.data_utils import rand_name |
Matthew Treinish | a83a16e | 2012-12-07 13:44:02 -0500 | [diff] [blame] | 19 | from tempest import exceptions |
Chris Yeoh | 01cb279 | 2013-02-09 22:25:37 +1030 | [diff] [blame] | 20 | from tempest.test import attr |
Vincent Hou | 6b8a7b7 | 2012-08-25 01:24:33 +0800 | [diff] [blame] | 21 | from tempest.tests.identity import base |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 22 | |
| 23 | |
Attila Fazekas | 0d0c616 | 2013-02-24 09:14:23 +0100 | [diff] [blame] | 24 | class TenantsTestJSON(base.BaseIdentityAdminTest): |
| 25 | _interface = 'json' |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 26 | |
Giampaolo Lauria | ea29495 | 2013-05-15 08:52:04 -0400 | [diff] [blame^] | 27 | @attr(type='gate') |
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 | |
Giampaolo Lauria | ea29495 | 2013-05-15 08:52:04 -0400 | [diff] [blame^] | 33 | @attr(type='gate') |
Rohit Karajgi | d2d3f79 | 2012-05-14 10:28:43 -0700 | [diff] [blame] | 34 | def test_list_tenant_request_without_token(self): |
Sean Dague | 46c4a2b | 2013-01-03 17:54:17 -0500 | [diff] [blame] | 35 | # Request to list tenants without a valid token should fail |
Rohit Karajgi | d2d3f79 | 2012-05-14 10:28:43 -0700 | [diff] [blame] | 36 | token = self.client.get_auth() |
| 37 | self.client.delete_token(token) |
| 38 | self.assertRaises(exceptions.Unauthorized, self.client.list_tenants) |
| 39 | self.client.clear_auth() |
| 40 | |
Giampaolo Lauria | ea29495 | 2013-05-15 08:52:04 -0400 | [diff] [blame^] | 41 | @attr(type='gate') |
Attila Fazekas | 40ec123 | 2013-01-08 11:45:32 +0100 | [diff] [blame] | 42 | def test_tenant_list_delete(self): |
Sean Dague | 46c4a2b | 2013-01-03 17:54:17 -0500 | [diff] [blame] | 43 | # Create several tenants and delete them |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 44 | tenants = [] |
Attila Fazekas | 40ec123 | 2013-01-08 11:45:32 +0100 | [diff] [blame] | 45 | for _ in xrange(3): |
| 46 | resp, tenant = self.client.create_tenant(rand_name('tenant-new')) |
| 47 | self.data.tenants.append(tenant) |
| 48 | tenants.append(tenant) |
| 49 | tenant_ids = map(lambda x: x['id'], tenants) |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 50 | resp, body = self.client.list_tenants() |
Attila Fazekas | 40ec123 | 2013-01-08 11:45:32 +0100 | [diff] [blame] | 51 | self.assertTrue(resp['status'].startswith('2')) |
| 52 | found = [tenant for tenant in body if tenant['id'] in tenant_ids] |
| 53 | self.assertEqual(len(found), len(tenants), 'Tenants not created') |
| 54 | |
| 55 | for tenant in tenants: |
| 56 | resp, body = self.client.delete_tenant(tenant['id']) |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 57 | self.assertTrue(resp['status'].startswith('2')) |
Attila Fazekas | 40ec123 | 2013-01-08 11:45:32 +0100 | [diff] [blame] | 58 | self.data.tenants.remove(tenant) |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 59 | |
| 60 | resp, body = self.client.list_tenants() |
Attila Fazekas | 40ec123 | 2013-01-08 11:45:32 +0100 | [diff] [blame] | 61 | found = [tenant for tenant in body if tenant['id'] in tenant_ids] |
| 62 | self.assertFalse(any(found), 'Tenants failed to delete') |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 63 | |
Giampaolo Lauria | ea29495 | 2013-05-15 08:52:04 -0400 | [diff] [blame^] | 64 | @attr(type='gate') |
Rohit Karajgi | d2d3f79 | 2012-05-14 10:28:43 -0700 | [diff] [blame] | 65 | def test_tenant_delete_by_unauthorized_user(self): |
Sean Dague | 46c4a2b | 2013-01-03 17:54:17 -0500 | [diff] [blame] | 66 | # Non-admin user should not be able to delete a tenant |
Rohit Karajgi | d2d3f79 | 2012-05-14 10:28:43 -0700 | [diff] [blame] | 67 | tenant_name = rand_name('tenant-') |
| 68 | resp, tenant = self.client.create_tenant(tenant_name) |
Attila Fazekas | 40ec123 | 2013-01-08 11:45:32 +0100 | [diff] [blame] | 69 | self.data.tenants.append(tenant) |
Rohit Karajgi | d2d3f79 | 2012-05-14 10:28:43 -0700 | [diff] [blame] | 70 | self.assertRaises(exceptions.Unauthorized, |
Zhongyue Luo | 79d8d36 | 2012-09-25 13:49:27 +0800 | [diff] [blame] | 71 | self.non_admin_client.delete_tenant, tenant['id']) |
Rohit Karajgi | d2d3f79 | 2012-05-14 10:28:43 -0700 | [diff] [blame] | 72 | |
Giampaolo Lauria | ea29495 | 2013-05-15 08:52:04 -0400 | [diff] [blame^] | 73 | @attr(type='gate') |
Rohit Karajgi | d2d3f79 | 2012-05-14 10:28:43 -0700 | [diff] [blame] | 74 | def test_tenant_delete_request_without_token(self): |
Sean Dague | 46c4a2b | 2013-01-03 17:54:17 -0500 | [diff] [blame] | 75 | # Request to delete a tenant without a valid token should fail |
Rohit Karajgi | d2d3f79 | 2012-05-14 10:28:43 -0700 | [diff] [blame] | 76 | tenant_name = rand_name('tenant-') |
| 77 | resp, tenant = self.client.create_tenant(tenant_name) |
Attila Fazekas | 40ec123 | 2013-01-08 11:45:32 +0100 | [diff] [blame] | 78 | self.data.tenants.append(tenant) |
Rohit Karajgi | d2d3f79 | 2012-05-14 10:28:43 -0700 | [diff] [blame] | 79 | token = self.client.get_auth() |
| 80 | self.client.delete_token(token) |
| 81 | self.assertRaises(exceptions.Unauthorized, self.client.delete_tenant, |
Zhongyue Luo | 79d8d36 | 2012-09-25 13:49:27 +0800 | [diff] [blame] | 82 | tenant['id']) |
Rohit Karajgi | d2d3f79 | 2012-05-14 10:28:43 -0700 | [diff] [blame] | 83 | self.client.clear_auth() |
| 84 | |
Giampaolo Lauria | ea29495 | 2013-05-15 08:52:04 -0400 | [diff] [blame^] | 85 | @attr(type='gate') |
Rohit Karajgi | d2d3f79 | 2012-05-14 10:28:43 -0700 | [diff] [blame] | 86 | def test_delete_non_existent_tenant(self): |
Sean Dague | 46c4a2b | 2013-01-03 17:54:17 -0500 | [diff] [blame] | 87 | # Attempt to delete a non existent tenant should fail |
Rohit Karajgi | d2d3f79 | 2012-05-14 10:28:43 -0700 | [diff] [blame] | 88 | self.assertRaises(exceptions.NotFound, self.client.delete_tenant, |
Zhongyue Luo | 79d8d36 | 2012-09-25 13:49:27 +0800 | [diff] [blame] | 89 | 'junk_tenant_123456abc') |
Rohit Karajgi | d2d3f79 | 2012-05-14 10:28:43 -0700 | [diff] [blame] | 90 | |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 91 | def test_tenant_create_with_description(self): |
Sean Dague | 46c4a2b | 2013-01-03 17:54:17 -0500 | [diff] [blame] | 92 | # Create tenant with a description |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 93 | tenant_name = rand_name('tenant-') |
| 94 | tenant_desc = rand_name('desc-') |
| 95 | resp, body = self.client.create_tenant(tenant_name, |
| 96 | description=tenant_desc) |
Attila Fazekas | 40ec123 | 2013-01-08 11:45:32 +0100 | [diff] [blame] | 97 | tenant = body |
| 98 | self.data.tenants.append(tenant) |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 99 | st1 = resp['status'] |
| 100 | tenant_id = body['id'] |
| 101 | desc1 = body['description'] |
| 102 | self.assertTrue(st1.startswith('2')) |
| 103 | self.assertEqual(desc1, tenant_desc, 'Description should have ' |
Zhongyue Luo | 79d8d36 | 2012-09-25 13:49:27 +0800 | [diff] [blame] | 104 | 'been sent in response for create') |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 105 | resp, body = self.client.get_tenant(tenant_id) |
| 106 | desc2 = body['description'] |
| 107 | self.assertEqual(desc2, tenant_desc, 'Description does not appear' |
Zhongyue Luo | 79d8d36 | 2012-09-25 13:49:27 +0800 | [diff] [blame] | 108 | 'to be set') |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 109 | self.client.delete_tenant(tenant_id) |
Attila Fazekas | 40ec123 | 2013-01-08 11:45:32 +0100 | [diff] [blame] | 110 | self.data.tenants.remove(tenant) |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 111 | |
| 112 | def test_tenant_create_enabled(self): |
Sean Dague | 46c4a2b | 2013-01-03 17:54:17 -0500 | [diff] [blame] | 113 | # Create a tenant that is enabled |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 114 | tenant_name = rand_name('tenant-') |
| 115 | resp, body = self.client.create_tenant(tenant_name, enabled=True) |
Attila Fazekas | 40ec123 | 2013-01-08 11:45:32 +0100 | [diff] [blame] | 116 | tenant = body |
| 117 | self.data.tenants.append(tenant) |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 118 | tenant_id = body['id'] |
| 119 | st1 = resp['status'] |
| 120 | en1 = body['enabled'] |
| 121 | self.assertTrue(st1.startswith('2')) |
| 122 | self.assertTrue(en1, 'Enable should be True in response') |
| 123 | resp, body = self.client.get_tenant(tenant_id) |
| 124 | en2 = body['enabled'] |
| 125 | self.assertTrue(en2, 'Enable should be True in lookup') |
| 126 | self.client.delete_tenant(tenant_id) |
Attila Fazekas | 40ec123 | 2013-01-08 11:45:32 +0100 | [diff] [blame] | 127 | self.data.tenants.remove(tenant) |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 128 | |
| 129 | def test_tenant_create_not_enabled(self): |
Sean Dague | 46c4a2b | 2013-01-03 17:54:17 -0500 | [diff] [blame] | 130 | # Create a tenant that is not enabled |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 131 | tenant_name = rand_name('tenant-') |
| 132 | resp, body = self.client.create_tenant(tenant_name, enabled=False) |
Attila Fazekas | 40ec123 | 2013-01-08 11:45:32 +0100 | [diff] [blame] | 133 | tenant = body |
| 134 | self.data.tenants.append(tenant) |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 135 | tenant_id = body['id'] |
| 136 | st1 = resp['status'] |
| 137 | en1 = body['enabled'] |
| 138 | self.assertTrue(st1.startswith('2')) |
Vincent Hou | 6b8a7b7 | 2012-08-25 01:24:33 +0800 | [diff] [blame] | 139 | self.assertEqual('false', str(en1).lower(), |
| 140 | 'Enable should be False in response') |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 141 | resp, body = self.client.get_tenant(tenant_id) |
| 142 | en2 = body['enabled'] |
Vincent Hou | 6b8a7b7 | 2012-08-25 01:24:33 +0800 | [diff] [blame] | 143 | self.assertEqual('false', str(en2).lower(), |
| 144 | 'Enable should be False in lookup') |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 145 | self.client.delete_tenant(tenant_id) |
Attila Fazekas | 40ec123 | 2013-01-08 11:45:32 +0100 | [diff] [blame] | 146 | self.data.tenants.remove(tenant) |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 147 | |
Giampaolo Lauria | ea29495 | 2013-05-15 08:52:04 -0400 | [diff] [blame^] | 148 | @attr(type='gate') |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 149 | def test_tenant_create_duplicate(self): |
Sean Dague | 46c4a2b | 2013-01-03 17:54:17 -0500 | [diff] [blame] | 150 | # Tenant names should be unique |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 151 | tenant_name = rand_name('tenant-dup-') |
| 152 | resp, body = self.client.create_tenant(tenant_name) |
Attila Fazekas | 40ec123 | 2013-01-08 11:45:32 +0100 | [diff] [blame] | 153 | tenant = body |
| 154 | self.data.tenants.append(tenant) |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 155 | tenant1_id = body.get('id') |
| 156 | |
Chris Yeoh | e04628e | 2013-02-25 17:12:21 +1030 | [diff] [blame] | 157 | self.addCleanup(self.client.delete_tenant, tenant1_id) |
| 158 | self.addCleanup(self.data.tenants.remove, tenant) |
| 159 | self.assertRaises(exceptions.Duplicate, self.client.create_tenant, |
| 160 | tenant_name) |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 161 | |
Giampaolo Lauria | ea29495 | 2013-05-15 08:52:04 -0400 | [diff] [blame^] | 162 | @attr(type='gate') |
Rohit Karajgi | d2d3f79 | 2012-05-14 10:28:43 -0700 | [diff] [blame] | 163 | def test_create_tenant_by_unauthorized_user(self): |
Sean Dague | 46c4a2b | 2013-01-03 17:54:17 -0500 | [diff] [blame] | 164 | # Non-admin user should not be authorized to create a tenant |
Rohit Karajgi | d2d3f79 | 2012-05-14 10:28:43 -0700 | [diff] [blame] | 165 | tenant_name = rand_name('tenant-') |
| 166 | self.assertRaises(exceptions.Unauthorized, |
Zhongyue Luo | 79d8d36 | 2012-09-25 13:49:27 +0800 | [diff] [blame] | 167 | self.non_admin_client.create_tenant, tenant_name) |
Rohit Karajgi | d2d3f79 | 2012-05-14 10:28:43 -0700 | [diff] [blame] | 168 | |
Giampaolo Lauria | ea29495 | 2013-05-15 08:52:04 -0400 | [diff] [blame^] | 169 | @attr(type='gate') |
Rohit Karajgi | d2d3f79 | 2012-05-14 10:28:43 -0700 | [diff] [blame] | 170 | def test_create_tenant_request_without_token(self): |
Sean Dague | 46c4a2b | 2013-01-03 17:54:17 -0500 | [diff] [blame] | 171 | # Create tenant request without a token should not be authorized |
Rohit Karajgi | d2d3f79 | 2012-05-14 10:28:43 -0700 | [diff] [blame] | 172 | tenant_name = rand_name('tenant-') |
| 173 | token = self.client.get_auth() |
| 174 | self.client.delete_token(token) |
| 175 | self.assertRaises(exceptions.Unauthorized, self.client.create_tenant, |
Zhongyue Luo | 79d8d36 | 2012-09-25 13:49:27 +0800 | [diff] [blame] | 176 | tenant_name) |
Rohit Karajgi | d2d3f79 | 2012-05-14 10:28:43 -0700 | [diff] [blame] | 177 | self.client.clear_auth() |
| 178 | |
Giampaolo Lauria | ea29495 | 2013-05-15 08:52:04 -0400 | [diff] [blame^] | 179 | @attr(type='gate') |
Rohit Karajgi | d2d3f79 | 2012-05-14 10:28:43 -0700 | [diff] [blame] | 180 | def test_create_tenant_with_empty_name(self): |
Sean Dague | 46c4a2b | 2013-01-03 17:54:17 -0500 | [diff] [blame] | 181 | # Tenant name should not be empty |
Rohit Karajgi | d2d3f79 | 2012-05-14 10:28:43 -0700 | [diff] [blame] | 182 | self.assertRaises(exceptions.BadRequest, self.client.create_tenant, |
| 183 | name='') |
| 184 | |
Rohit Karajgi | d2d3f79 | 2012-05-14 10:28:43 -0700 | [diff] [blame] | 185 | def test_create_tenants_name_length_over_64(self): |
Sean Dague | 46c4a2b | 2013-01-03 17:54:17 -0500 | [diff] [blame] | 186 | # Tenant name length should not be greater than 64 characters |
David Kranz | 28e35c5 | 2012-07-10 10:14:38 -0400 | [diff] [blame] | 187 | tenant_name = 'a' * 65 |
Rohit Karajgi | d2d3f79 | 2012-05-14 10:28:43 -0700 | [diff] [blame] | 188 | self.assertRaises(exceptions.BadRequest, self.client.create_tenant, |
Zhongyue Luo | 79d8d36 | 2012-09-25 13:49:27 +0800 | [diff] [blame] | 189 | tenant_name) |
Rohit Karajgi | d2d3f79 | 2012-05-14 10:28:43 -0700 | [diff] [blame] | 190 | |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 191 | def test_tenant_update_name(self): |
Sean Dague | 46c4a2b | 2013-01-03 17:54:17 -0500 | [diff] [blame] | 192 | # Update name attribute of a tenant |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 193 | t_name1 = rand_name('tenant-') |
| 194 | resp, body = self.client.create_tenant(t_name1) |
Attila Fazekas | 40ec123 | 2013-01-08 11:45:32 +0100 | [diff] [blame] | 195 | tenant = body |
| 196 | self.data.tenants.append(tenant) |
| 197 | |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 198 | t_id = body['id'] |
| 199 | resp1_name = body['name'] |
| 200 | |
| 201 | t_name2 = rand_name('tenant2-') |
| 202 | resp, body = self.client.update_tenant(t_id, name=t_name2) |
| 203 | st2 = resp['status'] |
| 204 | resp2_name = body['name'] |
| 205 | self.assertTrue(st2.startswith('2')) |
| 206 | self.assertNotEqual(resp1_name, resp2_name) |
| 207 | |
| 208 | resp, body = self.client.get_tenant(t_id) |
| 209 | resp3_name = body['name'] |
| 210 | |
| 211 | self.assertNotEqual(resp1_name, resp3_name) |
| 212 | self.assertEqual(t_name1, resp1_name) |
| 213 | self.assertEqual(resp2_name, resp3_name) |
| 214 | |
| 215 | self.client.delete_tenant(t_id) |
Attila Fazekas | 40ec123 | 2013-01-08 11:45:32 +0100 | [diff] [blame] | 216 | self.data.tenants.remove(tenant) |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 217 | |
| 218 | def test_tenant_update_desc(self): |
Sean Dague | 46c4a2b | 2013-01-03 17:54:17 -0500 | [diff] [blame] | 219 | # Update description attribute of a tenant |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 220 | t_name = rand_name('tenant-') |
| 221 | t_desc = rand_name('desc-') |
| 222 | resp, body = self.client.create_tenant(t_name, description=t_desc) |
Attila Fazekas | 40ec123 | 2013-01-08 11:45:32 +0100 | [diff] [blame] | 223 | tenant = body |
| 224 | self.data.tenants.append(tenant) |
| 225 | |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 226 | t_id = body['id'] |
| 227 | resp1_desc = body['description'] |
| 228 | |
| 229 | t_desc2 = rand_name('desc2-') |
| 230 | resp, body = self.client.update_tenant(t_id, description=t_desc2) |
| 231 | st2 = resp['status'] |
Gordon Chung | ad87360 | 2013-02-18 19:26:27 -0500 | [diff] [blame] | 232 | resp2_desc = body['description'] |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 233 | self.assertTrue(st2.startswith('2')) |
| 234 | self.assertNotEqual(resp1_desc, resp2_desc) |
| 235 | |
| 236 | resp, body = self.client.get_tenant(t_id) |
| 237 | resp3_desc = body['description'] |
| 238 | |
| 239 | self.assertNotEqual(resp1_desc, resp3_desc) |
| 240 | self.assertEqual(t_desc, resp1_desc) |
| 241 | self.assertEqual(resp2_desc, resp3_desc) |
| 242 | |
| 243 | self.client.delete_tenant(t_id) |
Attila Fazekas | 40ec123 | 2013-01-08 11:45:32 +0100 | [diff] [blame] | 244 | self.data.tenants.remove(tenant) |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 245 | |
| 246 | def test_tenant_update_enable(self): |
Sean Dague | 46c4a2b | 2013-01-03 17:54:17 -0500 | [diff] [blame] | 247 | # Update the enabled attribute of a tenant |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 248 | t_name = rand_name('tenant-') |
| 249 | t_en = False |
| 250 | resp, body = self.client.create_tenant(t_name, enabled=t_en) |
Attila Fazekas | 40ec123 | 2013-01-08 11:45:32 +0100 | [diff] [blame] | 251 | tenant = body |
| 252 | self.data.tenants.append(tenant) |
| 253 | |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 254 | t_id = body['id'] |
| 255 | resp1_en = body['enabled'] |
| 256 | |
| 257 | t_en2 = True |
| 258 | resp, body = self.client.update_tenant(t_id, enabled=t_en2) |
| 259 | st2 = resp['status'] |
Gordon Chung | ad87360 | 2013-02-18 19:26:27 -0500 | [diff] [blame] | 260 | resp2_en = body['enabled'] |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 261 | self.assertTrue(st2.startswith('2')) |
| 262 | self.assertNotEqual(resp1_en, resp2_en) |
| 263 | |
| 264 | resp, body = self.client.get_tenant(t_id) |
| 265 | resp3_en = body['enabled'] |
| 266 | |
| 267 | self.assertNotEqual(resp1_en, resp3_en) |
Vincent Hou | 6b8a7b7 | 2012-08-25 01:24:33 +0800 | [diff] [blame] | 268 | self.assertEqual('false', str(resp1_en).lower()) |
chris fattarsi | 9ba7b0e | 2012-05-07 13:55:51 -0700 | [diff] [blame] | 269 | self.assertEqual(resp2_en, resp3_en) |
| 270 | |
| 271 | self.client.delete_tenant(t_id) |
Attila Fazekas | 40ec123 | 2013-01-08 11:45:32 +0100 | [diff] [blame] | 272 | self.data.tenants.remove(tenant) |
Vincent Hou | 6b8a7b7 | 2012-08-25 01:24:33 +0800 | [diff] [blame] | 273 | |
| 274 | |
Attila Fazekas | 0d0c616 | 2013-02-24 09:14:23 +0100 | [diff] [blame] | 275 | class TenantsTestXML(TenantsTestJSON): |
| 276 | _interface = 'xml' |