Michael Foord <mich...@voidspace.org.uk> added the comment:

Note that instead of try finally constructs you can create a cleanup function 
and call self.addCleanup(...). This reduces extra levels of indentation in your 
test code.

In the new code in "test_ismount", from "# Non-existent mountpoint" on, I would 
turn this into a new test. (Does os.symlink *always* exist - if it is platform 
dependent you should skip the test if it isn't available.)

----------

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue11503>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to