git.iko.soy/unity8
on xenial help sign in
update tests
commit44eced403391306e6547fcd5fce283ae6952ab9b authorMichael Zanetti <michael.zanetti@canonical.com> dateMon 2 Sep 2013, 12:47 (13 years ago) parent4af4357
2 files changed+10−38
compare with xenial
unifiedsplit
plugins/Unity/Launcher/backend/launcherbackend.cpp+8−4 tests/plugins/Unity/Launcher/launcherbackendtest.cpp+2−34
plugins/Unity/Launcher/backend/launcherbackend.cpp+8−4
@@ -71,6 +71,12 @@ void LauncherBackend::setStoredApplications(const QStringList &appIds)
7171 }
7272 }
7373 m_storedApps = appIds;
74+ Q_FOREACH(const QString &appId, m_storedApps) {
75+ if (!m_itemCache.contains(appId)) {
76+ LauncherBackendItem *item = new LauncherBackendItem();
77+ m_itemCache.insert(appId, item);
78+ }
79+ }
7480 syncToAccounts();
7581}
7682
@@ -199,7 +205,7 @@ void LauncherBackend::syncFromAccounts()
199205 m_storedApps.clear();
200206
201207 qDebug() << "syncing from accounts" << m_user << m_accounts;
202− if (m_user != "" && m_accounts != nullptr) {
208+ if (m_accounts && m_user.isEmpty()) {
203209 qDebug() << "loading from user" << m_user << m_accounts;
204210 QVariant variant = m_accounts->getUserProperty(m_user, "launcher-items");
205211 qDebug() << "loaded launcher-items" << variant;
@@ -207,7 +213,7 @@ void LauncherBackend::syncFromAccounts()
207213 defaults = isDefaultsItem(apps);
208214 }
209215
210− if (defaults) {
216+ if (m_accounts && defaults) { // Checking accounts as it'll be null when !useStorage
211217 QGSettings gSettings("com.canonical.Unity.Launcher", "/com/canonical/unity/launcher/");
212218 qDebug() << "Got launcher gSettings" << gSettings.get("favorites");
213219 Q_FOREACH(const QString &entry, gSettings.get("favorites").toStringList()) {
@@ -329,5 +335,3 @@ bool LauncherBackend::isDefaultsItem(const QList<QVariantMap> &apps) const
329335
330336 return (apps.size() == 1 && apps[0].value("defaults").toBool());
331337}
332−
333−
tests/plugins/Unity/Launcher/launcherbackendtest.cpp+2−34
@@ -15,6 +15,7 @@
1515 *
1616 * Authors:
1717 * Michael Terry <michael.terry@canonical.com>
18+ * Michael Zanetti <michael.zanetti@canonical.com>
1819 */
1920
2021#include "launcherbackend.h"
@@ -32,46 +33,13 @@ private Q_SLOTS:
3233 LauncherBackend backend(false);
3334
3435 backend.setStoredApplications(QStringList() << "relative.path" << "/full/path");
35− QCOMPARE(backend.storedApplications(), QStringList() << "/usr/share/applications/relative.path" << "/full/path");
36−
37− QCOMPARE(backend.desktopFile("one.desktop"), QString("/usr/share/applications/one.desktop"));
38− QCOMPARE(backend.desktopFile("two"), QString("/usr/share/applications/two"));
39− QCOMPARE(backend.desktopFile("/full"), QString("/full"));
40−
41− QCOMPARE(backend.desktopFile("/usr/share/applications/one.desktop"), QString("/usr/share/applications/one.desktop"));
42− }
43−
44− void testDesktopReading()
45− {
46− LauncherBackend backend(false);
47−
48− QCOMPARE(backend.displayName(SRCDIR "/rel-icon.desktop"), QString("Relative Icon"));
49− QCOMPARE(backend.icon(SRCDIR "/rel-icon.desktop"), QString("image://gicon/rel-icon"));
50−
51− QCOMPARE(backend.displayName(SRCDIR "/abs-icon.desktop"), QString("Absolute Icon"));
52− QCOMPARE(backend.icon(SRCDIR "/abs-icon.desktop"), QString("/path/to/icon.png"));
53−
54− QCOMPARE(backend.displayName(SRCDIR "/no-icon.desktop"), QString("No Icon"));
55− QCOMPARE(backend.icon(SRCDIR "/no-icon.desktop"), QString(""));
56−
57− QCOMPARE(backend.displayName(SRCDIR "/no-name.desktop"), QString(""));
58− QCOMPARE(backend.icon(SRCDIR "/no-name.desktop"), QString("image://gicon/no-name"));
59−
60− QCOMPARE(backend.displayName(SRCDIR "/no-exist.desktop"), QString(""));
61− QCOMPARE(backend.icon(SRCDIR "/no-exist.desktop"), QString(""));
36+ QCOMPARE(backend.storedApplications(), QStringList() << "relative.path" << "/full/path");
6237 }
6338
6439 void testPinning()
6540 {
6641 LauncherBackend backend(false);
6742
68− // Confirm that default entries are all pinned
69− auto defaultApps = backend.storedApplications();
70− QVERIFY(defaultApps.size() > 0);
71− for (auto app: defaultApps) {
72− QCOMPARE(backend.isPinned(app), true);
73− }
74−
7543 backend.setStoredApplications(QStringList() << "one" << "two");
7644 QCOMPARE(backend.isPinned("one"), false);
7745 QCOMPARE(backend.isPinned("two"), false);