mirror of
https://github.com/nextcloud/desktop.git
synced 2024-12-22 13:44:34 +03:00
b27eb06386
seems we have an issue with Windows and QTimer instances used to detect network timeout workaround, find cause of https://github.com/nextcloud/desktop/issues/7184 Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
186 lines
8 KiB
C++
186 lines
8 KiB
C++
/*
|
|
* 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 "syncenginetestutils.h"
|
|
#include <syncengine.h>
|
|
#include <localdiscoverytracker.h>
|
|
|
|
using namespace OCC;
|
|
|
|
struct FakeBrokenXmlPropfindReply : FakePropfindReply {
|
|
FakeBrokenXmlPropfindReply(FileInfo &remoteRootFileInfo, QNetworkAccessManager::Operation op,
|
|
const QNetworkRequest &request, QObject *parent)
|
|
: FakePropfindReply(remoteRootFileInfo, op, request, parent) {
|
|
QVERIFY(payload.size() > 50);
|
|
// turncate the XML
|
|
payload.chop(20);
|
|
}
|
|
};
|
|
|
|
struct MissingPermissionsPropfindReply : FakePropfindReply {
|
|
MissingPermissionsPropfindReply(FileInfo &remoteRootFileInfo, QNetworkAccessManager::Operation op,
|
|
const QNetworkRequest &request, QObject *parent)
|
|
: FakePropfindReply(remoteRootFileInfo, op, request, parent) {
|
|
// If the propfind contains a single file without permissions, this is a server error
|
|
const char toRemove[] = "<oc:permissions>RDNVCKW</oc:permissions>";
|
|
auto pos = payload.indexOf(toRemove, payload.size()/2);
|
|
QVERIFY(pos > 0);
|
|
payload.remove(pos, sizeof(toRemove) - 1);
|
|
}
|
|
};
|
|
|
|
|
|
enum ErrorKind : int {
|
|
// Lower code are corresponding to HTML error code
|
|
InvalidXML = 1000,
|
|
Timeout,
|
|
};
|
|
|
|
Q_DECLARE_METATYPE(ErrorCategory)
|
|
|
|
class TestRemoteDiscovery : public QObject
|
|
{
|
|
Q_OBJECT
|
|
|
|
private slots:
|
|
void initTestCase()
|
|
{
|
|
AbstractNetworkJob::enableTimeout = true;
|
|
|
|
OCC::Logger::instance()->setLogFlush(true);
|
|
OCC::Logger::instance()->setLogDebug(true);
|
|
|
|
QStandardPaths::setTestModeEnabled(true);
|
|
}
|
|
|
|
void testRemoteDiscoveryError_data()
|
|
{
|
|
qRegisterMetaType<ErrorCategory>();
|
|
QTest::addColumn<int>("errorKind");
|
|
QTest::addColumn<QString>("expectedErrorString");
|
|
QTest::addColumn<bool>("syncSucceeds");
|
|
|
|
QString itemErrorMessage = "Internal Server Fake Error";
|
|
|
|
QTest::newRow("400") << 400 << itemErrorMessage << false;
|
|
QTest::newRow("401") << 401 << itemErrorMessage << false;
|
|
QTest::newRow("403") << 403 << itemErrorMessage << true;
|
|
QTest::newRow("404") << 404 << itemErrorMessage << true;
|
|
QTest::newRow("500") << 500 << itemErrorMessage << true;
|
|
QTest::newRow("503") << 503 << itemErrorMessage << true;
|
|
// 200 should be an error since propfind should return 207
|
|
QTest::newRow("200") << 200 << itemErrorMessage << false;
|
|
QTest::newRow("InvalidXML") << +InvalidXML << "Unknown error" << false;
|
|
QTest::newRow("Timeout") << +Timeout << "Operation canceled" << false;
|
|
}
|
|
|
|
|
|
// Check what happens when there is an error.
|
|
void testRemoteDiscoveryError()
|
|
{
|
|
QFETCH(int, errorKind);
|
|
QFETCH(QString, expectedErrorString);
|
|
QFETCH(bool, syncSucceeds);
|
|
|
|
FakeFolder fakeFolder{ FileInfo::A12_B12_C12_S12() };
|
|
|
|
// Do Some change as well
|
|
fakeFolder.localModifier().insert("A/z1");
|
|
fakeFolder.localModifier().insert("B/z1");
|
|
fakeFolder.localModifier().insert("C/z1");
|
|
fakeFolder.remoteModifier().insert("A/z2");
|
|
fakeFolder.remoteModifier().insert("B/z2");
|
|
fakeFolder.remoteModifier().insert("C/z2");
|
|
|
|
auto oldLocalState = fakeFolder.currentLocalState();
|
|
auto oldRemoteState = fakeFolder.currentRemoteState();
|
|
|
|
QString errorFolder = "dav/files/admin/B";
|
|
QString fatalErrorPrefix = "Server replied with an error while reading directory \"B\" : ";
|
|
fakeFolder.setServerOverride([&](QNetworkAccessManager::Operation op, const QNetworkRequest &req, QIODevice *)
|
|
-> QNetworkReply *{
|
|
if (req.attribute(QNetworkRequest::CustomVerbAttribute).toString() == "PROPFIND" && req.url().path().endsWith(errorFolder)) {
|
|
if (errorKind == InvalidXML) {
|
|
return new FakeBrokenXmlPropfindReply(fakeFolder.remoteModifier(), op, req, this);
|
|
} else if (errorKind == Timeout) {
|
|
return new FakeHangingReply(op, req, this);
|
|
} else if (errorKind < 1000) {
|
|
return new FakeErrorReply(op, req, this, errorKind);
|
|
}
|
|
}
|
|
return nullptr;
|
|
});
|
|
|
|
// So the test that test timeout finishes fast
|
|
QScopedValueRollback<int> setHttpTimeout(AbstractNetworkJob::httpTimeout, errorKind == Timeout ? 1 : 10000);
|
|
|
|
ItemCompletedSpy completeSpy(fakeFolder);
|
|
QSignalSpy errorSpy(&fakeFolder.syncEngine(), &SyncEngine::syncError);
|
|
QCOMPARE(fakeFolder.syncOnce(), syncSucceeds);
|
|
|
|
// The folder B should not have been sync'ed (and in particular not removed)
|
|
QCOMPARE(oldLocalState.children["B"], fakeFolder.currentLocalState().children["B"]);
|
|
QCOMPARE(oldRemoteState.children["B"], fakeFolder.currentRemoteState().children["B"]);
|
|
if (!syncSucceeds) {
|
|
QCOMPARE(errorSpy.size(), 1);
|
|
QCOMPARE(errorSpy[0][0].toString(), QString(fatalErrorPrefix + expectedErrorString));
|
|
} else {
|
|
QCOMPARE(completeSpy.findItem("B")->_instruction, CSYNC_INSTRUCTION_IGNORE);
|
|
QVERIFY(completeSpy.findItem("B")->_errorString.contains(expectedErrorString));
|
|
|
|
// The other folder should have been sync'ed as the sync just ignored the faulty dir
|
|
QCOMPARE(fakeFolder.currentRemoteState().children["A"], fakeFolder.currentLocalState().children["A"]);
|
|
QCOMPARE(fakeFolder.currentRemoteState().children["C"], fakeFolder.currentLocalState().children["C"]);
|
|
QCOMPARE(completeSpy.findItem("A/z1")->_instruction, CSYNC_INSTRUCTION_NEW);
|
|
}
|
|
|
|
//
|
|
// Check the same discovery error on the sync root
|
|
//
|
|
errorFolder = "dav/files/admin/";
|
|
fatalErrorPrefix = "Server replied with an error while reading directory \"\" : ";
|
|
errorSpy.clear();
|
|
QVERIFY(!fakeFolder.syncOnce());
|
|
QCOMPARE(errorSpy.size(), 1);
|
|
QCOMPARE(errorSpy[0][0].toString(), QString(fatalErrorPrefix + expectedErrorString));
|
|
}
|
|
|
|
void testMissingData()
|
|
{
|
|
FakeFolder fakeFolder{ FileInfo() };
|
|
fakeFolder.remoteModifier().insert("good");
|
|
fakeFolder.remoteModifier().insert("noetag");
|
|
fakeFolder.remoteModifier().find("noetag")->etag.clear();
|
|
fakeFolder.remoteModifier().insert("nofileid");
|
|
fakeFolder.remoteModifier().find("nofileid")->fileId.clear();
|
|
fakeFolder.remoteModifier().mkdir("nopermissions");
|
|
fakeFolder.remoteModifier().insert("nopermissions/A");
|
|
|
|
fakeFolder.setServerOverride([&](QNetworkAccessManager::Operation op, const QNetworkRequest &req, QIODevice *)
|
|
-> QNetworkReply *{
|
|
if (req.attribute(QNetworkRequest::CustomVerbAttribute).toString() == "PROPFIND" && req.url().path().endsWith("nopermissions"))
|
|
return new MissingPermissionsPropfindReply(fakeFolder.remoteModifier(), op, req, this);
|
|
return nullptr;
|
|
});
|
|
|
|
ItemCompletedSpy completeSpy(fakeFolder);
|
|
QVERIFY(!fakeFolder.syncOnce());
|
|
|
|
QCOMPARE(completeSpy.findItem("good")->_instruction, CSYNC_INSTRUCTION_NEW);
|
|
QCOMPARE(completeSpy.findItem("noetag")->_instruction, CSYNC_INSTRUCTION_ERROR);
|
|
QCOMPARE(completeSpy.findItem("nofileid")->_instruction, CSYNC_INSTRUCTION_ERROR);
|
|
QCOMPARE(completeSpy.findItem("nopermissions")->_instruction, CSYNC_INSTRUCTION_NEW);
|
|
QCOMPARE(completeSpy.findItem("nopermissions/A")->_instruction, CSYNC_INSTRUCTION_ERROR);
|
|
QVERIFY(completeSpy.findItem("noetag")->_errorString.contains("ETag"));
|
|
QVERIFY(completeSpy.findItem("nofileid")->_errorString.contains("file id"));
|
|
QVERIFY(completeSpy.findItem("nopermissions/A")->_errorString.contains("permission"));
|
|
}
|
|
};
|
|
|
|
QTEST_GUILESS_MAIN(TestRemoteDiscovery)
|
|
#include "testremotediscovery.moc"
|