kallithea/tests/models/test_notifications.py test failure

Thomas De Schampheleire patrickdepinguin at gmail.com
Tue Aug 2 20:03:57 UTC 2016


On Tue, Aug 2, 2016 at 9:38 PM, Mads Kiilerich <mads at kiilerich.com> wrote:
> On 08/02/2016 09:30 PM, Thomas De Schampheleire wrote:
>>
>> Hi Mads,
>>
>> Since commit
>> changeset:   6084:552800808386
>> user:        Mads Kiilerich <madski at unity3d.com>
>> date:        Thu Jul 28 16:31:40 2016 +0200
>> summary:     tests: add test coverage of PR comment @mention
>>
>> I get test failures on the e-mail tests, due to escaping of single quotes.
>> It is unclear to me how your change is causing this, but it is 100%
>> deterministic.
>>
>> Do you have a clue?
>>
>> The type of diff between out and ref, is:
>>
>> E     - ation. Don\'t reply to this mail.
>> E     ?           -
>> E     + ation. Don't reply to this mail.
>
>
> Weird. It passes here.
>
> What OS?

This is Gentoo Linux, Python 2.7.10.

>
> What locale settings do you have?

LANG=en_US.utf8

>
> Does it work for you on 3a221939a19f?

Yes, it does. I bisected and came out on the mentioned revision. Then
double-checked manually.

>
> I am on US English Fedora Linux.

/Thomas


More information about the kallithea-general mailing list