On 08/07/2014 05:40 PM, Martin Basti wrote:
On 07/08/14 17:05, Martin Basti wrote:
On 07/08/14 16:27, Petr Viktorin wrote:
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.
Please wait, I will fix the names then, it'll be fast.
Updated patch attached.
ACK, pushed to:
master: 4b5a4882497ce7c3ecdf8f898fc695b2309df1b5
ipa-4-1: 4b5a4882497ce7c3ecdf8f898fc695b2309df1b5
ipa-4-0: 2fa1555722ed875a32d3480ea08c5ad420a015a6
--
Petr³
_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel
