Unit tests for the DownloadUpdateTask class

This commit is contained in:
Jan Dalheimer 2013-12-15 12:18:42 +01:00
parent f273334212
commit 3e8bcc1cf6
14 changed files with 325 additions and 59 deletions

View File

@ -125,6 +125,7 @@ private:
private: private:
friend class UpdateCheckerTest; friend class UpdateCheckerTest;
friend class DownloadUpdateTaskTest;
std::shared_ptr<QTranslator> m_qt_translator; std::shared_ptr<QTranslator> m_qt_translator;
std::shared_ptr<QTranslator> m_mmc_translator; std::shared_ptr<QTranslator> m_mmc_translator;

View File

@ -45,19 +45,12 @@ void DownloadUpdateTask::executeTask()
findCurrentVersionInfo(); findCurrentVersionInfo();
} }
void DownloadUpdateTask::findCurrentVersionInfo() void DownloadUpdateTask::processChannels()
{ {
setStatus(tr("Finding information about the current version."));
auto checker = MMC->updateChecker(); auto checker = MMC->updateChecker();
// This runs after we've tried loading the channel list.
// If the channel list doesn't need to be loaded, this will be called immediately.
// If the channel list does need to be loaded, this will be called when it's done.
auto processFunc = [this, &checker] () -> void
{
// Now, check the channel list again. // Now, check the channel list again.
if (checker->hasChannels()) if (!checker->hasChannels())
{ {
// We still couldn't load the channel list. Give up. Call loadVersionInfo and return. // We still couldn't load the channel list. Give up. Call loadVersionInfo and return.
QLOG_INFO() << "Reloading the channel list didn't work. Giving up."; QLOG_INFO() << "Reloading the channel list didn't work. Giving up.";
@ -74,26 +67,32 @@ void DownloadUpdateTask::findCurrentVersionInfo()
if (channel.id == channelId) if (channel.id == channelId)
{ {
QLOG_INFO() << "Found matching channel."; QLOG_INFO() << "Found matching channel.";
m_cRepoUrl = channel.url; m_cRepoUrl = preparePath(channel.url);
break; break;
} }
} }
// Now that we've done that, load version info. // Now that we've done that, load version info.
loadVersionInfo(); loadVersionInfo();
}; }
if (checker->hasChannels()) void DownloadUpdateTask::findCurrentVersionInfo()
{
setStatus(tr("Finding information about the current version."));
auto checker = MMC->updateChecker();
if (!checker->hasChannels())
{ {
// Load the channel list and wait for it to finish loading. // Load the channel list and wait for it to finish loading.
QLOG_INFO() << "No channel list entries found. Will try reloading it."; QLOG_INFO() << "No channel list entries found. Will try reloading it.";
QObject::connect(checker.get(), &UpdateChecker::channelListLoaded, processFunc); QObject::connect(checker.get(), &UpdateChecker::channelListLoaded, this, &DownloadUpdateTask::processChannels);
checker->updateChanList(); checker->updateChanList();
} }
else else
{ {
processFunc(); processChannels();
} }
} }
@ -152,12 +151,24 @@ void DownloadUpdateTask::parseDownloadedVersionInfo()
{ {
setStatus(tr("Reading file lists.")); setStatus(tr("Reading file lists."));
parseVersionInfo(NEW_VERSION, &m_nVersionFileList); setStatus(tr("Reading file list for new version."));
QLOG_DEBUG() << "Reading file list for new version.";
QString error;
if (!parseVersionInfo(std::dynamic_pointer_cast<ByteArrayDownload>(
m_vinfoNetJob->first())->m_data, &m_nVersionFileList, &error))
{
emitFailed(error);
return;
}
// If there is a second entry in the network job's list, load it as the current version's info. // If there is a second entry in the network job's list, load it as the current version's info.
if (m_vinfoNetJob->size() >= 2 && m_vinfoNetJob->operator[](1)->m_status != Job_Failed) if (m_vinfoNetJob->size() >= 2 && m_vinfoNetJob->operator[](1)->m_status != Job_Failed)
{ {
parseVersionInfo(CURRENT_VERSION, &m_cVersionFileList); setStatus(tr("Reading file list for current version."));
QLOG_DEBUG() << "Reading file list for current version.";
QString error;
parseVersionInfo(std::dynamic_pointer_cast<ByteArrayDownload>(
m_vinfoNetJob->operator[](1))->m_data, &m_cVersionFileList, &error);
} }
// We don't need this any more. // We don't need this any more.
@ -167,26 +178,15 @@ void DownloadUpdateTask::parseDownloadedVersionInfo()
processFileLists(); processFileLists();
} }
void DownloadUpdateTask::parseVersionInfo(VersionInfoFileEnum vfile, VersionFileList* list) bool DownloadUpdateTask::parseVersionInfo(const QByteArray &data, VersionFileList* list, QString *error)
{ {
if (vfile == CURRENT_VERSION) setStatus(tr("Reading file list for current version."));
else if (vfile == NEW_VERSION) setStatus(tr("Reading file list for new version."));
QLOG_DEBUG() << "Reading file list for" << (vfile == NEW_VERSION ? "new" : "current") << "version.";
QByteArray data;
{
ByteArrayDownloadPtr dl = std::dynamic_pointer_cast<ByteArrayDownload>(
vfile == NEW_VERSION ? m_vinfoNetJob->first() : m_vinfoNetJob->operator[](1));
data = dl->m_data;
}
QJsonParseError jsonError; QJsonParseError jsonError;
QJsonDocument jsonDoc = QJsonDocument::fromJson(data, &jsonError); QJsonDocument jsonDoc = QJsonDocument::fromJson(data, &jsonError);
if (jsonError.error != QJsonParseError::NoError) if (jsonError.error != QJsonParseError::NoError)
{ {
QLOG_ERROR() << "Failed to parse version info JSON:" << jsonError.errorString() << "at" << jsonError.offset; *error = QString("Failed to parse version info JSON: %1 at %2").arg(jsonError.errorString()).arg(jsonError.offset);
return; QLOG_ERROR() << error;
return false;
} }
QJsonObject json = jsonDoc.object(); QJsonObject json = jsonDoc.object();
@ -213,11 +213,11 @@ void DownloadUpdateTask::parseVersionInfo(VersionInfoFileEnum vfile, VersionFile
QString type = sourceObj.value("SourceType").toString(); QString type = sourceObj.value("SourceType").toString();
if (type == "http") if (type == "http")
{ {
file.sources.append(FileSource("http", sourceObj.value("Url").toString())); file.sources.append(FileSource("http", preparePath(sourceObj.value("Url").toString())));
} }
else if (type == "httpc") else if (type == "httpc")
{ {
file.sources.append(FileSource("httpc", sourceObj.value("Url").toString(), sourceObj.value("CompressionType").toString())); file.sources.append(FileSource("httpc", preparePath(sourceObj.value("Url").toString()), sourceObj.value("CompressionType").toString()));
} }
else else
{ {
@ -229,6 +229,8 @@ void DownloadUpdateTask::parseVersionInfo(VersionInfoFileEnum vfile, VersionFile
list->append(file); list->append(file);
} }
return true;
} }
void DownloadUpdateTask::processFileLists() void DownloadUpdateTask::processFileLists()
@ -312,7 +314,7 @@ void DownloadUpdateTask::processFileLists()
writeInstallScript(m_operationList, PathCombine(m_updateFilesDir.path(), "file_list.xml")); writeInstallScript(m_operationList, PathCombine(m_updateFilesDir.path(), "file_list.xml"));
} }
void DownloadUpdateTask::writeInstallScript(UpdateOperationList& opsList, QString scriptFile) bool DownloadUpdateTask::writeInstallScript(UpdateOperationList& opsList, QString scriptFile)
{ {
// Build the base structure of the XML document. // Build the base structure of the XML document.
QDomDocument doc; QDomDocument doc;
@ -377,7 +379,15 @@ void DownloadUpdateTask::writeInstallScript(UpdateOperationList& opsList, QStrin
else else
{ {
emitFailed(tr("Failed to write update script file.")); emitFailed(tr("Failed to write update script file."));
return false;
} }
return true;
}
QString DownloadUpdateTask::preparePath(const QString &path)
{
return QString(path).replace("$PWD", qApp->applicationDirPath());
} }
void DownloadUpdateTask::fileDownloadFinished() void DownloadUpdateTask::fileDownloadFinished()

View File

@ -34,7 +34,8 @@ public:
*/ */
QString updateFilesDir(); QString updateFilesDir();
protected: public:
// TODO: We should probably put these data structures into a separate header... // TODO: We should probably put these data structures into a separate header...
/*! /*!
@ -59,6 +60,7 @@ protected:
/*! /*!
* Structure that describes an entry in a GoUpdate version's `Files` list. * Structure that describes an entry in a GoUpdate version's `Files` list.
*/ */
struct VersionFileEntry struct VersionFileEntry
{ {
QString path; QString path;
@ -69,6 +71,8 @@ protected:
typedef QList<VersionFileEntry> VersionFileList; typedef QList<VersionFileEntry> VersionFileList;
protected:
friend class DownloadUpdateTaskTest;
/*! /*!
* Structure that describes an operation to perform when installing updates. * Structure that describes an operation to perform when installing updates.
@ -119,6 +123,13 @@ protected:
*/ */
virtual void findCurrentVersionInfo(); virtual void findCurrentVersionInfo();
/*!
* This runs after we've tried loading the channel list.
* If the channel list doesn't need to be loaded, this will be called immediately.
* If the channel list does need to be loaded, this will be called when it's done.
*/
void processChannels();
/*! /*!
* Downloads the version info files from the repository. * Downloads the version info files from the repository.
* The files for both the current build, and the build that we're updating to need to be downloaded. * The files for both the current build, and the build that we're updating to need to be downloaded.
@ -142,7 +153,7 @@ protected:
/*! /*!
* Loads the file list from the given version info JSON object into the given list. * Loads the file list from the given version info JSON object into the given list.
*/ */
virtual void parseVersionInfo(VersionInfoFileEnum vfile, VersionFileList* list); virtual bool parseVersionInfo(const QByteArray &data, VersionFileList* list, QString *error);
/*! /*!
* Takes a list of file entries for the current version's files and the new version's files * Takes a list of file entries for the current version's files and the new version's files
@ -153,7 +164,7 @@ protected:
/*! /*!
* Takes the operations list and writes an install script for the updater to the update files directory. * Takes the operations list and writes an install script for the updater to the update files directory.
*/ */
virtual void writeInstallScript(UpdateOperationList& opsList, QString scriptFile); virtual bool writeInstallScript(UpdateOperationList& opsList, QString scriptFile);
VersionFileList m_downloadList; VersionFileList m_downloadList;
UpdateOperationList m_operationList; UpdateOperationList m_operationList;
@ -181,6 +192,11 @@ protected:
*/ */
QTemporaryDir m_updateFilesDir; QTemporaryDir m_updateFilesDir;
/*!
* Substitutes $PWD for the application directory
*/
static QString preparePath(const QString &path);
protected slots: protected slots:
void vinfoDownloadFinished(); void vinfoDownloadFinished();
void vinfoDownloadFailed(); void vinfoDownloadFailed();

View File

@ -21,6 +21,7 @@ endmacro()
add_unit_test(pathutils tst_pathutils.cpp) add_unit_test(pathutils tst_pathutils.cpp)
add_unit_test(userutils tst_userutils.cpp) add_unit_test(userutils tst_userutils.cpp)
add_unit_test(UpdateChecker tst_UpdateChecker.cpp) add_unit_test(UpdateChecker tst_UpdateChecker.cpp)
add_unit_test(DownloadUpdateTask tst_DownloadUpdateTask.cpp)
# Tests END # # Tests END #

View File

@ -15,9 +15,14 @@ struct TestsInternal
f.open(QFile::ReadOnly); f.open(QFile::ReadOnly);
return f.readAll(); return f.readAll();
} }
static QString readFileUtf8(const QString &fileName)
{
return QString::fromUtf8(readFile(fileName));
}
}; };
#define MULTIMC_GET_TEST_FILE(file) TestsInternal::readFile(QFINDTESTDATA( file )) #define MULTIMC_GET_TEST_FILE(file) TestsInternal::readFile(QFINDTESTDATA( file ))
#define MULTIMC_GET_TEST_FILE_UTF8(file) TestsInternal::readFileUtf8(QFINDTESTDATA( file ))
#define QTEST_GUILESS_MAIN_MULTIMC(TestObject) \ #define QTEST_GUILESS_MAIN_MULTIMC(TestObject) \
int main(int argc, char *argv[]) \ int main(int argc, char *argv[]) \

43
tests/data/1.json Normal file
View File

@ -0,0 +1,43 @@
{
"ApiVersion": 0,
"Id": 1,
"Name": "1.0.1",
"Files": [
{
"Path": "fileOne",
"Sources": [
{
"SourceType": "http",
"Url": "file://$PWD/tests/data/fileOneA"
}
],
"Executable": true,
"Perms": 493,
"MD5": "9eb84090956c484e32cb6c08455a667b"
},
{
"Path": "fileTwo",
"Sources": [
{
"SourceType": "http",
"Url": "file://$PWD/tests/data/fileTwo"
}
],
"Executable": false,
"Perms": 644,
"MD5": "38f94f54fa3eb72b0ea836538c10b043"
},
{
"Path": "fileThree",
"Sources": [
{
"SourceType": "http",
"Url": "file://$PWD/tests/data/fileThree"
}
],
"Executable": false,
"Perms": "750",
"MD5": "f12df554b21e320be6471d7154130e70"
}
]
}

31
tests/data/2.json Normal file
View File

@ -0,0 +1,31 @@
{
"ApiVersion": 0,
"Id": 1,
"Name": "1.0.1",
"Files": [
{
"Path": "fileOne",
"Sources": [
{
"SourceType": "http",
"Url": "file://$PWD/tests/data/fileOneB"
}
],
"Executable": true,
"Perms": 493,
"MD5": "42915a71277c9016668cce7b82c6b577"
},
{
"Path": "fileTwo",
"Sources": [
{
"SourceType": "http",
"Url": "file://$PWD/tests/data/fileTwo"
}
],
"Executable": false,
"Perms": 644,
"MD5": "38f94f54fa3eb72b0ea836538c10b043"
}
]
}

View File

@ -5,7 +5,7 @@
"id": "develop", "id": "develop",
"name": "Develop", "name": "Develop",
"description": "The channel called \"develop\"", "description": "The channel called \"develop\"",
"url": "http://example.org/stuff" "url": "file://$PWD/tests/data/"
}, },
{ {
"id": "stable", "id": "stable",

1
tests/data/fileOneA Normal file
View File

@ -0,0 +1 @@
stuff

3
tests/data/fileOneB Normal file
View File

@ -0,0 +1,3 @@
stuff
more stuff that came in the new version

1
tests/data/fileThree Normal file
View File

@ -0,0 +1 @@
this is yet another file

1
tests/data/fileTwo Normal file
View File

@ -0,0 +1 @@
some other stuff

View File

@ -0,0 +1,17 @@
<update version="3">
<install>
<file>
<source>sourceOne</source>
<dest>destOne</dest>
<mode>0777</mode>
</file>
<file>
<source>MultiMC.exe</source>
<dest>M/u/l/t/i/M/C/e/x/e</dest>
<mode>0644</mode>
</file>
</install>
<uninstall>
<file>toDelete.abc</file>
</uninstall>
</update>

View File

@ -0,0 +1,136 @@
#include <QTest>
#include <QSignalSpy>
#include "TestUtil.h"
#include "logic/updater/DownloadUpdateTask.h"
#include "logic/updater/UpdateChecker.h"
Q_DECLARE_METATYPE(DownloadUpdateTask::VersionFileList)
bool operator==(const DownloadUpdateTask::FileSource &f1, const DownloadUpdateTask::FileSource &f2)
{
return f1.type == f2.type &&
f1.url == f2.url &&
f1.compressionType == f2.compressionType;
}
bool operator==(const DownloadUpdateTask::VersionFileEntry &v1, const DownloadUpdateTask::VersionFileEntry &v2)
{
return v1.path == v2.path &&
v1.mode == v2.mode &&
v1.sources == v2.sources &&
v1.md5 == v2.md5;
}
QDebug operator<<(QDebug dbg, const DownloadUpdateTask::FileSource &f)
{
dbg.nospace() << "FileSource(type=" << f.type << " url=" << f.url << " comp=" << f.compressionType << ")";
return dbg.maybeSpace();
}
QDebug operator<<(QDebug dbg, const DownloadUpdateTask::VersionFileEntry &v)
{
dbg.nospace() << "VersionFileEntry(path=" << v.path << " mode=" << v.mode << " md5=" << v.md5 << " sources=" << v.sources << ")";
return dbg.maybeSpace();
}
class DownloadUpdateTaskTest : public QObject
{
Q_OBJECT
private
slots:
void initTestCase()
{
}
void cleanupTestCase()
{
}
void test_writeInstallScript()
{
DownloadUpdateTask task(QUrl::fromLocalFile(QDir::current().absoluteFilePath("tests/data/")).toString(), 0);
DownloadUpdateTask::UpdateOperationList ops;
ops << DownloadUpdateTask::UpdateOperation::CopyOp("sourceOne", "destOne", 0777)
<< DownloadUpdateTask::UpdateOperation::CopyOp("MultiMC.exe", "M/u/l/t/i/M/C/e/x/e")
<< DownloadUpdateTask::UpdateOperation::DeleteOp("toDelete.abc");
const QString script = QDir::temp().absoluteFilePath("MultiMCUpdateScript.xml");
QVERIFY(task.writeInstallScript(ops, script));
QCOMPARE(TestsInternal::readFileUtf8(script), MULTIMC_GET_TEST_FILE_UTF8("tests/data/tst_DownloadUpdateTask-test_writeInstallScript.xml"));
}
void test_parseVersionInfo_data()
{
QTest::addColumn<QByteArray>("data");
QTest::addColumn<DownloadUpdateTask::VersionFileList>("list");
QTest::addColumn<QString>("error");
QTest::addColumn<bool>("ret");
QTest::newRow("one") << MULTIMC_GET_TEST_FILE("tests/data/1.json")
<< (DownloadUpdateTask::VersionFileList()
<< DownloadUpdateTask::VersionFileEntry{"fileOne", 493,
(DownloadUpdateTask::FileSourceList() << DownloadUpdateTask::FileSource("http", "file://" + qApp->applicationDirPath() + "/tests/data/fileOneA")),
"9eb84090956c484e32cb6c08455a667b"}
<< DownloadUpdateTask::VersionFileEntry{"fileTwo", 644,
(DownloadUpdateTask::FileSourceList() << DownloadUpdateTask::FileSource("http", "file://" + qApp->applicationDirPath() + "/tests/data/fileTwo")),
"38f94f54fa3eb72b0ea836538c10b043"}
<< DownloadUpdateTask::VersionFileEntry{"fileThree", 750,
(DownloadUpdateTask::FileSourceList() << DownloadUpdateTask::FileSource("http", "file://" + qApp->applicationDirPath() + "/tests/data/fileThree")),
"f12df554b21e320be6471d7154130e70"})
<< QString()
<< true;
QTest::newRow("two") << MULTIMC_GET_TEST_FILE("tests/data/2.json")
<< (DownloadUpdateTask::VersionFileList()
<< DownloadUpdateTask::VersionFileEntry{"fileOne", 493,
(DownloadUpdateTask::FileSourceList() << DownloadUpdateTask::FileSource("http", "file://" + qApp->applicationDirPath() + "/tests/data/fileOneB")),
"42915a71277c9016668cce7b82c6b577"}
<< DownloadUpdateTask::VersionFileEntry{"fileTwo", 644,
(DownloadUpdateTask::FileSourceList() << DownloadUpdateTask::FileSource("http", "file://" + qApp->applicationDirPath() + "/tests/data/fileTwo")),
"38f94f54fa3eb72b0ea836538c10b043"})
<< QString()
<< true;
}
void test_parseVersionInfo()
{
QFETCH(QByteArray, data);
QFETCH(DownloadUpdateTask::VersionFileList, list);
QFETCH(QString, error);
QFETCH(bool, ret);
DownloadUpdateTask::VersionFileList outList;
QString outError;
bool outRet = DownloadUpdateTask("", 0).parseVersionInfo(data, &outList, &outError);
QCOMPARE(outRet, ret);
QCOMPARE(outList, list);
QCOMPARE(outError, error);
}
void test_processFileLists()
{
// TODO create unit test for this
}
void test_masterTest()
{
QLOG_INFO() << "#####################";
MMC->m_version.build = 1;
MMC->m_version.channel = "develop";
MMC->updateChecker()->setChannelListUrl(QUrl::fromLocalFile(QDir::current().absoluteFilePath("tests/data/channels.json")).toString());
MMC->updateChecker()->setCurrentChannel("develop");
DownloadUpdateTask task(QUrl::fromLocalFile(QDir::current().absoluteFilePath("tests/data/")).toString(), 2);
QSignalSpy succeededSpy(&task, SIGNAL(succeeded()));
task.start();
QVERIFY(succeededSpy.wait());
}
};
QTEST_GUILESS_MAIN_MULTIMC(DownloadUpdateTaskTest)
#include "tst_DownloadUpdateTask.moc"