[Freeipa-devel] [PATCH] 208 Fix raw format for ACI commands

[Martin Kosek]

On Fri, 2012-02-03 at 10:45 -0500, Rob Crittenden wrote:
> Rob Crittenden wrote:
> > Martin Kosek wrote:
> >> ACI plugins (permission, selfservice and delegation) were not
> >> prepared to serve ACIs in a raw format, i.e. raw "aci" attribute
> >> taken from LDAP. This patch fixes all these plugins and their
> >> commands to provide provide this format. Few ACI raw format unit
> >> tests were added for all these plugins.
> >>
> >> https://fedorahosted.org/freeipa/ticket/2010
> >> https://fedorahosted.org/freeipa/ticket/2223
> >> https://fedorahosted.org/freeipa/ticket/2228
> >> https://fedorahosted.org/freeipa/ticket/2232
> >
> > Showing a raw delegation entry doesn't include the cn but it does for
> > permission and selfservice.
> >
> > The domain is hardcoded in the tests (glad to see this affect someone
> > else, it often gets me). The tests otherwise work.
> 
> Martin reminded me that selfservice entries exist only as an ACI which 
> is why no cn is displayed.
> 
> ACK if the tests are fixed.
> 
> rob

I fixed the tests to use api.env.basedn as a basedn instead of the
hard-coded value.

Pushed to master, ipa-2-2.

Martin