[Freeipa-devel] [PATCH 0102] update idrange tests to reflect disabled modification of local ID ranges
Tomas Babej
tbabej at redhat.com
Mon Nov 23 11:58:45 UTC 2015
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
More information about the Freeipa-devel
mailing list