Fix test on systems where C:\ is not writeable
authorHannah von Reth <hannah.vonreth@owncloud.com>
Thu, 26 Nov 2020 14:03:11 +0000 (15:03 +0100)
committerKevin Ottens <kevin.ottens@nextcloud.com>
Tue, 15 Dec 2020 09:59:27 +0000 (10:59 +0100)
src/gui/folderman.cpp
test/testfolderman.cpp

index 04d6eef60a2c4264141b6d11cf261f7aa8a3fb5e..87ecc4fb9e75410fd0568c4e1ce5daab2aa980ee 100644 (file)
@@ -1503,8 +1503,10 @@ static QString canonicalPath(const QString &path)
 QString FolderMan::checkPathValidityForNewFolder(const QString &path, const QUrl &serverUrl) const
 {
     QString recursiveValidity = checkPathValidityRecursive(path);
-    if (!recursiveValidity.isEmpty())
+    if (!recursiveValidity.isEmpty()) {
+        qCDebug(lcFolderMan) << path << recursiveValidity;
         return recursiveValidity;
+    }
 
     // check if the local directory isn't used yet in another ownCloud sync
     Qt::CaseSensitivity cs = Qt::CaseSensitive;
index 21a475327fe4fa2067b2f26cb4607cd97fbf4e82..66db0128ba5c7d17aa17cb9da9258e89a9d6524e 100644 (file)
@@ -27,6 +27,9 @@ class TestFolderMan: public QObject
 private slots:
     void testCheckPathValidityForNewFolder()
     {
+#ifdef Q_OS_WIN
+        Utility::NtfsPermissionLookupRAII ntfs_perm;
+#endif
         QTemporaryDir dir;
         ConfigFile::setConfDir(dir.path()); // we don't want to pollute the user's config file
         QVERIFY(dir.isValid());