diff options
| author | termie <github@anarkystic.com> | 2011-11-14 10:29:56 -0800 |
|---|---|---|
| committer | termie <github@anarkystic.com> | 2011-11-14 10:29:56 -0800 |
| commit | d0009db73564aee8eb8cd34df21457e7d3d8c851 (patch) | |
| tree | d7df0a7179cc4591c21eb12cdc4786afd4380e32 | |
| parent | 8ff5606b3d63a6ab31e69f5605b1fbe4ce59dda4 (diff) | |
| download | keystone-d0009db73564aee8eb8cd34df21457e7d3d8c851.tar.gz keystone-d0009db73564aee8eb8cd34df21457e7d3d8c851.tar.xz keystone-d0009db73564aee8eb8cd34df21457e7d3d8c851.zip | |
add crud tests
| -rw-r--r-- | keystonelight/backends/kvs.py | 4 | ||||
| -rw-r--r-- | keystonelight/service.py | 1 | ||||
| -rw-r--r-- | tests/test_identity_api.py | 5 |
3 files changed, 8 insertions, 2 deletions
diff --git a/keystonelight/backends/kvs.py b/keystonelight/backends/kvs.py index e7fa369f..f71f7bbd 100644 --- a/keystonelight/backends/kvs.py +++ b/keystonelight/backends/kvs.py @@ -56,6 +56,10 @@ class KvsIdentity(object): self.db.set('user-%s' % id, user) return user + def delete_user(self, id): + self.db.delete('user-%s' % id) + return None + def create_tenant(self, id, tenant): self.db.set('tenant-%s' % id, tenant) self.db.set('tenant_name-%s' % tenant['name'], tenant) diff --git a/keystonelight/service.py b/keystonelight/service.py index 758d87ba..a089e94f 100644 --- a/keystonelight/service.py +++ b/keystonelight/service.py @@ -137,7 +137,6 @@ class IdentityController(BaseApplication): return self.identity_api.delete_user(context, user_id=user_id) - class Router(wsgi.Router): def __init__(self, options): self.options = options diff --git a/tests/test_identity_api.py b/tests/test_identity_api.py index 8700b7bc..ff7291d8 100644 --- a/tests/test_identity_api.py +++ b/tests/test_identity_api.py @@ -74,7 +74,7 @@ class IdentityApi(test.TestCase): data = json.loads(resp.body) self.assertDictEquals(self.tenant_bar, data[0]) - def test_create_user(self): + def test_crud_user(self): token_id = self.options['admin_token'] c = client.TestClient(self.app, token=token_id) user_ref = models.User() @@ -91,3 +91,6 @@ class IdentityApi(test.TestCase): self.assertEquals(del_resp.body, '') delget_resp = c.get_user(user_id=data['id']) + self.assertEquals(delget_resp.body, '') + # TODO(termie): we should probably return not founds instead of None + #self.assertEquals(delget_resp.status, '404 Not Found') |