245 lines
12 KiB
C++
245 lines
12 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 "testutils/syncenginetestutils.h"
|
|
#include "testutils/testutils.h"
|
|
|
|
#include <syncengine.h>
|
|
#include <localdiscoverytracker.h>
|
|
|
|
#include <QtTest>
|
|
|
|
using namespace OCC;
|
|
|
|
class TestLocalDiscovery : public QObject
|
|
{
|
|
Q_OBJECT
|
|
|
|
private Q_SLOTS:
|
|
void initTestCase_data()
|
|
{
|
|
QTest::addColumn<Vfs::Mode>("vfsMode");
|
|
QTest::addColumn<bool>("filesAreDehydrated");
|
|
|
|
QTest::newRow("Vfs::Mode::Off") << Vfs::Mode::Off << false;
|
|
|
|
if (VfsPluginManager::instance().isVfsPluginAvailable(Vfs::Mode::WindowsCfApi)) {
|
|
QTest::newRow("Vfs::Mode::WindowsCfApi dehydrated") << Vfs::Mode::WindowsCfApi << true;
|
|
|
|
// TODO: the hydrated version will fail due to an issue in the winvfs plugin, so leave it disabled for now.
|
|
// QTest::newRow("Vfs::Mode::WindowsCfApi hydrated") << Vfs::Mode::WindowsCfApi << false;
|
|
} else if (Utility::isWindows()) {
|
|
qWarning("Skipping Vfs::Mode::WindowsCfApi");
|
|
}
|
|
}
|
|
|
|
/// Check correct behavior when local discovery is partially drawn from the db
|
|
void testLocalDiscoveryStyle()
|
|
{
|
|
QFETCH_GLOBAL(Vfs::Mode, vfsMode);
|
|
QFETCH_GLOBAL(bool, filesAreDehydrated);
|
|
|
|
FakeFolder fakeFolder(FileInfo::A12_B12_C12_S12(), vfsMode, filesAreDehydrated);
|
|
|
|
LocalDiscoveryTracker tracker;
|
|
connect(&fakeFolder.syncEngine(), &SyncEngine::itemCompleted, &tracker, &LocalDiscoveryTracker::slotItemCompleted);
|
|
connect(&fakeFolder.syncEngine(), &SyncEngine::finished, &tracker, &LocalDiscoveryTracker::slotSyncFinished);
|
|
|
|
// More subdirectories are useful for testing
|
|
fakeFolder.localModifier().mkdir(QStringLiteral("A/X"));
|
|
fakeFolder.localModifier().mkdir(QStringLiteral("A/Y"));
|
|
fakeFolder.localModifier().insert(QStringLiteral("A/X/x1"));
|
|
fakeFolder.localModifier().insert(QStringLiteral("A/Y/y1"));
|
|
tracker.addTouchedPath(QStringLiteral("A/X"));
|
|
|
|
tracker.startSyncFullDiscovery();
|
|
QVERIFY(fakeFolder.applyLocalModificationsAndSync());
|
|
|
|
QCOMPARE(fakeFolder.currentLocalState(), fakeFolder.currentRemoteState());
|
|
QVERIFY(tracker.localDiscoveryPaths().empty());
|
|
|
|
// Test begins
|
|
fakeFolder.localModifier().insert(QStringLiteral("A/a3"));
|
|
fakeFolder.localModifier().insert(QStringLiteral("A/X/x2"));
|
|
fakeFolder.localModifier().insert(QStringLiteral("A/Y/y2"));
|
|
fakeFolder.localModifier().insert(QStringLiteral("B/b3"));
|
|
fakeFolder.remoteModifier().insert(QStringLiteral("C/c3"));
|
|
fakeFolder.remoteModifier().appendByte(QStringLiteral("C/c1"));
|
|
tracker.addTouchedPath(QStringLiteral("A/X"));
|
|
|
|
fakeFolder.syncEngine().setLocalDiscoveryOptions(LocalDiscoveryStyle::DatabaseAndFilesystem, tracker.localDiscoveryPaths());
|
|
tracker.startSyncPartialDiscovery();
|
|
QVERIFY(fakeFolder.applyLocalModificationsAndSync());
|
|
|
|
QVERIFY(fakeFolder.currentRemoteState().find(QStringLiteral("A/a3")));
|
|
QVERIFY(fakeFolder.currentRemoteState().find(QStringLiteral("A/X/x2")));
|
|
QVERIFY(!fakeFolder.currentRemoteState().find(QStringLiteral("A/Y/y2")));
|
|
QVERIFY(!fakeFolder.currentRemoteState().find(QStringLiteral("B/b3")));
|
|
QVERIFY(fakeFolder.currentLocalState().find(QStringLiteral("C/c3")));
|
|
QCOMPARE(fakeFolder.syncEngine().lastLocalDiscoveryStyle(), LocalDiscoveryStyle::DatabaseAndFilesystem);
|
|
QVERIFY(tracker.localDiscoveryPaths().empty());
|
|
|
|
QVERIFY(fakeFolder.applyLocalModificationsAndSync());
|
|
QCOMPARE(fakeFolder.currentLocalState(), fakeFolder.currentRemoteState());
|
|
QCOMPARE(fakeFolder.syncEngine().lastLocalDiscoveryStyle(), LocalDiscoveryStyle::FilesystemOnly);
|
|
QVERIFY(tracker.localDiscoveryPaths().empty());
|
|
}
|
|
|
|
void testLocalDiscoveryDecision()
|
|
{
|
|
QFETCH_GLOBAL(Vfs::Mode, vfsMode);
|
|
QFETCH_GLOBAL(bool, filesAreDehydrated);
|
|
|
|
FakeFolder fakeFolder(FileInfo::A12_B12_C12_S12(), vfsMode, filesAreDehydrated);
|
|
auto &engine = fakeFolder.syncEngine();
|
|
|
|
QVERIFY(engine.shouldDiscoverLocally(QString()));
|
|
QVERIFY(engine.shouldDiscoverLocally(QStringLiteral("A")));
|
|
QVERIFY(engine.shouldDiscoverLocally(QStringLiteral("A/X")));
|
|
|
|
fakeFolder.syncEngine().setLocalDiscoveryOptions(LocalDiscoveryStyle::DatabaseAndFilesystem,
|
|
{QStringLiteral("A/X"), QStringLiteral("A/X space"), QStringLiteral("A/X/beta"), QStringLiteral("foo bar space/touch"), QStringLiteral("foo/"),
|
|
QStringLiteral("zzz"), QStringLiteral("zzzz")});
|
|
|
|
QVERIFY(engine.shouldDiscoverLocally(QString()));
|
|
QVERIFY(engine.shouldDiscoverLocally(QStringLiteral("A")));
|
|
QVERIFY(engine.shouldDiscoverLocally(QStringLiteral("A/X")));
|
|
QVERIFY(!engine.shouldDiscoverLocally(QStringLiteral("B")));
|
|
QVERIFY(!engine.shouldDiscoverLocally(QStringLiteral("A B")));
|
|
QVERIFY(!engine.shouldDiscoverLocally(QStringLiteral("B/X")));
|
|
QVERIFY(engine.shouldDiscoverLocally(QStringLiteral("foo bar space")));
|
|
QVERIFY(engine.shouldDiscoverLocally(QStringLiteral("foo")));
|
|
QVERIFY(!engine.shouldDiscoverLocally(QStringLiteral("foo bar")));
|
|
QVERIFY(!engine.shouldDiscoverLocally(QStringLiteral("foo bar/touch")));
|
|
// These are within QStringLiteral("A/X") so they should be discovered
|
|
QVERIFY(engine.shouldDiscoverLocally(QStringLiteral("A/X/alpha")));
|
|
QVERIFY(engine.shouldDiscoverLocally(QStringLiteral("A/X beta")));
|
|
QVERIFY(engine.shouldDiscoverLocally(QStringLiteral("A/X/Y")));
|
|
QVERIFY(engine.shouldDiscoverLocally(QStringLiteral("A/X space")));
|
|
QVERIFY(engine.shouldDiscoverLocally(QStringLiteral("A/X space/alpha")));
|
|
QVERIFY(!engine.shouldDiscoverLocally(QStringLiteral("A/Xylo/foo")));
|
|
QVERIFY(engine.shouldDiscoverLocally(QStringLiteral("zzzz/hello")));
|
|
QVERIFY(!engine.shouldDiscoverLocally(QStringLiteral("zzza/hello")));
|
|
|
|
QEXPECT_FAIL("", "There is a possibility of false positives if the set contains a path "
|
|
"which is a prefix, and that prefix is followed by a character less than '/'", Continue);
|
|
QVERIFY(!engine.shouldDiscoverLocally(QStringLiteral("A/X o")));
|
|
|
|
fakeFolder.syncEngine().setLocalDiscoveryOptions(
|
|
LocalDiscoveryStyle::DatabaseAndFilesystem,
|
|
{});
|
|
|
|
QVERIFY(!engine.shouldDiscoverLocally(QString()));
|
|
}
|
|
|
|
// Check whether item success and item failure adjusts the
|
|
// tracker correctly.
|
|
void testTrackerItemCompletion()
|
|
{
|
|
QFETCH_GLOBAL(Vfs::Mode, vfsMode);
|
|
QFETCH_GLOBAL(bool, filesAreDehydrated);
|
|
|
|
FakeFolder fakeFolder(FileInfo::A12_B12_C12_S12(), vfsMode, filesAreDehydrated);
|
|
|
|
LocalDiscoveryTracker tracker;
|
|
connect(&fakeFolder.syncEngine(), &SyncEngine::itemCompleted, &tracker, &LocalDiscoveryTracker::slotItemCompleted);
|
|
connect(&fakeFolder.syncEngine(), &SyncEngine::finished, &tracker, &LocalDiscoveryTracker::slotSyncFinished);
|
|
auto trackerContains = [&](const char *path) {
|
|
return tracker.localDiscoveryPaths().find(QString::fromLatin1(path)) != tracker.localDiscoveryPaths().end();
|
|
};
|
|
|
|
tracker.addTouchedPath(QStringLiteral("A/spurious"));
|
|
|
|
fakeFolder.localModifier().insert(QStringLiteral("A/a3"));
|
|
tracker.addTouchedPath(QStringLiteral("A/a3"));
|
|
|
|
fakeFolder.localModifier().insert(QStringLiteral("A/a4"));
|
|
fakeFolder.serverErrorPaths().append(QStringLiteral("A/a4"));
|
|
// We're not adding a4 as touched, it's in the same folder as a3 and will be seen.
|
|
// And due to the error it should be added to the explicit list while a3 gets removed.
|
|
|
|
fakeFolder.syncEngine().setLocalDiscoveryOptions(LocalDiscoveryStyle::DatabaseAndFilesystem, tracker.localDiscoveryPaths());
|
|
tracker.startSyncPartialDiscovery();
|
|
QVERIFY(!fakeFolder.applyLocalModificationsAndSync());
|
|
|
|
QVERIFY(fakeFolder.currentRemoteState().find(QStringLiteral("A/a3")));
|
|
QVERIFY(!fakeFolder.currentRemoteState().find(QStringLiteral("A/a4")));
|
|
QVERIFY(!trackerContains("A/a3"));
|
|
QVERIFY(trackerContains("A/a4"));
|
|
QVERIFY(trackerContains("A/spurious")); // not removed since overall sync not successful
|
|
|
|
fakeFolder.syncEngine().setLocalDiscoveryOptions(LocalDiscoveryStyle::FilesystemOnly);
|
|
tracker.startSyncFullDiscovery();
|
|
QVERIFY(!fakeFolder.applyLocalModificationsAndSync());
|
|
|
|
QVERIFY(!fakeFolder.currentRemoteState().find(QStringLiteral("A/a4")));
|
|
QVERIFY(trackerContains("A/a4")); // had an error, still here
|
|
QVERIFY(!trackerContains("A/spurious")); // removed due to full discovery
|
|
|
|
fakeFolder.serverErrorPaths().clear();
|
|
fakeFolder.syncJournal().wipeErrorBlacklist();
|
|
tracker.addTouchedPath(QStringLiteral("A/newspurious")); // will be removed due to successful sync
|
|
|
|
fakeFolder.syncEngine().setLocalDiscoveryOptions(LocalDiscoveryStyle::DatabaseAndFilesystem, tracker.localDiscoveryPaths());
|
|
tracker.startSyncPartialDiscovery();
|
|
QVERIFY(fakeFolder.applyLocalModificationsAndSync());
|
|
|
|
QVERIFY(fakeFolder.currentRemoteState().find(QStringLiteral("A/a4")));
|
|
QVERIFY(tracker.localDiscoveryPaths().empty());
|
|
}
|
|
|
|
void testDirectoryAndSubDirectory()
|
|
{
|
|
QFETCH_GLOBAL(Vfs::Mode, vfsMode);
|
|
QFETCH_GLOBAL(bool, filesAreDehydrated);
|
|
|
|
FakeFolder fakeFolder(FileInfo::A12_B12_C12_S12(), vfsMode, filesAreDehydrated);
|
|
|
|
fakeFolder.localModifier().mkdir(QStringLiteral("A/newDir"));
|
|
fakeFolder.localModifier().mkdir(QStringLiteral("A/newDir/subDir"));
|
|
fakeFolder.localModifier().insert(QStringLiteral("A/newDir/subDir/file"), 10_B);
|
|
|
|
// Only "A" was modified according to the file system tracker
|
|
fakeFolder.syncEngine().setLocalDiscoveryOptions(
|
|
LocalDiscoveryStyle::DatabaseAndFilesystem,
|
|
{ QStringLiteral("A") });
|
|
|
|
QVERIFY(fakeFolder.applyLocalModificationsAndSync());
|
|
|
|
QCOMPARE(fakeFolder.currentLocalState(), fakeFolder.currentRemoteState());
|
|
QVERIFY(fakeFolder.currentRemoteState().find(QStringLiteral("A/newDir/subDir/file")));
|
|
}
|
|
|
|
// Tests the behavior of invalid filename detection
|
|
void testServerBlacklist()
|
|
{
|
|
QFETCH_GLOBAL(Vfs::Mode, vfsMode);
|
|
QFETCH_GLOBAL(bool, filesAreDehydrated);
|
|
|
|
FakeFolder fakeFolder(FileInfo::A12_B12_C12_S12(), vfsMode, filesAreDehydrated);
|
|
QCOMPARE(fakeFolder.currentLocalState(), fakeFolder.currentRemoteState());
|
|
|
|
auto cap = TestUtils::testCapabilities();
|
|
cap.insert(
|
|
{{QStringLiteral("files"), QVariantMap{{QStringLiteral("blacklisted_files"), QVariantList{QStringLiteral(".foo"), QStringLiteral("bar")}}}}});
|
|
fakeFolder.account()->setCapabilities({fakeFolder.account()->url(), cap});
|
|
fakeFolder.localModifier().insert(QStringLiteral("C/.foo"));
|
|
fakeFolder.localModifier().insert(QStringLiteral("C/bar"));
|
|
fakeFolder.localModifier().insert(QStringLiteral("C/moo"));
|
|
fakeFolder.localModifier().insert(QStringLiteral("C/.moo"));
|
|
|
|
QVERIFY(fakeFolder.applyLocalModificationsAndSync());
|
|
QVERIFY(fakeFolder.currentRemoteState().find(QStringLiteral("C/moo")));
|
|
QVERIFY(fakeFolder.currentRemoteState().find(QStringLiteral("C/.moo")));
|
|
QVERIFY(!fakeFolder.currentRemoteState().find(QStringLiteral("C/.foo")));
|
|
QVERIFY(!fakeFolder.currentRemoteState().find(QStringLiteral("C/bar")));
|
|
}
|
|
};
|
|
|
|
QTEST_GUILESS_MAIN(TestLocalDiscovery)
|
|
#include "testlocaldiscovery.moc"
|