From ff343c999dffa985dc86044fbb6a447845505fcc Mon Sep 17 00:00:00 2001
From: Jean-Christophe Fillion-Robin <jchris.fillionr@kitware.com>
Date: Sat, 13 Jan 2024 14:14:54 -0500
Subject: [PATCH] BUG: Fix ctkUtilsCopyDirRecursivelyTest on windows creating
 hidden files

---
 .../Cpp/ctkUtilsCopyDirRecursivelyTest1.cpp   | 62 ++++++++++++++++++-
 1 file changed, 59 insertions(+), 3 deletions(-)

diff --git a/Libs/Core/Testing/Cpp/ctkUtilsCopyDirRecursivelyTest1.cpp b/Libs/Core/Testing/Cpp/ctkUtilsCopyDirRecursivelyTest1.cpp
index bdd136c955..e4b36afc0b 100644
--- a/Libs/Core/Testing/Cpp/ctkUtilsCopyDirRecursivelyTest1.cpp
+++ b/Libs/Core/Testing/Cpp/ctkUtilsCopyDirRecursivelyTest1.cpp
@@ -32,6 +32,11 @@
 #include <cstdlib>
 #include <iostream>
 
+// Windows includes
+#ifdef Q_OS_WIN
+#include <windows.h> // For SetFileAttributesW
+#endif
+
 namespace
 {
 //-----------------------------------------------------------------------------
@@ -56,6 +61,49 @@ bool createFile(int line, const QDir& dir, const QString& relativePath, const QS
   return true;
 }
 
+//-----------------------------------------------------------------------------
+bool hideFile(int line, const QDir& dir, const QString& relativePath)
+{
+  QString filePath = QFileInfo(dir, relativePath).filePath();
+
+  if (!QFile::exists(filePath))
+    {
+    std::cerr << "Line " << line << " - File" << qPrintable(filePath) << "does not exist" <<std::endl;
+    return false;
+    }
+
+#ifdef Q_OS_WIN
+  const wchar_t* filePathW = filePath.toStdWString().c_str();
+  DWORD dwAttrs = GetFileAttributesW(filePathW);
+  if ((dwAttrs & FILE_ATTRIBUTE_HIDDEN) == 0)
+    {
+    SetFileAttributesW(filePathW, dwAttrs | FILE_ATTRIBUTE_HIDDEN);
+    }
+#endif
+
+  return true;
+}
+
+//-----------------------------------------------------------------------------
+bool isFileHidden(int line, const QDir& dir, const QString& relativePath)
+{
+  QString filePath = QFileInfo(dir, relativePath).filePath();
+
+  if (!QFile::exists(filePath))
+    {
+    std::cerr << "Line " << line << " - File" << qPrintable(filePath) << "does not exist" <<std::endl;
+    return false;
+    }
+
+  if (!QFileInfo(filePath).isHidden())
+    {
+    std::cerr << "Line " << line << " - File" << qPrintable(filePath) << "is expected to be hidden" << std::endl;
+    return false;
+    }
+
+  return true;
+}
+
 } // end of anonymous namespace
 
 
@@ -112,15 +160,23 @@ int ctkUtilsCopyDirRecursivelyTest1(int argc, char * argv [] )
     {
     return EXIT_FAILURE;
     }
-  if (!createFile(__LINE__, tmp, "foo/zoo", ".hidden.txt"))
+  if (!(createFile(__LINE__, tmp, "foo/zoo", ".hidden.txt")
+        && hideFile(__LINE__, tmp, "foo/zoo/.hidden.txt")
+        && isFileHidden(__LINE__, tmp, "foo/zoo/.hidden.txt")))
     {
     return EXIT_FAILURE;
     }
-  if (!createFile(__LINE__, tmp, "foo/.hidden", "c.txt"))
+  if (!(createFile(__LINE__, tmp, "foo/.hidden", "c.txt")
+        && hideFile(__LINE__, tmp, "foo/.hidden")
+        && isFileHidden(__LINE__, tmp, "foo/.hidden")))
     {
     return EXIT_FAILURE;
     }
-  if (!createFile(__LINE__, tmp, "foo/.hidden", ".hidden.txt"))
+  if (!(createFile(__LINE__, tmp, "foo/.hidden", ".hidden.txt")
+        && hideFile(__LINE__, tmp, "foo/.hidden")
+        && hideFile(__LINE__, tmp, "foo/.hidden/.hidden.txt")
+        && isFileHidden(__LINE__, tmp, "foo/.hidden")
+        && isFileHidden(__LINE__, tmp, "foo/.hidden/.hidden.txt")))
     {
     return EXIT_FAILURE;
     }