[Freeipa-devel] [PATCH] 0008 Fixes different behaviour of permission-mod and show.

Tomas Babej tbabej at redhat.com
Mon Aug 27 12:07:46 UTC 2012


On 08/23/2012 02:46 PM, Rob Crittenden wrote:
> Tomas Babej wrote:
>> On 08/22/2012 05:15 PM, Rob Crittenden wrote:
>>> Tomas Babej wrote:
>>>> Hi,
>>>>
>>>> Both commands now produce the same output regarding
>>>> the attributelevelrights.
>>>>
>>>> https://fedorahosted.org/freeipa/ticket/2875
>>>
>>> I think some unit tests would be helpful so we don't regress and we
>>> know which other commands this fixes.
>>>
>>> rob
>>
>> I ran the tests for the permission plugin (test_permission_plugin.py)
>> and all of them passed. Tested on clean VM with newly built IPA from the
>> master, so there should be no regression. Results themselves attached.
>>
>> Tomas
>
> Right, but those tests all passed prior to your fix as well. We need a 
> test that does a permission-mod and confirms that the rights contains 
> the full list of attributes (and perhaps testing any other commands 
> that were similarly fixed).
>
> rob
>

I added unit tests for permission-mod and permission-show.

Tomas
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-tbabej-0008-2-Fixes-different-behaviour-of-permission-mod-and-show.patch
Type: text/x-patch
Size: 5874 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20120827/bd955bcc/attachment.bin>


More information about the Freeipa-devel mailing list