Message102643
Thank you very much for those tests. I think you can simplify them a bit. For example, you can use assertRaises. You also might be able to use the test_support.temp_cwd context manager in your context manager, even though you don't need the cwd part. I've attached an alternate, simpler patch to fix this bug, based on a similar fix I did in pydoc. The disadvantage of my patch is that it contains a hardcoding of the name of the function doing the import. I think this is acceptable given the much greater simplicity of my patch. I may be missing some subtlety, though, that the twisted folks know about. Or perhaps people will just find the hardcoding itself objectionable. | |
| Date | User | Action | Args | | 2010-04-08 20:28:21 | r.david.murray | set | recipients: + r.david.murray, rbcollins, michael.foord, slmnhq, chris.jerdonek | | 2010-04-08 20:28:21 | r.david.murray | set | messageid: <1270758501.34.0.5188532463.issue7559@psf.upfronthosting.co.za> | | 2010-04-08 20:28:19 | r.david.murray | link | issue7559 messages | | 2010-04-08 20:28:19 | r.david.murray | create | | |