diff options
author | Rob Crittenden <rcritten@redhat.com> | 2010-12-17 16:57:28 -0500 |
---|---|---|
committer | Simo Sorce <ssorce@redhat.com> | 2010-12-17 18:04:37 -0500 |
commit | 34534a026f39e5c5c139d23ab70db72009789e5b (patch) | |
tree | 7e64d9a94bd2a6c5ac54e54f46e600b539727212 /tests/test_xmlrpc/test_selfservice_plugin.py | |
parent | 7035ffe49ca8456a1efc155c9cb22ec01a881ba2 (diff) | |
download | freeipa-34534a026f39e5c5c139d23ab70db72009789e5b.tar.gz freeipa-34534a026f39e5c5c139d23ab70db72009789e5b.tar.xz freeipa-34534a026f39e5c5c139d23ab70db72009789e5b.zip |
Don't use camel-case LDAP attributes in ACI and don't clear enrolledBy
We keep LDAP attributes lower-case elsewhere in the API we should do the
same with all access controls.
There were two ACIs pointing at the manage_host_keytab permission. This
isn't allowed in general and we have decided separately to not clear out
enrolledBy when a host is unenrolled so dropping it is the obvious thing
to do.
ticket 597
Diffstat (limited to 'tests/test_xmlrpc/test_selfservice_plugin.py')
-rw-r--r-- | tests/test_xmlrpc/test_selfservice_plugin.py | 12 |
1 files changed, 7 insertions, 5 deletions
diff --git a/tests/test_xmlrpc/test_selfservice_plugin.py b/tests/test_xmlrpc/test_selfservice_plugin.py index 897bd0da4..30b5d7644 100644 --- a/tests/test_xmlrpc/test_selfservice_plugin.py +++ b/tests/test_xmlrpc/test_selfservice_plugin.py @@ -68,6 +68,8 @@ class test_selfservice(Declarative): ), + # Note that we add postalCode but expect postalcode. This tests + # the attrs normalizer. dict( desc='Create %r' % selfservice1, command=( @@ -80,7 +82,7 @@ class test_selfservice(Declarative): value=selfservice1, summary=u'Added selfservice "%s"' % selfservice1, result=dict( - attrs=[u'street', u'c', u'l', u'st', u'postalCode'], + attrs=[u'street', u'c', u'l', u'st', u'postalcode'], permissions=[u'write'], selfaci=True, aciname=selfservice1, @@ -108,7 +110,7 @@ class test_selfservice(Declarative): value=selfservice1, summary=None, result={ - 'attrs': [u'street', u'c', u'l', u'st', u'postalCode'], + 'attrs': [u'street', u'c', u'l', u'st', u'postalcode'], 'permissions': [u'write'], 'selfaci': True, 'aciname': selfservice1, @@ -126,7 +128,7 @@ class test_selfservice(Declarative): summary=u'1 selfservice matched', result=[ { - 'attrs': [u'street', u'c', u'l', u'st', u'postalCode'], + 'attrs': [u'street', u'c', u'l', u'st', u'postalcode'], 'permissions': [u'write'], 'selfaci': True, 'aciname': selfservice1, @@ -145,7 +147,7 @@ class test_selfservice(Declarative): value=selfservice1, summary=u'Modified selfservice "%s"' % selfservice1, result=dict( - attrs=[u'street', u'c', u'l', u'st', u'postalCode'], + attrs=[u'street', u'c', u'l', u'st', u'postalcode'], permissions=[u'read'], selfaci=True, aciname=selfservice1, @@ -161,7 +163,7 @@ class test_selfservice(Declarative): value=selfservice1, summary=None, result={ - 'attrs': [u'street', u'c', u'l', u'st', u'postalCode'], + 'attrs': [u'street', u'c', u'l', u'st', u'postalcode'], 'permissions': [u'read'], 'selfaci': True, 'aciname': selfservice1, |