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. Peace, R. -- Roberto Polli Community Manager Babel S.r.l. - http://www.babel.it T: +39.06.9826.9651 M: +39.340.652.2736 F: +39.06.9826.9680 P.zza S.Benedetto da Norcia, 33 - 00040 Pomezia (Roma) CONFIDENZIALE: Questo messaggio ed i suoi allegati sono di carattere confidenziale per i destinatari in indirizzo. E' vietato l'inoltro non autorizzato a destinatari diversi da quelli indicati nel messaggio originale. Se ricevuto per errore, l'uso del contenuto e' proibito; si prega di comunicarlo al mittente e cancellarlo immediatamente. -- 389-devel mailing list 389-devel@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/389-devel
