[Freeipa-devel] [PATCH 0101, 106] Tests: host plugin (Allow to add host if AAAA record exists)

[Petr Viktorin]

On 08/07/2014 02:33 PM, Martin Basti wrote:
> On 28/07/14 14:11, Petr Viktorin wrote:
>> On 07/09/2014 06:29 PM, Martin Basti wrote:
>>> Patch attached.
>>> Ticket: https://fedorahosted.org/freeipa/ticket/4164
>>>
>>
>> Looks & works fine for me.
>> Can you also add a test for this?
>>
>>
> Tests attached.
> I also added tests with --ip-address parameter.
>

This works, thanks!
I have some comments however:

Variables like `name5` can have more descriptive names, so that you can 
look at the test definition and actually know what's being tested.

Some of the tests are independent (both in the sense that they don't 
need the other tests to be run, and that they test a different thing 
than the others); those can be in separate classes.


There are better things to do in IPA than making the tests perfect, so 
ACK if you want to push this as is.

-- 
Petr³