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:
Looks & works fine for me.
Can you also add a test for this?
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.
Freeipa-devel mailing list