[389-devel] Please review bug 47586: CI tests: test case for 47490

[thierry bordaz]

On 11/15/2013 04:16 PM, Roberto Polli wrote:
> On Friday 15 November 2013 15:18:26 thierry bordaz wrote:
>> https://fedorahosted.org/389/attachment/ticket/47586/0001-Ticket-47586-CI-te
>> sts-test-case-for-47490.patch
> 1- I'd replace the following:
> -from lib389._constants import DN_DM
> +from lib389 import DN_DM
>
> Files starting with _ are "private" and are just facilities for the developer.
> We could consider to rename _constants to constants
>
>
> 2- To isolate testing tools from the library I would put
> args_standalone & co in a proper bugfix harness file/class that we can add to
> lib389, eg:
>
> from lib389.bugfix import args_standalone
> Otherwise people/projects using lib389 will be full of our testing code.
Hi Roberto,

    Thanks for reviewing it. I agree that _constants make no sense, I
    will change that.

    I have a question regarding your 2nd point.
    Lib389 offers an interface to create an instance 'createInstance'.
    This function takes a dict as argument containing the description of
    the instance.
    For me implementation of that dict should be done by the lib389 user.
    What lib389 can offer, is a dict template with default values.  Is
    that this template that you recommend to put into lib389.bugfix ?

    Note: currently ticket/constant.py contains a arg_standalone. I
    think it is not a good idea, as simply a arg_template should be
    provided.

regards
thierry
>
>
>
> Peace,
> R.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.fedoraproject.org/pipermail/389-devel/attachments/20131118/ae8066f8/attachment.html>