more unittests for preferences.

Tomaz Canabrava tcanabrava at kde.org
Wed Sep 21 10:09:24 PDT 2016


Ugh, I probably was sleeping when I wrote that.

Em 21 de set de 2016 13:47, "Dirk Hohndel" <dirk at hohndel.org> escreveu:

> On Wed, Sep 21, 2016 at 09:44:55AM -0700, Dirk Hohndel wrote:
> > On Wed, Sep 21, 2016 at 01:23:04PM -0300, Tomaz Canabrava wrote:
> > > I'm on my last two days in brazil, things here are *crazy* right now.
> > > My next patch will be from munich - probably in a mini-hackaton with
> > > robert. :)
> >
> > I love patches that add tests that fail :-)
> >
> > Now I need to figure out WHY it fails and whether that means the test is
> > wrong or it actually identified an issue.
>
> Umm. Tomaz, did you even run your tests?
>
>
> diff --git a/tests/testpreferences.cpp b/tests/testpreferences.cpp
> index 72ea73f88c63..14a40cb81277 100644
> --- a/tests/testpreferences.cpp
> +++ b/tests/testpreferences.cpp
> @@ -61,7 +61,7 @@ void TestPreferences::testPreferences()
>         // Why this is short and not bool?
>         cloud->setSaveUserIdLocal(1);
>         TEST(cloud->saveUserIdLocal(), (short)1);
> -       cloud->setSavePasswordLocal(0);
> +       cloud->setSaveUserIdLocal(0);
>         TEST(cloud->saveUserIdLocal(), (short)0);
>
>         cloud->setUserId("Tomaz");
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.subsurface-divelog.org/pipermail/subsurface/attachments/20160921/4f33753f/attachment-0001.html>


More information about the subsurface mailing list