[Freeipa-devel] [PATCH 0031] Deprecate options --dom-sid and --dom-name in idrange-mod
Tomas Babej
tbabej at redhat.com
Fri May 31 10:25:34 UTC 2013
On 05/29/2013 03:24 PM, Ana Krivokapic wrote:
> Hello,
>
> This patch addresses ticket https://fedorahosted.org/freeipa/ticket/3636
>
>
>
> _______________________________________________
> Freeipa-devel mailing list
> Freeipa-devel at redhat.com
> https://www.redhat.com/mailman/listinfo/freeipa-devel
I edited your patch to use newly introduced prompt_param method as
agreed in my patches 53-55 thread.
The functional part itself looks good, the tests though, are dependent
on the environment. The particular
code branch of tests that is being executed depends on the fact whether
any trust is estabilished on that
particular FreeIPA instance the test suite is being run on.
I suggest you create a mock trust LDAP entry as in my patch 57 that has
been just pushed to master,
and test both cases (whether the interactive prompt behaves correctly
both with the trust estabilished
and without it).
Maybe we should move the setUpClass/tearDownClass logic to tests/util.py
to avoid code duplication.
Attaching the updated patch (apply on top of tbabej-55-3).
Tomas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20130531/22bdae0e/attachment.htm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-akrivoka-0030-Require-rid-base-and-secondary-rid-base-options-in-i.patch
Type: text/x-patch
Size: 5320 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20130531/22bdae0e/attachment.bin>
More information about the Freeipa-devel
mailing list