➜

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 eli.bendersky
Recipients brett.cannon, eli.bendersky, giampaolo.rodola, ncoghlan, r.david.murray
Date 2011-01-29.05:22:07
SpamBayes Score 0.00010011035
Marked as misclassified No
Message-id <1296278527.81.0.489936707258.issue11049@psf.upfronthosting.co.za>
In-reply-to
Content 
Giampaolo,

The patch looks good. Some minor comments below:

* class TestCase(unittest.TestCase): -> why not give the test class a more useful name?
* import_fresh_module is tested similarly to import_module - can't a test be added that it indeed performs its "special sauce"?
* test_DirsOnSysPath: you may want to use assertIn and assertNotIn here. [these were added to Python in 3.1]
* test_python_is_optimized: you may want to use assertIsInstance here.
History
Date User Action Args
2011-01-29 05:22:07eli.benderskysetrecipients: + eli.bendersky, brett.cannon, ncoghlan, giampaolo.rodola, r.david.murray
2011-01-29 05:22:07eli.benderskysetmessageid: <1296278527.81.0.489936707258.issue11049@psf.upfronthosting.co.za>
2011-01-29 05:22:07eli.benderskylinkissue11049 messages
2011-01-29 05:22:07eli.benderskycreate