Merge lp:~ken-vandine/ubuntu-system-settings/rtm_click_framework_check into lp:ubuntu-system-settings/rtm-14.09

[Ken VanDine]

Include supported frameworks and arch when checking for updates

Include supported frameworks and arch when checking for updates

[lp:~ken-vandine/ubuntu-system-settings/rtm_click_framework_check]

1013. By Ken VanDine on 2015-04-29

Added tests to ensure framework and arch are included in check for click updates

1014. By Ken VanDine on 2015-04-29

Use rawHeader on the request to pass framework and arch

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:1013
http://jenkins.qa.ubuntu.com/job/ubuntu-system-settings-rtm-14.09-ci/75/
Executed test runs:
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-14.09-touch/54
    SUCCESS: http://jenkins.qa.ubuntu.com/job/ubuntu-system-settings-rtm-14.09-14.09-armhf-ci/76
        deb: http://jenkins.qa.ubuntu.com/job/ubuntu-system-settings-rtm-14.09-14.09-armhf-ci/76/artifact/work/output/*zip*/output.zip
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-14.09-mako/55
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-14.09-armhf/54
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-14.09-armhf/54/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/20026

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/ubuntu-system-settings-rtm-14.09-ci/75/rebuild

review: Needs Fixing (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:1014
http://jenkins.qa.ubuntu.com/job/ubuntu-system-settings-rtm-14.09-ci/76/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-14.09-touch/55/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/ubuntu-system-settings-rtm-14.09-14.09-armhf-ci/77/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-14.09-armhf/55/console

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/ubuntu-system-settings-rtm-14.09-ci/76/rebuild

review: Needs Fixing (continuous-integration)

[lp:~ken-vandine/ubuntu-system-settings/rtm_click_framework_check]

1015. By Ken VanDine on 2015-04-29

Don't use QByteArray::fromStdString, it was just introduced in 5.4

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:1015
http://jenkins.qa.ubuntu.com/job/ubuntu-system-settings-rtm-14.09-ci/77/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-14.09-touch/56/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/ubuntu-system-settings-rtm-14.09-14.09-armhf-ci/78/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-14.09-armhf/56/console

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/ubuntu-system-settings-rtm-14.09-ci/77/rebuild

review: Needs Fixing (continuous-integration)

[lp:~ken-vandine/ubuntu-system-settings/rtm_click_framework_check]

1016. By Ken VanDine on 2015-04-29

Added build depends on ubuntu-sdk-libs to get click frameworks needed by tests

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:1016
http://jenkins.qa.ubuntu.com/job/ubuntu-system-settings-rtm-14.09-ci/78/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-14.09-touch/57
    SUCCESS: http://jenkins.qa.ubuntu.com/job/ubuntu-system-settings-rtm-14.09-14.09-armhf-ci/79
        deb: http://jenkins.qa.ubuntu.com/job/ubuntu-system-settings-rtm-14.09-14.09-armhf-ci/79/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-14.09-mako/56
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-14.09-armhf/57
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-14.09-armhf/57/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/20042

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/ubuntu-system-settings-rtm-14.09-ci/78/rebuild

review: Approve (continuous-integration)

[lp:~ken-vandine/ubuntu-system-settings/rtm_click_framework_check]

1017. By Ken VanDine on 2015-04-29

fixed url for click updates

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:1017
http://jenkins.qa.ubuntu.com/job/ubuntu-system-settings-rtm-14.09-ci/79/
Executed test runs:
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-14.09-touch/58
    SUCCESS: http://jenkins.qa.ubuntu.com/job/ubuntu-system-settings-rtm-14.09-14.09-armhf-ci/81
        deb: http://jenkins.qa.ubuntu.com/job/ubuntu-system-settings-rtm-14.09-14.09-armhf-ci/81/artifact/work/output/*zip*/output.zip
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-14.09-mako/57
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-14.09-armhf/58
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-14.09-armhf/58/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/20051

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/ubuntu-system-settings-rtm-14.09-ci/79/rebuild

review: Needs Fixing (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:1017
http://jenkins.qa.ubuntu.com/job/ubuntu-system-settings-rtm-14.09-ci/80/
Executed test runs:
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-14.09-touch/59
    SUCCESS: http://jenkins.qa.ubuntu.com/job/ubuntu-system-settings-rtm-14.09-14.09-armhf-ci/82
        deb: http://jenkins.qa.ubuntu.com/job/ubuntu-system-settings-rtm-14.09-14.09-armhf-ci/82/artifact/work/output/*zip*/output.zip
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-14.09-mako/58
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-14.09-armhf/59
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-14.09-armhf/59/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/20064

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/ubuntu-system-settings-rtm-14.09-ci/80/rebuild

review: Needs Fixing (continuous-integration)

[lp:~ken-vandine/ubuntu-system-settings/rtm_click_framework_check]

1018. By Ken VanDine on 2015-04-30

cleaned up based on review feedback

[Manuel de la Peña (mandel)]

review: Approve

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:1017
http://jenkins.qa.ubuntu.com/job/ubuntu-system-settings-rtm-14.09-ci/81/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-14.09-touch/60
    SUCCESS: http://jenkins.qa.ubuntu.com/job/ubuntu-system-settings-rtm-14.09-14.09-armhf-ci/83
        deb: http://jenkins.qa.ubuntu.com/job/ubuntu-system-settings-rtm-14.09-14.09-armhf-ci/83/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-14.09-mako/59
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-14.09-armhf/60
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-14.09-armhf/60/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/20082

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/ubuntu-system-settings-rtm-14.09-ci/81/rebuild

review: Approve (continuous-integration)

[lp:~ken-vandine/ubuntu-system-settings/rtm_click_framework_check]

1019. By Ken VanDine on 2015-04-30

Fixed function rename missed in merge from trunk

=== modified file 'debian/control'
--- debian/control	2015-02-17 12:55:11 +0000
+++ debian/control	2015-04-30 14:58:55 +0000
@@ -40,6 +40,7 @@
                pep8,
                python3-pep8,
                pyflakes,
+               ubuntu-sdk-libs,
 Standards-Version: 3.9.6
 Homepage: https://launchpad.net/ubuntu-system-settings
 # If you aren't a member of ~system-settings-touch but need to upload packaging

=== modified file 'plugins/system-update/network/network.cpp'
--- plugins/system-update/network/network.cpp	2014-09-18 13:50:15 +0000
+++ plugins/system-update/network/network.cpp	2015-04-30 14:58:55 +0000
@@ -17,6 +17,7 @@
  */
 
 #include "network.h"
+#include <sstream>
 #include <QJsonDocument>
 #include <QJsonObject>
 #include <QJsonArray>
@@ -25,9 +26,13 @@
 #include <QUrl>
 #include <QProcessEnvironment>
 
-#define URL_APPS "https://myapps.developer.ubuntu.com/dev/api/click-metadata/"
-
-#define APPS_DATA "APPS_DATA"
+namespace {
+    const QString URL_APPS = "https://search.apps.ubuntu.com/api/v1/click-metadata";
+    const QString APPS_DATA = "APPS_DATA";
+    constexpr static const char* FRAMEWORKS_FOLDER {"/usr/share/click/frameworks/"};
+    constexpr static const char* FRAMEWORKS_PATTERN {"*.framework"};
+    constexpr static const int FRAMEWORKS_EXTENSION_LENGTH = 10; // strlen(".framework")
+}
 
 namespace UpdatePlugin {
 
@@ -39,10 +44,55 @@
                      this, SLOT(onReply(QNetworkReply*)));
 }
 
+std::string Network::getArchitecture()
+{
+    static const std::string deb_arch {architectureFromDpkg()};
+    return deb_arch;
+}
+
+std::vector<std::string> Network::getAvailableFrameworks()
+{
+    std::vector<std::string> result;
+    for (auto f: listFolder(FRAMEWORKS_FOLDER, FRAMEWORKS_PATTERN)) {
+        result.push_back(f.substr(0, f.size()-FRAMEWORKS_EXTENSION_LENGTH));
+    }
+    return result;
+}
+
+std::string Network::architectureFromDpkg()
+{
+    QString program("dpkg");
+    QStringList arguments;
+    arguments << "--print-architecture";
+    QProcess archDetector;
+    archDetector.start(program, arguments);
+    if(!archDetector.waitForFinished()) {
+        qWarning() << "Architecture detection failed.";
+    }
+    auto output = archDetector.readAllStandardOutput();
+    auto ostr = QString::fromUtf8(output);
+
+    return ostr.trimmed().toStdString();
+}
+
+std::vector<std::string> Network::listFolder(const std::string& folder, const std::string& pattern)
+{
+    std::vector<std::string> result;
+
+    QDir dir(QString::fromStdString(folder), QString::fromStdString(pattern),
+                                    QDir::Unsorted, QDir::Readable | QDir::Files);
+    QStringList entries = dir.entryList();
+    for (int i = 0; i < entries.size(); ++i) {
+        QString filename = entries.at(i);
+        result.push_back(filename.toStdString());
+    }
+
+    return result;
+}
+
 void Network::checkForNewVersions(QHash<QString, Update*> &apps)
 {
     m_apps = apps;
-
     QJsonObject serializer;
     QJsonArray array;
     foreach(QString id, m_apps.keys()) {
@@ -50,13 +100,18 @@
     }
 
     serializer.insert("name", array);
+    std::stringstream frameworks;
+    for (auto f: getAvailableFrameworks()) {
+        frameworks << "," << f;
+    }
     QJsonDocument doc(serializer);
-
     QByteArray content = doc.toJson();
 
     QString urlApps = getUrlApps();
     QNetworkRequest request;
     request.setHeader(QNetworkRequest::ContentTypeHeader, "application/json");
+    request.setRawHeader(QByteArray("X-Ubuntu-Frameworks"), QByteArray(frameworks.str().c_str()));
+    request.setRawHeader(QByteArray("X-Ubuntu-Architecture"), QByteArray(getArchitecture().c_str()));
     request.setUrl(QUrl(urlApps));
     RequestObject* reqObject = new RequestObject(QString(APPS_DATA));
     request.setOriginatingObject(reqObject);

=== modified file 'plugins/system-update/network/network.h'
--- plugins/system-update/network/network.h	2014-10-07 17:50:03 +0000
+++ plugins/system-update/network/network.h	2015-04-30 14:58:55 +0000
@@ -51,6 +51,8 @@
     void checkForNewVersions(QHash<QString, Update*> &apps);
     void getClickToken(Update *app, const QString &url,
                        const QString &authHeader);
+    virtual std::vector<std::string> getAvailableFrameworks();
+    virtual std::string getArchitecture();
 
 Q_SIGNALS:
     void updatesFound();
@@ -68,6 +70,11 @@
     QHash<QString, Update*> m_apps;
 
     QString getUrlApps();
+
+protected:
+    virtual std::string architectureFromDpkg();
+    virtual std::vector<std::string> listFolder(const std::string &folder, const std::string &pattern);
+
 };
 
 }

=== modified file 'tests/plugins/system-update/CMakeLists.txt'
--- tests/plugins/system-update/CMakeLists.txt	2014-10-23 13:27:05 +0000
+++ tests/plugins/system-update/CMakeLists.txt	2015-04-30 14:58:55 +0000
@@ -25,6 +25,12 @@
     ../../../plugins/system-update/update.cpp
 )
 
+add_executable(tst-network
+    tst_network.cpp
+    ../../../plugins/system-update/update.cpp
+    ../../../plugins/system-update/network/network.cpp
+)
+
 # set the path to the library folder
 include_directories(/usr/include/apt-pkg/)
 
@@ -36,6 +42,10 @@
 target_link_libraries(tst-update apt-pkg update-plugin)
 add_test(NAME tst-update COMMAND ${XVFB_CMD} ${CMAKE_CURRENT_BINARY_DIR}/tst-update)
 
+qt5_use_modules(tst-network Qml Quick Core DBus Xml Network Test)
+target_link_libraries(tst-network apt-pkg update-plugin)
+add_test(NAME tst-network COMMAND ${XVFB_CMD} ${CMAKE_CURRENT_BINARY_DIR}/tst-network)
+
 add_custom_command(
     TARGET tst-update-manager
     COMMAND ${CMAKE_COMMAND} -E copy ${CMAKE_CURRENT_LIST_DIR}/click.result ${CMAKE_CURRENT_BINARY_DIR}/click.result

=== added file 'tests/plugins/system-update/tst_network.cpp'
--- tests/plugins/system-update/tst_network.cpp	1970-01-01 00:00:00 +0000
+++ tests/plugins/system-update/tst_network.cpp	2015-04-30 14:58:55 +0000
@@ -0,0 +1,58 @@
+/*
+ * This file is part of system-settings
+ *
+ * Copyright (C) 2015 Canonical Ltd.
+ *
+ * This program is free software: you can redistribute it and/or modify it
+ * under the terms of the GNU General Public License version 3, as published
+ * by the Free Software Foundation.
+ *
+ * This program is distributed in the hope that it will be useful, but
+ * WITHOUT ANY WARRANTY; without even the implied warranties of
+ * MERCHANTABILITY, SATISFACTORY QUALITY, or FITNESS FOR A PARTICULAR
+ * PURPOSE.  See the GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License along
+ * with this program.  If not, see <http://www.gnu.org/licenses/>.
+ */
+
+#include <QDebug>
+#include <QtQml>
+#include <QtQml/QQmlContext>
+#include <QObject>
+#include <QTest>
+#include <QString>
+#include "network/network.h"
+#include "update.h"
+
+using namespace UpdatePlugin;
+
+class NetworkTest: public QObject
+{
+    Q_OBJECT
+
+public:
+    NetworkTest() {};
+
+private Q_SLOTS:
+    void testArch();
+    void testFrameworks();
+
+};
+
+void NetworkTest::testFrameworks()
+{
+    Network net;
+    auto frameworks = net.getAvailableFrameworks();
+    QCOMPARE(frameworks.empty(), false);
+}
+
+void NetworkTest::testArch()
+{
+    Network net;
+    auto arch = net.getArchitecture();
+    QCOMPARE(arch.empty(), false);
+}
+
+QTEST_MAIN(NetworkTest)
+#include "tst_network.moc"