[issue13447] Add tests for Tools/scripts/reindent.py

2011-11-21 Thread Ezio Melotti
Ezio Melotti added the comment: +1 What you describe sounds all good to me, except that I would keep all tests in the same dir (i.e. Tools/tests (or Tools/test?)) instead of having different test dirs for each subdir (so test_pygettext.py would go in Tools/tests and not in Tools/i18n/tests).

[issue13447] Add tests for Tools/scripts/reindent.py

2011-11-21 Thread Éric Araujo
New submission from Éric Araujo : When people find bugs in reindent.py (#12930 is a recent one), we have to try to reproduce the bug manually, apply a fix and test manually again. The alternative is to only read the code and trust that it works. I find both of these positions unsatisfactory