On 10/24/2013 04:38 PM, Tomas Babej wrote:
On 10/24/2013 01:29 PM, Petr Viktorin wrote:
In ADTrustBase, it looks like if test_install_adtrust or
test_configure_dns_and_time fail, it doesn't make much sense to run
the other tests.
If that's the case they can go in an install() classmethod. Same with
test_establish_trust* in the subclasses.
I made them part of the install() classmethod.
As for the test_estabilish_trust, I would have to still override that in
each class that uses it, since all of them use it in a slightly
I'd prefer an "establish_trust" classmethod called from install(), but
it's not really important in this case.
Also, there's a typo in test_estabilish_trusts several times.
Typo fixed. Also attaching a patch that fixes the same type in the other
parts of the codebase.
ACK, ACK, pushed to
Thanks for your patience!
Freeipa-devel mailing list