more unittests for preferences.

Dirk Hohndel dirk at hohndel.org
Wed Sep 21 09:47:24 PDT 2016


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");


More information about the subsurface mailing list