Message169104
Personally I prefer to have the test case create the file(s) used in the test dynamically, writing them to the temporary working directory. Since these are Python modules, you could use the helpers from script_helpers for this.
Otherwise I think your patch is OK. I don't think we have a doc convention for "unfortunate implementation detail" :( We have in the past put notes in the doc that say "this may change in the future", though that is usually about desired enhancements. But perhaps it could apply here as well. |
|
Date |
User |
Action |
Args |
2012-08-25 00:54:59 | r.david.murray | set | recipients:
+ r.david.murray, asvetlov, chris.jerdonek |
2012-08-25 00:54:59 | r.david.murray | set | messageid: <1345856099.31.0.469527722962.issue14649@psf.upfronthosting.co.za> |
2012-08-25 00:54:58 | r.david.murray | link | issue14649 messages |
2012-08-25 00:54:57 | r.david.murray | create | |
|