On 11/10/2015 12:13 PM, Filip Škola wrote:



On Tue, 10 Nov 2015 10:52:45 +0100
Milan Kubík <mku...@redhat.com> wrote:

On 11/09/2015 04:35 PM, Filip Škola wrote:
Another patch was applied in the meantime.

Attaching an updated version.


On Mon, 9 Nov 2015 13:35:02 +0100
Milan Kubík <mku...@redhat.com> wrote:

On 11/06/2015 11:32 AM, Filip Škola wrote:
the patch doesn't apply.

Please fix this.

TestDeniedBindWithExpiredPrincipal.teardown_class] Undefined variable

Also, use the version numbers for your changed patches.

Thanks for the patch. Several issues:

1. Use dict.items instead of dict.iteritems, for python3 compatibility

2. What is the purpose of TestPrepare class? The 'purge' methods do not call any ipa commands. Tracker.make_fixture should be used to make the Tracked resources clean themselves up when they're out of scope.

3. Why reference the resources by hardcoded name if they have a fixture representation?

4. Rewrite {create,delete}_test_group to a fixture. You may want to use different scope (or not).

5. In `def atest_rename_to_invalid_login(self, user):` - use pytest.skipif decorator and provide a reason if you must,
do not obfuscate method name in order not to run it.

Milan Kubik

Manage your subscription for the Freeipa-devel mailing list:
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

Reply via email to