Message189271
Just a quick note on the directory naming: idlelib/idle_test sounds like a good option to me.
Putting it under idlelib makes it easy for distros to carve it out along with the rest of Idle, as well as making it clear that it falls under the purview of PEP 434
Expanding Itest -> idle_test is a combination of "Itest" being somewhat cryptic in the first place, and then the potential for 1/I/l/i glyph confusion (depending on font) making it worse.
Other than that, +1 to what Ezio said (I wouldn't worry about moving any more existing tests for this patch though - starting to move self-tests out of the individual IDLE files to the new test suite can be a good next step) |
|
Date |
User |
Action |
Args |
2013-05-15 07:11:01 | ncoghlan | set | recipients:
+ ncoghlan, terry.reedy, ned.deily, ezio.melotti, roger.serwy, r.david.murray, Todd.Rovito, tshepang, francismb, JayKrish, Tomoki.Imai |
2013-05-15 07:11:01 | ncoghlan | set | messageid: <1368601861.51.0.967615798706.issue15392@psf.upfronthosting.co.za> |
2013-05-15 07:11:01 | ncoghlan | link | issue15392 messages |
2013-05-15 07:11:01 | ncoghlan | create | |
|