My comments (a github like platform for comment could be really useful)
line: comment
lib389/__init__.py:1: the module is lib389, not dirsrv
lib389/brooker.py:795: python variable naming convention: I would get stick
with the "_" instead of camelCase and change whenever possible.
tests/dsadmin_test.py: I renamed it lib389_test.py, you can merge my changes
tests/dsadmin_test.py:39: why remove the addbackend_harn?
tests/replica_test.py:119: you're using Backend.delete in a class that should
test just Replica. I would use harness and the standard python-ldap methods in
setup/teardown, so that we can change the Backend and Replica class without at
least breaking the tests.
Let me know + 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.