| Age | Commit message (Collapse) | Author |
|---|
|
The code assumes that prefs.cloud_base_url is non NULL. Allowing that to
be NULL makes no sense during normal operation of the app. Yet, most of
the tests don't initialize the prefs at all.
Making things worse, if we do correctly initialize the prefs (so as to
reasonably approximate the behavior when running the app), things break
because some of the reference outputs assume that the prefs are unset.
This deserves fixing.
For now, simply make sure that cloud_base_url is set for all the tests
that try to parse files.
Additionally, the semantics how cloud_base_url is saved to disk have
changed, so adjust the test for those prefs accordingly.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
We know the preference is never empty, so stop testing for this. But
don't maintain two different preferences with basically the same
content. Instead add the '/git' suffix where needed and keep this all in
one place.
Simplify the extraction of the branch name from the cloud URL.
Also a typo fix and a new comment.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
qPref* set/get functions are static.
remove creation of qPref* instances, reference
static functions.
Signed-off-by: Jan Iversen <jan@casacondor.com>
|
|
Signed-off-by: Rolf Eike Beer <eike@sf-mail.de>
|
|
With removal of the git_local_only from the preferences (see
ae653703a5d3f), the users choice, in the mobile app, was not
stored any more in between sessions. This resulted in issue
1725.
So, in order to store that user preference, we need a new
preference. This is added here, but its not yet hooked up
in the app yet. This deals only with the preference handling.
And adapted tests are included.
Signed-off-by: Jan Mulder <jlmulder@xs4all.nl>
|
|
Remove tests for removed properties.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
Add signal testing of all variables
this commit contains all qPref* that work directly
followup commit will do changes to qPref* to make signals work
Signed-off-by: Jan Iversen <jani@apache.org>
|
|
reduce number of includes by removing qPref* includes in qPref.h
Signed-off-by: Jan Iversen <jani@apache.org>
|
|
qml declaration of cloud_status (defined in pref.h) does not
belong in qPref.h but in qPrefCloudStorage
Signed-off-by: Jan Iversen <jani@apache.org>
|
|
Remove all the code to handle the preferences itself.
Signed-off-by: Jan Mulder <jlmulder@xs4all.nl>
|
|
It's a mamber of the cloud storage authentication class, used to hold the
proposed new password until the backend has accepted it.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
It's the current state of the app, so it should be a global variable, not a
preference.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
This reverts commit b7a1bb670ce11b22e52361f4152b4853b97e13f5.
Signed-off-by: Jan Iversen <jani@apache.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
Add special property loadFromCloud + mail to qPrefCloudStorage
this variable is used by qmlmanager.cpp
Signed-off-by: Jan Iversen <jani@apache.org>
|
|
test_multiple did not test correctly, correct.
Signed-off-by: Jan Iversen <jani@apache.org>
|
|
update tests to reflect that git_url is derived from base_url
Signed-off-by: Jan Iversen <jani@apache.org>
|
|
Again, entirely script based.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
Remove CloudStorage test in testpreferences
add the same CloudStorage tests to testqPrefCloudStorage
Signed-off-by: Jan Iversen <jani@apache.org>
|
|
add test cases to secure struct preferences and qPrefCloudStorage work together
Signed-off-by: Jan Iversen <jani@apache.org>
|
Dirk Hohndel
jan Iversen
Rolf Eike Beer
Jan Mulder
jan Iversen