<p dir="ltr">Ugh, I probably was sleeping when I wrote that. <br>
</p>
<div class="gmail_extra"><br><div class="gmail_quote">Em 21 de set de 2016 13:47, "Dirk Hohndel" <<a href="mailto:dirk@hohndel.org">dirk@hohndel.org</a>> escreveu:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Wed, Sep 21, 2016 at 09:44:55AM -0700, Dirk Hohndel wrote:<br>
> On Wed, Sep 21, 2016 at 01:23:04PM -0300, Tomaz Canabrava wrote:<br>
> > I'm on my last two days in brazil, things here are *crazy* right now.<br>
> > My next patch will be from munich - probably in a mini-hackaton with<br>
> > robert. :)<br>
><br>
> I love patches that add tests that fail :-)<br>
><br>
> Now I need to figure out WHY it fails and whether that means the test is<br>
> wrong or it actually identified an issue.<br>
<br>
Umm. Tomaz, did you even run your tests?<br>
<br>
<br>
diff --git a/tests/testpreferences.cpp b/tests/testpreferences.cpp<br>
index 72ea73f88c63..14a40cb81277 100644<br>
--- a/tests/testpreferences.cpp<br>
+++ b/tests/testpreferences.cpp<br>
@@ -61,7 +61,7 @@ void TestPreferences::<wbr>testPreferences()<br>
        // Why this is short and not bool?<br>
        cloud->setSaveUserIdLocal(1);<br>
        TEST(cloud->saveUserIdLocal(), (short)1);<br>
-       cloud->setSavePasswordLocal(0)<wbr>;<br>
+       cloud->setSaveUserIdLocal(0);<br>
        TEST(cloud->saveUserIdLocal(), (short)0);<br>
<br>
        cloud->setUserId("Tomaz");<br>
</blockquote></div></div>