Message160090
Only two comments, otherwise LGTM (and I can't believe the solution was to go back through the import system just to pull out the cached module; the things we would change if we were doing this from scratch).
One, you have some "XXX False" markers in the tests. Should those get deleted or replaced with something?
Two, in your first test (at least) you only test what is in sys.modules once instead of after each attempted import. I would repeat the test after each import. |
|
Date |
User |
Action |
Args |
2012-05-06 16:24:00 | brett.cannon | set | recipients:
+ brett.cannon, pitrou, nadeem.vawda, Arfrever, skrah, eric.snow |
2012-05-06 16:24:00 | brett.cannon | set | messageid: <1336321440.61.0.280194133984.issue14583@psf.upfronthosting.co.za> |
2012-05-06 16:24:00 | brett.cannon | link | issue14583 messages |
2012-05-06 16:23:59 | brett.cannon | create | |
|