diff options
author | Theo Chatzimichos <tampakrap@gentoo.org> | 2013-08-25 22:39:07 +0200 |
---|---|---|
committer | Theo Chatzimichos <tampakrap@gentoo.org> | 2013-08-28 12:55:43 +0200 |
commit | cd036ce2ef18fe4e8c81a2f58efd311aadac9ee4 (patch) | |
tree | 9f22f55214ff1368b764f772e385626554942df8 | |
parent | Merge pull request #80 from tampakrap/pep8 (diff) | |
download | identity.gentoo.org-cd036ce2ef18fe4e8c81a2f58efd311aadac9ee4.tar.gz identity.gentoo.org-cd036ce2ef18fe4e8c81a2f58efd311aadac9ee4.tar.bz2 identity.gentoo.org-cd036ce2ef18fe4e8c81a2f58efd311aadac9ee4.zip |
Add tests for ldapuser and get_bound_ldapuser
-rw-r--r-- | okupy/accounts/models.py | 3 | ||||
-rw-r--r-- | okupy/tests/unit/test_ldapuser.py | 89 |
2 files changed, 89 insertions, 3 deletions
diff --git a/okupy/accounts/models.py b/okupy/accounts/models.py index b56c20f..277f470 100644 --- a/okupy/accounts/models.py +++ b/okupy/accounts/models.py @@ -83,9 +83,6 @@ class LDAPUser(ldapdb.models.Model): is_infra = ACLField(db_column='gentooACL') is_retired = ACLField(db_column='gentooACL') - def __str__(self): - return self.username - def __unicode__(self): return self.username diff --git a/okupy/tests/unit/test_ldapuser.py b/okupy/tests/unit/test_ldapuser.py new file mode 100644 index 0000000..f793009 --- /dev/null +++ b/okupy/tests/unit/test_ldapuser.py @@ -0,0 +1,89 @@ +# vim:fileencoding=utf8:et:ts=4:sts=4:sw=4:ft=python + +from django.conf import settings +from django.test import TestCase + +from base64 import b64encode +from Crypto import Random +from mockldap import MockLdap +from passlib.hash import ldap_md5_crypt + +from okupy import OkupyError +from okupy.accounts.models import LDAPUser +from okupy.common.ldap_helpers import get_bound_ldapuser +from okupy.common.test_helpers import ldap_users, set_request +from okupy.crypto.ciphers import cipher +from okupy.tests import vars + +import ldap + + +class LDAPUserUnitTests(TestCase): + @classmethod + def setUpClass(cls): + cls.mockldap = MockLdap(vars.DIRECTORY) + + def setUp(self): + self.mockldap.start() + self.ldapobject = self.mockldap[settings.AUTH_LDAP_SERVER_URI] + + def tearDown(self): + self.mockldap.stop() + + def test_return_unicode_username(self): + alice = LDAPUser.objects.get(username='alice') + self.assertEqual(alice.__unicode__(), u'alice') + self.assertTrue(isinstance(alice.__unicode__(), unicode)) + + def test_get_bound_ldapuser_from_request(self): + secondary_password = Random.get_random_bytes(48) + secondary_password_crypt = ldap_md5_crypt.encrypt(b64encode( + secondary_password)) + self.ldapobject.directory[ldap_users('alice')[0]][ + 'userPassword'].append(secondary_password_crypt) + request = set_request('/', user=vars.USER_ALICE) + request.session['secondary_password'] = cipher.encrypt( + secondary_password) + user = get_bound_ldapuser(request) + self.assertEqual(user.username, vars.USER_ALICE.username) + + def test_get_bound_ldapuser_bind_as_is_properly_set_from_request(self): + secondary_password = Random.get_random_bytes(48) + secondary_password_crypt = ldap_md5_crypt.encrypt(b64encode( + secondary_password)) + self.ldapobject.directory[ldap_users('alice')[0]][ + 'userPassword'].append(secondary_password_crypt) + request = set_request('/', user=vars.USER_ALICE) + request.session['secondary_password'] = cipher.encrypt( + secondary_password) + get_bound_ldapuser(request) + self.assertEqual(settings.DATABASES['ldap_alice']['PASSWORD'], + b64encode(secondary_password)) + + def test_get_bound_ldapuser_bind_as_is_properly_set_from_password(self): + request = set_request('/', user=vars.USER_ALICE) + get_bound_ldapuser(request, password='ldaptest') + self.assertTrue(ldap_md5_crypt.verify(settings.DATABASES['ldap_alice'][ + 'PASSWORD'], ldap_users('alice')[1]['userPassword'][0])) + + def test_get_bound_ldapuser_password_set(self): + request = set_request('/', user=vars.USER_ALICE) + user = get_bound_ldapuser(request, password='ldaptest') + self.assertEqual(user.username, vars.USER_ALICE.username) + + def test_get_bound_ldapuser_no_password_available(self): + request = set_request('/', user=vars.USER_ALICE) + self.assertRaises(OkupyError, get_bound_ldapuser, request) + + def test_get_bound_ldapuser_invalid_secondary_password(self): + secondary_password = Random.get_random_bytes(48) + request = set_request('/', user=vars.USER_ALICE) + request.session['secondary_password'] = cipher.encrypt( + secondary_password) + self.assertRaises(ldap.INVALID_CREDENTIALS, get_bound_ldapuser, + request) + + def test_get_bound_ldapuser_invalid_given_password(self): + request = set_request('/', user=vars.USER_ALICE) + self.assertRaises(ldap.INVALID_CREDENTIALS, get_bound_ldapuser, + request, 'test') |