[Freeipa-devel] [PATCH 0102] update idrange tests to reflect disabled modification of local ID ranges
Tomas Babej
tbabej at redhat.com
Wed Nov 25 13:29:11 UTC 2015
On 11/23/2015 12:58 PM, Tomas Babej wrote:
>
>
> On 11/20/2015 06:41 PM, Milan Kubík wrote:
>> On 11/20/2015 04:06 PM, Martin Babinsky wrote:
>>> When I fixed https://fedorahosted.org/freeipa/ticket/4826 I forgot to
>>> fix the corresponding xmlrpc tests.
>>>
>>> This oversight bit me today when I ran in-tree tests on my VM.
>>>
>>> Here is the patch that makes idrange tests green and shiny again.
>>>
>> Tests are now passing, ACK
>>
>
> Tests are now passing, however, the code paths that have been checked by
> these tests are no longer executed (as an modifications to the local
> ranges are prohibited now).
>
> This essentially means that we are testing the same thing multiple times
> (i.e. local range cannot be dealt with), and the test names are
> misleading currently.
>
> I think we should either reduce the set of (effectively) duplicate
> tests, or, better, introduce their replacements that are using allowed
> range types, if possible.
>
> Tomas
>
I created a ticket so that this does not get lost:
https://fedorahosted.org/freeipa/ticket/5493
More information about the Freeipa-devel
mailing list