Add unit test for downloads

owncloud · Feb 3, 2022 · 57484ad · 57484ad
1 parent c49429d
commit 57484ad
Show file tree

Hide file tree

Showing 3 changed files with 69 additions and 5 deletions.
diff --git a/src/gui/updater/ocupdater.cpp b/src/gui/updater/ocupdater.cpp
@@ -175,7 +175,7 @@ QString OCUpdater::statusString() const
     }
 }
 
-int OCUpdater::downloadState() const
+OCUpdater::DownloadState OCUpdater::downloadState() const
 {
     return _state;
 }
@@ -360,12 +360,12 @@ void NSISUpdater::versionInfoArrived(const UpdateInfo &info)
         qCInfo(lcUpdater) << "Client is on latest version!";
         setDownloadState(UpToDate);
     } else {
-        QString url = info.downloadUrl();
+        const QString url = info.downloadUrl();
         if (url.isEmpty()) {
             showNoUrlDialog(info);
         } else {
             _targetFile = ConfigFile::configPath() + url.mid(url.lastIndexOf('/') + 1);
-            if (QFile(_targetFile).exists()) {
+            if (QFile::exists(_targetFile)) {
                 setDownloadState(DownloadComplete);
             } else {
                 auto request = QNetworkRequest(QUrl(url));

diff --git a/src/gui/updater/ocupdater.h b/src/gui/updater/ocupdater.h
@@ -97,6 +97,8 @@ class OCUpdater : public Updater
         DownloadFailed,
         DownloadTimedOut,
         UpdateOnlyAvailableThroughSystem };
+    Q_ENUM(DownloadState);
+
     explicit OCUpdater(const QUrl &url);
 
     void setUpdateUrl(const QUrl &url);
@@ -106,7 +108,7 @@ class OCUpdater : public Updater
     void checkForUpdate() override;
 
     QString statusString() const;
-    int downloadState() const;
+    DownloadState downloadState() const;
     void setDownloadState(DownloadState state);
 
 signals:
@@ -134,7 +136,7 @@ private slots:
 
 private:
     QUrl _updateUrl;
-    int _state;
+    DownloadState _state;
     QNetworkAccessManager *_accessManager;
     QTimer *_timeoutWatchdog; /** Timer to guard the timeout of an individual network request */
     UpdateInfo _updateInfo;
@@ -162,6 +164,8 @@ private slots:
     void versionInfoArrived(const UpdateInfo &info) override;
     QScopedPointer<QTemporaryFile> _file;
     QString _targetFile;
+
+    friend class TestUpdater;
 };
 
 /**

diff --git a/test/testupdater.cpp b/test/testupdater.cpp
@@ -0,0 +1,60 @@
+/*
+   This software is in the public domain, furnished "as is", without technical
+   support, and with no warranty, express or implied, as to its usefulness for
+   any purpose.
+*/
+
+#include <QtTest>
+
+#include "updater/updater.h"
+#include "updater/ocupdater.h"
+
+namespace OCC {
+
+class TestUpdater : public QObject
+{
+    Q_OBJECT
+
+private slots:
+    void testVersionToInt()
+    {
+        auto lowVersion = Updater::Helper::versionToInt(1, 2, 80, 3000);
+        QCOMPARE(Updater::Helper::stringVersionToInt("1.2.80.3000"), lowVersion);
+
+        auto highVersion = Updater::Helper::versionToInt(999, 2, 80, 3000);
+        auto currVersion = Updater::Helper::currentVersionToInt();
+        QVERIFY(currVersion > 0);
+        QVERIFY(currVersion > lowVersion);
+        QVERIFY(currVersion < highVersion);
+    }
+
+    void testDownload_data()
+    {
+        QTest::addColumn<QString>("url");
+        QTest::addColumn<OCUpdater::DownloadState>("result");
+        // a redirect to attic
+        QTest::newRow("redirect") << "https://download.owncloud.com/desktop/stable/ownCloud-2.2.4.6408-setup.exe" << OCUpdater::DownloadComplete;
+        QTest::newRow("broken url") << "https://&" << OCUpdater::DownloadFailed;
+    }
+
+    void testDownload()
+    {
+        QFETCH(QString, url);
+        QFETCH(OCUpdater::DownloadState, result);
+        UpdateInfo info;
+        info.setDownloadUrl(url);
+        info.setVersionString("ownCloud 2.2.4 (build 6408)");
+        // esnure we do the update
+        info.setVersion("100.2.4.6408");
+        auto *updater = new NSISUpdater({});
+        QSignalSpy downloadSpy(updater, &NSISUpdater::slotDownloadFinished);
+        updater->versionInfoArrived(info);
+        downloadSpy.wait();
+        QCOMPARE(updater->downloadState(), result);
+        updater->deleteLater();
+    }
+};
+}
+
+QTEST_MAIN(OCC::TestUpdater)
+#include "testupdater.moc"