Message240836
Thanks for the quick response, Berker! Let me address your comments in the order you made them.
#1: removing checking for "kot" in the exception args.
This line was causing the tests to fail, which is how I noticed it. The reason it fails (I think) is caused by the changes made in the patch. Namely, the fact that we're strictly sticking to longMessage=False means the custom message passed by the user (in this particular case that's "Message") is be the *only* thing sent to AssertionError. Thus checking for the presence of "kot" in this error's args doesn't seem to make much sense and will simply keep breaking our tests.
Now it goes without saying that I may have missed something, please let me know if that's the case.
#2
For some reason I got the impression that the tests should be using the newer method names, but since using the old ones didn't break them on my machine, I've updated the patch file accordingly. I'll upload it pending your feedback regarding #1.
Glad to help!
Ilia. |
|
Date |
User |
Action |
Args |
2015-04-14 03:17:11 | copper-head | set | recipients:
+ copper-head, ezio.melotti, michael.foord, berker.peksag, Dhara |
2015-04-14 03:17:11 | copper-head | set | messageid: <1428981431.45.0.751613169823.issue20362@psf.upfronthosting.co.za> |
2015-04-14 03:17:11 | copper-head | link | issue20362 messages |
2015-04-14 03:17:11 | copper-head | create | |
|