[Freeipa-devel] [PATCH] 0517 test_realmdomains_plugin: Add default ACI to expected output

Martin Kosek mkosek at redhat.com
Fri Apr 11 10:46:40 UTC 2014


On 04/11/2014 12:41 PM, Petr Viktorin wrote:
> On 04/10/2014 03:55 PM, Petr Viktorin wrote:
>> Subject: [PATCH] 0516 Add managed read permissions to realmdomains
>>
>> Read access is given to all authenticated users.
>>
> 
> Jenkins tells me this breaks tests. Since realmdomains ACIs are set on a single
> entry, not a container, realmdomains_show --all will include the ACI in the
> output. As it should, since we're asking for all LDAP attributes.
> Test fix attached.
> 

Good boy Jenkins. I did not see this coming, I mostly test the patches in my
LDAP browser and CLI as we did not step to the functional testing part yet.

This fixed the test for me, ACK.

Pushed to master: db85ea32c185fee270ed900b2c60353b234454d8

Martin




More information about the Freeipa-devel mailing list