[issue21624] Idle: polish htests
Terry J. Reedy
report at bugs.python.org
Wed Jun 11 08:29:53 CEST 2014
Terry J. Reedy added the comment:
Refinement 1: in doing coverage for UndoDelegator, I noticed that the htest function is counted as missing (uncovered). Both of the following in .coveragerc work to ignore the block: name prefix; comment suffix.
def htest_.*:
.*# htest #
The second is more practical for alphabetical order in htest. It is also less work to change. The particular form marks it as not a normal comment.
Refinement 2: put all imports that are specific to the htest function at the top of the function. Since the function is only called once per process, there is no efficiency consideration. I decided that after the tkinter import at the top got me (wasting time) looking through UndoDelegator for a widget call that might be the source of the leak. The changes can wait until we edit the file anyway, or at least write a test.
However, the docstring at the top of htest.py should be changed.
----------
_______________________________________
Python tracker <report at bugs.python.org>
<http://bugs.python.org/issue21624>
_______________________________________
More information about the Python-bugs-list
mailing list