skip single test or whole test file for macOS: they are broken

would need careful analyze to fix those

until we can get them fixed, better make the other tests be required in
the CI

Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>

test/CMakeLists.txt

@@ -47,9 +47,9 @@ nextcloud_add_test(Utility)
 
 if (NOT APPLE)
     nextcloud_add_test(SyncEngine)
+    nextcloud_add_test(SyncVirtualFiles)
 endif()
 
-nextcloud_add_test(SyncVirtualFiles)
 nextcloud_add_test(SyncMove)
 nextcloud_add_test(SyncDelete)
 nextcloud_add_test(SyncConflict)

test/testfolderwatcher.cpp

@@ -331,6 +331,10 @@ private slots:
     }
     void testDetectLockFilesExternally()
     {
+#if defined Q_OS_MACOS
+        QSKIP("not reliable on macOS");
+#endif
+
         QStringList listOfOfficeFiles = {QString(_rootPath + "/document.docx"), QString(_rootPath + "/document.odt")};
         std::sort(std::begin(listOfOfficeFiles), std::end(listOfOfficeFiles));
 

test/testsyncmove.cpp

@@ -976,6 +976,10 @@ private slots:
 
     void testMovedWithError()
     {
+#if defined Q_OS_MACOS
+        QSKIP("not reliable on macOS");
+#endif
+
         QFETCH(Vfs::Mode, vfsMode);
         const auto getName = [vfsMode] (const QString &s)
         {