<p dir="ltr"></p>
<p dir="ltr">On Aug 2, 2016 22:13, "Mads Kiilerich" <<a href="mailto:mads@kiilerich.com">mads@kiilerich.com</a>> wrote:<br>
><br>
> On 08/02/2016 10:03 PM, Thomas De Schampheleire wrote:<br>
>><br>
>> On Tue, Aug 2, 2016 at 9:38 PM, Mads Kiilerich <<a href="mailto:mads@kiilerich.com">mads@kiilerich.com</a>> wrote:<br>
>>><br>
>>> On 08/02/2016 09:30 PM, Thomas De Schampheleire wrote:<br>
>>>><br>
>>>> Hi Mads,<br>
>>>><br>
>>>> Since commit<br>
>>>> changeset:   6084:552800808386<br>
>>>> user:        Mads Kiilerich <<a href="mailto:madski@unity3d.com">madski@unity3d.com</a>><br>
>>>> date:        Thu Jul 28 16:31:40 2016 +0200<br>
>>>> summary:     tests: add test coverage of PR comment @mention<br>
>>>><br>
>>>> I get test failures on the e-mail tests, due to escaping of single quotes.<br>
>>>> It is unclear to me how your change is causing this, but it is 100%<br>
>>>> deterministic.<br>
>>>><br>
>>>> Do you have a clue?<br>
>>>><br>
>>>> The type of diff between out and ref, is:<br>
>>>><br>
>>>> E     - ation. Don\'t reply to this mail.<br>
>>>> E     ?           -<br>
>>>> E     + ation. Don't reply to this mail.<br>
><br>
><br>
> Is it failing like that on all the text mails, or only the new one related to this change?</p>
<p dir="ltr">On all.</p>
<p dir="ltr">><br>
> I think my answer must be that I have no clue and don't have the problem. Please, you can reproduce the problem - can you dive into it?</p>
<p dir="ltr">What I already tried:<br>
- remove the added comments (FIXME) from the commit<br>
- remove the addition of the is_mention clause.</p>
<p dir="ltr">With these changes, the problem still occurred.</p>
<p dir="ltr">Reverting the changes in ref too fixes the problem. What I don't recall is whether the html files itself were fine or not -- perhaps the problem is in the comparison by pytest.</p>
<p dir="ltr">I can't continue on this tonight, I can see if I can find more later, unless Dominik found it by then.</p>
<p dir="ltr">/Thomas<br></p>