Tim Hatch wrote: > >> I have been running [7648] for the last two days on two installations, >> without any surprises. They are very low-traffic internal sites, so >> this >> might not mean much, but it's still an additional data point. > > At first glance it looks like [7648] reduced test coverage and might > be ignoring a potential bug with unicode passing through to the > notification engine. Eli and I would like till tomorrow evening to > look at this and add new tests before calling it "ready to release."
If you're referring to the "test_mimebody_none"-test I'm not quite sure what's supposed to happen when non-ascii text is encountered. But it at least looks like it's supposed to simply blow up: http://trac.edgewall.org/browser/trunk/trac/notification.py#L394 That makes the "mime_encoding=none" option look pretty useless. but anways, I don't have any problem with waiting with the release until you've had some time to take a closer look at this and why the coverage dropped. / Jonas --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Trac Development" group. To post to this group, send email to [email protected] To unsubscribe from this group, send email to [EMAIL PROTECTED] For more options, visit this group at http://groups.google.com/group/trac-dev?hl=en -~----------~----~----~----~------~----~------~--~---
