R. David Murray added the comment: Thanks, Milan.
I made some tweaks...mostly documentation and code style (your code style wasn't wrong, I just have some slightly different preferences with regards to line folding). I also eliminated the authobjects test method, since I didn't see that it added anything to duplicate the code from the login method to test them...they get tested via the login methods. Instead I added the loop to the test for the auth method, so that we test that calling the public API with the auth objects that are documented as part of the public API work. That is, if someone were to change login such that the method names changed but login still worked, the auth test method will throw an error because the documented method names changed. I'm attaching the final patch here so you can look at the differences via the reitveld patch-diff function, if you want to. ---------- resolution: -> fixed stage: needs patch -> resolved status: open -> closed Added file: http://bugs.python.org/file35842/smtplib_auth.patch _______________________________________ Python tracker <rep...@bugs.python.org> <http://bugs.python.org/issue15014> _______________________________________ _______________________________________________ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
