<div dir="ltr">Hi Khaled,<br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Nov 21, 2016 at 7:19 PM, Khaled Hosny <span dir="ltr"><<a href="mailto:khaledhosny@eglug.org" target="_blank">khaledhosny@eglug.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I’m having a failure on Linux in this test, I can reproduce it locally<br>
but I have no idea how to debug it:<br>
<a href="http://ci.libreoffice.org/job/lo_gerrit/2956/Config=linux_gcc_release_64/console" rel="noreferrer" target="_blank">http://ci.libreoffice.org/job/<wbr>lo_gerrit/2956/Config=linux_<wbr>gcc_release_64/console</a><br>
<br>
(for <a href="https://gerrit.libreoffice.org/#/c/31053/" rel="noreferrer" target="_blank">https://gerrit.libreoffice.<wbr>org/#/c/31053/</a>).<br>
<br>
The test apparently checks if a mouse event at a certain position does<br>
something, and since I’m changing how line spacing is calculated it<br>
might have caused some vertical shift. But I don’t understand the code<br>
of the test that much and don’t know where to start, any tips?<br>
<br></blockquote><div><br></div><div>It's a hit-test. Basically, it simulates a mouse click (by invoking the relevant API) and validates that the cursor position is updated as a result.</div><div><br></div><div>I guess you need to rework the test to take into account the new spacing so the mouse would end up in the text-edit control. Line 355 looks relevant.</div><div><br></div><div>From 'git blame' Miklos is the author (CC-ed).</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Regards,<br>
Khaled<br>
______________________________<wbr>_________________<br>
LibreOffice mailing list<br>
<a href="mailto:LibreOffice@lists.freedesktop.org">LibreOffice@lists.freedesktop.<wbr>org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/libreoffice" rel="noreferrer" target="_blank">https://lists.freedesktop.org/<wbr>mailman/listinfo/libreoffice</a><br>
</blockquote></div><br></div></div>