Message184582
Thanks for the feedback :-D
Changes:
Utilize test.support.temp_cwd()
Removed imports that this made unnecessary
Awesome trick!
Cut line length down. Don't worry about nitpicking. Anything to get this to be as good as possible.
I added an explanation to the test class explaining why test_getting_all_methods_from_class does not appear. I also agree this is fragile. I have an idea for how to fix this and will upload it as a separate patch (in this thread) because I am not sure what people will think. |
|
Date |
User |
Action |
Args |
2013-03-19 02:58:41 | Matt.Bachmann | set | recipients:
+ Matt.Bachmann, r.david.murray, raduv |
2013-03-19 02:58:41 | Matt.Bachmann | set | messageid: <1363661921.39.0.196534191219.issue17464@psf.upfronthosting.co.za> |
2013-03-19 02:58:41 | Matt.Bachmann | link | issue17464 messages |
2013-03-19 02:58:40 | Matt.Bachmann | create | |
|