<br><br><div class="gmail_quote">On Sun, Aug 5, 2012 at 5:17 PM, David Faure <span dir="ltr"><<a href="mailto:faure@kde.org" target="_blank">faure@kde.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On Friday 03 August 2012 16:34:23 Vishesh Handa wrote:<br>
> I know<br>
><br>
> These tests are a pain to maintain cause they used to check the string<br>
> value of the queries. One of my commits broke them. I'm in the process of<br>
> writing new tests which actually runt the queries and test them properly.<br>
<br>
</div>Thanks.<br>
<br>
If these new tests will take time to write, or won't go into kdelibs-4.9-<br>
branch, please use QEXPECT_FAIL there so that the unittests are green again.<br>
As all books on the topic say: unittests should always be green. Known<br>
failures should be marked as such, or fixed, obviously.<br></blockquote><div><br>Oh. I generally write failing unit tests as a kind of TODO list. I'll mark them with QEXPECT_FAIL.<br><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="HOEnZb"><div class="h5"><br>
--<br>
David Faure, <a href="mailto:faure@kde.org">faure@kde.org</a>, <a href="http://www.davidfaure.fr" target="_blank">http://www.davidfaure.fr</a><br>
Sponsored by Nokia to work on KDE, incl. KDE Frameworks 5<br>
<br>
</div></div></blockquote></div><br><br clear="all"><br>-- <br><span style="color:rgb(192,192,192)">Vishesh Handa</span><br><br>