Message137046
Actually, I only commented on the patch format and not on the actual contents of the patch, sorry :)
Your test method missed the self parameter, and the test case needed to be added to the testcases_other list for the test to be actually run. I attached a corrected patch.
And, now that I tested it, the test runs correctly on Linux, so maybe this is a Windows specific issue? |
|
Date |
User |
Action |
Args |
2011-05-27 11:28:08 | petri.lehtinen | set | recipients:
+ petri.lehtinen, terry.reedy, jnoller, ezio.melotti, cool-RR |
2011-05-27 11:28:08 | petri.lehtinen | set | messageid: <1306495688.92.0.911727596169.issue11969@psf.upfronthosting.co.za> |
2011-05-27 11:28:08 | petri.lehtinen | link | issue11969 messages |
2011-05-27 11:28:08 | petri.lehtinen | create | |
|