mirrors/nextcloud-desktop
1
0
Fork 0
mirror of https://github.com/nextcloud/desktop.git synced 2024-11-26 06:55:59 +03:00
Code Issues Projects Releases Packages Wiki Activity
nextcloud-desktop/test/testfolderman.cpp

Ignoring revisions in .git-blame-ignore-revs. Click here to bypass and see the normal blame view.

318 lines
14 KiB
C++
Raw Normal View History

FolderMan::checkPathValidityForNewFolder factorize the function to check the validity of a new folder out of the folder wizard to the FolderMan. There is even a test for it now. That way we can also use it in the normal wizard
2015-07-13 15:35:19 +03:00
/*
* 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.
*
*/
Fix test build... again. The issue was unrelated to moc! We just need to make sure QT_VERSION etc. are actually defined when we use them for a test!
2015-07-15 10:47:42 +03:00
#include <qglobal.h>
FolderMan::checkPathValidityForNewFolder factorize the function to check the validity of a new folder out of the folder wizard to the FolderMan. There is even a test for it now. That way we can also use it in the normal wizard
2015-07-13 15:35:19 +03:00
#include <QTemporaryDir>
#include <QtTest>
Move Utility to a new common static library Now that csync builds as C++, this will avoid having to implement functionalities needed by csync mandatorily in csync itself. This library is built as part of libocsync and symbols exported through it. This requires a relicense of Utility as LGPL. All classes moved into this library from src/libsync will need to be relicensed as well.
2017-08-16 09:36:52 +03:00
#include "common/utility.h"
FolderMan::checkPathValidityForNewFolder factorize the function to check the validity of a new folder out of the folder wizard to the FolderMan. There is even a test for it now. That way we can also use it in the normal wizard
2015-07-13 15:35:19 +03:00
#include "folderman.h"
Make the AccountState a construction argument of the Folder This will help moving the SyncEngine construction in the constructor and allow moving functionalities from Folder to SyncEngine or its delegated objects.
2016-03-16 21:07:40 +03:00
#include "account.h"
FolderMan::checkPathValidityForNewFolder factorize the function to check the validity of a new folder out of the folder wizard to the FolderMan. There is even a test for it now. That way we can also use it in the normal wizard
2015-07-13 15:35:19 +03:00
#include "accountstate.h"
Implement 'Leave this share' context menu entry. Fix incorrect sharing state for incoming and my shares in custom state icons and local database. Signed-off-by: allexzander <blackslayer4@gmail.com>
2022-10-21 17:11:29 +03:00
#include <accountmanager.h>
TestFolderMan: don't pollute user's settings
2016-10-11 12:55:20 +03:00
#include "configfile.h"
Implement 'Leave this share' context menu entry. Fix incorrect sharing state for incoming and my shares in custom state icons and local database. Signed-off-by: allexzander <blackslayer4@gmail.com>
2022-10-21 17:11:29 +03:00
#include "syncenginetestutils.h"
Once client gets 401/403 from the server, check if remote wipe was requested. - When the the users logs because of 401 or 403 errors, it checks if the server requested the remote wipe. If yes, locally deletes account and folders connected to the account and notify the server. If no, proceeds to ask the user to login again. - The app password is restored in the keychain. - WIP: The change also includes a test class for RemoteWipe. Signed-off-by: Camila San <hello@camila.codes>
2019-07-24 14:56:21 +03:00
#include "testhelper.h"
Fix the folderman test, adopting the checkPathValidity method tests.
2016-10-07 17:24:09 +03:00
FolderMan::checkPathValidityForNewFolder factorize the function to check the validity of a new folder out of the folder wizard to the FolderMan. There is even a test for it now. That way we can also use it in the normal wizard
2015-07-13 15:35:19 +03:00
using namespace OCC;
class TestFolderMan: public QObject
{
Q_OBJECT
FolderMan _fm;
Implement 'Leave this share' context menu entry. Fix incorrect sharing state for incoming and my shares in custom state icons and local database. Signed-off-by: allexzander <blackslayer4@gmail.com>
2022-10-21 17:11:29 +03:00
signals:
void incomingShareDeleted();
FolderMan::checkPathValidityForNewFolder factorize the function to check the validity of a new folder out of the folder wizard to the FolderMan. There is even a test for it now. That way we can also use it in the normal wizard
2015-07-13 15:35:19 +03:00
private slots:
Implement 'Leave this share' context menu entry. Fix incorrect sharing state for incoming and my shares in custom state icons and local database. Signed-off-by: allexzander <blackslayer4@gmail.com>
2022-10-21 17:11:29 +03:00
void testLeaveShare()
{
prevents modifying real user config file Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
2022-11-23 18:56:35 +03:00
QTemporaryDir dir;
ConfigFile::setConfDir(dir.path()); // we don't want to pollute the user's config file
Implement 'Leave this share' context menu entry. Fix incorrect sharing state for incoming and my shares in custom state icons and local database. Signed-off-by: allexzander <blackslayer4@gmail.com>
2022-10-21 17:11:29 +03:00
constexpr auto firstSharePath = "A/sharedwithme_A.txt";
constexpr auto secondSharePath = "A/B/sharedwithme_B.data";
QScopedPointer<FakeQNAM> fakeQnam(new FakeQNAM({}));
OCC::AccountPtr account = OCC::Account::create();
account->setCredentials(new FakeCredentials{fakeQnam.data()});
account->setUrl(QUrl(("http://example.de")));
OCC::AccountManager::instance()->addAccount(account);
FakeFolder fakeFolder{FileInfo{}};
fakeFolder.remoteModifier().mkdir("A");
fakeFolder.remoteModifier().insert(firstSharePath, 100);
const auto firstShare = fakeFolder.remoteModifier().find(firstSharePath);
QVERIFY(firstShare);
firstShare->permissions.setPermission(OCC::RemotePermissions::IsShared);
fakeFolder.remoteModifier().mkdir("A/B");
fakeFolder.remoteModifier().insert(secondSharePath, 100);
const auto secondShare = fakeFolder.remoteModifier().find(secondSharePath);
QVERIFY(secondShare);
secondShare->permissions.setPermission(OCC::RemotePermissions::IsShared);
FolderMan *folderman = FolderMan::instance();
QCOMPARE(folderman, &_fm);
OCC::AccountState *accountState = OCC::AccountManager::instance()->accounts().first().data();
const auto folder = folderman->addFolder(accountState, folderDefinition(fakeFolder.localPath()));
QVERIFY(folder);
auto realFolder = FolderMan::instance()->folderForPath(fakeFolder.localPath());
QVERIFY(realFolder);
QVERIFY(fakeFolder.syncOnce());
QCOMPARE(fakeFolder.currentLocalState(), fakeFolder.currentRemoteState());
fakeQnam->setOverride([this, accountState, &fakeFolder](QNetworkAccessManager::Operation op, const QNetworkRequest &req, QIODevice *device) {
Q_UNUSED(device);
QNetworkReply *reply = nullptr;
if (op != QNetworkAccessManager::DeleteOperation) {
reply = new FakeErrorReply(op, req, this, 405);
return reply;
}
if (req.url().path().isEmpty()) {
reply = new FakeErrorReply(op, req, this, 404);
return reply;
}
const auto filePathRelative = req.url().path().remove(accountState->account()->davPath());
const auto foundFileInRemoteFolder = fakeFolder.remoteModifier().find(filePathRelative);
if (filePathRelative.isEmpty() || !foundFileInRemoteFolder) {
reply = new FakeErrorReply(op, req, this, 404);
return reply;
}
fakeFolder.remoteModifier().remove(filePathRelative);
reply = new FakePayloadReply(op, req, {}, nullptr);
emit incomingShareDeleted();
fix code format Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
2022-11-23 18:57:14 +03:00
Implement 'Leave this share' context menu entry. Fix incorrect sharing state for incoming and my shares in custom state icons and local database. Signed-off-by: allexzander <blackslayer4@gmail.com>
2022-10-21 17:11:29 +03:00
return reply;
});
QSignalSpy incomingShareDeletedSignal(this, &TestFolderMan::incomingShareDeleted);
// verify first share gets deleted
folderman->leaveShare(fakeFolder.localPath() + firstSharePath);
QCOMPARE(incomingShareDeletedSignal.count(), 1);
QVERIFY(!fakeFolder.remoteModifier().find(firstSharePath));
QVERIFY(fakeFolder.syncOnce());
QCOMPARE(fakeFolder.currentLocalState(), fakeFolder.currentRemoteState());
// verify no share gets deleted
folderman->leaveShare(fakeFolder.localPath() + "A/B/notsharedwithme_B.data");
QCOMPARE(incomingShareDeletedSignal.count(), 1);
QVERIFY(fakeFolder.remoteModifier().find("A/B/sharedwithme_B.data"));
QVERIFY(fakeFolder.syncOnce());
QCOMPARE(fakeFolder.currentLocalState(), fakeFolder.currentRemoteState());
// verify second share gets deleted
folderman->leaveShare(fakeFolder.localPath() + secondSharePath);
QCOMPARE(incomingShareDeletedSignal.count(), 2);
QVERIFY(!fakeFolder.remoteModifier().find(secondSharePath));
QVERIFY(fakeFolder.syncOnce());
QCOMPARE(fakeFolder.currentLocalState(), fakeFolder.currentRemoteState());
OCC::AccountManager::instance()->deleteAccount(accountState);
}
FolderMan::checkPathValidityForNewFolder factorize the function to check the validity of a new folder out of the folder wizard to the FolderMan. There is even a test for it now. That way we can also use it in the normal wizard
2015-07-13 15:35:19 +03:00
void testCheckPathValidityForNewFolder()
{
Fix test on systems where C:\ is not writeable
2020-11-26 17:03:11 +03:00
#ifdef Q_OS_WIN
Utility::NtfsPermissionLookupRAII ntfs_perm;
#endif
FolderMan::checkPathValidityForNewFolder factorize the function to check the validity of a new folder out of the folder wizard to the FolderMan. There is even a test for it now. That way we can also use it in the normal wizard
2015-07-13 15:35:19 +03:00
QTemporaryDir dir;
TestFolderMan: don't pollute user's settings
2016-10-11 12:55:20 +03:00
ConfigFile::setConfDir(dir.path()); // we don't want to pollute the user's config file
FolderMan::checkPathValidityForNewFolder factorize the function to check the validity of a new folder out of the folder wizard to the FolderMan. There is even a test for it now. That way we can also use it in the normal wizard
2015-07-13 15:35:19 +03:00
QVERIFY(dir.isValid());
QDir dir2(dir.path());
QVERIFY(dir2.mkpath("sub/ownCloud1/folder/f"));
QVERIFY(dir2.mkpath("ownCloud2"));
QVERIFY(dir2.mkpath("sub/free"));
QVERIFY(dir2.mkpath("free2/sub"));
Wizards: allow adding a folder in a non-existing directory and create that folder The owncloud wizard already created the directory, but the recent addition of FolderMan::checkPathValidityForNewFolder stopped allowing unexisting directories. So change FolderMan::checkPathValidityForNewFolder to allow non existing directory and whange the FolderWizard to create the directory if it does not exist. Issue #3492
2015-07-28 13:14:52 +03:00
{
QFile f(dir.path() + "/sub/file.txt");
f.open(QFile::WriteOnly);
f.write("hello");
}
Fix FolderManTest on macOS On macOS /var is a symlink to /private/var and we have to make sure that we use the canonical path before and after it enters the code to make sure we compare paths correctly.
2017-02-09 20:17:01 +03:00
QString dirPath = dir2.canonicalPath();
FolderMan::checkPathValidityForNewFolder factorize the function to check the validity of a new folder out of the folder wizard to the FolderMan. There is even a test for it now. That way we can also use it in the normal wizard
2015-07-13 15:35:19 +03:00
Fix the folderman test, adopting the checkPathValidity method tests.
2016-10-07 17:24:09 +03:00
AccountPtr account = Account::create();
QUrl url("http://example.de");
Use auto to avoiding repeating type names Signed-off-by: Kevin Ottens <kevin.ottens@nextcloud.com>
2020-05-18 21:54:23 +03:00
auto *cred = new HttpCredentialsTest("testuser", "secret");
Fix the folderman test, adopting the checkPathValidity method tests.
2016-10-07 17:24:09 +03:00
account->setCredentials(cred);
account->setUrl( url );
AccountStatePtr newAccountState(new AccountState(account));
FolderMan::checkPathValidityForNewFolder factorize the function to check the validity of a new folder out of the folder wizard to the FolderMan. There is even a test for it now. That way we can also use it in the normal wizard
2015-07-13 15:35:19 +03:00
FolderMan *folderman = FolderMan::instance();
QCOMPARE(folderman, &_fm);
Fix FolderManTest on macOS On macOS /var is a symlink to /private/var and we have to make sure that we use the canonical path before and after it enters the code to make sure we compare paths correctly.
2017-02-09 20:17:01 +03:00
QVERIFY(folderman->addFolder(newAccountState.data(), folderDefinition(dirPath + "/sub/ownCloud1")));
QVERIFY(folderman->addFolder(newAccountState.data(), folderDefinition(dirPath + "/ownCloud2")));
FolderMan::checkPathValidityForNewFolder factorize the function to check the validity of a new folder out of the folder wizard to the FolderMan. There is even a test for it now. That way we can also use it in the normal wizard
2015-07-13 15:35:19 +03:00
Invoke 'bool Folder::isSyncRunning() const' from tests. Signed-off-by: allexzander <blackslayer4@gmail.com>
2021-07-28 08:48:38 +03:00
const auto folderList = folderman->map();
for (const auto &folder : folderList) {
QVERIFY(!folder->isSyncRunning());
}
FolderMan::checkPathValidityForNewFolder factorize the function to check the validity of a new folder out of the folder wizard to the FolderMan. There is even a test for it now. That way we can also use it in the normal wizard
2015-07-13 15:35:19 +03:00
// those should be allowed
Make sure Folder is deleted from the list and the SyncJournalDB is closed for every folder of the account that has been removed. Signed-off-by: alex-z <blackslayer4@gmail.com>
2022-09-08 14:21:30 +03:00
// QString FolderMan::checkPathValidityForNewFolder(const QString& path, const QUrl &serverUrl, bool forNewDirectory).second
Fix the folderman test, adopting the checkPathValidity method tests.
2016-10-07 17:24:09 +03:00
Check folder invalidity reson when deciding to enable or disable the 'Finish' button in a Wizard page. Signed-off-by: alex-z <blackslayer4@gmail.com>
2022-09-08 13:00:13 +03:00
QCOMPARE(folderman->checkPathValidityForNewFolder(dirPath + "/sub/free").second, QString());
QCOMPARE(folderman->checkPathValidityForNewFolder(dirPath + "/free2/").second, QString());
Wizards: allow adding a folder in a non-existing directory and create that folder The owncloud wizard already created the directory, but the recent addition of FolderMan::checkPathValidityForNewFolder stopped allowing unexisting directories. So change FolderMan::checkPathValidityForNewFolder to allow non existing directory and whange the FolderWizard to create the directory if it does not exist. Issue #3492
2015-07-28 13:14:52 +03:00
// Not an existing directory -> Ok
Check folder invalidity reson when deciding to enable or disable the 'Finish' button in a Wizard page. Signed-off-by: alex-z <blackslayer4@gmail.com>
2022-09-08 13:00:13 +03:00
QCOMPARE(folderman->checkPathValidityForNewFolder(dirPath + "/sub/bliblablu").second, QString());
QCOMPARE(folderman->checkPathValidityForNewFolder(dirPath + "/sub/free/bliblablu").second, QString());
Make sure Folder is deleted from the list and the SyncJournalDB is closed for every folder of the account that has been removed. Signed-off-by: alex-z <blackslayer4@gmail.com>
2022-09-08 14:21:30 +03:00
// QCOMPARE(folderman->checkPathValidityForNewFolder(dirPath + "/sub/bliblablu/some/more").second, QString());
FolderMan::checkPathValidityForNewFolder factorize the function to check the validity of a new folder out of the folder wizard to the FolderMan. There is even a test for it now. That way we can also use it in the normal wizard
2015-07-13 15:35:19 +03:00
Wizards: allow adding a folder in a non-existing directory and create that folder The owncloud wizard already created the directory, but the recent addition of FolderMan::checkPathValidityForNewFolder stopped allowing unexisting directories. So change FolderMan::checkPathValidityForNewFolder to allow non existing directory and whange the FolderWizard to create the directory if it does not exist. Issue #3492
2015-07-28 13:14:52 +03:00
// A file -> Error
Check folder invalidity reson when deciding to enable or disable the 'Finish' button in a Wizard page. Signed-off-by: alex-z <blackslayer4@gmail.com>
2022-09-08 13:00:13 +03:00
QVERIFY(!folderman->checkPathValidityForNewFolder(dirPath + "/sub/file.txt").second.isNull());
FolderMan::checkPathValidityForNewFolder factorize the function to check the validity of a new folder out of the folder wizard to the FolderMan. There is even a test for it now. That way we can also use it in the normal wizard
2015-07-13 15:35:19 +03:00
Fix the folderman test, adopting the checkPathValidity method tests.
2016-10-07 17:24:09 +03:00
// There are folders configured in those folders, url needs to be taken into account: -> ERROR
QUrl url2(url);
const QString user = account->credentials()->user();
url2.setUserName(user);
// The following both fail because they refer to the same account (user and url)
Check folder invalidity reson when deciding to enable or disable the 'Finish' button in a Wizard page. Signed-off-by: alex-z <blackslayer4@gmail.com>
2022-09-08 13:00:13 +03:00
QVERIFY(!folderman->checkPathValidityForNewFolder(dirPath + "/sub/ownCloud1", url2).second.isNull());
QVERIFY(!folderman->checkPathValidityForNewFolder(dirPath + "/ownCloud2/", url2).second.isNull());
Fix the folderman test, adopting the checkPathValidity method tests.
2016-10-07 17:24:09 +03:00
// Now it will work because the account is different
QUrl url3("http://anotherexample.org");
url3.setUserName("dummy");
Check folder invalidity reson when deciding to enable or disable the 'Finish' button in a Wizard page. Signed-off-by: alex-z <blackslayer4@gmail.com>
2022-09-08 13:00:13 +03:00
QCOMPARE(folderman->checkPathValidityForNewFolder(dirPath + "/sub/ownCloud1", url3).second, QString());
QCOMPARE(folderman->checkPathValidityForNewFolder(dirPath + "/ownCloud2/", url3).second, QString());
Fix the folderman test, adopting the checkPathValidity method tests.
2016-10-07 17:24:09 +03:00
Check folder invalidity reson when deciding to enable or disable the 'Finish' button in a Wizard page. Signed-off-by: alex-z <blackslayer4@gmail.com>
2022-09-08 13:00:13 +03:00
QVERIFY(!folderman->checkPathValidityForNewFolder(dirPath).second.isNull());
QVERIFY(!folderman->checkPathValidityForNewFolder(dirPath + "/sub/ownCloud1/folder").second.isNull());
QVERIFY(!folderman->checkPathValidityForNewFolder(dirPath + "/sub/ownCloud1/folder/f").second.isNull());
FolderMan::checkPathValidityForNewFolder factorize the function to check the validity of a new folder out of the folder wizard to the FolderMan. There is even a test for it now. That way we can also use it in the normal wizard
2015-07-13 15:35:19 +03:00
Fix the TestFolderMan on Windows
2017-12-15 16:50:41 +03:00
#ifndef Q_OS_WIN // no links on windows, no permissions
FolderMan::checkPathValidityForNewFolder factorize the function to check the validity of a new folder out of the folder wizard to the FolderMan. There is even a test for it now. That way we can also use it in the normal wizard
2015-07-13 15:35:19 +03:00
// make a bunch of links
Fix FolderManTest on macOS On macOS /var is a symlink to /private/var and we have to make sure that we use the canonical path before and after it enters the code to make sure we compare paths correctly.
2017-02-09 20:17:01 +03:00
QVERIFY(QFile::link(dirPath + "/sub/free", dirPath + "/link1"));
QVERIFY(QFile::link(dirPath + "/sub", dirPath + "/link2"));
QVERIFY(QFile::link(dirPath + "/sub/ownCloud1", dirPath + "/link3"));
QVERIFY(QFile::link(dirPath + "/sub/ownCloud1/folder", dirPath + "/link4"));
FolderMan::checkPathValidityForNewFolder factorize the function to check the validity of a new folder out of the folder wizard to the FolderMan. There is even a test for it now. That way we can also use it in the normal wizard
2015-07-13 15:35:19 +03:00
// Ok
Make sure Folder is deleted from the list and the SyncJournalDB is closed for every folder of the account that has been removed. Signed-off-by: alex-z <blackslayer4@gmail.com>
2022-09-08 14:21:30 +03:00
QVERIFY(folderman->checkPathValidityForNewFolder(dirPath + "/link1").second.isNull());
QVERIFY(folderman->checkPathValidityForNewFolder(dirPath + "/link2/free").second.isNull());
FolderMan::checkPathValidityForNewFolder factorize the function to check the validity of a new folder out of the folder wizard to the FolderMan. There is even a test for it now. That way we can also use it in the normal wizard
2015-07-13 15:35:19 +03:00
// Not Ok
Make sure Folder is deleted from the list and the SyncJournalDB is closed for every folder of the account that has been removed. Signed-off-by: alex-z <blackslayer4@gmail.com>
2022-09-08 14:21:30 +03:00
QVERIFY(!folderman->checkPathValidityForNewFolder(dirPath + "/link2").second.isNull());
Fix the folderman test, adopting the checkPathValidity method tests.
2016-10-07 17:24:09 +03:00
// link 3 points to an existing sync folder. To make it fail, the account must be the same
Make sure Folder is deleted from the list and the SyncJournalDB is closed for every folder of the account that has been removed. Signed-off-by: alex-z <blackslayer4@gmail.com>
2022-09-08 14:21:30 +03:00
QVERIFY(!folderman->checkPathValidityForNewFolder(dirPath + "/link3", url2).second.isNull());
Fix the folderman test, adopting the checkPathValidity method tests.
2016-10-07 17:24:09 +03:00
// while with a different account, this is fine
Make sure Folder is deleted from the list and the SyncJournalDB is closed for every folder of the account that has been removed. Signed-off-by: alex-z <blackslayer4@gmail.com>
2022-09-08 14:21:30 +03:00
QCOMPARE(folderman->checkPathValidityForNewFolder(dirPath + "/link3", url3).second, QString());
Fix the folderman test, adopting the checkPathValidity method tests.
2016-10-07 17:24:09 +03:00
Make sure Folder is deleted from the list and the SyncJournalDB is closed for every folder of the account that has been removed. Signed-off-by: alex-z <blackslayer4@gmail.com>
2022-09-08 14:21:30 +03:00
QVERIFY(!folderman->checkPathValidityForNewFolder(dirPath + "/link4").second.isNull());
QVERIFY(!folderman->checkPathValidityForNewFolder(dirPath + "/link3/folder").second.isNull());
Wizards: allow adding a folder in a non-existing directory and create that folder The owncloud wizard already created the directory, but the recent addition of FolderMan::checkPathValidityForNewFolder stopped allowing unexisting directories. So change FolderMan::checkPathValidityForNewFolder to allow non existing directory and whange the FolderWizard to create the directory if it does not exist. Issue #3492
2015-07-28 13:14:52 +03:00
// test some non existing sub path (error)
Make sure Folder is deleted from the list and the SyncJournalDB is closed for every folder of the account that has been removed. Signed-off-by: alex-z <blackslayer4@gmail.com>
2022-09-08 14:21:30 +03:00
QVERIFY(!folderman->checkPathValidityForNewFolder(dirPath + "/sub/ownCloud1/some/sub/path").second.isNull());
QVERIFY(!folderman->checkPathValidityForNewFolder(dirPath + "/ownCloud2/blublu").second.isNull());
QVERIFY(!folderman->checkPathValidityForNewFolder(dirPath + "/sub/ownCloud1/folder/g/h").second.isNull());
QVERIFY(!folderman->checkPathValidityForNewFolder(dirPath + "/link3/folder/neu_folder").second.isNull());
Wizards: allow adding a folder in a non-existing directory and create that folder The owncloud wizard already created the directory, but the recent addition of FolderMan::checkPathValidityForNewFolder stopped allowing unexisting directories. So change FolderMan::checkPathValidityForNewFolder to allow non existing directory and whange the FolderWizard to create the directory if it does not exist. Issue #3492
2015-07-28 13:14:52 +03:00
// Subfolder of links
Make sure Folder is deleted from the list and the SyncJournalDB is closed for every folder of the account that has been removed. Signed-off-by: alex-z <blackslayer4@gmail.com>
2022-09-08 14:21:30 +03:00
QVERIFY(folderman->checkPathValidityForNewFolder(dirPath + "/link1/subfolder").second.isNull());
QVERIFY(folderman->checkPathValidityForNewFolder(dirPath + "/link2/free/subfolder").second.isNull());
Wizards: allow adding a folder in a non-existing directory and create that folder The owncloud wizard already created the directory, but the recent addition of FolderMan::checkPathValidityForNewFolder stopped allowing unexisting directories. So change FolderMan::checkPathValidityForNewFolder to allow non existing directory and whange the FolderWizard to create the directory if it does not exist. Issue #3492
2015-07-28 13:14:52 +03:00
// Should not have the rights
Make sure Folder is deleted from the list and the SyncJournalDB is closed for every folder of the account that has been removed. Signed-off-by: alex-z <blackslayer4@gmail.com>
2022-09-08 14:21:30 +03:00
QVERIFY(!folderman->checkPathValidityForNewFolder("/").second.isNull());
QVERIFY(!folderman->checkPathValidityForNewFolder("/usr/bin/somefolder").second.isNull());
Fix the TestFolderMan on Windows
2017-12-15 16:50:41 +03:00
#endif
FolderWizard: Don't crash when typing invalid drive #7041 When the user typed "x:" where the drive x didn't exist, the validation function would loop forever. Now it shows a "path doesn't exist" error.
2019-03-14 11:12:46 +03:00
#ifdef Q_OS_WIN // drive-letter tests
if (!QFileInfo("v:/").exists()) {
Check folder invalidity reson when deciding to enable or disable the 'Finish' button in a Wizard page. Signed-off-by: alex-z <blackslayer4@gmail.com>
2022-09-08 13:00:13 +03:00
QVERIFY(!folderman->checkPathValidityForNewFolder("v:").second.isNull());
QVERIFY(!folderman->checkPathValidityForNewFolder("v:/").second.isNull());
QVERIFY(!folderman->checkPathValidityForNewFolder("v:/foo").second.isNull());
FolderWizard: Don't crash when typing invalid drive #7041 When the user typed "x:" where the drive x didn't exist, the validation function would loop forever. Now it shows a "path doesn't exist" error.
2019-03-14 11:12:46 +03:00
}
if (QFileInfo("c:/").isWritable()) {
Check folder invalidity reson when deciding to enable or disable the 'Finish' button in a Wizard page. Signed-off-by: alex-z <blackslayer4@gmail.com>
2022-09-08 13:00:13 +03:00
QVERIFY(folderman->checkPathValidityForNewFolder("c:").second.isNull());
QVERIFY(folderman->checkPathValidityForNewFolder("c:/").second.isNull());
QVERIFY(folderman->checkPathValidityForNewFolder("c:/foo").second.isNull());
FolderWizard: Don't crash when typing invalid drive #7041 When the user typed "x:" where the drive x didn't exist, the validation function would loop forever. Now it shows a "path doesn't exist" error.
2019-03-14 11:12:46 +03:00
}
#endif
Fix the TestFolderMan on Windows
2017-12-15 16:50:41 +03:00
// Invalid paths
Check folder invalidity reson when deciding to enable or disable the 'Finish' button in a Wizard page. Signed-off-by: alex-z <blackslayer4@gmail.com>
2022-09-08 13:00:13 +03:00
QVERIFY(!folderman->checkPathValidityForNewFolder("").second.isNull());
FolderMan::checkPathValidityForNewFolder: make sure to work when folder points to deleted folders Note that we also needed to adjust the server url to contains the user name in the folder wizard. (As checkPathValidityForNewFolder expect the user name) Issue #6654
2018-07-16 13:46:03 +03:00
// REMOVE ownCloud2 from the filesystem, but keep a folder sync'ed to it.
QDir(dirPath + "/ownCloud2/").removeRecursively();
Check folder invalidity reson when deciding to enable or disable the 'Finish' button in a Wizard page. Signed-off-by: alex-z <blackslayer4@gmail.com>
2022-09-08 13:00:13 +03:00
QVERIFY(!folderman->checkPathValidityForNewFolder(dirPath + "/ownCloud2/blublu").second.isNull());
QVERIFY(!folderman->checkPathValidityForNewFolder(dirPath + "/ownCloud2/sub/subsub/sub").second.isNull());
Wizards: Never propose an existing folder for syncing #5597
2017-03-20 16:28:32 +03:00
}
void testFindGoodPathForNewSyncFolder()
{
// SETUP
QTemporaryDir dir;
ConfigFile::setConfDir(dir.path()); // we don't want to pollute the user's config file
QVERIFY(dir.isValid());
QDir dir2(dir.path());
QVERIFY(dir2.mkpath("sub/ownCloud1/folder/f"));
QVERIFY(dir2.mkpath("ownCloud"));
QVERIFY(dir2.mkpath("ownCloud2"));
QVERIFY(dir2.mkpath("ownCloud2/foo"));
QVERIFY(dir2.mkpath("sub/free"));
QVERIFY(dir2.mkpath("free2/sub"));
QString dirPath = dir2.canonicalPath();
AccountPtr account = Account::create();
QUrl url("http://example.de");
Use auto to avoiding repeating type names Signed-off-by: Kevin Ottens <kevin.ottens@nextcloud.com>
2020-05-18 21:54:23 +03:00
auto *cred = new HttpCredentialsTest("testuser", "secret");
Wizards: Never propose an existing folder for syncing #5597
2017-03-20 16:28:32 +03:00
account->setCredentials(cred);
account->setUrl( url );
FolderMan::checkPathValidityForNewFolder: make sure to work when folder points to deleted folders Note that we also needed to adjust the server url to contains the user name in the folder wizard. (As checkPathValidityForNewFolder expect the user name) Issue #6654
2018-07-16 13:46:03 +03:00
url.setUserName(cred->user());
Wizards: Never propose an existing folder for syncing #5597
2017-03-20 16:28:32 +03:00
AccountStatePtr newAccountState(new AccountState(account));
FolderMan *folderman = FolderMan::instance();
QCOMPARE(folderman, &_fm);
QVERIFY(folderman->addFolder(newAccountState.data(), folderDefinition(dirPath + "/sub/ownCloud/")));
QVERIFY(folderman->addFolder(newAccountState.data(), folderDefinition(dirPath + "/ownCloud2/")));
// TEST
QCOMPARE(folderman->findGoodPathForNewSyncFolder(dirPath + "/oc", url),
QString(dirPath + "/oc"));
QCOMPARE(folderman->findGoodPathForNewSyncFolder(dirPath + "/ownCloud", url),
QString(dirPath + "/ownCloud3"));
QCOMPARE(folderman->findGoodPathForNewSyncFolder(dirPath + "/ownCloud2", url),
QString(dirPath + "/ownCloud22"));
QCOMPARE(folderman->findGoodPathForNewSyncFolder(dirPath + "/ownCloud2/foo", url),
QString(dirPath + "/ownCloud2/foo"));
QCOMPARE(folderman->findGoodPathForNewSyncFolder(dirPath + "/ownCloud2/bar", url),
QString(dirPath + "/ownCloud2/bar"));
QCOMPARE(folderman->findGoodPathForNewSyncFolder(dirPath + "/sub", url),
QString(dirPath + "/sub2"));
FolderMan::checkPathValidityForNewFolder: make sure to work when folder points to deleted folders Note that we also needed to adjust the server url to contains the user name in the folder wizard. (As checkPathValidityForNewFolder expect the user name) Issue #6654
2018-07-16 13:46:03 +03:00
// REMOVE ownCloud2 from the filesystem, but keep a folder sync'ed to it.
// We should still not suggest this folder as a new folder.
QDir(dirPath + "/ownCloud2/").removeRecursively();
QCOMPARE(folderman->findGoodPathForNewSyncFolder(dirPath + "/ownCloud", url),
QString(dirPath + "/ownCloud3"));
QCOMPARE(folderman->findGoodPathForNewSyncFolder(dirPath + "/ownCloud2", url),
QString(dirPath + "/ownCloud22"));
FolderMan::checkPathValidityForNewFolder factorize the function to check the validity of a new folder out of the folder wizard to the FolderMan. There is even a test for it now. That way we can also use it in the normal wizard
2015-07-13 15:35:19 +03:00
}
};
Implement 'Leave this share' context menu entry. Fix incorrect sharing state for incoming and my shares in custom state icons and local database. Signed-off-by: allexzander <blackslayer4@gmail.com>
2022-10-21 17:11:29 +03:00
QTEST_GUILESS_MAIN(TestFolderMan)
Simplify the build of auto tests Remove all configure_files: - Move all tests to cpp files - Use the QTEST_MAIN macro instead of a generated main.cpp - Include test*.moc in the cpp to let CMAKE_AUTOMOC call moc - Pass info through add_definitions instead of generating oc_bin.h with them This makes sure that build errors points to the original test source file instead of the generated one in the build directory to be able to jump and fix errors directly from the IDE's error pane.
2016-03-30 18:58:15 +03:00
#include "testfolderman.moc"
Reference in a new issue Copy permalink