This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author copper-head
Recipients Dhara, berker.peksag, copper-head, ezio.melotti, michael.foord
Date 2015-04-14.03:17:11
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1428981431.45.0.751613169823.issue20362@psf.upfronthosting.co.za>
In-reply-to
Content
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.
History
Date User Action Args
2015-04-14 03:17:11copper-headsetrecipients: + copper-head, ezio.melotti, michael.foord, berker.peksag, Dhara
2015-04-14 03:17:11copper-headsetmessageid: <1428981431.45.0.751613169823.issue20362@psf.upfronthosting.co.za>
2015-04-14 03:17:11copper-headlinkissue20362 messages
2015-04-14 03:17:11copper-headcreate