This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author ezio.melotti
Recipients eric.araujo, eric.snow, ezio.melotti, francismb, nadeem.vawda, tshepang
Date 2012-02-23.00:43:12
SpamBayes Score 3.9286743e-08
Marked as misclassified No
Message-id <1329957793.92.0.268557684031.issue13447@psf.upfronthosting.co.za>
In-reply-to
Content
> I went for something even simpler: one new file Lib/test/test_tools.py.

I'm fine with this option too; if it grows too big we can always split it in several modules later.

> My only concern is communication: how do we tell people working on a
> tool that they should write a test in test_tools?

I don't think it's a big deal, eventually people will learn that it's there and we can always point to that file when a new issue is opened or during the patch review.
Adding a check to `make patchcheck` will also help (it's still better to add tests later than leaving the code untested), but maybe it should be displayed only if something is actually changed in the Tools dir (since that doesn't happen often, having always a "Tools modified... no" will just add noise to the output).
History
Date User Action Args
2012-02-23 00:43:14ezio.melottisetrecipients: + ezio.melotti, nadeem.vawda, eric.araujo, tshepang, eric.snow, francismb
2012-02-23 00:43:13ezio.melottisetmessageid: <1329957793.92.0.268557684031.issue13447@psf.upfronthosting.co.za>
2012-02-23 00:43:13ezio.melottilinkissue13447 messages
2012-02-23 00:43:12ezio.melotticreate