Martin Panter added the comment: >From the current documentation and limited experience with the module, >ReadError (or a subclass) sounds best. I would only expect OSError only for >OS-level things, like file not found, disk error, etc.
The patches look good. One last suggestion is to use assertRaisesRegex, to be sure that you are getting exactly the exception you expect. Too many times I have found tests in the test suite that were completely broken, but happened to pass because the error happened to match the exception being tested. ---------- _______________________________________ Python tracker <rep...@bugs.python.org> <http://bugs.python.org/issue24259> _______________________________________ _______________________________________________ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com