Code review comment for lp:~piastucki/bzr-xmloutput/fix-auth

Revision history for this message
Guillermo Gonzalez (verterok) wrote :

The changes looks good.
Could you add some tests checking the behavior of the UIFactory? Also, please could you add tests for at least the 2 cases with the new "auth" command?

Thanks.

review: Needs Fixing

« Back to merge proposal